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

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

Hi Cristian,
 
> Can you please explain why you need to add the new
> RTE_SCHED_CMAN_NONE value to this enumeration and what is the ABI
> breakage that you are trying to avoid?
> 


> The library expects the struct rte_sched_subport_params:: cman_params to
> be NULL when the CMAN is disabled, why is this not enough to detect that
> CMAN is disabled? I suggest to keep the current approach.
Well, that above was just another approach. I'll have look to follow your advice, because there should be a third case when no RED or PIR config was found in files. 

So I used third enum to detect this case, like here:
> +	else if (params->cman_params->cman_mode ==
> RTE_SCHED_CMAN_NONE)
> +		return 1;

Also I got this advice to move RTE_SCHED_CMAN_NONE to the bottom to remove ABI warnings, because structure in header file has changed without change in map files (?) for that?

Here is some more detailed log with errors:

          1 data member change:
            type of 'rte_sched_cman_params* rte_sched_subport_params::cman_params' changed:
              in pointed to type 'struct rte_sched_cman_params' at rte_sched.h:166:1:
                type size hasn't changed
                1 data member change:
                  type of 'rte_sched_cman_mode rte_sched_cman_params::cman_mode' changed:
                    type size hasn't changed
                    1 enumerator insertion:
                      'rte_sched_cman_mode::RTE_SCHED_CMAN_NONE' value '0'
                    2 enumerator changes:
                      'rte_sched_cman_mode::RTE_SCHED_CMAN_RED' from value '0' to '1' at rte_sched.h:128:1
                      'rte_sched_cman_mode::RTE_SCHED_CMAN_PIE' from value '1' to '2' at rte_sched.h:128:1

Error: ABI issue reported for 'abidiff --suppr devtools/libabigail.abignore --no-added-syms --headers-dir1 reference/usr/local/include --headers-dir2 install/usr/local/include reference/dump/librte_sched.dump install/dump/librte_sched.dump'
ABIDIFF_ABI_CHANGE, this change requires a review (abidiff flagged this as a potential issue).

Kind Regards,
/Marcin

--------------------------------------------------------------
Intel Research and Development Ireland Limited
Registered in Ireland
Registered Office: Collinstown Industrial Park, Leixlip, County Kildare
Registered Number: 308263


This e-mail and any attachments may contain confidential material for the sole
use of the intended recipient(s). Any review or distribution by others is
strictly prohibited. If you are not the intended recipient, please contact the
sender and delete all copies.