diff mbox series

[8/8] vmxnet3: Fix merge error in initialization for rxDataRing feature

Message ID 20220503042257.15626-9-pagupta@vmware.com (mailing list archive)
State Changes Requested, archived
Delegated to: Andrew Rybchenko
Headers
Series vmxnet3 version V5 and V6 |

Checks

Context Check Description
ci/checkpatch warning coding style issues
ci/Intel-compilation success Compilation OK
ci/intel-Testing success Testing PASS
ci/iol-mellanox-Performance success Performance Testing PASS
ci/iol-intel-Functional success Functional Testing PASS
ci/github-robot: build success github build: passed
ci/iol-intel-Performance success Performance Testing PASS
ci/iol-aarch64-unit-testing success Testing PASS
ci/iol-x86_64-unit-testing success Testing PASS
ci/iol-x86_64-compile-testing success Testing PASS
ci/iol-aarch64-compile-testing success Testing PASS
ci/iol-abi-testing warning Testing issues

Commit Message

Pankaj Gupta May 3, 2022, 4:22 a.m. UTC 
  Fix merge error in initialization for rxDataRing feature.

Tested, using testpmd, for different hardware version on ESXi 7.0 Update 2.

Signed-off-by: Pankaj Gupta <pagupta@vmware.com>
---
 drivers/net/vmxnet3/vmxnet3_ethdev.c | 5 +++++
 1 file changed, 5 insertions(+)

Comments

Andrew Rybchenko May 4, 2022, 3:09 p.m. UTC | #1 
On 5/3/22 07:22, Pankaj Gupta wrote:
> Fix merge error in initialization for rxDataRing feature.

Is it a bug fix? If so, it requires corresponding "Fixes:" tag and Cc to
stable@dpdk.org in order to be backported to stable releases.

> 
> Tested, using testpmd, for different hardware version on ESXi 7.0 Update 2.
> 
> Signed-off-by: Pankaj Gupta <pagupta@vmware.com>
> ---
>   drivers/net/vmxnet3/vmxnet3_ethdev.c | 5 +++++
>   1 file changed, 5 insertions(+)
> 
> diff --git a/drivers/net/vmxnet3/vmxnet3_ethdev.c b/drivers/net/vmxnet3/vmxnet3_ethdev.c
> index 9955f121f6..6ced76ae2a 100644
> --- a/drivers/net/vmxnet3/vmxnet3_ethdev.c
> +++ b/drivers/net/vmxnet3/vmxnet3_ethdev.c
> @@ -876,6 +876,11 @@ vmxnet3_setup_driver_shared(struct rte_eth_dev *dev)
>   		rqd->conf.rxRingSize[1]   = rxq->cmd_ring[1].size;
>   		rqd->conf.compRingSize    = rxq->comp_ring.size;
>   
> +		if (VMXNET3_VERSION_GE_3(hw)) {
> +			rqd->conf.rxDataRingBasePA = rxq->data_ring.basePA;
> +			rqd->conf.rxDataRingDescSize = rxq->data_desc_size;
> +		}
> +
>   		if (hw->intr.lsc_only)
>   			rqd->conf.intrIdx = 1;
>   		else
Pankaj Gupta May 4, 2022, 8:37 p.m. UTC | #2 
Hi Andrew,
Changes done in this patch was accidentally removed in commit 046f1161956777e3afb13504acbe8df2ec3a383c.
This was noticed later and we are trying to address this here.



Thanks,
Pankaj

From: Andrew Rybchenko <andrew.rybchenko@oktetlabs.ru>
Date: Wednesday, May 4, 2022 at 8:09 AM
To: Pankaj Gupta <pagupta@vmware.com>, Jochen Behrens <jbehrens@vmware.com>, Yong Wang <yongwang@vmware.com>
Cc: dev@dpdk.org <dev@dpdk.org>
Subject: Re: [PATCH 8/8] vmxnet3: Fix merge error in initialization for rxDataRing feature
⚠ External Email

On 5/3/22 07:22, Pankaj Gupta wrote:
> Fix merge error in initialization for rxDataRing feature.

Is it a bug fix? If so, it requires corresponding "Fixes:" tag and Cc to
stable@dpdk.org in order to be backported to stable releases.

>
> Tested, using testpmd, for different hardware version on ESXi 7.0 Update 2.
>
> Signed-off-by: Pankaj Gupta <pagupta@vmware.com>
> ---
>   drivers/net/vmxnet3/vmxnet3_ethdev.c | 5 +++++
>   1 file changed, 5 insertions(+)
>
> diff --git a/drivers/net/vmxnet3/vmxnet3_ethdev.c b/drivers/net/vmxnet3/vmxnet3_ethdev.c
> index 9955f121f6..6ced76ae2a 100644
> --- a/drivers/net/vmxnet3/vmxnet3_ethdev.c
> +++ b/drivers/net/vmxnet3/vmxnet3_ethdev.c
> @@ -876,6 +876,11 @@ vmxnet3_setup_driver_shared(struct rte_eth_dev *dev)
>               rqd->conf.rxRingSize[1]   = rxq->cmd_ring[1].size;
>               rqd->conf.compRingSize    = rxq->comp_ring.size;
>
> +             if (VMXNET3_VERSION_GE_3(hw)) {
> +                     rqd->conf.rxDataRingBasePA = rxq->data_ring.basePA;
> +                     rqd->conf.rxDataRingDescSize = rxq->data_desc_size;
> +             }
> +
>               if (hw->intr.lsc_only)
>                       rqd->conf.intrIdx = 1;
>               else
Andrew Rybchenko May 5, 2022, 8:37 a.m. UTC | #3 
Hi Pakai,

On 5/4/22 23:37, Pankaj Gupta wrote:
> Hi Andrew,
> 
> Changes done in this patch was accidentally removed in commit 
> 046f1161956777e3afb13504acbe8df2ec3a383c.
> 
> This was noticed later and we are trying to address this here.

If so, you should add:

Fixes: 046f11619567 ("net/vmxnet3: support MSI-X interrupt")
Cc: stable@dpdk.org

> 
> Thanks,
> 
> Pankaj
> 
> *From: *Andrew Rybchenko <andrew.rybchenko@oktetlabs.ru>
> *Date: *Wednesday, May 4, 2022 at 8:09 AM
> *To: *Pankaj Gupta <pagupta@vmware.com>, Jochen Behrens 
> <jbehrens@vmware.com>, Yong Wang <yongwang@vmware.com>
> *Cc: *dev@dpdk.org <dev@dpdk.org>
> *Subject: *Re: [PATCH 8/8] vmxnet3: Fix merge error in initialization 
> for rxDataRing feature
> 
> ⚠External Email
> 
> On 5/3/22 07:22, Pankaj Gupta wrote:
>> Fix merge error in initialization for rxDataRing feature.
> 
> Is it a bug fix? If so, it requires corresponding "Fixes:" tag and Cc to
> stable@dpdk.org in order to be backported to stable releases.
> 
>>
>> Tested, using testpmd, for different hardware version on ESXi 7.0 Update 2.
>>
>> Signed-off-by: Pankaj Gupta <pagupta@vmware.com>
>> ---
>>   drivers/net/vmxnet3/vmxnet3_ethdev.c | 5 +++++
>>   1 file changed, 5 insertions(+)
>>
>> diff --git a/drivers/net/vmxnet3/vmxnet3_ethdev.c b/drivers/net/vmxnet3/vmxnet3_ethdev.c
>> index 9955f121f6..6ced76ae2a 100644
>> --- a/drivers/net/vmxnet3/vmxnet3_ethdev.c
>> +++ b/drivers/net/vmxnet3/vmxnet3_ethdev.c
>> @@ -876,6 +876,11 @@ vmxnet3_setup_driver_shared(struct rte_eth_dev *dev)
>>               rqd->conf.rxRingSize[1]   = rxq->cmd_ring[1].size;
>>               rqd->conf.compRingSize    = rxq->comp_ring.size;
>>
>> +             if (VMXNET3_VERSION_GE_3(hw)) {
>> +                     rqd->conf.rxDataRingBasePA = rxq->data_ring.basePA;
>> +                     rqd->conf.rxDataRingDescSize = rxq->data_desc_size;
>> +             }
>> +
>>               if (hw->intr.lsc_only)
>>                       rqd->conf.intrIdx = 1;
>>               else
> 
> 
> ________________________________
> 
> ⚠External Email: This email originated from outside of the organization. 
> Do not click links or open attachments unless you recognize the sender.
>
diff mbox series

Patch

diff --git a/drivers/net/vmxnet3/vmxnet3_ethdev.c b/drivers/net/vmxnet3/vmxnet3_ethdev.c
index 9955f121f6..6ced76ae2a 100644
--- a/drivers/net/vmxnet3/vmxnet3_ethdev.c
+++ b/drivers/net/vmxnet3/vmxnet3_ethdev.c
@@ -876,6 +876,11 @@  vmxnet3_setup_driver_shared(struct rte_eth_dev *dev)
 		rqd->conf.rxRingSize[1]   = rxq->cmd_ring[1].size;
 		rqd->conf.compRingSize    = rxq->comp_ring.size;
 
+		if (VMXNET3_VERSION_GE_3(hw)) {
+			rqd->conf.rxDataRingBasePA = rxq->data_ring.basePA;
+			rqd->conf.rxDataRingDescSize = rxq->data_desc_size;
+		}
+
 		if (hw->intr.lsc_only)
 			rqd->conf.intrIdx = 1;
 		else