RE: [PATCH 03/10] dpaa_eth: add configurable bpool thresholds

[Madalin-Cristian Bucur <madalin.bucur@freescale.com>]

> -----Original Message-----
> From: Joe Perches [mailto:joe@perches.com]
> On Wed, 2015-07-22 at 19:16 +0300, Madalin Bucur wrote:
> > Allow the user to tweak the refill threshold and the total number
> > of buffers in the buffer pool. The provided values are for one CPU.
>=20
> Any value in making these module parameters instead?

I expect one would (hardly ever) change these to improve some corner
cases then use them with the new values. It may help in the tuning process
but afterwards the bloat to the bootcmd would probably be  a nuisance.

> > +config FSL_DPAA_ETH_MAX_BUF_COUNT
> > +	int "Maximum number of buffers in private bpool"
> > +	range 64 2048
> > +	default "128"
> > +	---help---
> > +	  The maximum number of buffers to be by default allocated in the
> DPAA-Ethernet private port's
> > +	  buffer pool. One needn't normally modify this, as it has probably
> been tuned for performance
> > +	  already. This cannot be lower than DPAA_ETH_REFILL_THRESHOLD.
> > +
> > +config FSL_DPAA_ETH_REFILL_THRESHOLD
> > +	int "Private bpool refill threshold"
> > +	range 32 FSL_DPAA_ETH_MAX_BUF_COUNT
> > +	default "80"
> > +	---help---
> > +	  The DPAA-Ethernet driver will start replenishing buffer pools whose
> count
> > +	  falls below this threshold. This must be related to
> DPAA_ETH_MAX_BUF_COUNT. One needn't normally
> > +	  modify this value unless one has very specific performance reasons.
> > +
> >  config FSL_DPAA_CS_THRESHOLD_1G
> >  	hex "Egress congestion threshold on 1G ports"
> >  	range 0x1000 0x10000000
>=20
>=20