From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <stable-bounces@dpdk.org>
Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124])
	by inbox.dpdk.org (Postfix) with ESMTP id 6C3F543A72
	for <public@inbox.dpdk.org>; Mon,  5 Feb 2024 14:33:15 +0100 (CET)
Received: from mails.dpdk.org (localhost [127.0.0.1])
	by mails.dpdk.org (Postfix) with ESMTP id 498EE402BE;
	Mon,  5 Feb 2024 14:33:15 +0100 (CET)
Received: from NAM12-MW2-obe.outbound.protection.outlook.com
 (mail-mw2nam12on2059.outbound.protection.outlook.com [40.107.244.59])
 by mails.dpdk.org (Postfix) with ESMTP id E954F402A2;
 Mon,  5 Feb 2024 14:33:13 +0100 (CET)
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none;
 b=ExhyEqO0ok/28PTez9VQ6QWFKKmSYS/51pmqOdRsPFsBY/sXo+dQf4uOUc7LbSW2JBz+JaZQ8EddazyCxPdGFgLUjhD037DJyHkqTsiEypC0Re4hLeNPMMumW0zZ77XFz/HLNh9w+yylbcOeb2cGUWluGW4oPldwkwD/gGuAGSJOvBUeDCHig7a/TOQzG5kAYtPv6cQTTduq8LQtQEfwpa2mPsd1lOaQhVRiRcUN/PH6rUhWDomn18gZCGy2XEWmf283+o+IWaqPgnu0ER30GQN47+ViPi00hrinCMAhQDE+2p4kIQcWIYoyMGUfPLOhXuTYpfjzEFvm84Ioi3p9Ng==
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=XOqcMapMzjCMMtTE7LeCn5MAk3V2ensmZdv+6y9gQ9M=;
 b=m6r+lDT0K7Uf57GTGhz2mwSoOuD988FzXDlHmJVylG8wz4Gh8Wt1AfAKJ25BUfJmMAB55KpI/PhCJwyIpVMmAq3b7DdqpGAwj3/ug2hXwI+WJWrq5YHByDLzyj+3xzwBg1pR5xVJdEL38qqltFoEeJk9C105Fh+dlbqbgHAwnMBUWnlwNJGxb7Q+Cn4YNPOyo2maJSELfjDm2ndIRaoCRex7CGkpypm0ioh9H69yrp6NP9iGFS0VmOaZzGFTcB6xSRWPQJDa2a9yrOncBR5ocRZVeKT+IdknOzYYIAxu+o8RUdrIgGHPLVftixYfTZPw47uiE7A+IX+VxKKlTc8zsg==
ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass
 smtp.mailfrom=amd.com; dmarc=pass action=none header.from=amd.com; dkim=pass
 header.d=amd.com; arc=none
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amd.com; s=selector1; 
 h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck;
 bh=XOqcMapMzjCMMtTE7LeCn5MAk3V2ensmZdv+6y9gQ9M=;
 b=GHHWwWxlng00Ig3dowf1amrJSHxvFgMHSjxkC4BmyhvmkkCYwHbP4KjLdSwEQ1s0MtQvuQwOO7YoBQSfz3TQBo8ceDWpLTG89sAT9usKhpS3/ljEejQFT9aP0D8QjuFheOX5wfAuNBtfgXgVqrO7WGGLBAbuMxrl2P1bewoOy04=
Received: from MW6PR12MB8999.namprd12.prod.outlook.com (2603:10b6:303:247::8)
 by LV2PR12MB5918.namprd12.prod.outlook.com (2603:10b6:408:174::21)
 with Microsoft SMTP Server (version=TLS1_2,
 cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7270.14; Mon, 5 Feb
 2024 13:33:11 +0000
Received: from MW6PR12MB8999.namprd12.prod.outlook.com
 ([fe80::b2d3:710f:1b2e:81ef]) by MW6PR12MB8999.namprd12.prod.outlook.com
 ([fe80::b2d3:710f:1b2e:81ef%4]) with mapi id 15.20.7270.012; Mon, 5 Feb 2024
 13:33:11 +0000
From: "Boyer, Andrew" <Andrew.Boyer@amd.com>
To: Stephen Hemminger <stephen@networkplumber.org>
CC: "Boyer, Andrew" <Andrew.Boyer@amd.com>, "dev@dpdk.org" <dev@dpdk.org>,
 "Patel, Neel" <Neel.Patel@amd.com>, "stable@dpdk.org" <stable@dpdk.org>
Subject: Re: [PATCH 04/13] net/ionic: fix missing volatile type for cqe
 pointers
Thread-Topic: [PATCH 04/13] net/ionic: fix missing volatile type for cqe
 pointers
Thread-Index: AQHaVg6sxEkbhOyxa0qCHcOWhnGDtLD4BjsAgAO9UoA=
Date: Mon, 5 Feb 2024 13:33:11 +0000
Message-ID: <0D66E753-A5A3-4F4A-A532-1BE2FD55493D@amd.com>
References: <20240202193238.62669-1-andrew.boyer@amd.com>
 <20240202193238.62669-5-andrew.boyer@amd.com>
 <20240202202636.450b1b38@hermes.local>
In-Reply-To: <20240202202636.450b1b38@hermes.local>
Accept-Language: en-US
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=amd.com;
x-ms-publictraffictype: Email
x-ms-traffictypediagnostic: MW6PR12MB8999:EE_|LV2PR12MB5918:EE_
x-ms-office365-filtering-correlation-id: f4d2025f-6ac1-4a58-b3c2-08dc264eff52
x-ms-exchange-senderadcheck: 1
x-ms-exchange-antispam-relay: 0
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info: 82X6VXAOyhCXG3Y/0FIx5/1DVXkZHRufZ4sOvoy87jnnN/lTEALCReu6uHpRjLYsKH86S8xLZVdJHSu+5bctxjZHgAj0C15/NuLHUb0mbAJhnjLeXscYEYMHYDPahpvtj5PPBcw4thLvNLtJkkuT934lmCNkHdQjk/kGSiC+TmmeHZTduaaFZp44e59K8gk0jKW/T1jKJ+ssnEq+osm5Wvm9HNQtNudqVvs8S50yTMp4qTWyBco/Z93AjV8KxzalpFVO4fp/4rMWgE7EOFmt647U2nF3YvZga6bMnFnrMBiQ36RzvLZGO0ExQh7Q1cAHaVN6Qdve7/rOa/2cw4DXfg4i+O3abTN4Tj6gWhNSFZgxuBnTEMnenJ4Vj65Ay5FHbbYPXBuTl8wkmEJiu9A3Px3GBqFU0Ne/yuP7SYkZZDk2M5YVctlnioX5341gkvXEqT4eP5lMqZi3KY4dD6SnZtUYFUueoZh6JerxilU7XDOstZlIWbwLCV8w9eKLZDB1nuEubxMkxr1lVLXTw2HvbAP7ai5lK7qZu2FZsX161i3bIPkbXkrGYnPWC+IU1QBgoKOvtuu3s0xrDy2m9au/g+i3VcK+b0U5gILekT2jEgIcAB13v1/Qi8yhwtPat34ZhFVmz8tLBEWLxLwDpnT73g==
x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:;
 IPV:NLI; SFV:NSPM; H:MW6PR12MB8999.namprd12.prod.outlook.com; PTR:; CAT:NONE;
 SFS:(13230031)(136003)(396003)(366004)(346002)(376002)(39860400002)(230922051799003)(230273577357003)(186009)(1800799012)(451199024)(64100799003)(41300700001)(26005)(53546011)(6512007)(6506007)(2616005)(5660300002)(66476007)(66946007)(66446008)(8676002)(4326008)(66556008)(64756008)(8936002)(2906002)(478600001)(33656002)(86362001)(76116006)(966005)(6486002)(54906003)(6916009)(316002)(83380400001)(38100700002)(122000001)(71200400001)(36756003)(38070700009)(45980500001);
 DIR:OUT; SFP:1101; 
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?w8dFyZ80h/3ME1ntot1z0HPn7TLpRhFikhqoEHcUaIwEVM1Yx4yzoh7H3k2G?=
 =?us-ascii?Q?A43+QgvVpPU2PDrh5cd/N44a1IrpJV0O3XEWpW8iJHLJIsviwiwFkxPGSz6T?=
 =?us-ascii?Q?ArSRDRxOWbsjOsmOm81IbQzFcx7+x/K3boLoeXbxHFVhkVgurrz0jMXs/zEB?=
 =?us-ascii?Q?Lern1VDtWyL8S+KHd6KvncXnzg5pK7/r2lbbjXbkzPtgS02lVqOZjeGO7kva?=
 =?us-ascii?Q?JB7GwYioyR6XxF9yurQ5uuoxhHMYWXCjE7YFORDx5vNrPp4oq1Gah9Scrgpx?=
 =?us-ascii?Q?rphL/96qz1pqZ9Ayv3JMEwG0tb7gopDlEo3uV4BGXl41FU2uNupp7SnBO07B?=
 =?us-ascii?Q?I3TBg2JHe5GNLrRyYto0ZgLii88vJr66Ol+MRIlMUHfFZKC478vbPfNE8cTs?=
 =?us-ascii?Q?bT01Yiuo55Egy1a6n82559aDhbl3c02L8Sp8oZLIDC3njoO5EnhOKQ2E1KrR?=
 =?us-ascii?Q?tD8FE1cdw2JqOTw95F29WhsUaATMTT9KCZLiEeNSZ7k/uEmXKWeKcfrZlSVc?=
 =?us-ascii?Q?QA9YrgjcQTpgDv3Qr44LHr50owUQqgd5Tyn/JiPrbhBarHHcayGMNVPirmkY?=
 =?us-ascii?Q?vJwkWma4SEn7zTPSv09u+Eq5qTIf0e4Fr8jN5F4QapcRMPpfxs/MSIoWwHb+?=
 =?us-ascii?Q?3wEZsuCz8xZYXNQWv/hVMBDUN6HhfibWHMFNdlSKIv5mVOa41GJkAGP24Nz4?=
 =?us-ascii?Q?okI0w/1djYF8/p7BYC0BB2atgUk5Tq79wNqlOg+cqh4NxjT6SUNQ/HcjX9Pw?=
 =?us-ascii?Q?2h/FxcHP8hJAGbKlI0JPLwioTfht51sTc73UHe7pGj+zvFUnc2UXzlxpPXsc?=
 =?us-ascii?Q?jZV4gfNt8hQZ0bYB+o/uMXfoobB7F/geoJ96bXQOYUmEBJ3U1ctxTIP+kmQH?=
 =?us-ascii?Q?7U3UlobJNQlXm+ZfhyUV4hFoRdWyKIEL5bPzizwwXgioYJGlE9J7GqRHusjv?=
 =?us-ascii?Q?urNufzZZw/s2Ve3kZo0mBaV4Rqlwe+jdIyx5lX4/HuuiDRj07lDsuUvp5SQ6?=
 =?us-ascii?Q?lw0NLpDTz8h7ZchTEvMwLbiomoFTbaveEug7DUsdAh59AC5Vx0OPPt8YN8yl?=
 =?us-ascii?Q?edPhyQC/QYGryssdlYtJ/xBkYN3IgbG2bd5omPkuERFBjqWTICb1jSpgFxtU?=
 =?us-ascii?Q?VQGXhiKUCzgMgrRLj5+uutlgTsYxUURznaLOk7sbbYHD4cEQ8W1OddSPPpYZ?=
 =?us-ascii?Q?nozwCJi/h5VvqcLq5+MhM7D3CTGqQlW0BurNEexNpwUO43pKruz2gnMnmCvL?=
 =?us-ascii?Q?SojAkrNZ8wyivZK5QuMxrfydtidqK9rDuKysivF3gywIIdDDJpMPlWqJIOWW?=
 =?us-ascii?Q?n/455yso5FKElp2WDfV95r5Wdhe4sgo3eOe05x1VkSh6eDzGc/954OUefOPC?=
 =?us-ascii?Q?FJScdyKHrCWBEkMXYXpag0aYdIsRHKtH2l8Bc1/uHHhD7THVRbhP+wHdgdlb?=
 =?us-ascii?Q?CVaCujemfezdrZOHlfj2nTpoGvaz26jBUtED2UD2MzudP0/0rbKOuIcalhOE?=
 =?us-ascii?Q?9bs2yNk7uD+uLT7SxFQryJMi7gNgTyZWgHqadNZiowTh+2+zwaSp22fEfJWZ?=
 =?us-ascii?Q?gJyn9n/Mvvu4gTSdD0JGMOgEt00ikd5Io4DVD/fE?=
Content-Type: text/plain; charset="us-ascii"
Content-ID: <0DFC73EAEC8CDD4990E3280B74B94907@namprd12.prod.outlook.com>
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: MW6PR12MB8999.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: f4d2025f-6ac1-4a58-b3c2-08dc264eff52
X-MS-Exchange-CrossTenant-originalarrivaltime: 05 Feb 2024 13:33:11.2556 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 3dd8961f-e488-4e60-8e11-a82d994e183d
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: MG62WWiYZktGTGbgGqnEjoanl2W2hVoSgNKdb5s9Qr4ub8dS10oC44ujAPceUbcS
X-MS-Exchange-Transport-CrossTenantHeadersStamped: LV2PR12MB5918
X-BeenThere: stable@dpdk.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: patches for DPDK stable branches <stable.dpdk.org>
List-Unsubscribe: <https://mails.dpdk.org/options/stable>,
 <mailto:stable-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://mails.dpdk.org/archives/stable/>
List-Post: <mailto:stable@dpdk.org>
List-Help: <mailto:stable-request@dpdk.org?subject=help>
List-Subscribe: <https://mails.dpdk.org/listinfo/stable>,
 <mailto:stable-request@dpdk.org?subject=subscribe>
Errors-To: stable-bounces@dpdk.org



> On Feb 2, 2024, at 11:26 PM, Stephen Hemminger <stephen@networkplumber.or=
g> wrote:
>=20
> Caution: This message originated from an External Source. Use proper caut=
ion when opening attachments, clicking links, or responding.
>=20
>=20
> On Fri, 2 Feb 2024 11:32:29 -0800
> Andrew Boyer <andrew.boyer@amd.com> wrote:
>=20
>> From: Neel Patel <neel.patel@amd.com>
>>=20
>> This memory may be changed by the hardware, so the volatile
>> keyword is required for correctness.
>>=20
>> Fixes: e86a6fcc7cf3 ("net/ionic: add optimized non-scattered Rx/Tx")
>> cc: stable@dpdk.org
>>=20
>> Signed-off-by: Andrew Boyer <andrew.boyer@amd.com>
>> Signed-off-by: Neel Patel <neel.patel@amd.com>
>=20
> In general barriers are better than volatile.
> Volatile is a compiler not hardware construct really.
>=20
> https://www.kernel.org/doc/html/latest/process/volatile-considered-harmfu=
l.html

Thanks for looking. The goal here is to prevent inappropriate compiler opti=
mization.

>From the link you posted:

> Pointers to data structures in coherent memory which might be modified by=
 I/O devices can, sometimes, legitimately be volatile. A ring buffer used b=
y a network adapter, where that adapter changes pointers to indicate which =
descriptors have been processed, is an example of this type of situation.


Doesn't that sound like what we are doing?

Thanks,
Andrew