On 11/25/2018 10:42 AM, Thomas Monjalon wrote: > 23/11/2018 23:29, Thomas Monjalon: >> 23/11/2018 21:07, Kevin Traynor: >>> 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: >>>>>> 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. >> >> I won't take any risk at this stage. >> The memory of the headaches we had in the past with iopl is too much strong. >> The real change of this patch is to register the PMD even without successful >> iopl. >> I would like to be sure it won't cause any regression. > > After more thinking, I decide to take the risk. > Take it as a gift to OVS :-) >
ok, thanks :-) If there's any subsequent changes needed, I will get them on an 18.11 stable branch straight away. > Series applied > >
