On 22.07.2019 21:10, William Tu wrote:
> On Mon, Jul 22, 2019 at 05:08:48PM +0300, Ilya Maximets wrote:
>> In case of failure of 'xsk_configure_all()', 'n_rxq' and 'xdpmode'
>> will remain in a new state. This will result in successful
>> reconfiguration (immediate return, because configuration is already
>> applied) if 'netdev_reconfigure()' will be called again.
>>
>> Same issue was fixed previously for netdev-dpdk using 'dev->started'
>> flag in commit:
>> 606f66507250 ("netdev-dpdk: Don't use PMD driver if not configured
>> successfully")
>>
>> Let's use similar approach with checking the 'dev->xsks' which only
>> exists if configuration was successful.
>>
>> Additionally implemented 'netdev_afxdp_construct()' function to
>> explicitly initialize all the specific fields and request the
>> reconfiguration.
>>
>> CC: William Tu <[email protected]>
>> Fixes: 0de1b425962d ("netdev-afxdp: add new netdev type for AF_XDP.")
>> Signed-off-by: Ilya Maximets <[email protected]>
>
> LGTM, thanks for the fix.
>
> Acked-by: William Tu <[email protected]>
Thanks!
Applied to master and branch-2.12.
Best regards, Ilya Maximets.
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
