* [dpdk-stable] [PATCH] app/testpmd: fix offloads default set error
@ 2019-06-10 6:45 Wei Zhao
2019-06-10 10:09 ` [dpdk-stable] [dpdk-dev] " Kevin Traynor
0 siblings, 1 reply; 8+ messages in thread
From: Wei Zhao @ 2019-06-10 6:45 UTC (permalink / raw)
To: dev; +Cc: stable, ferruh.yigit, Wei Zhao
There is no need to use default offloads configuration
if offloads configuration has been pass down from uplayer.
Fixes: 5e91aeef218c ("app/testpmd: fix offload flags after port config")
Signed-off-by: Wei Zhao <wei.zhao1@intel.com>
---
app/test-pmd/testpmd.c | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/app/test-pmd/testpmd.c b/app/test-pmd/testpmd.c
index 0148b0a..b5f5801 100644
--- a/app/test-pmd/testpmd.c
+++ b/app/test-pmd/testpmd.c
@@ -2815,7 +2815,8 @@ rxtx_port_config(struct rte_port *port)
for (qid = 0; qid < nb_rxq; qid++) {
offloads = port->rx_conf[qid].offloads;
port->rx_conf[qid] = port->dev_info.default_rxconf;
- port->rx_conf[qid].offloads |= offloads;
+ if (offloads)
+ port->rx_conf[qid].offloads = offloads;
/* Check if any Rx parameters have been passed */
if (rx_pthresh != RTE_PMD_PARAM_UNSET)
@@ -2839,7 +2840,8 @@ rxtx_port_config(struct rte_port *port)
for (qid = 0; qid < nb_txq; qid++) {
offloads = port->tx_conf[qid].offloads;
port->tx_conf[qid] = port->dev_info.default_txconf;
- port->tx_conf[qid].offloads |= offloads;
+ if (offloads)
+ port->tx_conf[qid].offloads = offloads;
/* Check if any Tx parameters have been passed */
if (tx_pthresh != RTE_PMD_PARAM_UNSET)
--
2.7.5
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [dpdk-stable] [dpdk-dev] [PATCH] app/testpmd: fix offloads default set error
2019-06-10 6:45 [dpdk-stable] [PATCH] app/testpmd: fix offloads default set error Wei Zhao
@ 2019-06-10 10:09 ` Kevin Traynor
2019-06-11 1:28 ` Zhao1, Wei
2019-06-11 1:31 ` Zhao1, Wei
0 siblings, 2 replies; 8+ messages in thread
From: Kevin Traynor @ 2019-06-10 10:09 UTC (permalink / raw)
To: Wei Zhao, dev; +Cc: stable, ferruh.yigit
On 10/06/2019 07:45, Wei Zhao wrote:
> There is no need to use default offloads configuration
> if offloads configuration has been pass down from uplayer.
>
It doesn't seem like the right fix to me. It means if a user
enable/disable one offload in the command line they are silently
changing from the default behaviour of other offloads? I could
understand if the user set all offloads in one go through a mask but not
when they are set seemingly independent.
It looks like you need to track which ones have been enabled/disabled by
the user, and use the default for for the other offloads.
> Fixes: 5e91aeef218c ("app/testpmd: fix offload flags after port config")
>
> Signed-off-by: Wei Zhao <wei.zhao1@intel.com>
> ---
> app/test-pmd/testpmd.c | 6 ++++--
> 1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/app/test-pmd/testpmd.c b/app/test-pmd/testpmd.c
> index 0148b0a..b5f5801 100644
> --- a/app/test-pmd/testpmd.c
> +++ b/app/test-pmd/testpmd.c
> @@ -2815,7 +2815,8 @@ rxtx_port_config(struct rte_port *port)
> for (qid = 0; qid < nb_rxq; qid++) {
> offloads = port->rx_conf[qid].offloads;
> port->rx_conf[qid] = port->dev_info.default_rxconf;
> - port->rx_conf[qid].offloads |= offloads;
> + if (offloads)
> + port->rx_conf[qid].offloads = offloads;
>
> /* Check if any Rx parameters have been passed */
> if (rx_pthresh != RTE_PMD_PARAM_UNSET)
> @@ -2839,7 +2840,8 @@ rxtx_port_config(struct rte_port *port)
> for (qid = 0; qid < nb_txq; qid++) {
> offloads = port->tx_conf[qid].offloads;
> port->tx_conf[qid] = port->dev_info.default_txconf;
> - port->tx_conf[qid].offloads |= offloads;
> + if (offloads)
> + port->tx_conf[qid].offloads = offloads;
>
> /* Check if any Tx parameters have been passed */
> if (tx_pthresh != RTE_PMD_PARAM_UNSET)
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [dpdk-stable] [dpdk-dev] [PATCH] app/testpmd: fix offloads default set error
2019-06-10 10:09 ` [dpdk-stable] [dpdk-dev] " Kevin Traynor
@ 2019-06-11 1:28 ` Zhao1, Wei
2019-06-11 14:41 ` Ferruh Yigit
2019-06-11 1:31 ` Zhao1, Wei
1 sibling, 1 reply; 8+ messages in thread
From: Zhao1, Wei @ 2019-06-11 1:28 UTC (permalink / raw)
To: Kevin Traynor, dev; +Cc: stable, Yigit, Ferruh
Hi, Kevin Traynor
This patch is based on Yigit Ferruh's suggestion in the following mail.
https://mails.dpdk.org/archives/dev/2019-May/132178.html
If we take your propose, it means we do need any patch at all.
Because the code implement now is just that scheme, we will do "or" for offloads.
Hello Ferruh,What is your comment?
> -----Original Message-----
> From: Kevin Traynor [mailto:ktraynor@redhat.com]
> Sent: Monday, June 10, 2019 6:10 PM
> To: Zhao1, Wei <wei.zhao1@intel.com>; dev@dpdk.org
> Cc: stable@dpdk.org; Yigit, Ferruh <ferruh.yigit@intel.com>
> Subject: Re: [dpdk-dev] [PATCH] app/testpmd: fix offloads default set error
>
> On 10/06/2019 07:45, Wei Zhao wrote:
> > There is no need to use default offloads configuration if offloads
> > configuration has been pass down from uplayer.
> >
>
> It doesn't seem like the right fix to me. It means if a user enable/disable one
> offload in the command line they are silently changing from the default
> behaviour of other offloads? I could understand if the user set all offloads in
> one go through a mask but not when they are set seemingly independent.
>
> It looks like you need to track which ones have been enabled/disabled by the
> user, and use the default for for the other offloads.
>
> > Fixes: 5e91aeef218c ("app/testpmd: fix offload flags after port
> > config")
> >
> > Signed-off-by: Wei Zhao <wei.zhao1@intel.com>
> > ---
> > app/test-pmd/testpmd.c | 6 ++++--
> > 1 file changed, 4 insertions(+), 2 deletions(-)
> >
> > diff --git a/app/test-pmd/testpmd.c b/app/test-pmd/testpmd.c index
> > 0148b0a..b5f5801 100644
> > --- a/app/test-pmd/testpmd.c
> > +++ b/app/test-pmd/testpmd.c
> > @@ -2815,7 +2815,8 @@ rxtx_port_config(struct rte_port *port)
> > for (qid = 0; qid < nb_rxq; qid++) {
> > offloads = port->rx_conf[qid].offloads;
> > port->rx_conf[qid] = port->dev_info.default_rxconf;
> > - port->rx_conf[qid].offloads |= offloads;
> > + if (offloads)
> > + port->rx_conf[qid].offloads = offloads;
> >
> > /* Check if any Rx parameters have been passed */
> > if (rx_pthresh != RTE_PMD_PARAM_UNSET) @@ -2839,7
> +2840,8 @@
> > rxtx_port_config(struct rte_port *port)
> > for (qid = 0; qid < nb_txq; qid++) {
> > offloads = port->tx_conf[qid].offloads;
> > port->tx_conf[qid] = port->dev_info.default_txconf;
> > - port->tx_conf[qid].offloads |= offloads;
> > + if (offloads)
> > + port->tx_conf[qid].offloads = offloads;
> >
> > /* Check if any Tx parameters have been passed */
> > if (tx_pthresh != RTE_PMD_PARAM_UNSET)
> >
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [dpdk-stable] [dpdk-dev] [PATCH] app/testpmd: fix offloads default set error
2019-06-10 10:09 ` [dpdk-stable] [dpdk-dev] " Kevin Traynor
2019-06-11 1:28 ` Zhao1, Wei
@ 2019-06-11 1:31 ` Zhao1, Wei
1 sibling, 0 replies; 8+ messages in thread
From: Zhao1, Wei @ 2019-06-11 1:31 UTC (permalink / raw)
To: Kevin Traynor, dev; +Cc: stable, Yigit, Ferruh
"If we take that propose, it means we do not need any patch at all."
Sorry for typo.
> -----Original Message-----
> From: Zhao1, Wei
> Sent: Tuesday, June 11, 2019 9:29 AM
> To: 'Kevin Traynor' <ktraynor@redhat.com>; dev@dpdk.org
> Cc: stable@dpdk.org; Yigit, Ferruh <ferruh.yigit@intel.com>
> Subject: RE: [dpdk-dev] [PATCH] app/testpmd: fix offloads default set error
>
> Hi, Kevin Traynor
> This patch is based on Yigit Ferruh's suggestion in the following mail.
> https://mails.dpdk.org/archives/dev/2019-May/132178.html
> If we take your propose, it means we do need any patch at all.
> Because the code implement now is just that scheme, we will do "or" for
> offloads.
> Hello Ferruh,What is your comment?
>
> > -----Original Message-----
> > From: Kevin Traynor [mailto:ktraynor@redhat.com]
> > Sent: Monday, June 10, 2019 6:10 PM
> > To: Zhao1, Wei <wei.zhao1@intel.com>; dev@dpdk.org
> > Cc: stable@dpdk.org; Yigit, Ferruh <ferruh.yigit@intel.com>
> > Subject: Re: [dpdk-dev] [PATCH] app/testpmd: fix offloads default set
> > error
> >
> > On 10/06/2019 07:45, Wei Zhao wrote:
> > > There is no need to use default offloads configuration if offloads
> > > configuration has been pass down from uplayer.
> > >
> >
> > It doesn't seem like the right fix to me. It means if a user
> > enable/disable one offload in the command line they are silently
> > changing from the default behaviour of other offloads? I could
> > understand if the user set all offloads in one go through a mask but not when
> they are set seemingly independent.
> >
> > It looks like you need to track which ones have been enabled/disabled
> > by the user, and use the default for for the other offloads.
> >
> > > Fixes: 5e91aeef218c ("app/testpmd: fix offload flags after port
> > > config")
> > >
> > > Signed-off-by: Wei Zhao <wei.zhao1@intel.com>
> > > ---
> > > app/test-pmd/testpmd.c | 6 ++++--
> > > 1 file changed, 4 insertions(+), 2 deletions(-)
> > >
> > > diff --git a/app/test-pmd/testpmd.c b/app/test-pmd/testpmd.c index
> > > 0148b0a..b5f5801 100644
> > > --- a/app/test-pmd/testpmd.c
> > > +++ b/app/test-pmd/testpmd.c
> > > @@ -2815,7 +2815,8 @@ rxtx_port_config(struct rte_port *port)
> > > for (qid = 0; qid < nb_rxq; qid++) {
> > > offloads = port->rx_conf[qid].offloads;
> > > port->rx_conf[qid] = port->dev_info.default_rxconf;
> > > - port->rx_conf[qid].offloads |= offloads;
> > > + if (offloads)
> > > + port->rx_conf[qid].offloads = offloads;
> > >
> > > /* Check if any Rx parameters have been passed */
> > > if (rx_pthresh != RTE_PMD_PARAM_UNSET) @@ -2839,7
> > +2840,8 @@
> > > rxtx_port_config(struct rte_port *port)
> > > for (qid = 0; qid < nb_txq; qid++) {
> > > offloads = port->tx_conf[qid].offloads;
> > > port->tx_conf[qid] = port->dev_info.default_txconf;
> > > - port->tx_conf[qid].offloads |= offloads;
> > > + if (offloads)
> > > + port->tx_conf[qid].offloads = offloads;
> > >
> > > /* Check if any Tx parameters have been passed */
> > > if (tx_pthresh != RTE_PMD_PARAM_UNSET)
> > >
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [dpdk-stable] [dpdk-dev] [PATCH] app/testpmd: fix offloads default set error
2019-06-11 1:28 ` Zhao1, Wei
@ 2019-06-11 14:41 ` Ferruh Yigit
2019-06-12 1:04 ` Zhao1, Wei
0 siblings, 1 reply; 8+ messages in thread
From: Ferruh Yigit @ 2019-06-11 14:41 UTC (permalink / raw)
To: Zhao1, Wei, Kevin Traynor, dev; +Cc: stable
On 6/11/2019 2:28 AM, Zhao1, Wei wrote:
> Hi, Kevin Traynor
> This patch is based on Yigit Ferruh's suggestion in the following mail.
> https://mails.dpdk.org/archives/dev/2019-May/132178.html
> If we take your propose, it means we do need any patch at all.
> Because the code implement now is just that scheme, we will do "or" for offloads.
> Hello Ferruh,What is your comment?
My comment was if the user not set the queue offload use the default one, so I
think we don't need to track individual offload options.
But also there is slightly difference between user not set the value and
'offloads' being zero. What if user set the offloads value to '0' to disable all
offloads, it will use default values in this case.
>
>> -----Original Message-----
>> From: Kevin Traynor [mailto:ktraynor@redhat.com]
>> Sent: Monday, June 10, 2019 6:10 PM
>> To: Zhao1, Wei <wei.zhao1@intel.com>; dev@dpdk.org
>> Cc: stable@dpdk.org; Yigit, Ferruh <ferruh.yigit@intel.com>
>> Subject: Re: [dpdk-dev] [PATCH] app/testpmd: fix offloads default set error
>>
>> On 10/06/2019 07:45, Wei Zhao wrote:
>>> There is no need to use default offloads configuration if offloads
>>> configuration has been pass down from uplayer.
>>>
>>
>> It doesn't seem like the right fix to me. It means if a user enable/disable one
>> offload in the command line they are silently changing from the default
>> behaviour of other offloads? I could understand if the user set all offloads in
>> one go through a mask but not when they are set seemingly independent.
>>
>> It looks like you need to track which ones have been enabled/disabled by the
>> user, and use the default for for the other offloads.
>>
>>> Fixes: 5e91aeef218c ("app/testpmd: fix offload flags after port
>>> config")
>>>
>>> Signed-off-by: Wei Zhao <wei.zhao1@intel.com>
>>> ---
>>> app/test-pmd/testpmd.c | 6 ++++--
>>> 1 file changed, 4 insertions(+), 2 deletions(-)
>>>
>>> diff --git a/app/test-pmd/testpmd.c b/app/test-pmd/testpmd.c index
>>> 0148b0a..b5f5801 100644
>>> --- a/app/test-pmd/testpmd.c
>>> +++ b/app/test-pmd/testpmd.c
>>> @@ -2815,7 +2815,8 @@ rxtx_port_config(struct rte_port *port)
>>> for (qid = 0; qid < nb_rxq; qid++) {
>>> offloads = port->rx_conf[qid].offloads;
>>> port->rx_conf[qid] = port->dev_info.default_rxconf;
>>> - port->rx_conf[qid].offloads |= offloads;
>>> + if (offloads)
>>> + port->rx_conf[qid].offloads = offloads;
>>>
>>> /* Check if any Rx parameters have been passed */
>>> if (rx_pthresh != RTE_PMD_PARAM_UNSET) @@ -2839,7
>> +2840,8 @@
>>> rxtx_port_config(struct rte_port *port)
>>> for (qid = 0; qid < nb_txq; qid++) {
>>> offloads = port->tx_conf[qid].offloads;
>>> port->tx_conf[qid] = port->dev_info.default_txconf;
>>> - port->tx_conf[qid].offloads |= offloads;
>>> + if (offloads)
>>> + port->tx_conf[qid].offloads = offloads;
>>>
>>> /* Check if any Tx parameters have been passed */
>>> if (tx_pthresh != RTE_PMD_PARAM_UNSET)
>>>
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [dpdk-stable] [dpdk-dev] [PATCH] app/testpmd: fix offloads default set error
2019-06-11 14:41 ` Ferruh Yigit
@ 2019-06-12 1:04 ` Zhao1, Wei
2019-07-02 17:56 ` Ferruh Yigit
0 siblings, 1 reply; 8+ messages in thread
From: Zhao1, Wei @ 2019-06-12 1:04 UTC (permalink / raw)
To: Yigit, Ferruh, Kevin Traynor, dev; +Cc: stable
> -----Original Message-----
> From: Yigit, Ferruh
> Sent: Tuesday, June 11, 2019 10:42 PM
> To: Zhao1, Wei <wei.zhao1@intel.com>; Kevin Traynor
> <ktraynor@redhat.com>; dev@dpdk.org
> Cc: stable@dpdk.org
> Subject: Re: [dpdk-dev] [PATCH] app/testpmd: fix offloads default set error
>
> On 6/11/2019 2:28 AM, Zhao1, Wei wrote:
> > Hi, Kevin Traynor
> > This patch is based on Yigit Ferruh's suggestion in the following mail.
> > https://mails.dpdk.org/archives/dev/2019-May/132178.html
> > If we take your propose, it means we do need any patch at all.
> > Because the code implement now is just that scheme, we will do "or" for
> offloads.
> > Hello Ferruh,What is your comment?
>
> My comment was if the user not set the queue offload use the default one, so I
> think we don't need to track individual offload options.
>
> But also there is slightly difference between user not set the value and
> 'offloads' being zero. What if user set the offloads value to '0' to disable all
> offloads, it will use default values in this case.
>
Yes, we can not distinguish between not set the value and using 0 to disable it, so if the bit is 0 how can we
Know the actual purpose of users? I think it is hard track individual offload as Kevin Traynor said.
> >
> >> -----Original Message-----
> >> From: Kevin Traynor [mailto:ktraynor@redhat.com]
> >> Sent: Monday, June 10, 2019 6:10 PM
> >> To: Zhao1, Wei <wei.zhao1@intel.com>; dev@dpdk.org
> >> Cc: stable@dpdk.org; Yigit, Ferruh <ferruh.yigit@intel.com>
> >> Subject: Re: [dpdk-dev] [PATCH] app/testpmd: fix offloads default set
> >> error
> >>
> >> On 10/06/2019 07:45, Wei Zhao wrote:
> >>> There is no need to use default offloads configuration if offloads
> >>> configuration has been pass down from uplayer.
> >>>
> >>
> >> It doesn't seem like the right fix to me. It means if a user
> >> enable/disable one offload in the command line they are silently
> >> changing from the default behaviour of other offloads? I could
> >> understand if the user set all offloads in one go through a mask but not
> when they are set seemingly independent.
> >>
> >> It looks like you need to track which ones have been enabled/disabled
> >> by the user, and use the default for for the other offloads.
> >>
> >>> Fixes: 5e91aeef218c ("app/testpmd: fix offload flags after port
> >>> config")
> >>>
> >>> Signed-off-by: Wei Zhao <wei.zhao1@intel.com>
> >>> ---
> >>> app/test-pmd/testpmd.c | 6 ++++--
> >>> 1 file changed, 4 insertions(+), 2 deletions(-)
> >>>
> >>> diff --git a/app/test-pmd/testpmd.c b/app/test-pmd/testpmd.c index
> >>> 0148b0a..b5f5801 100644
> >>> --- a/app/test-pmd/testpmd.c
> >>> +++ b/app/test-pmd/testpmd.c
> >>> @@ -2815,7 +2815,8 @@ rxtx_port_config(struct rte_port *port)
> >>> for (qid = 0; qid < nb_rxq; qid++) {
> >>> offloads = port->rx_conf[qid].offloads;
> >>> port->rx_conf[qid] = port->dev_info.default_rxconf;
> >>> - port->rx_conf[qid].offloads |= offloads;
> >>> + if (offloads)
> >>> + port->rx_conf[qid].offloads = offloads;
> >>>
> >>> /* Check if any Rx parameters have been passed */
> >>> if (rx_pthresh != RTE_PMD_PARAM_UNSET) @@ -2839,7
> >> +2840,8 @@
> >>> rxtx_port_config(struct rte_port *port)
> >>> for (qid = 0; qid < nb_txq; qid++) {
> >>> offloads = port->tx_conf[qid].offloads;
> >>> port->tx_conf[qid] = port->dev_info.default_txconf;
> >>> - port->tx_conf[qid].offloads |= offloads;
> >>> + if (offloads)
> >>> + port->tx_conf[qid].offloads = offloads;
> >>>
> >>> /* Check if any Tx parameters have been passed */
> >>> if (tx_pthresh != RTE_PMD_PARAM_UNSET)
> >>>
> >
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [dpdk-stable] [dpdk-dev] [PATCH] app/testpmd: fix offloads default set error
2019-06-12 1:04 ` Zhao1, Wei
@ 2019-07-02 17:56 ` Ferruh Yigit
2019-07-02 18:02 ` Ferruh Yigit
0 siblings, 1 reply; 8+ messages in thread
From: Ferruh Yigit @ 2019-07-02 17:56 UTC (permalink / raw)
To: Zhao1, Wei, Kevin Traynor, dev; +Cc: stable
On 6/12/2019 2:04 AM, Zhao1, Wei wrote:
>
>
>> -----Original Message-----
>> From: Yigit, Ferruh
>> Sent: Tuesday, June 11, 2019 10:42 PM
>> To: Zhao1, Wei <wei.zhao1@intel.com>; Kevin Traynor
>> <ktraynor@redhat.com>; dev@dpdk.org
>> Cc: stable@dpdk.org
>> Subject: Re: [dpdk-dev] [PATCH] app/testpmd: fix offloads default set error
>>
>> On 6/11/2019 2:28 AM, Zhao1, Wei wrote:
>>> Hi, Kevin Traynor
>>> This patch is based on Yigit Ferruh's suggestion in the following mail.
>>> https://mails.dpdk.org/archives/dev/2019-May/132178.html
>>> If we take your propose, it means we do need any patch at all.
>>> Because the code implement now is just that scheme, we will do "or" for
>> offloads.
>>> Hello Ferruh,What is your comment?
>>
>> My comment was if the user not set the queue offload use the default one, so I
>> think we don't need to track individual offload options.
>>
>> But also there is slightly difference between user not set the value and
>> 'offloads' being zero. What if user set the offloads value to '0' to disable all
>> offloads, it will use default values in this case.
>>
>
> Yes, we can not distinguish between not set the value and using 0 to disable it, so if the bit is 0 how can we
> Know the actual purpose of users? I think it is hard track individual offload as Kevin Traynor said.
Yes it is hard to track, and I believe this is better than the existing code, so:
Reviewed-by: Ferruh Yigit <ferruh.yigit@intel.com>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [dpdk-stable] [dpdk-dev] [PATCH] app/testpmd: fix offloads default set error
2019-07-02 17:56 ` Ferruh Yigit
@ 2019-07-02 18:02 ` Ferruh Yigit
0 siblings, 0 replies; 8+ messages in thread
From: Ferruh Yigit @ 2019-07-02 18:02 UTC (permalink / raw)
To: Zhao1, Wei, Kevin Traynor, dev; +Cc: stable
On 7/2/2019 6:56 PM, Ferruh Yigit wrote:
> On 6/12/2019 2:04 AM, Zhao1, Wei wrote:
>>
>>
>>> -----Original Message-----
>>> From: Yigit, Ferruh
>>> Sent: Tuesday, June 11, 2019 10:42 PM
>>> To: Zhao1, Wei <wei.zhao1@intel.com>; Kevin Traynor
>>> <ktraynor@redhat.com>; dev@dpdk.org
>>> Cc: stable@dpdk.org
>>> Subject: Re: [dpdk-dev] [PATCH] app/testpmd: fix offloads default set error
>>>
>>> On 6/11/2019 2:28 AM, Zhao1, Wei wrote:
>>>> Hi, Kevin Traynor
>>>> This patch is based on Yigit Ferruh's suggestion in the following mail.
>>>> https://mails.dpdk.org/archives/dev/2019-May/132178.html
>>>> If we take your propose, it means we do need any patch at all.
>>>> Because the code implement now is just that scheme, we will do "or" for
>>> offloads.
>>>> Hello Ferruh,What is your comment?
>>>
>>> My comment was if the user not set the queue offload use the default one, so I
>>> think we don't need to track individual offload options.
>>>
>>> But also there is slightly difference between user not set the value and
>>> 'offloads' being zero. What if user set the offloads value to '0' to disable all
>>> offloads, it will use default values in this case.
>>>
>>
>> Yes, we can not distinguish between not set the value and using 0 to disable it, so if the bit is 0 how can we
>> Know the actual purpose of users? I think it is hard track individual offload as Kevin Traynor said.
>
> Yes it is hard to track, and I believe this is better than the existing code, so:
>
> Reviewed-by: Ferruh Yigit <ferruh.yigit@intel.com>
>
Applied to dpdk-next-net/master, thanks.
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2019-07-02 18:02 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-06-10 6:45 [dpdk-stable] [PATCH] app/testpmd: fix offloads default set error Wei Zhao
2019-06-10 10:09 ` [dpdk-stable] [dpdk-dev] " Kevin Traynor
2019-06-11 1:28 ` Zhao1, Wei
2019-06-11 14:41 ` Ferruh Yigit
2019-06-12 1:04 ` Zhao1, Wei
2019-07-02 17:56 ` Ferruh Yigit
2019-07-02 18:02 ` Ferruh Yigit
2019-06-11 1:31 ` Zhao1, Wei
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).