| Message ID | MEYP282MB3499CEC6B8938D9CE48D7FCECDCC9@MEYP282MB3499.AUSP282.PROD.OUTLOOK.COM |
|---|---|
| State | Superseded |
| Headers | show |
| Series | Fix ALB parameters type and value mismatch. | expand |
| Context | Check | Description |
|---|---|---|
| ovsrobot/apply-robot | warning | apply and check: warning |
| ovsrobot/github-robot-_Build_and_Test | success | github build: passed |
| ovsrobot/intel-ovs-compilation | success | test: success |
Hi Lin, On 15/05/2022 08:09, miterv@outlook.com wrote: > From: linhuang <linhuang@ruijie.com.cn> > > The ALB parameters should never be negative. > So it's to use unsigned smap_get versions to get it properly, and > update VLOG formatting. > > Fixes: 5bf84282482a ("Adding support for PMD auto load balancing") > Signed-off-by: Lin Huang <linhuang@ruijie.com.cn> The CI is complaining about some items: http://patchwork.ozlabs.org/project/openvswitch/list/?series=300268 Here the From: and Signed-off-by: email addresses should match exactly > --- > lib/dpif-netdev.c | 22 +++++++++++----------- > 1 file changed, 11 insertions(+), 11 deletions(-) > > diff --git a/lib/dpif-netdev.c b/lib/dpif-netdev.c > index 61929049c..904f4ec92 100644 > --- a/lib/dpif-netdev.c > +++ b/lib/dpif-netdev.c > @@ -4777,8 +4777,8 @@ dpif_netdev_set_config(struct dpif *dpif, const struct smap *other_config) > uint32_t insert_min, cur_min; > uint32_t tx_flush_interval, cur_tx_flush_interval; > uint64_t rebalance_intvl; > - uint8_t rebalance_load, cur_rebalance_load; > - uint8_t rebalance_improve; > + uint8_t cur_rebalance_load; > + uint32_t rebalance_load, rebalance_improve; > bool log_autolb = false; > enum sched_assignment_type pmd_rxq_assign_type; > > @@ -4879,8 +4879,8 @@ dpif_netdev_set_config(struct dpif *dpif, const struct smap *other_config) > > struct pmd_auto_lb *pmd_alb = &dp->pmd_alb; > > - rebalance_intvl = smap_get_int(other_config, "pmd-auto-lb-rebal-interval", > - ALB_REBALANCE_INTERVAL); > + rebalance_intvl = smap_get_ullong(other_config, "pmd-auto-lb-rebal-interval", This line is getting flagged as being too long now that it has some extra characters, so it needs an extra break. > + ALB_REBALANCE_INTERVAL); > > /* Input is in min, convert it to msec. */ > rebalance_intvl = > @@ -4893,21 +4893,21 @@ dpif_netdev_set_config(struct dpif *dpif, const struct smap *other_config) > log_autolb = true; > } > > - rebalance_improve = smap_get_int(other_config, > - "pmd-auto-lb-improvement-threshold", > - ALB_IMPROVEMENT_THRESHOLD); > + rebalance_improve = smap_get_uint(other_config, > + "pmd-auto-lb-improvement-threshold", > + ALB_IMPROVEMENT_THRESHOLD); > if (rebalance_improve > 100) { > rebalance_improve = ALB_IMPROVEMENT_THRESHOLD; > } > if (rebalance_improve != pmd_alb->rebalance_improve_thresh) { > pmd_alb->rebalance_improve_thresh = rebalance_improve; > VLOG_INFO("PMD auto load balance improvement threshold set to " > - "%"PRIu8"%%", rebalance_improve); > + "%"PRIu32"%%", rebalance_improve); > log_autolb = true; > } > > - rebalance_load = smap_get_int(other_config, "pmd-auto-lb-load-threshold", > - ALB_LOAD_THRESHOLD); > + rebalance_load = smap_get_uint(other_config, "pmd-auto-lb-load-threshold", > + ALB_LOAD_THRESHOLD); > if (rebalance_load > 100) { > rebalance_load = ALB_LOAD_THRESHOLD; > } > @@ -4915,7 +4915,7 @@ dpif_netdev_set_config(struct dpif *dpif, const struct smap *other_config) > if (rebalance_load != cur_rebalance_load) { > atomic_store_relaxed(&pmd_alb->rebalance_load_thresh, > rebalance_load); > - VLOG_INFO("PMD auto load balance load threshold set to %"PRIu8"%%", > + VLOG_INFO("PMD auto load balance load threshold set to %"PRIu32"%%", > rebalance_load); > log_autolb = true; > }
diff --git a/lib/dpif-netdev.c b/lib/dpif-netdev.c index 61929049c..904f4ec92 100644 --- a/lib/dpif-netdev.c +++ b/lib/dpif-netdev.c @@ -4777,8 +4777,8 @@ dpif_netdev_set_config(struct dpif *dpif, const struct smap *other_config) uint32_t insert_min, cur_min; uint32_t tx_flush_interval, cur_tx_flush_interval; uint64_t rebalance_intvl; - uint8_t rebalance_load, cur_rebalance_load; - uint8_t rebalance_improve; + uint8_t cur_rebalance_load; + uint32_t rebalance_load, rebalance_improve; bool log_autolb = false; enum sched_assignment_type pmd_rxq_assign_type; @@ -4879,8 +4879,8 @@ dpif_netdev_set_config(struct dpif *dpif, const struct smap *other_config) struct pmd_auto_lb *pmd_alb = &dp->pmd_alb; - rebalance_intvl = smap_get_int(other_config, "pmd-auto-lb-rebal-interval", - ALB_REBALANCE_INTERVAL); + rebalance_intvl = smap_get_ullong(other_config, "pmd-auto-lb-rebal-interval", + ALB_REBALANCE_INTERVAL); /* Input is in min, convert it to msec. */ rebalance_intvl = @@ -4893,21 +4893,21 @@ dpif_netdev_set_config(struct dpif *dpif, const struct smap *other_config) log_autolb = true; } - rebalance_improve = smap_get_int(other_config, - "pmd-auto-lb-improvement-threshold", - ALB_IMPROVEMENT_THRESHOLD); + rebalance_improve = smap_get_uint(other_config, + "pmd-auto-lb-improvement-threshold", + ALB_IMPROVEMENT_THRESHOLD); if (rebalance_improve > 100) { rebalance_improve = ALB_IMPROVEMENT_THRESHOLD; } if (rebalance_improve != pmd_alb->rebalance_improve_thresh) { pmd_alb->rebalance_improve_thresh = rebalance_improve; VLOG_INFO("PMD auto load balance improvement threshold set to " - "%"PRIu8"%%", rebalance_improve); + "%"PRIu32"%%", rebalance_improve); log_autolb = true; } - rebalance_load = smap_get_int(other_config, "pmd-auto-lb-load-threshold", - ALB_LOAD_THRESHOLD); + rebalance_load = smap_get_uint(other_config, "pmd-auto-lb-load-threshold", + ALB_LOAD_THRESHOLD); if (rebalance_load > 100) { rebalance_load = ALB_LOAD_THRESHOLD; } @@ -4915,7 +4915,7 @@ dpif_netdev_set_config(struct dpif *dpif, const struct smap *other_config) if (rebalance_load != cur_rebalance_load) { atomic_store_relaxed(&pmd_alb->rebalance_load_thresh, rebalance_load); - VLOG_INFO("PMD auto load balance load threshold set to %"PRIu8"%%", + VLOG_INFO("PMD auto load balance load threshold set to %"PRIu32"%%", rebalance_load); log_autolb = true; }