[dpdk-dev] [PATCH v3 0/2] IOPL related fixes
Maxime Coquelin
maxime.coquelin at redhat.com
Fri Nov 23 17:25:05 CET 2018
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 at 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(-)
>>>
>>
>>
More information about the dev
mailing list