From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 89E7EA0093 for ; Fri, 22 May 2020 09:37:07 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 6A6E91D916; Fri, 22 May 2020 09:37:07 +0200 (CEST) Received: from us-smtp-delivery-1.mimecast.com (us-smtp-2.mimecast.com [205.139.110.61]) by dpdk.org (Postfix) with ESMTP id D84281D8F0 for ; Fri, 22 May 2020 09:37:04 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1590133024; 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: in-reply-to:in-reply-to:references:references; bh=alSolqSUTozqCguq2NmGA6ypDwh/rDcXvixzduvD0YE=; b=aOT4krPO4ghWOASb0ntI0/IDyNEOAWpgXZRSsWxTlQvKN7rbNBbnULhSyt1RTsrkhTwcd6 Gl56zpbBQ2Li5U9nWkvq3o/OyevIwRYLPjYglcaYYJkR43FaDe9IIX1CP/9zQi95q/cj2v qkYNJ13t9LMgPN5pkJ40ecbrGMwNpE8= Received: from mail-vk1-f200.google.com (mail-vk1-f200.google.com [209.85.221.200]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-246-mhPMCehgMaedOxYkELueRQ-1; Fri, 22 May 2020 03:37:00 -0400 X-MC-Unique: mhPMCehgMaedOxYkELueRQ-1 Received: by mail-vk1-f200.google.com with SMTP id u190so3913051vke.0 for ; Fri, 22 May 2020 00:37:00 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=alSolqSUTozqCguq2NmGA6ypDwh/rDcXvixzduvD0YE=; b=nNCOwMVcWdDjG2owhn3dfuihBNf4x+d/NaWLyq3XkUD0qmxlupYPQJyMv5/GIgV3OS d1C4/JpBrdoKA1P0PyQcr/gDfQIxHW9QBgHHtpLc0aV3/VYmmRMQ3STJe+rHOiegaxoP 8LLuk7LJBf7uNrcGBPJpKeIK/whkekagiTse/wd6xTWmDkHSyV7e6S9DzMAnFfcwiECw CiCx5mQj7h2hebTZ7sB3d17fycFBCeCblSwyggZ7DwKFSggpbbgkpcn1gyQTGb6UhTC7 0QFTyY+xPWhNvs/OvjEN4K366zorVviOr/wDPbWUPHbn97kjiQ5M4UYFs0fRbGjFZ59C LLGA== X-Gm-Message-State: AOAM53189t5+BY2HfgViQrnF1gwQ3vR0U97acsOjYysL9VGEag4y0JHr x3Koo0Mz15hVhdLejKYEFv9eMk+z+kNqWF5WJgw3sztECelS2QCgns18RY9z5RMi6vfoTvWrVV8 rZuju11eBhjpavaOcWDI0egA= X-Received: by 2002:a9f:3273:: with SMTP id y48mr10009467uad.53.1590133019986; Fri, 22 May 2020 00:36:59 -0700 (PDT) X-Google-Smtp-Source: ABdhPJzTrq92cHP7Dcdw3UQ8/Wx+JJKUEUFxETQgr7AGgnvH5g2qXEihp+8UvfstxVMoJFQgR7vs6vfFO9uSm91dPaE= X-Received: by 2002:a9f:3273:: with SMTP id y48mr10009457uad.53.1590133019630; Fri, 22 May 2020 00:36:59 -0700 (PDT) MIME-Version: 1.0 References: <1589859720-16224-1-git-send-email-wangyunjian@huawei.com> In-Reply-To: From: David Marchand Date: Fri, 22 May 2020 09:36:48 +0200 Message-ID: To: "Burakov, Anatoly" Cc: wangyunjian , Luca Boccassi , Kevin Traynor , dev , Hemant Agrawal , Sachin Saxena , "Lilijun (Jerry)" , xudingke , dpdk stable X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset="UTF-8" Subject: Re: [dpdk-stable] [dpdk-dev] [PATCH 1/2] vfio: fix check for vfio_group_fd X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.15 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 Sender: "stable" On Thu, May 21, 2020 at 2:54 PM Burakov, Anatoly wrote: > > On 19-May-20 8:43 AM, David Marchand wrote: > > On Tue, May 19, 2020 at 5:42 AM wangyunjian wrote: > >> > >> From: Yunjian Wang > >> > >> The issue is that a file descriptor at 0 is a valid one. Currently > >> the file not found, the return value will be set to 0. As a result, > >> it is impossible to distinguish between a correct descriptor and a > >> failed return value. Fix it to return -ENOENT instead of 0. > >> > >> Fixes: b758423bc4fe ("vfio: fix race condition with sysfs") > >> Fixes: ff0b67d1c868 ("vfio: DMA mappinge") > >> Cc: stable@dpdk.org > > > > This patch reverts the (marked for stable) fix b758423bc4fe ("vfio: > > fix race condition with sysfs") and comes with a different fix. > > > > - Kevin, Luca, I would put this b758423bc4fe backport on hold. > > > > - Anatoly, I don't want to put 20.05 at risk. > > My simple question for 20.05 is, should we revert b758423bc4fe? > > No, let's not revert anything. > > As far as i can tell, this patch is a more complete fix, but it > essentially does the same thing, just in a different (and better) way. > Still, i haven't reviewed it in detail. > > > > > - This patchset will go to 20.08 as I don't feel confident in taking it now. > > > > Yes, let's leave it for 20.08. Thanks for the analysis Anatoly. -- David Marchand