From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id D9602461AD for ; Thu, 6 Feb 2025 16:15:21 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id CE4ED40E27; Thu, 6 Feb 2025 16:15:21 +0100 (CET) Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by mails.dpdk.org (Postfix) with ESMTP id 4DBA5402F1 for ; Thu, 6 Feb 2025 16:15:19 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1738854918; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=tflXWDwX/4NkSERa0pxRuzhNb59F/a2/weGKkvLCGE4=; b=OgaO1uNBtAxUX5pD9miL6tvr8rs1hCL0J3fhvL8GCbZ4qGFcdRDnf4RtmV4gjWrZHRyFbn cq1sldyTyzRIZTEw/T3UP/qGcYZMdajFBFVlSVWGNdpjnYBL5kRPXbheIVlZHcL0Chbnd/ aFL5kc7nlnyxS7WWIw1JwMQh4hNOa5Y= Received: from mail-lf1-f71.google.com (mail-lf1-f71.google.com [209.85.167.71]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-674-45Q8sxRRPhual6EMeLGyCg-1; Thu, 06 Feb 2025 10:15:15 -0500 X-MC-Unique: 45Q8sxRRPhual6EMeLGyCg-1 X-Mimecast-MFC-AGG-ID: 45Q8sxRRPhual6EMeLGyCg Received: by mail-lf1-f71.google.com with SMTP id 2adb3069b0e04-5440c445282so598105e87.2 for ; Thu, 06 Feb 2025 07:15:15 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1738854914; x=1739459714; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=tflXWDwX/4NkSERa0pxRuzhNb59F/a2/weGKkvLCGE4=; b=X0UUjcx8VV/Y/dFmsYBut/OVP32uGo38Gb7pCFS0z486UhWTpZAU1UhUMow1Pe7YJo iGJhXCakUlblOquDzIEmWkki7i0m8+knAWP0Bosf9ga0zyoJmTgh5aEhm32iOnX4119D xpXhBHHSxnW9RvkVEWwecSBZmoiRRBEnC/mxGI1NEBUOfKeHFOsIeRESt8xdGrDtPieF DxL3Rlrr6Zbn5vNfpBvPPnyEgx9LvJCWZZIsSEnhi33JVMW5jl0G42MelKasV3fe+h0Y ryveppDhhT49ZKhzOhV6WLvXHhICg05V7CJ1SL5cU7Id0w7R7DlMCAS4tkj0ijLDFaHG mcKA== X-Forwarded-Encrypted: i=1; AJvYcCXNJzdR0Ob5+klbrCG5nNSsnYo0cq3yBTjkpwT6xVLT4x3LBA5wrVVkcreBVHQ6lLp4k7fRZDc=@dpdk.org X-Gm-Message-State: AOJu0YyIs3oP4qyekyNoBle6l/2iy+ew5nBlwjRAEWMZCYIqzDC65V4C 0JVmSFOeU+Os5S667nj7DPpX3vSCJn6obaMhrQm2wBSd72XgvEDv2DhcBgyvfD0SlJTVd03E4pa 9Uqu0DZzAtl8dwTZMMR6nIWopSXcxzo1RiI9aHHYE2dSZPsxGg4e/zZCe9A/ZaZPcO6iFJNkTn2 VkmEV5jtWisQeoFMalZo8= X-Gm-Gg: ASbGncuyvOjjlGH7nYYSA8lNczWE2+F4UThRiQ+0A8WEbWFjo00wi3mcWZRU7eNdexc YlJBKdYEBk7Ycfo3gdYqPwqYx+1TCC43/KE0wEU5daSYwkVaqEQq8ZHBWlIGMZ5g+ X-Received: by 2002:a05:6512:220a:b0:542:8da7:242f with SMTP id 2adb3069b0e04-54405a10653mr2534278e87.10.1738854913980; Thu, 06 Feb 2025 07:15:13 -0800 (PST) X-Google-Smtp-Source: AGHT+IGAgpKDazhHIUnIE+g55VXBN2sCbfZ4KyLMuUFas3LgwCWZK/0pNOJLBiC38QmspoXb1iZrEDqqSF6Iozl7IT0= X-Received: by 2002:a05:6512:220a:b0:542:8da7:242f with SMTP id 2adb3069b0e04-54405a10653mr2534273e87.10.1738854913606; Thu, 06 Feb 2025 07:15:13 -0800 (PST) MIME-Version: 1.0 References: <20250206084154.2230766-1-maxime.coquelin@redhat.com> In-Reply-To: <20250206084154.2230766-1-maxime.coquelin@redhat.com> From: David Marchand Date: Thu, 6 Feb 2025 16:15:02 +0100 X-Gm-Features: AWEUYZmn1x1TyTV9mJUyDEQWOLEI4x7lWpUbIaUAuXjsLVUwjrkfvQsh-wacqjs Message-ID: Subject: Re: [PATCH] vhost: fix FD entries cleanup To: Maxime Coquelin Cc: dev@dpdk.org, chenbox@nvidia.com, stable@dpdk.org X-Mimecast-Spam-Score: 0 X-Mimecast-MFC-PROC-ID: _tkLSPEN5ZqeogMONAeroa6Q5OOC78pEwbITyEeKd9U_1738854914 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: patches for DPDK stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: stable-bounces@dpdk.org On Thu, Feb 6, 2025 at 9:42=E2=80=AFAM Maxime Coquelin wrote: > > With the recent rework of the FD manager to use epoll, > an error message is emitted with Vhost-user at FD entry > removal: > > ERR|VHOST_FDMAN: could not remove 102 fd from 101 epfd: No such file or d= irectory > > It occurs because the read callback closes the file > descriptor before it is removed from the FD set. > This patch defers the close() after the FD entry is removed > from the set. > > Fixes: 0e38b42bf61c ("vhost: manage FD with epoll") > Cc: stable@dpdk.org > > Signed-off-by: David Marchand > Signed-off-by: Maxime Coquelin LGTM. --=20 David Marchand