From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <dev-bounces@dpdk.org>
Received: from dpdk.org (dpdk.org [92.243.14.124])
	by dpdk.space (Postfix) with ESMTP id 2FB8BA0096
	for <public@inbox.dpdk.org>; Sat, 13 Apr 2019 08:43:39 +0200 (CEST)
Received: from [92.243.14.124] (localhost [127.0.0.1])
	by dpdk.org (Postfix) with ESMTP id D53721B147;
	Sat, 13 Apr 2019 08:43:37 +0200 (CEST)
Received: from mx0b-0016f401.pphosted.com (mx0b-0016f401.pphosted.com
 [67.231.156.173]) by dpdk.org (Postfix) with ESMTP id AF6B11B144
 for <dev@dpdk.org>; Sat, 13 Apr 2019 08:43:36 +0200 (CEST)
Received: from pps.filterd (m0045851.ppops.net [127.0.0.1])
 by mx0b-0016f401.pphosted.com (8.16.0.27/8.16.0.27) with SMTP id
 x3D6eDGF028777; Fri, 12 Apr 2019 23:43:33 -0700
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=marvell.com;
 h=from : to : cc :
 subject : date : message-id : references : in-reply-to : content-type :
 content-transfer-encoding : mime-version; s=pfpt0818;
 bh=rdf0JahTMkKRvsUp0UGlvr0yIQa4cv8rdD7o9HzBPlg=;
 b=Hy+1PqnRAh0QlcR+wgepf9Dt8htG07UQqyUpMaYtUtVG2XckBTOJIhMaEQNw9X3cPQh7
 M3cNe9UE7N+fxQJCfTyjZHU0ldJFCGu4js4f5sESp9YRqdrB23LO4rtfXwuZrm4kKL1H
 PdXOL11gpcXaGwMrm3Z9QBS2zPpx01Yp4UTRD0boBthyxztZqXmuM3m61rqUxaRNs4ZQ
 XNRDbzcKIDQTXOSx1vl0j7yMoO1KqNGaWneQlH4YuCsG/S+U0InedAC+kICXXFitlKym
 UdPRrIX0rASOIlbPSj+5+/NTIKL03O8XIF4KA109xui19wEKhvA5ky5cI1pMkg22H7so ag== 
Received: from sc-exch02.marvell.com ([199.233.58.182])
 by mx0b-0016f401.pphosted.com with ESMTP id 2rtthau7e5-1
 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-SHA384 bits=256 verify=NOT);
 Fri, 12 Apr 2019 23:43:33 -0700
Received: from SC-EXCH01.marvell.com (10.93.176.81) by SC-EXCH02.marvell.com
 (10.93.176.82) with Microsoft SMTP Server (TLS) id 15.0.1367.3; Fri, 12 Apr
 2019 23:43:31 -0700
Received: from NAM05-DM3-obe.outbound.protection.outlook.com (104.47.49.52) by
 SC-EXCH01.marvell.com (10.93.176.81) with Microsoft SMTP Server
 (TLS) id
 15.0.1367.3 via Frontend Transport; Fri, 12 Apr 2019 23:43:31 -0700
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
 d=marvell.onmicrosoft.com; s=selector1-marvell-com;
 h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck;
 bh=rdf0JahTMkKRvsUp0UGlvr0yIQa4cv8rdD7o9HzBPlg=;
 b=eAuB5NqPNDdGwNNlp8wzqbVfvT5bjsAWA/oh1qCgl36M62CBs7AoSB34mSWvsw2Tz0mTskIYPqaPkfvBDO5pcBrHbYjuBzFYWGT4Wu+zRjk2KkFILZEYQNR4LWlWeS909KXSD2ssyfGVLl01vLAr0bTho+KKPeHgoOncw7uvCfE=
Received: from BYAPR18MB2424.namprd18.prod.outlook.com (20.179.91.149) by
 BYAPR18MB2885.namprd18.prod.outlook.com (20.179.58.159) with Microsoft SMTP
 Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id
 15.20.1792.17; Sat, 13 Apr 2019 06:43:29 +0000
Received: from BYAPR18MB2424.namprd18.prod.outlook.com
 ([fe80::6dd3:c056:b23b:ab4e]) by BYAPR18MB2424.namprd18.prod.outlook.com
 ([fe80::6dd3:c056:b23b:ab4e%7]) with mapi id 15.20.1771.019; Sat, 13 Apr 2019
 06:43:29 +0000
From: Jerin Jacob Kollanukkaran <jerinj@marvell.com>
To: Yongseok Koh <yskoh@mellanox.com>, "bruce.richardson@intel.com"
 <bruce.richardson@intel.com>, Pavan Nikhilesh Bhagavatula
 <pbhagavatula@marvell.com>, "shahafs@mellanox.com" <shahafs@mellanox.com>
CC: "dev@dpdk.org" <dev@dpdk.org>, "thomas@monjalon.net" <thomas@monjalon.net>,
 "gavin.hu@arm.com" <gavin.hu@arm.com>,
 "Honnappa.Nagarahalli@arm.com" <Honnappa.Nagarahalli@arm.com>
Thread-Topic: [EXT] [PATCH 2/6] meson: change default cache line size for
 cortex-a72
Thread-Index: AQHU8Yb7IExSSGsjf0GUl5EIdkLmL6Y5ovtQ
Date: Sat, 13 Apr 2019 06:43:29 +0000
Message-ID:
 <BYAPR18MB24248C9353FC4990051E7DE8C8290@BYAPR18MB2424.namprd18.prod.outlook.com>
References: <20190412232451.30197-1-yskoh@mellanox.com>
 <20190412232451.30197-3-yskoh@mellanox.com>
In-Reply-To: <20190412232451.30197-3-yskoh@mellanox.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach: 
X-MS-TNEF-Correlator: 
x-originating-ip: [122.182.194.144]
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: 8fed4e00-579e-48dd-c25a-08d6bfdb56a3
x-microsoft-antispam: BCL:0; PCL:0;
 RULEID:(2390118)(7020095)(4652040)(8989299)(4534185)(4627221)(201703031133081)(201702281549075)(8990200)(5600139)(711020)(4605104)(2017052603328)(7193020);
 SRVR:BYAPR18MB2885; 
x-ms-traffictypediagnostic: BYAPR18MB2885:
x-ms-exchange-purlcount: 1
x-microsoft-antispam-prvs: <BYAPR18MB28851E378AA050523418D3F9C8290@BYAPR18MB2885.namprd18.prod.outlook.com>
x-forefront-prvs: 00064751B6
x-forefront-antispam-report: SFV:NSPM;
 SFS:(10009020)(346002)(376002)(366004)(396003)(39860400002)(136003)(199004)(189003)(13464003)(476003)(66066001)(7696005)(8676002)(2501003)(106356001)(486006)(105586002)(446003)(11346002)(76176011)(6306002)(6436002)(99286004)(86362001)(9686003)(55016002)(53936002)(97736004)(229853002)(68736007)(81166006)(81156014)(110136005)(54906003)(316002)(71200400001)(52536014)(14454004)(6116002)(6246003)(33656002)(14444005)(256004)(966005)(5660300002)(71190400001)(6506007)(2906002)(3846002)(53546011)(8936002)(478600001)(25786009)(7736002)(26005)(186003)(74316002)(305945005)(102836004)(4326008);
 DIR:OUT; SFP:1101; SCL:1; SRVR:BYAPR18MB2885;
 H:BYAPR18MB2424.namprd18.prod.outlook.com; FPR:; SPF:None; LANG:en;
 PTR:InfoNoRecords; A:1; MX:1; 
received-spf: None (protection.outlook.com: marvell.com does not designate
 permitted sender hosts)
x-ms-exchange-senderadcheck: 1
x-microsoft-antispam-message-info: wmvnmiczAiu59XZW7IxnwZ+bAiCOOlx7zW4sbOYlMwc1HvHIJUG4Bf1N4UFCCxE+ertFmcrGpRQ5JpZxqvUaBCqlGLrfm25WfjfS0tZPji3Ia5MDSKnU9bPic8QiJp0j4lzi4yQKb3w+ifd9IvPyzI9lDjKEMFHSF6KQg3pCGzzv+mEUmCcazG+f6Mnpvsm7nfu5g6TaS66QwHTjnSIn8CJBLgkH1Lue4h2JWK8F8J3OAtNczL/eWx5FBkd9MmeHyY7MaaMtn6Sdrj0gfT4U9MbrULdViT1NFJCCtEXNcy0O69sTviMg9FkFV6XAXbQ9/lNY1Zf82I4IL36gUj0PWosbKO1i1AhBaWV/Mr4Uc95fhkIDcbMpwQJPViTEHCI3ILIj9yRDcYvUUepu14CZr8jHRyBWrQ9G4mWpIliBv1M=
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: 8fed4e00-579e-48dd-c25a-08d6bfdb56a3
X-MS-Exchange-CrossTenant-originalarrivaltime: 13 Apr 2019 06:43:29.0688 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 70e1fb47-1155-421d-87fc-2e58f638b6e0
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BYAPR18MB2885
X-OriginatorOrg: marvell.com
X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10434:, ,
 definitions=2019-04-13_02:, , signatures=0
Subject: Re: [dpdk-dev] [EXT] [PATCH 2/6] meson: change default cache line
 size for cortex-a72
X-BeenThere: dev@dpdk.org
X-Mailman-Version: 2.1.15
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
Sender: "dev" <dev-bounces@dpdk.org>
Message-ID: <20190413064329.3oVb8tvLNzitYRkzWApQTrR3xGnrsfm5RDgpwub8V0E@z>



> -----Original Message-----
> From: Yongseok Koh <yskoh@mellanox.com>
> Sent: Saturday, April 13, 2019 4:55 AM
> To: bruce.richardson@intel.com; Jerin Jacob Kollanukkaran
> <jerinj@marvell.com>; Pavan Nikhilesh Bhagavatula
> <pbhagavatula@marvell.com>; shahafs@mellanox.com
> Cc: dev@dpdk.org; thomas@monjalon.net; gavin.hu@arm.com;
> Honnappa.Nagarahalli@arm.com
> Subject: [EXT] [PATCH 2/6] meson: change default cache line size for cort=
ex-a72
>=20
> ----------------------------------------------------------------------
> Per the email discussion [1], the default cache line size of armv8
> cortex-a72 is changed to 64 bytes.

IMO, In git commit you remove the reference to specific discussion and
Update the reason correctly.


>=20
> [1] https://mails.dpdk.org/archives/dev/2019-January/123218.html
>=20
> Signed-off-by: Yongseok Koh <yskoh@mellanox.com>
> ---
>  config/arm/meson.build | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
>=20
> diff --git a/config/arm/meson.build b/config/arm/meson.build index
> e00b894523..73c581948c 100644
> --- a/config/arm/meson.build
> +++ b/config/arm/meson.build
> @@ -51,6 +51,8 @@ flags_dpaa2 =3D [
>  	['RTE_MAX_LCORE', 16],
>  	['RTE_LIBRTE_DPAA2_USE_PHYS_IOVA', false]]  flags_default_extra =3D []
> +flags_cortex_a72_extra =3D [
> +	['RTE_CACHE_LINE_SIZE', 64]]
>  flags_thunderx_extra =3D [
>  	['RTE_MACHINE', '"thunderx"'],
>  	['RTE_USE_C11_MEM_MODEL', false]]
> @@ -73,7 +75,7 @@ machine_args_generic =3D [
>  	['0xd03', ['-mcpu=3Dcortex-a53']],
>  	['0xd04', ['-mcpu=3Dcortex-a35']],
>  	['0xd07', ['-mcpu=3Dcortex-a57']],
> -	['0xd08', ['-mcpu=3Dcortex-a72']],
> +	['0xd08', ['-mcpu=3Dcortex-a72'], flags_cortex_a72_extra],
>  	['0xd09', ['-mcpu=3Dcortex-a73']],
>  	['0xd0a', ['-mcpu=3Dcortex-a75']]]

I think, flags_cortex_a72_extra() can be changed to flags_vendor_arm_extra =
or something similar
And update the following CPUs also not just cortex-a72.

	['0xd03', ['-mcpu=3Dcortex-a53']],
	['0xd04', ['-mcpu=3Dcortex-a35']],
	['0xd05', ['-mcpu=3Dcortex-a55']],
	['0xd07', ['-mcpu=3Dcortex-a57']],
	['0xd08', ['-mcpu=3Dcortex-a72']],
	['0xd09', ['-mcpu=3Dcortex-a73']],
	['0xd0a', ['-mcpu=3Dcortex-a75']],
	['0xd0b', ['-mcpu=3Dcortex-a76']],


>=20
> --
> 2.21.0.196.g041f5ea