On Thu, Mar 3, 2016 at 4:02 PM, Wiles, Keith <keith.wiles at intel.com> wrote: >>On Thu, Feb 25, 2016 at 11:12 PM, Wiles, Keith <keith.wiles at intel.com> >>wrote: >>>>On Thu, Feb 25, 2016 at 01:09:16PM -0600, Keith Wiles wrote: >>>>> A number of short options for EAL are missing long options >>>>> and this patch adds those missing options. >>>>> >>>>> The missing long options are for: >>>>> -c add --coremask >>>>> -d add --driver >>>>> -l add --corelist >>>>> -m add --memsize >>>>> -n add --mem-channels >>>>> -r add --mem-ranks >>>>> -v add --version >>>>> Add an alias for --lcores using --lcore-map [snip] >>No strong opinion on this. >> >>Just, why "memsize" with no - but "mem-channels" ? >>And why cut down to mem rather than memory ? > > I debated on mem-size, but I noticed in a couple places some used memsize. I > can change them to any thing someone wants. If you want memory-channels and > memory-ranks I am good with that too.
Ok, since I can see no real argument against, let's go with explicit options : --memory-size --memory-channels --memory-ranks The best would be then to align all other existing long options (but keeping compat with existing ones). Deal ? -- David Marchand