HI Suanming, snipped > >> /* true if x is a power of 2 */ > >> #define POWEROF2(x) ((((x)-1) & (x)) == 0) @@ -413,6 +416,18 @@ > >> struct > > Can we use ` RTE_IS_POWER_OF_2(n) ' instead of ` POWEROF2`? > > I'm sorry, but that line is not add by this patch this time. > > Maybe another commit is more suitable to fix the previous code. Ok
Snipped > > > > As suggested in v4 can you update the `pdump.rst` on the new behaviour? > > As noted in the cover letter in v5. The `exit with primary` configuration > should be > made as default or not is still not confirmed from the maintainer. > > Since `exit with primary` is now removed in the patch and made as default per > Anatoly's suggestion and not get more information from the maintainer, the > update of the doc is also got hung up. I am not clear with this, if there is change in behaviour of the application it has to be documented. Thanks Vipin Varghese
