From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by dpdk.org (Postfix) with ESMTP id 40F0C325F for ; Fri, 23 Nov 2018 21:07:31 +0100 (CET) Received: from smtp.corp.redhat.com (int-mx01.intmail.prod.int.phx2.redhat.com [10.5.11.11]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id ECDB183F3F; Fri, 23 Nov 2018 20:07:29 +0000 (UTC) Received: from ktraynor.remote.csb (unknown [10.36.118.7]) by smtp.corp.redhat.com (Postfix) with ESMTP id 38CFE604CD; Fri, 23 Nov 2018 20:07:22 +0000 (UTC) To: Maxime Coquelin , Ilya Maximets , dev@dpdk.org, David Marchand Cc: Tiwei Bie , Zhihong Wang , Thomas Monjalon , Ferruh Yigit , Ian Stokes , Bruce Richardson References: <20181123143620.10480-1-i.maximets@samsung.com> <20181123153920.12398-1-i.maximets@samsung.com> <4b4f7faf-fb21-74d6-f76e-105ba78ff59c@redhat.com> <69a9bf2c-7664-4e63-48a4-fe2f73ce3c86@samsung.com> From: Kevin Traynor Organization: Red Hat Message-ID: <2f95ce88-8f61-10c0-9131-49e2a5d10048@redhat.com> Date: Fri, 23 Nov 2018 20:07:20 +0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit X-Scanned-By: MIMEDefang 2.79 on 10.5.11.11 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.27]); Fri, 23 Nov 2018 20:07:30 +0000 (UTC) Subject: Re: [dpdk-dev] [PATCH v3 0/2] IOPL related fixes X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 23 Nov 2018 20:07:31 -0000 On 11/23/2018 04:25 PM, Maxime Coquelin wrote: > > > On 11/23/18 4:53 PM, Ilya Maximets wrote: >> On 23.11.2018 18:41, Maxime Coquelin wrote: >>> Hi, >>> >>> On 11/23/18 4:39 PM, Ilya Maximets wrote: >>>> Patches primary targeted to fix OVS unit test failures with >>>> DPDK 18.11 due to following error: >>>> 'IOPL call failed - cannot use virtio PMD'. >>> >>> You mention v18.11, do you mean this is a regression? >> >> Kind of. But not really a bug. It's just a message that shows up >> every time you starting the app as a non-root user. >> >> The message itself was introduced long time ago, but it wasn't >> printed for unclear reasons. It's probably some change in logging >> subsystem uncovered it. >> >> I just checked and found that message appears starting from v18.02. >> But OVS stuck with 17.11 LTS where this message exists, but not >> printed for some reason. That's why this wasn't an issue previously. > > Thanks for the clarification. > In that case, we may want it in v18.11 LTS, but I think it can wait or > v18.11.1. > It would be better if it can be in 18.11. Upstream OVS will point at a release tarball, and at present an 18.11.1 release is not due until mid-January. If it's too late for 18.11, alternatively we could do an earlier 18.11.1 release. > Once in the next-virtio tree, I'll send it to stable@dpdk.org. > > Regards, > Maxime > >> P.S. OVS tests just checks the output for error messages. >> Tests themselves works fine. They are not actually using DPDK. >> >> Best regards, Ilya Maximets. >> >>> >>>> >>>> We can avoid test failures in OVS by filtering the output like this: >>>> https://patchwork.ozlabs.org/project/openvswitch/list/?series=77706 >>>> but it still looks very inconvenient for me to have this >>>> message in the output of every command for the DPDK linked app. >>>> >>>> Version 3: >>>> * Splitted in two patches. [David Marchand] >>>> * Fixed typo in commit message. [David Marchand] >>>> >>>> Version 2: >>>> * Fixed possible fd leak on BSD. >>>> >>>> Ilya Maximets (2): >>>> eal/bsd: fix possible IOPL fd leak >>>> net/virtio: avoid annoying IOPL call related errors >>>> >>>> drivers/net/virtio/virtio_ethdev.c | 11 ++++++----- >>>> lib/librte_eal/bsdapp/eal/eal.c | 6 ++++-- >>>> 2 files changed, 10 insertions(+), 7 deletions(-) >>>> >>> >>>