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 sta...@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(-)



Reply via email to