RE: [PATCH v11] sched: enable CMAN at runtime

["Danilewicz, MarcinX" <marcinx.danilewicz@intel.com>]

Hi Thomas,

Could you merge this change with the latest DPDK? 

Thanks!

Kind Regards,
/Marcin

> -----Original Message-----
> From: Singh, Jasvinder <jasvinder.singh@intel.com>
> Sent: Tuesday, July 26, 2022 3:49 PM
> To: Danilewicz, MarcinX <marcinx.danilewicz@intel.com>; dev@dpdk.org;
> Dumitrescu, Cristian <cristian.dumitrescu@intel.com>
> Cc: Ajmera, Megha <megha.ajmera@intel.com>
> Subject: RE: [PATCH v11] sched: enable CMAN at runtime
> 
> 
> 
> > -----Original Message-----
> > From: Danilewicz, MarcinX <marcinx.danilewicz@intel.com>
> > Sent: Friday, July 15, 2022 3:51 PM
> > To: dev@dpdk.org; Singh, Jasvinder <jasvinder.singh@intel.com>;
> > Dumitrescu, Cristian <cristian.dumitrescu@intel.com>
> > Cc: Ajmera, Megha <megha.ajmera@intel.com>
> > Subject: [PATCH v11] sched: enable CMAN at runtime
> >
> > Added changes to enable CMAN (RED or PIE) at init from profile
> > configuration file.
> >
> > By default CMAN code is enable but not in use, when there is no RED or
> > PIE profile configured.
> >
> > Signed-off-by: Marcin Danilewicz <marcinx.danilewicz@intel.com>
> > ---
> > Log: v2 change in rte_sched.h to avoid ABI breakage.
> >      v3 changes from comments
> >      v4 rebase to 22.07-rc1
> >      v5 rebase to main latest
> >      v6 commit message fixed
> >      v7 changes from comments
> >      v8 with changes from comments
> >      v9 changes from comments
> >         tmgr.c
> >         cman_params set to null
> >         qos_sched/cfg_file.c
> >         removed redundant cman_params to NULL assignement
> >         subport_params[].cman_params assigned
> >         only when CMAN enabled
> >      v10 removed ip_pipeline app build error from change
> >         in tmgr.c
> >      v11 added cman_enabled flag instead
> >         of testing cman_params.cman_mode variable
> > ---
> 
> Acked-by: Jasvinder Singh <jasvinder.singh@intel.com>