[PATCH v1] ethdev: beautify the Rx desc number check indent

Wang, Haiyue haiyue.wang at intel.com
Tue Feb 15 12:41:16 CET 2022


> -----Original Message-----
> From: Yigit, Ferruh <ferruh.yigit at intel.com>
> Sent: Tuesday, February 15, 2022 17:20
> To: Wang, Haiyue <haiyue.wang at intel.com>; dev at dpdk.org
> Cc: Thomas Monjalon <thomas at monjalon.net>; Andrew Rybchenko <andrew.rybchenko at oktetlabs.ru>
> Subject: Re: [PATCH v1] ethdev: beautify the Rx desc number check indent
> 
> On 2/15/2022 7:04 AM, Haiyue Wang wrote:
> > Align to Tx desc number check indent, which may help to understand Rx
> > descriptor limits member in the alignment comparison view. And remove
> > the extra empty line.
> >
> > Signed-off-by: Haiyue Wang <haiyue.wang at intel.com>
> > ---
> >   lib/ethdev/rte_ethdev.c | 5 ++---
> >   1 file changed, 2 insertions(+), 3 deletions(-)
> >
> > diff --git a/lib/ethdev/rte_ethdev.c b/lib/ethdev/rte_ethdev.c
> > index 70c850a2f1..54f1e14c6b 100644
> > --- a/lib/ethdev/rte_ethdev.c
> > +++ b/lib/ethdev/rte_ethdev.c
> > @@ -1799,9 +1799,8 @@ rte_eth_rx_queue_setup(uint16_t port_id, uint16_t rx_queue_id,
> >   	}
> >
> >   	if (nb_rx_desc > dev_info.rx_desc_lim.nb_max ||
> > -			nb_rx_desc < dev_info.rx_desc_lim.nb_min ||
> > -			nb_rx_desc % dev_info.rx_desc_lim.nb_align != 0) {
> > -
> > +	    nb_rx_desc < dev_info.rx_desc_lim.nb_min ||
> > +	    nb_rx_desc % dev_info.rx_desc_lim.nb_align != 0) {
> >   		RTE_ETHDEV_LOG(ERR,
> >   			"Invalid value for nb_rx_desc(=%hu), should be: <= %hu, >= %hu, and a product
> of %hu\n",
> >   			nb_rx_desc, dev_info.rx_desc_lim.nb_max,
> 
> Hi Haiyue,
> 
> Initial code matches to DPDK coding convention, updated code doesn't:
> https://doc.dpdk.org/guides/contributing/coding_style.html

I just read that 'rte_eth_tx_queue_setup' does it as above. ;-)

> 
> And I don't think syntax only updates are good idea,
> it corrupts the git history with minimal gain, instead
> better to fix them while doing a functional changes in
> those lines.

Got it.


More information about the dev mailing list