From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 86CD6A0C4B; Thu, 25 Nov 2021 09:12:02 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 69E6E40DFD; Thu, 25 Nov 2021 09:12:02 +0100 (CET) Received: from esa2.mentor.iphmx.com (esa2.mentor.iphmx.com [68.232.141.98]) by mails.dpdk.org (Postfix) with ESMTP id A964840DF5 for ; Thu, 25 Nov 2021 09:12:00 +0100 (CET) IronPort-SDR: f+hdLTMTBZRyYMYcG98SAJL7SDzWWYkzIkx7Y3UzZ9H/briCfPXllCkrtjcf9RYKnUI3g5YSjD ZvFK/n7BzO7VzbpmKvW8+cFQZ0v92s/WBMo5YucgDow6mCAT2y9IXS/lsQFmX7yXuDj54zXzxW aELq5PJjsojMUrQmLYLDSrpLzYt10S6doEtN75jlPgUTzSX8bQK5aZi4lqX521me5KysdxdofN 6mWsMtIIxujYk7Oj6EklRtQE0aRhXEnGZTQxdlic6+2gdIF9wKchTYnGcqUT7CvgRKD5Z9rE+P owQK8MurGMD9Y8wB95G290wl X-IronPort-AV: E=Sophos;i="5.87,262,1631606400"; d="scan'208";a="68909173" Received: from orw-gwy-02-in.mentorg.com ([192.94.38.167]) by esa2.mentor.iphmx.com with ESMTP; 25 Nov 2021 00:11:59 -0800 IronPort-SDR: 2JS1Hz2Ir0FVAxvfVIoEFupW8jNVRwVPZsqunQC4I3HxTyV1AYPumgRS8U3TfRe9AycVK8w7mL 7EYm3oEoR/Dl3mn3bzZRO9KiUbwt17VejNJpyWfSL3uRv5GEfRdP5xX3UfO9n1jJQ9aJVYc1Hf Txd5usmr1VnMtso2is/tfAEH3+ZClJ9OySYriXxrgQTLeI4mRibBsPx7DrxEyMCKYe/0Hw5osC yeJ64zGLZqII3Pjwbzegkkhal5HnN15OMUM9n7x9i/Qb0+aKCH/HBMqnFp5dIYRdaFRFDYY7zB dvA= From: "Awan, Arsalan" To: "Namburu, Chandu-babu" , "dev@dpdk.org" CC: "Somalapuram, Amaranath" , "Sebastian, Selwin" , "Giriyapura, Maheshwaramurthy" Subject: Re: [PATCH] net/axgbe: fix PCI ID to distinguish v1000 vs e3000 Thread-Topic: [PATCH] net/axgbe: fix PCI ID to distinguish v1000 vs e3000 Thread-Index: AQHX1iiECaNqQqg8g0i/N/Z2zni9Z6v//FMAgARpSiqAD5Skcw== Date: Thu, 25 Nov 2021 08:11:54 +0000 Message-ID: <1637827914612.88281@mentor.com> References: <20211110114519.8216-1-Arsalan_Awan@mentor.com>, , <1636971324930.27524@mentor.com> In-Reply-To: <1636971324930.27524@mentor.com> Accept-Language: en-US, en-IE Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-ms-exchange-transport-fromentityheader: Hosted x-originating-ip: [192.94.31.225] Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Hi Chandu,=0A= =0A= Do we have an update on this?=0A= =0A= Can we go ahead with this patch for now as it does not break anything else,= but fixes the problem.=0A= =0A= DPDK will be working fine as is on the e3000.=0A= =0A= This patch only adjusts the code for the v1000 and it works fine and there = isn't a difference in the approach of fixing the problem compared to that b= eing done currently to distinguish between the two platforms i.e. e3000 & v= 1000.=0A= =0A= Hoping to hear from you soon.=0A= =0A= Thanks,=0A= Arsalan=0A= ________________________________________=0A= From: Awan, Arsalan=0A= Sent: Monday, November 15, 2021 3:15 PM=0A= To: Namburu, Chandu-babu; dev@dpdk.org=0A= Cc: Somalapuram, Amaranath; Sebastian, Selwin; Giriyapura, Maheshwaramurthy= =0A= Subject: Re: [PATCH] net/axgbe: fix PCI ID to distinguish v1000 vs e3000=0A= =0A= Thanks, Chandu!=0A= =0A= Appreciate you guys looking into this!=0A= =0A= Looking forward to hear from you soon.=0A= =0A= Thanks and regards,=0A= Arsalan=0A= ________________________________________=0A= From: Namburu, Chandu-babu =0A= Sent: Friday, November 12, 2021 7:48 PM=0A= To: Awan, Arsalan; dev@dpdk.org=0A= Cc: Somalapuram, Amaranath; Sebastian, Selwin; Giriyapura, Maheshwaramurthy= =0A= Subject: RE: [PATCH] net/axgbe: fix PCI ID to distinguish v1000 vs e3000=0A= =0A= [AMD Official Use Only]=0A= =0A= Hi Arsalan Awan,=0A= =0A= Thank you for sharing the patch. DPDK not working on V1000 is regression du= e to "bus/pci: optimize bus scan" patch. As RV Root Complex device does not= have any Linux kernel driver assigned, this device is removed from PCI sca= n list and not found using pcie_search_device() routine.=0A= =0A= Patch submitted is potential fix for the issue, but we are checking if this= is the best way to fix regression. Will update you soon on the fix shared.= =0A= =0A= Regards,=0A= Chandu=0A= =0A= -----Original Message-----=0A= From: Arsalan H. Awan =0A= Sent: Wednesday, November 10, 2021 5:15 PM=0A= To: dev@dpdk.org=0A= Cc: Namburu, Chandu-babu ; Somalapuram, Amaranath ; Sebastian, Selwin ; Giriy= apura, Maheshwaramurthy ; Arsalan H. A= wan =0A= Subject: [PATCH] net/axgbe: fix PCI ID to distinguish v1000 vs e3000=0A= =0A= The RV PCI Root Complex Device is not visible to DPDK on the PCI bus via pc= i_search_device on some OSs including Ubuntu and Yocto. This makes it impos= sible to determine which machine DPDK/axgbe is running on. As a result, DPD= K/axgbe does not work on v1000.=0A= =0A= Let's use the Raven Internal PCIe GPP Bridge ID that DPDK can see on the PC= I bus that exists on the v1000 to differentiate between RV AXGBE and SNOWY = AXGBE, and set the registers accordingly. This fixes DPDK not working on v1= 000 platform.=0A= =0A= Signed-off-by: Arsalan H. Awan =0A= ---=0A= drivers/net/axgbe/axgbe_ethdev.c | 6 +++---=0A= 1 file changed, 3 insertions(+), 3 deletions(-)=0A= =0A= diff --git a/drivers/net/axgbe/axgbe_ethdev.c b/drivers/net/axgbe/axgbe_eth= dev.c=0A= index 7d40c18a86..7b1b46990f 100644=0A= --- a/drivers/net/axgbe/axgbe_ethdev.c=0A= +++ b/drivers/net/axgbe/axgbe_ethdev.c=0A= @@ -170,7 +170,7 @@ static const struct axgbe_xstats axgbe_xstats_strings[]= =3D {=0A= =0A= /* The set of PCI devices this driver supports */=0A= #define AMD_PCI_VENDOR_ID 0x1022=0A= -#define AMD_PCI_RV_ROOT_COMPLEX_ID 0x15d0=0A= +#define AMD_PCI_RV_INTERNAL_PCIE_GPP_BRIDGE_ID 0x15db=0A= #define AMD_PCI_AXGBE_DEVICE_V2A 0x1458 #define AMD_PCI_AXGBE_DEVICE_V2B = 0x1459=0A= =0A= @@ -2178,9 +2178,9 @@ eth_axgbe_dev_init(struct rte_eth_dev *eth_dev)=0A= pdata->pci_dev =3D pci_dev;=0A= =0A= /*=0A= - * Use root complex device ID to differentiate RV AXGBE vs SNOWY AX= GBE=0A= + * Use Raven Internal PCIe GPP Bridge device ID to differentiate RV= =0A= +AXGBE vs SNOWY AXGBE=0A= */=0A= - if (pci_search_device(AMD_PCI_RV_ROOT_COMPLEX_ID)) {=0A= + if (pci_search_device(AMD_PCI_RV_INTERNAL_PCIE_GPP_BRIDGE_ID)) {=0A= pdata->xpcs_window_def_reg =3D PCS_V2_RV_WINDOW_DEF;=0A= pdata->xpcs_window_sel_reg =3D PCS_V2_RV_WINDOW_SELECT;=0A= } else {=0A= --=0A= 2.17.1=0A= =0A=