Hi Thomas,

> I don't understand what would be improved by adding a new parameter.
> I think being able to handle the 2 syntaxes within the same option is nice.

Well, -c option has been handling everything as hex since forever. Our own 
documentation usually has coremasks without the 0x prefix. That means that 
there's probably a lot of legacy code (including our own internal validation 
code, and probably a lot of customers' code too) that depend on this behavior 
and that will fail in an unobvious way (e.g. you specified -c 3, that gave you 
cores 0,1 previously, now it gives you core 3). So I personally would much 
prefer leaving coremask as a *coremask* (e.g. a hex mask of cores), and add a 
new parameter to have a core *list* that would be interchangeable with coremask.

Best regards,
Anatoly Burakov
DPDK SW Engineer



Reply via email to