[dpdk-dev] [PATCH v5 10/10] qede: Enable PMD build

[Bruce Richardson]

On Wed, Apr 20, 2016 at 10:51:06AM +0200, Thomas Monjalon wrote:
> 2016-04-20 00:14, Harish Patil:
> > >2016-03-31 19:15, Rasesh Mody:
> > >> --- a/config/common_base
> > >> +++ b/config/common_base
> > >> +CONFIG_RTE_LIBRTE_QEDE_RX_COAL_US=24
> > >> +CONFIG_RTE_LIBRTE_QEDE_TX_COAL_US=48
> > >
> > >It looks to be some tuning which could be done at runtime. Isn't it?
> >  
> > That’s right. Can you please suggest if there is any better option to make
> > it a runtime? There is no PMD API for that.
> 
> There are some devargs for that.
> For PCI dev, it can be passed in the whitelist option.
> We should remove this limitation by having a devargs API (and command line
> options) independent of whitelisting.

But back to the original setting. Are these likely to be values that are tunable
or need to be tunable by the user? If not, I see little reason to make them
run-time configurable - they could be defines inside the driver itself.

/Bruce