[ovs-dev] [PATCH v6 0/4] Add PMD auto load balance parameters.
Kevin Traynor
ktraynor at redhat.com
Mon Feb 1 15:41:53 UTC 2021
On 21/12/2020 13:10, Kevin Traynor wrote:
> v6:
> - Split patches to add log to existing interval param as a seperate fix first
> - Fix reference in vswitchd.xml
> - Kept Acks as minor changes
>
> v5:
> - Make naming more consistent in vswitchd.xml/NEWS/commit msgs
> - Added Co-Authored-By
>
> v4:
> - Changed naming of params
> - Updated default #defines names
> - Used atomic for param accessed in the pmd thread
> - Revert to default if param range is exceeded
> - Only set params if they are changed
> - Add logging for params when they are changed
> - Add log for auto load balance enabled/disabled when param changed
>
A question has been raised about whether this series should be
backported to 2.13/2/14.
> Christophe Fontaine (1):
> dpif-netdev: Add parameters to configure PMD auto load balance.
>
^ This is adding new params for auto load balance and it is not a direct
fix for existing functionality.
OTOH, I have heard that as the hardcoded load threshold is 95%, this is
too high and params are needed to make the feature usable, so in that
sense it could be considered a fix for the feature.
> Kevin Traynor (3):
> dpif-netdev: Add log for PMD auto load balance interval parameter.
^ This is a clearer fix to 2.13/2.14 so I think it is a good candidate
to be backported.
> dpif-netdev: Add PMD auto load balance status log.
^ This is a small stats improvement, if the new params are backported I
would take this too, if not, then it's not worth to backport as it would
have conflicts.
What do others think?
Aside from NEWS, it is a clean backport, I can test and send backports
if needed.
> AUTHORS: Add Christophe Fontaine.
>
> AUTHORS.rst | 1 +
> NEWS | 1 +
> lib/dpif-netdev.c | 69 +++++++++++++++++++++++++++++++++++---------
> vswitchd/vswitch.xml | 29 +++++++++++++++++--
> 4 files changed, 85 insertions(+), 15 deletions(-)
>
More information about the dev
mailing list