Re: [PATCH 3/4] app/testpmd: check queue count for related options

2024-03-13 Thread Ferruh Yigit
On 3/13/2024 11:10 AM, David Marchand wrote: > On Wed, Mar 13, 2024 at 11:52 AM Ferruh Yigit wrote: >> >> On 3/13/2024 7:37 AM, David Marchand wrote: >>> On Tue, Mar 12, 2024 at 5:59 PM Ferruh Yigit wrote: On 3/8/2024 2:48 PM, David Marchand wrote: > Checking the number of rxq/txq i

Re: [PATCH 3/4] app/testpmd: check queue count for related options

2024-03-13 Thread David Marchand
On Wed, Mar 13, 2024 at 11:52 AM Ferruh Yigit wrote: > > On 3/13/2024 7:37 AM, David Marchand wrote: > > On Tue, Mar 12, 2024 at 5:59 PM Ferruh Yigit wrote: > >> > >> On 3/8/2024 2:48 PM, David Marchand wrote: > >>> Checking the number of rxq/txq in the middle of option parsing is > >>> confusing

Re: [PATCH 3/4] app/testpmd: check queue count for related options

2024-03-13 Thread Ferruh Yigit
On 3/13/2024 7:37 AM, David Marchand wrote: > On Tue, Mar 12, 2024 at 5:59 PM Ferruh Yigit wrote: >> >> On 3/8/2024 2:48 PM, David Marchand wrote: >>> Checking the number of rxq/txq in the middle of option parsing is >>> confusing. Move the check where nb_rxq / nb_txq are modified. >>> >>> Signed-

Re: [PATCH 3/4] app/testpmd: check queue count for related options

2024-03-13 Thread David Marchand
On Tue, Mar 12, 2024 at 5:59 PM Ferruh Yigit wrote: > > On 3/8/2024 2:48 PM, David Marchand wrote: > > Checking the number of rxq/txq in the middle of option parsing is > > confusing. Move the check where nb_rxq / nb_txq are modified. > > > > Signed-off-by: David Marchand > > --- > > app/test-pm

Re: [PATCH 3/4] app/testpmd: check queue count for related options

2024-03-12 Thread Ferruh Yigit
On 3/8/2024 2:48 PM, David Marchand wrote: > Checking the number of rxq/txq in the middle of option parsing is > confusing. Move the check where nb_rxq / nb_txq are modified. > > Signed-off-by: David Marchand > --- > app/test-pmd/parameters.c | 8 > 1 file changed, 4 insertions(+), 4 de

[PATCH 3/4] app/testpmd: check queue count for related options

2024-03-08 Thread David Marchand
Checking the number of rxq/txq in the middle of option parsing is confusing. Move the check where nb_rxq / nb_txq are modified. Signed-off-by: David Marchand --- app/test-pmd/parameters.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/app/test-pmd/parameters.c b/app/