From mboxrd@z Thu Jan 1 00:00:00 1970 From: Maxime Coquelin Subject: Re: [PATCH v3 0/2] IOPL related fixes Date: Fri, 23 Nov 2018 17:25:05 +0100 Message-ID: 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> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit Cc: Tiwei Bie , Zhihong Wang , Thomas Monjalon , Ferruh Yigit , Ian Stokes , Kevin Traynor , Bruce Richardson To: Ilya Maximets , dev@dpdk.org, David Marchand Return-path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by dpdk.org (Postfix) with ESMTP id E87BB1B556 for ; Fri, 23 Nov 2018 17:25:37 +0100 (CET) In-Reply-To: <69a9bf2c-7664-4e63-48a4-fe2f73ce3c86@samsung.com> Content-Language: en-US List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 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. 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(-) >>> >> >>