On 3/18/21 12:34 PM, Kevin Traynor wrote:
> Previously auto load balance did not trigger a reassignment when
> there was any cross-numa polling as an rxq could be polled from a
> different numa after reassign and it could impact estimates.
> 
> In the case where there is only one numa with pmds available, the
> same numa will always poll before and after reassignment, so estimates
> are valid. Allow PMD auto load balance to trigger a reassignment in
> this case.
> 
> Acked-by: Eelco Chaudron <[email protected]>
> Acked-by: David Marchand <[email protected]>
> Tested-by: Sunil Pai G <[email protected]>
> Acked-by: Flavio Leitner <[email protected]>
> Signed-off-by: Kevin Traynor <[email protected]>
> ---
> https://github.com/kevintraynor/ovs/actions/runs/664214192
> v3:
> - Updated docs and logs as per Ilya suggestion
> v2:
> - Same logic as v1, combined two "ifs" as per David suggestion
> - Updated comments/logs
> - Updated the doc note that said it will not work for cross NUMA to
>   include new condition
> - Kept Eelco's Ack, as no logic changed
> ---
>  Documentation/topics/dpdk/pmd.rst |  4 +++-
>  lib/dpif-netdev.c                 | 19 ++++++++++++++++---
>  2 files changed, 19 insertions(+), 4 deletions(-)

Thanks!  I added a NEWS entry and applied to master.

Best regards, Ilya Maximets.
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to