From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <dev-bounces@dpdk.org>
Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124])
	by inbox.dpdk.org (Postfix) with ESMTP id C40A842536;
	Thu,  7 Sep 2023 15:23:52 +0200 (CEST)
Received: from mails.dpdk.org (localhost [127.0.0.1])
	by mails.dpdk.org (Postfix) with ESMTP id 5678D402AF;
	Thu,  7 Sep 2023 15:23:52 +0200 (CEST)
Received: from mgamail.intel.com (mgamail.intel.com [192.55.52.115])
 by mails.dpdk.org (Postfix) with ESMTP id 948934026C
 for <dev@dpdk.org>; Thu,  7 Sep 2023 15:23:50 +0200 (CEST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple;
 d=intel.com; i=@intel.com; q=dns/txt; s=Intel;
 t=1694093030; x=1725629030;
 h=from:to:cc:subject:date:message-id:references:
 in-reply-to:content-transfer-encoding:mime-version;
 bh=CfxDfVSBeCP6e4oEtqRwXhHTFxd7RMXK6jzb6+hXY7s=;
 b=kWI2DlVLRxf8pMKJ8VTw2aO05txCCoyzl0bt9OYBUl3CFYrYLGOwt7II
 tk7900yLsCmkLW/bauE+Fdv/Ciai4yM/vsuUoAUq9VxdAhe1MQrw6PQ6i
 1k8knpp1ZjjOmfWNDFp3svL5FGxLn447f9WPnKEssL3h+w4hmq3SmCwhM
 +Tlk9aYuwpz3jgz0Ejz51Z9iAWTff6Or+wk5dEF93emA5gByY9W/GRILo
 RzQX5gAtWbnk9vxLukf9ZANDWkXStY/fQ5X/+0YQA5CSmjQX2ZnHevppT
 A0rlL4VAI1yW8XS164zaoldr5zQFVVwndz73OTLRt69J6PKrrNN68tZ3j g==;
X-IronPort-AV: E=McAfee;i="6600,9927,10826"; a="377265973"
X-IronPort-AV: E=Sophos;i="6.02,235,1688454000"; d="scan'208";a="377265973"
Received: from fmsmga008.fm.intel.com ([10.253.24.58])
 by fmsmga103.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384;
 07 Sep 2023 06:23:49 -0700
X-ExtLoop1: 1
X-IronPort-AV: E=McAfee;i="6600,9927,10826"; a="807547922"
X-IronPort-AV: E=Sophos;i="6.02,235,1688454000"; d="scan'208";a="807547922"
Received: from orsmsx601.amr.corp.intel.com ([10.22.229.14])
 by fmsmga008.fm.intel.com with ESMTP/TLS/AES256-GCM-SHA384;
 07 Sep 2023 06:23:48 -0700
Received: from orsmsx611.amr.corp.intel.com (10.22.229.24) by
 ORSMSX601.amr.corp.intel.com (10.22.229.14) with Microsoft SMTP Server
 (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id
 15.1.2507.32; Thu, 7 Sep 2023 06:23:48 -0700
Received: from orsmsx610.amr.corp.intel.com (10.22.229.23) by
 ORSMSX611.amr.corp.intel.com (10.22.229.24) with Microsoft SMTP Server
 (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id
 15.1.2507.32; Thu, 7 Sep 2023 06:23:47 -0700
Received: from ORSEDG602.ED.cps.intel.com (10.7.248.7) by
 orsmsx610.amr.corp.intel.com (10.22.229.23) with Microsoft SMTP Server
 (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id
 15.1.2507.32 via Frontend Transport; Thu, 7 Sep 2023 06:23:47 -0700
Received: from NAM11-DM6-obe.outbound.protection.outlook.com (104.47.57.170)
 by edgegateway.intel.com (134.134.137.103) with Microsoft SMTP Server
 (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id
 15.1.2507.27; Thu, 7 Sep 2023 06:23:26 -0700
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none;
 b=cYArOLolMBKO3Vk81ZSSUibdifzCPx5jyLKhVMUQ4U1xcBan4+OFzgH6OBYl+MSgcOgFQbUq2OSO5Bml9dGeOZUU1lFd7QVUnPhaEL9fUMREc3sW70x8ItAYnzNnAGFNG6pAmQgohmwyMooP++7jTBR0eNYBBF3YHWUOtm4WDP+6rseLaevxox1DcTv1/cGh9RZZaigQCVhtIqNXKjMYULEfo89aJI+q7jjbxDdXc54EHMNKuopuIurzjl0af9T1mkZJRTvMAUWHVBta+SgmRMoWFlcEKhoDZS2hpUHJg33k1lmUMEnE8CAHTd1nhSJ9LFflhmDlqSIJKJR8Mp7fSg==
ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; 
 s=arcselector9901;
 h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1;
 bh=CQGY4jVTo9lTF4IkM8tRvCPeQr2sGj4ojnjuy42T95w=;
 b=fkxo3Ly/k004ppMqeqPB05Q+pG/mF202vVuXKflZFWgEbqdSDdhrMx2AiuYrJUoCROT/Uxrmxat0J95DCFfkO8QJ0nUqFHyrvAaJTr92KB7SxMcxIymgNBK5BoRXTQymlq59Arz3ZuS7WKlhne+gaG4L3U1SfrNWX8+DH/j0SS4sjlfKQk5SblSitaadCyXN+nsQbC6NUlUyys1CyrrMnHq/EIbIH+1BftsD/S9J6R+WpFSfaN49DXO7Guy3WRfLjPDMEa9YfZ72pzV9w8BcnhOcH3//kxvb9E7WURxAMHiq88m5lRW6yEc1I5+/I98M+P/AZoYqNbAVw6xIOpUpow==
ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass
 smtp.mailfrom=intel.com; dmarc=pass action=none header.from=intel.com;
 dkim=pass header.d=intel.com; arc=none
Received: from CH3PR11MB8362.namprd11.prod.outlook.com (2603:10b6:610:175::6)
 by DM4PR11MB7328.namprd11.prod.outlook.com (2603:10b6:8:104::9) with
 Microsoft SMTP Server (version=TLS1_2,
 cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6745.20; Thu, 7 Sep
 2023 13:23:19 +0000
Received: from CH3PR11MB8362.namprd11.prod.outlook.com
 ([fe80::88c2:332c:5e8a:25e]) by CH3PR11MB8362.namprd11.prod.outlook.com
 ([fe80::88c2:332c:5e8a:25e%7]) with mapi id 15.20.6768.029; Thu, 7 Sep 2023
 13:23:19 +0000
From: "Xia, Chenbo" <chenbo.xia@intel.com>
To: David Marchand <david.marchand@redhat.com>, "dev@dpdk.org" <dev@dpdk.org>
CC: "thomas@monjalon.net" <thomas@monjalon.net>, "ferruh.yigit@amd.com"
 <ferruh.yigit@amd.com>, "nipun.gupta@amd.com" <nipun.gupta@amd.com>,
 "Richardson, Bruce" <bruce.richardson@intel.com>, "McDaniel, Timothy"
 <timothy.mcdaniel@intel.com>, Gaetan Rivet <grive@u256.net>
Subject: RE: [PATCH v2 11/15] pci: define some extended capability constants
Thread-Topic: [PATCH v2 11/15] pci: define some extended capability constants
Thread-Index: AQHZ1CPMFKvAafipTUiz1MTFQ29rLLAPctDA
Date: Thu, 7 Sep 2023 13:23:18 +0000
Message-ID: <CH3PR11MB8362AC018087C118D30CCE979CEEA@CH3PR11MB8362.namprd11.prod.outlook.com>
References: <20230803075038.307012-1-david.marchand@redhat.com>
 <20230821113549.3191921-1-david.marchand@redhat.com>
 <20230821113549.3191921-12-david.marchand@redhat.com>
In-Reply-To: <20230821113549.3191921-12-david.marchand@redhat.com>
Accept-Language: en-US, zh-CN
Content-Language: en-US
X-MS-Has-Attach: 
X-MS-TNEF-Correlator: 
authentication-results: dkim=none (message not signed)
 header.d=none;dmarc=none action=none header.from=intel.com;
x-ms-publictraffictype: Email
x-ms-traffictypediagnostic: CH3PR11MB8362:EE_|DM4PR11MB7328:EE_
x-ms-office365-filtering-correlation-id: 99c8eb89-a648-476a-4204-08dbafa599ec
x-ms-exchange-senderadcheck: 1
x-ms-exchange-antispam-relay: 0
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info: 3nUVVS5/mb5uYkdq2pegHh11teCmQriqE6mjRzrfq99LxHWIX4ICn0WcCkdvsf3THwmE6XKOMBmx2WKIuATwpDcgUBSr2odKnmsbBb0tlsESMn8S8VuXRjrdIemUnzl5EuAOHqIEcSBdGEKVwtN3FMxGDj2qCleAnlNgV4nNeNdi/+kkunM01lyen6hRn5L/VjOpo3W9pf4J/pPXA0Qoayj0Zej7EG2whj9wrCwXLhUF082Xu1woK6SO/yZEnOTOrnqtUj4nSxIB3pPuNwawRDVNeCxExcbycE8lQFaPxD86dMU6g9jGIPYy9jRngeuMRHSfXyvERIbyRIAMg6jh0608WD9daDfmZ1ZmeckkNdrxIv4b/e/o+eUA5/za32HFedOdELhEV+6Rxc1P606VNR+0gLsWRTl3sdVonSZqLnNbITWIeDuX/4o3Ij1ct0/zIuk5LjdEeoe+Q7qQ9NCH89v2Ey9LWwbZhxPB2kizwuCr9JujnEl1ZN6BlWCd7EKQxjL+Y8GeNrJ6GcDkzrPrJ6eRViESwy1ymvbgdVwtlJNAjq7MFkYuZEQhcR6r47sQ3Jp6lgUfRAamF5WfbQQIuX3COMwuhsExkefWXuZRRNhPnmZrVhBZ/XsMQ7pxe1u6
x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:;
 IPV:NLI; SFV:NSPM; H:CH3PR11MB8362.namprd11.prod.outlook.com; PTR:; CAT:NONE;
 SFS:(13230031)(366004)(39860400002)(136003)(376002)(396003)(346002)(1800799009)(186009)(451199024)(55016003)(41300700001)(478600001)(2906002)(66556008)(82960400001)(38070700005)(66946007)(76116006)(54906003)(66476007)(316002)(66446008)(64756008)(38100700002)(122000001)(110136005)(8936002)(5660300002)(8676002)(4326008)(52536014)(86362001)(53546011)(6506007)(9686003)(7696005)(26005)(83380400001)(71200400001)(33656002);
 DIR:OUT; SFP:1102; 
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?m/ehsfA1ITfnzzY8yqf5G69aLx9jWyFbFC4QlDogxCSSh8I5Xs72dydLh5/j?=
 =?us-ascii?Q?uSwgs30iZn9esErVjWEQrdOPP54pOXHN8oDvkFBi3lYJrTkM3VpLxI4SabcQ?=
 =?us-ascii?Q?jelYwcC9KTj+fmLC4uiDWNiveknWDxIQJw5MMSrjNq/lOhOOT+jBg7uF7PgU?=
 =?us-ascii?Q?ll2ANSyKu2NalojJD3DcEp+0Iw51an4q8D2AMHMfjzysO5pVYt+uJNQwlKiH?=
 =?us-ascii?Q?ocoFbgcN7L6vBAfKOi65U3DFf8x+KRlpHLe8Yx+YiE5hrsu1ZarbZof3kp67?=
 =?us-ascii?Q?W8Ginr3MMO2BxX+NGF24Fg19aV5gX8i7Gl2ra47Mgp4F87q/0Mz43Z8/3HQj?=
 =?us-ascii?Q?GZsTuyBbRjIRdLSHQZzmpajY41HKhEI6a7UeFArG8C9NhggZxJSbDCMyJuye?=
 =?us-ascii?Q?Ipg9A/7K2j1YWHqWnMhGTQIUJ8wl7GqAIvlrcRMp1t6YVFebe//G4ITS5b3s?=
 =?us-ascii?Q?qLVRyyUNtKkgY40n356/NIqqI9fQsBt69LHkT1JPd16rOWFZ2N3rjI07X/SU?=
 =?us-ascii?Q?UjFQC60r32Mgj/A+AKarFOtkdc6NHvi8XlbXVGvFKv6RmeCKjLRC0MGPZONe?=
 =?us-ascii?Q?6YLP9PciD/a5iYmgCW1QDjpg+tKRrt8MexAhRkIn3FbysvdShI/0SoWNmSG6?=
 =?us-ascii?Q?1PvPo6yez5FbebuSEQ4i1DLvNhonHnFdKvzz3na2iRnbr1gfM7AL/zQ/s038?=
 =?us-ascii?Q?iFa9Say93c41VhYD0n71kd7+OLEPLCLX8MyOUkBvFEQsg5jbO8zmGpF3xsP9?=
 =?us-ascii?Q?si3ztzfMIXrtgOUjycE5cL/aVLQsYEwIplM0EKidK3ALS4Alc5Bgod6m7wX9?=
 =?us-ascii?Q?OMsj1fmhTnjMh+NCC2yyyPx1I5cP50dexPc0eY5ST3Itr//GVcsnP+Ny5Ot3?=
 =?us-ascii?Q?BKecc+bXJGq0WLfp+Wwt4oNt8hF33gCjMiosIKN52PFs0UdbshQizwgW+DNM?=
 =?us-ascii?Q?HIVj0fjnkHNFNWWGqFPcJJyzGat5b0k148S5y+3oc8FRjEtjJdRods7osqeb?=
 =?us-ascii?Q?I9JXVeKFiH1aCBm239N9a/UfFtuHFuF/2z9ZK0HRLdJse3AlKak90TUaTGTy?=
 =?us-ascii?Q?JEPqapgYe42PIddphB15v+D82Zw0MxunAHcjM+U+LYqO0hHgbPHw+yONSsR1?=
 =?us-ascii?Q?+4Hzree73S0bdzZsG1R7YmQgEFgeoMBXnGdRZQYARqlIIQxeRsX53w6yMn4u?=
 =?us-ascii?Q?YRKJ8mb3VWiSPgOIMgo2GOUIKRMol/BXK/XAJt8nOMD6JWpK3qDxiIaQIaZY?=
 =?us-ascii?Q?rZNHGJiMHfCl4oX1LzV5IBoJY4uGWtM+RNqSg1ftj+hefc+xx9+ERbfbLG/z?=
 =?us-ascii?Q?gI91ue7StPPz7PB324hz6UnogMZawSH8ZMVY15Jo2IVfYW19vImGMLuYojYa?=
 =?us-ascii?Q?8YVdMgfPTSeZBcXmPn6aTfuHftFHde49qzCtdH5fZepjnmh/7ltri9/O+IHs?=
 =?us-ascii?Q?/ah8LtCVkLjqOU4eXDRrv+1GuVSXjsqYj9VYY414rEIyGKXmZjgyvNrT/Klx?=
 =?us-ascii?Q?oFEuc3xc8s9FoVT3UT4Q07gDqfp+ucDI8OJx/AogcVLXmQ20QjcjAXC4A9gV?=
 =?us-ascii?Q?hJWAil18xqN5oHgRNvVYr+dKRwbLRvUd0LHI7obg?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: CH3PR11MB8362.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 99c8eb89-a648-476a-4204-08dbafa599ec
X-MS-Exchange-CrossTenant-originalarrivaltime: 07 Sep 2023 13:23:18.9477 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: L1b/fUY9usIeO2qiAbqMbTH2k/seFh9DxYqtPgeveIjwKg6FNQuPADxDFmr3cOC7ZgPIIwPmPr66mEe2MvUkSg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM4PR11MB7328
X-OriginatorOrg: intel.com
X-BeenThere: dev@dpdk.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: DPDK patches and discussions <dev.dpdk.org>
List-Unsubscribe: <https://mails.dpdk.org/options/dev>,
 <mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://mails.dpdk.org/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <https://mails.dpdk.org/listinfo/dev>,
 <mailto:dev-request@dpdk.org?subject=subscribe>
Errors-To: dev-bounces@dpdk.org

Hi David,

> -----Original Message-----
> From: David Marchand <david.marchand@redhat.com>
> Sent: Monday, August 21, 2023 7:36 PM
> To: dev@dpdk.org
> Cc: thomas@monjalon.net; ferruh.yigit@amd.com; Xia, Chenbo
> <chenbo.xia@intel.com>; nipun.gupta@amd.com; Richardson, Bruce
> <bruce.richardson@intel.com>; McDaniel, Timothy
> <timothy.mcdaniel@intel.com>; Gaetan Rivet <grive@u256.net>
> Subject: [PATCH v2 11/15] pci: define some extended capability constants
>=20
> Define some PCI extended capability constants and use them in existing
> drivers.
>=20
> Signed-off-by: David Marchand <david.marchand@redhat.com>
> Acked-by: Bruce Richardson <bruce.richardson@intel.com>
> ---
>  drivers/event/dlb2/pf/dlb2_main.c | 7 ++-----
>  lib/pci/rte_pci.h                 | 4 +++-
>  2 files changed, 5 insertions(+), 6 deletions(-)
>=20
> diff --git a/drivers/event/dlb2/pf/dlb2_main.c
> b/drivers/event/dlb2/pf/dlb2_main.c
> index 8d960edef6..29e3001627 100644
> --- a/drivers/event/dlb2/pf/dlb2_main.c
> +++ b/drivers/event/dlb2/pf/dlb2_main.c
> @@ -27,9 +27,6 @@
>  #define NO_OWNER_VF 0	/* PF ONLY! */
>  #define NOT_VF_REQ false /* PF ONLY! */
>=20
> -#define DLB2_PCI_EXT_CAP_ID_PRI   0x13
> -#define DLB2_PCI_EXT_CAP_ID_ACS   0xD
> -
>  #define DLB2_PCI_PRI_CTRL_ENABLE         0x1
>  #define DLB2_PCI_PRI_ALLOC_REQ           0xC
>  #define DLB2_PCI_PRI_CTRL                0x4
> @@ -263,7 +260,7 @@ dlb2_pf_reset(struct dlb2_dev *dlb2_dev)
>  	if (rte_pci_read_config(pdev, &slt_word2, 2, off) !=3D 2)
>  		slt_word2 =3D 0;
>=20
> -	off =3D DLB2_PCI_EXT_CAP_ID_PRI;
> +	off =3D RTE_PCI_EXT_CAP_ID_PRI;
>  	pri_cap_offset =3D rte_pci_find_ext_capability(pdev, off);
>=20
>  	if (pri_cap_offset >=3D 0) {
> @@ -490,7 +487,7 @@ dlb2_pf_reset(struct dlb2_dev *dlb2_dev)
>  		}
>  	}
>=20
> -	off =3D DLB2_PCI_EXT_CAP_ID_ACS;
> +	off =3D RTE_PCI_EXT_CAP_ID_ACS;
>  	acs_cap_offset =3D rte_pci_find_ext_capability(pdev, off);
>=20
>  	if (acs_cap_offset >=3D 0) {
> diff --git a/lib/pci/rte_pci.h b/lib/pci/rte_pci.h
> index a82f073a7d..d2f7a96f17 100644
> --- a/lib/pci/rte_pci.h
> +++ b/lib/pci/rte_pci.h
> @@ -97,9 +97,11 @@ extern "C" {
>=20
>  #define RTE_PCI_EXT_CAP_ID_ERR		0x01	/* Advanced Error
> Reporting */
>  #define RTE_PCI_EXT_CAP_ID_DSN		0x03	/* Device Serial Number */
> +#define RTE_PCI_EXT_CAP_ID_ACS		0x0d	/* Access Control Services
> */
>  #define RTE_PCI_EXT_CAP_ID_SRIOV	0x10	/* SR-IOV*/

Maybe we could also do the small clean-up: it should be one space after 'IO=
V' :)

With this fixed:

Reviewed-by: Chenbo Xia <chenbo.xia@intel.com>

> +#define RTE_PCI_EXT_CAP_ID_PRI		0x13	/* Page Request Interface
> */
>=20
> -/* Single Root I/O Virtualization */
> +/* Single Root I/O Virtualization (RTE_PCI_EXT_CAP_ID_SRIOV) */
>  #define RTE_PCI_SRIOV_CAP		0x04	/* SR-IOV Capabilities */
>  #define RTE_PCI_SRIOV_CTRL		0x08	/* SR-IOV Control */
>  #define RTE_PCI_SRIOV_INITIAL_VF	0x0c	/* Initial VFs */
> --
> 2.41.0