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 1D56E43032; Fri, 11 Aug 2023 10:52:48 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id E7F2240F16; Fri, 11 Aug 2023 10:52:47 +0200 (CEST) Received: from mgamail.intel.com (mgamail.intel.com [134.134.136.24]) by mails.dpdk.org (Postfix) with ESMTP id 101BA40E03 for ; Fri, 11 Aug 2023 10:52:45 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1691743966; x=1723279966; h=date:from:to:cc:subject:message-id:references: content-transfer-encoding:in-reply-to:mime-version; bh=mfoKqw/EV68C2UgDiXie6vygqOiz0PYoy5JJM84t0Z8=; b=gmWME4wvqQgDREnTI4nLL5/iJvYMW7TD9mdHg52tFCuSlSQqYtl7mHba O9t8Z5GZ3k1YibcCTptQU1Odh3zIQLeYrBKzDjf0DUV+YCyqvO5Nb/+Zh AdzlA7GanNUnwiD9ns494HXsyFwe1tmDRG7Wro3HSdU9ayOgOcJdqNGmW bZ6k01rEJs3oMh/WK7iIcLECbPOip3fIbS/WXefLcbOo1ZG4JglparnIt snKr5WumEz1tynsWM/pgY7SCL+Te7Yr6zkO4o0crHhOhZgk3PQGC5emi+ jz3x8MQK9dl/O/Z35P48S7FXp/iSCKjYZjvoTJsb2e/Yb0q+c2CQqoEds Q==; X-IronPort-AV: E=McAfee;i="6600,9927,10798"; a="374401473" X-IronPort-AV: E=Sophos;i="6.01,165,1684825200"; d="scan'208";a="374401473" Received: from orsmga006.jf.intel.com ([10.7.209.51]) by orsmga102.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 11 Aug 2023 01:52:44 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6600,9927,10798"; a="709463256" X-IronPort-AV: E=Sophos;i="6.01,165,1684825200"; d="scan'208";a="709463256" Received: from orsmsx602.amr.corp.intel.com ([10.22.229.15]) by orsmga006.jf.intel.com with ESMTP; 11 Aug 2023 01:52:43 -0700 Received: from orsmsx601.amr.corp.intel.com (10.22.229.14) by ORSMSX602.amr.corp.intel.com (10.22.229.15) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.27; Fri, 11 Aug 2023 01:52:43 -0700 Received: from ORSEDG601.ED.cps.intel.com (10.7.248.6) 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.27 via Frontend Transport; Fri, 11 Aug 2023 01:52:43 -0700 Received: from NAM10-BN7-obe.outbound.protection.outlook.com (104.47.70.103) by edgegateway.intel.com (134.134.137.102) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2507.27; Fri, 11 Aug 2023 01:52:42 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=g+WAaC4/foVu/yKOOvV+KuxB8rRbAVkjk0oR8G2MsPCb73nCJVo5bbvZm1xjPr8CNmskw2qeI2EyGFGIxM2YEtw0Yb2jUTzlsPoA2mFtmF9i3B0O2dXqL4RAmG5RQ4VNfTc1kvJGewUx+3I1nBBz4DLkkgHxWoHsFnDzbaWfno4AQ7vshMF8UFA4mpfzijcLy9cbyKz/H3fX6AGMN6UTNz7CRaKbs1yz0kGs8L+7sIhKTsG8uF0XkUZ5FeEUTFa61JTH4Q8+COWUTi8N+890OgyOtAXw0piDhkp+jAjkBk628S++6KX9Di61Knxb+yrP54Oema9TbROtCzGlFjphoQ== 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=8PnDwUn5SY9zpy/2jUeT1A3o3nWPUKlhMMLNnlgxjyA=; b=Tz6POUJEOs0TWTmjhE7Uf7aI8aMTbVSVP67lTLU+CzitsbDmB1QN/20x6ap36gYxLEX7HkWVYaNuad5gtR8pe8npKrYH4Pi0P14aGpUAyZWmBHTM19/uzO89jbhuUX3XFG62n6v41gj6ADmHuuSGNJfd9+IduQ6YxATWYLfN2UceZ+MGC6inzipqGpN1qwert964lOsHgCb6pkzopS0rxWXXeF0WjMhu0h3Y2qD5obxYvm7BrW9sIKssuwli7Tb5oc7BqLzvOO9HeBo5yw7MXyYoE3tQbpZ1n75lAjEQMzrG0rESShp/wbH8ZCrysHPb8ahuZk7Pp8eS12X7L3lFYg== 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 Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=intel.com; Received: from DS0PR11MB7309.namprd11.prod.outlook.com (2603:10b6:8:13e::17) by PH7PR11MB7962.namprd11.prod.outlook.com (2603:10b6:510:245::7) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6652.30; Fri, 11 Aug 2023 08:52:40 +0000 Received: from DS0PR11MB7309.namprd11.prod.outlook.com ([fe80::9802:65f0:c441:780f]) by DS0PR11MB7309.namprd11.prod.outlook.com ([fe80::9802:65f0:c441:780f%7]) with mapi id 15.20.6652.029; Fri, 11 Aug 2023 08:52:40 +0000 Date: Fri, 11 Aug 2023 09:52:34 +0100 From: Bruce Richardson To: Tyler Retzlaff CC: Morten =?iso-8859-1?Q?Br=F8rup?= , "Stephen Hemminger" , Thomas Monjalon , David Marchand , , Ali Alnubani , Subject: Re: [PATCH v5] build: update DPDK to use C11 standard Message-ID: References: <20230731103858.1491751-1-bruce.richardson@intel.com> <20230810074839.446f655d@hermes.local> <60418966.matp6XCIr4@thomas> <20230810100258.0a38bdb9@hermes.local> <98CBD80474FA8B44BF855DF32C47DC35D87AE6@smartserver.smartshare.dk> <20230810223443.GA5743@linuxonhyperv3.guj3yctzbm1etfxqx2vob5hsef.xx.internal.cloudapp.net> Content-Type: text/plain; charset="utf-8" Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20230810223443.GA5743@linuxonhyperv3.guj3yctzbm1etfxqx2vob5hsef.xx.internal.cloudapp.net> X-ClientProxiedBy: DUZPR01CA0075.eurprd01.prod.exchangelabs.com (2603:10a6:10:3c2::7) To DS0PR11MB7309.namprd11.prod.outlook.com (2603:10b6:8:13e::17) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: DS0PR11MB7309:EE_|PH7PR11MB7962:EE_ X-MS-Office365-Filtering-Correlation-Id: f12648bf-2523-4f0b-4720-08db9a4851d3 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: KmoUgaLtpAUQbWYpM1cc0UFpD3LGmhAMnENVT5MctiXFN/iTwYAP+6ektuzN7vDKD5ToFEQ0TblVAGLtB0ZNxLEEfBtdFLdCbq3BK8jyw/iXKaOarCrOlkbTNb7h8sksELbpudyX7ikSdsPKVatcLuxD5UCdbbKMXyQDZigELsDBV6Orlub1W1us3l/eJ4nheyxHLJqUZRBPOa7CSNiqXP7t7g1LWwvDYVTaua/AEZWH1wfysN1w9SQnV+SS5CWoNhMxgSVjH57v2ESy+HKM3kfDPD3VnxD4geJzpj8J0VZw5jgMsfLsWjXzh5z8qS7HyFZqkZGEmcQyEY53h7cJqpRz11BcjljzbA1M2cLoYusIg6a5pFjmQfEh/0PAckG2ZwmswdpcofK0e3MksVe5fpYNrN9GRDxqmdT1sDsKSqXtqhZhaMmO+xGBf4RjfX79jQEkNQD0ZzbpATVH27/2o510Tcxiyn2pdBWO37t9067H6JUsyO2iEiH39nVXbyMOTyre5vr4lOA+rs5irqaGkOmIVh6OrvoI/5pxWg37lpfFirWX+tUhFlOkEDJcssJB X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:DS0PR11MB7309.namprd11.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230028)(39860400002)(346002)(366004)(136003)(396003)(376002)(1800799006)(186006)(451199021)(6512007)(478600001)(6486002)(6916009)(4326008)(66556008)(66476007)(66946007)(5660300002)(316002)(6666004)(54906003)(41300700001)(8676002)(8936002)(6506007)(53546011)(26005)(86362001)(44832011)(38100700002)(82960400001)(83380400001)(2906002)(66574015)(15650500001); DIR:OUT; SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?K2F1bTR5T1VoUVM3TDB0OWpYM0hNNGtXbDBXNXhyTUVBN01jZ2h0dWx1dFZr?= =?utf-8?B?NkVNcTJTTHk4dS9TenhjQk9pL3hmajFpZThpeHRBZ2FqNVYyOVczMmhrWFNs?= =?utf-8?B?MmhLZzJ4dDBGVWQ5K0o2TGlLY3hudjZlS255NCtZemxuOEJNeDlsZEV1UVlD?= =?utf-8?B?YThScGplN1QxQk5nMlluUHpkSzRTUUszMno1aitnWmRNaFluWWFBVDhDbSsx?= =?utf-8?B?ZHJMNWRienA4Q2cyTUk4LysvUXZ4ODJzTjN6NW1GbjFvVTdCTkgyVmdqSkNP?= =?utf-8?B?RVhIMTlORGV6Z2VlQjRlUERhbHNLekgyQm9URmdxWWo1cGhaeUJpZzlnajhu?= =?utf-8?B?RDZzUUJJVmVpL2djVSs5OFdpYzZHOE56dEticlZRTWhWK2NNYjBxUVFrUkxi?= =?utf-8?B?alBXenQzOEZsWFBPTElwUHJOYnVSRnBqei83akw1QWhocFROYmNmRms1azNt?= =?utf-8?B?ZjR2NUNDVEs1VitkY0NxNXFJMkw0NUxrR1JqYlR5c0FCYjZLaXBkR0tXbm41?= =?utf-8?B?eitxeDFlOCtESWpMam9yQjVsUitPNkZDd3YyK0ZaVGFQak1ndW9NY3RXbE5x?= =?utf-8?B?aS94RkhnUWJMVWlhaWRHYkRURSsxaDhnRTMzVFNvV1M2MmN5UEpOdWVwYlJi?= =?utf-8?B?aEhUR3BnMlZZbkFSZWFxZklMNXNVQkZrZEdrV0NYY2hOaDJGNW43a0IxNGlz?= =?utf-8?B?Z0ZIQnZWbFdsK3c1VWF0S2xRVVR3ZVV0elNVK3V1ZkVxL29kMWpJNnQxODJy?= =?utf-8?B?M0VLQWpEcXlMNkhnU01JbXdyY2RpVVdEeXY2b3NBZU5zbTdIS2NmRnFCL000?= =?utf-8?B?STRZMHJsUTV6NzdDRldhM1dXZ1p6UTFzbjhqRHpJMm5JMkgzcTlTVXhVRmtE?= =?utf-8?B?a2N4SkYwdThScXlDT2RCWk13eEFDTnROTU5Fa0dJK0Q3c3hqcGRPbjR1ME0x?= =?utf-8?B?T2dVeVpDT1lGUWwxRlJvL253VXc3dDZxbDBOeFVpT3RBT3hGSnlLemZ5VE9q?= =?utf-8?B?eW93WUdUR3lMK3lvcTd2SHVKQTNZWHNrb0paQ0lacGVPZTlzVnlBNFRydXUx?= =?utf-8?B?blRlak9rUlRiTjRlbXNxTTBSTHVqcGJ3KytHZUVtK3hOZlhzdnRWNVRGM3NK?= =?utf-8?B?OHRHaWV3QXNRY3UyQTBxSmI2b2orS01wU28yUTcycGJUSFVraW5DUXlSU3RG?= =?utf-8?B?RmxLcjF4QlFSV3d2OExJR0MyVHlKbHpFVkRkWGdRcmR4OVhwT1RUbitRWHZX?= =?utf-8?B?Q2xEWTZXOWRsUUlWYS9uV3NBQUJwb2MrNkFzeGt5TFpsWGs0bC84Mkc0ZmN2?= =?utf-8?B?bUh2SGdNV0hSTkphZGRQRlV6ODlDelcxTitSOEZ0ZEhOUWkvcGlpNkc4TE1W?= =?utf-8?B?MXhadHRqLzMweWNKeHpoc3pLZmJlNkZmeTVFOFBiYVF2YllPclNhdVFVaHIy?= =?utf-8?B?WGR0eVAwTDRwcy82cUh3MHRQWEJSVVRDczkvRUdyN0ZQRGtoOFlxaHRUTHJO?= =?utf-8?B?WnVnM0ltMmJheWJNaHhlZ2FTK1VHOGRjNXFVRzd2eXJFdnZJRmozL2VaWWhi?= =?utf-8?B?V1dSTzloRXFwbS9HMjdYR01HMW5OSGRLMlFYdGFHcm1Yc1BkNjVOTnRUaytk?= =?utf-8?B?RWF0R0NSZjNWV1BqaFdKY0xFRDNGbW85U1RnT3FmZHczd1VNY3NyWDdyQkpQ?= =?utf-8?B?R3I0aE9abzVYS1dsUzQ3MlVHd3prNlpsN2FWaVovbWQ5NzljeGtPTWtRR2ky?= =?utf-8?B?NE1tL2FMeHVtWGJjLzRPaUU5dHVjek1NRUE5RFhpRHRKRnBOSVNnanlyZndK?= =?utf-8?B?anN0YVZGSGNxSWpVQW5iMmkxYk5sSEVKZVZjWkhwN09ZdzlUTm1sYWx5ODYv?= =?utf-8?B?emNHbGRWeE1LUzB4dklubloyWjNVUUZ5cmhHdkNELzU0eGRmWFcyb2hXQmZl?= =?utf-8?B?YXEzRWkvaXZVOEVYdXNRLzRSRkhiT1JIb3ViWnZhcU9IWURRenl1aytDV01J?= =?utf-8?B?eGFpSGMyVEV0eG55aGozeFJhbkxsSVdSTnNKS2lpQVpJdFdhWTM0K24xVkc5?= =?utf-8?B?bGxkRGZiUGxsTXpuRkZrTkZ0QWNxdlpxSnFndGRaVk5KajAvTUx1Nk9STHlQ?= =?utf-8?B?anMxeWRzNFVld2huQ0w2bC9SaDRMdjZGc0U0WXA0QUVMSTlRUnAvNUNwUGRy?= =?utf-8?B?WlE9PQ==?= X-MS-Exchange-CrossTenant-Network-Message-Id: f12648bf-2523-4f0b-4720-08db9a4851d3 X-MS-Exchange-CrossTenant-AuthSource: DS0PR11MB7309.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 11 Aug 2023 08:52:40.6472 (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: /RDOKRZ5Bji0cEj4XzcGD9UmKgLc3XhI4+cZ31cJkQx8b8C7Bv7TnPF5vIFlcVLCe5jlOeaoAGTQBVvY/0D3gwhxJB5TtbcCjVBBO0Q8Aj4= X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH7PR11MB7962 X-OriginatorOrg: intel.com 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 On Thu, Aug 10, 2023 at 03:34:43PM -0700, Tyler Retzlaff wrote: > On Thu, Aug 10, 2023 at 08:17:23PM +0200, Morten Brørup wrote: > > > From: Stephen Hemminger [mailto:stephen@networkplumber.org] > > > Sent: Thursday, 10 August 2023 19.03 > > > > > > On Thu, 10 Aug 2023 18:49:09 +0200 > > > Thomas Monjalon wrote: > > > > > > > 10/08/2023 18:35, Bruce Richardson: > > > > > On Thu, Aug 10, 2023 at 07:48:39AM -0700, Stephen Hemminger wrote: > > > > > > On Thu, 10 Aug 2023 15:34:43 +0200 > > > > > > Thomas Monjalon wrote: > > > > > > > > > > > > > 03/08/2023 15:36, David Marchand: > > > > > > > > On Wed, Aug 2, 2023 at 2:32 PM Bruce Richardson > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > As previously announced, DPDK 23.11 will require a C11 > > > supporting > > > > > > > > > compiler and will use the C11 standard in all builds. > > > > > > > > > > > > > > > > > > Forcing use of the C standard, rather than the standard with > > > > > > > > > GNU extensions, means that some posix definitions which are > > > not in > > > > > > > > > the C standard are unavailable by default. We fix this by > > > ensuring > > > > > > > > > the correct defines or cflags are passed to the components > > > that > > > > > > > > > need them. > > > > > > > > > > > > > > > > > > Signed-off-by: Bruce Richardson > > > > > > > > > Acked-by: Morten Brørup > > > > > > > > > Acked-by: Tyler Retzlaff > > > > > > > > Tested-by: Ali Alnubani > > > > > > > > > > > > > > > > The CI results look good. > > > > > > > > > > > > > > > > Applied, thanks! > > > > > > > > > > > > > > The compiler support is updated, that's fine. > > > > > > > Should we go further and document some major Linux distributions? > > > > > > > One concern is to make clear RHEL 7 is not supported anymore. > > > > > > > Should it be a release note? > > > > > > > > > > > > > > > > > Well, DPDK currently is still building fine on Centos 7 for me, so > > > let's > > > > > hold off on claiming anything until it's definitely broken. > > > > > > > > > > > > > > > > > > > > > > > > Should be addressed in linux/sys_reqs.rst as well as deprecation > > > notice. > > > > > > Also, is it possible to add automated check in build for compiler > > > version? > > > > > > > > > > I'd be a little careful about what we claim, and I think current docs > > > are > > > > > accurate vs our original plans. What we didn't plan to support was > > > the GCC > > > > > and Clang compiler versions in RHEL 7, but if one installs an updated > > > GCC, > > > > > for example, the build should be fine on RHEL 7. > > > > > > > > > > Now, though, we are having to re-evaluate our use of stdatomics, > > > which > > > > > means we may not actually break RHEL 7 compatibility after all. We'll > > > have > > > > > to "watch this space" as the saying goes! > > > > > > > > > > Overall, I think the approach of build-time checks is the best, but > > > not > > > > > for specific versions, but instead for capabilities. If/when we add > > > support > > > > > for stdatomics to DPDK builds on Linux/BSD, at that point we put in > > > the > > > > > initial compiler checks a suitable check for them being present and > > > output > > > > > a suitable error if not found. > > > > Exactly. Capabilities checks is the right way to go when cross compiling. > > > > > > > > > > OK looks good > > > > > > Note: RHEL 7 official end of maintenance support is not until June 2024. > > > > > > > It was agreed to abandon RHEL 7, mainly driven by the need for C11 stdatomic.h, which is not supported by the GCC C compiler included with RHEL 7. So it pains me to admit that Stephen has a valid point here, after it turned out that the GCC g++ is not C11 compatible. > > we would substantially reduce porting delta to retain C11, there are a > number of other things that help with portability from C11 that we can > utilized that i hadn't brought up before since it had been resolved to > adopt it. > > it would be really unfortunate to say we aren't going to require C11 > since that would cause me to have to bring a lot more conditional > compile into the tree. > > just fyi > As far as I know we are requiring C11, and the meson.build file on main tree currently has set the minimum for that. We just haven't got code in there yet that uses the C11 atomics, which is the bit that is missing from the RHEL 7 compiler. I agree that we should not actually actively support RHEL 7 - I just wouldn't call attention to the fact that it's no longer supported if it still happens to work. Once we actually break it, I'm all for documenting that it's not supported. [NOTE: I both cases, I'm not saying that we call it out explicitly as supported either - I'm just talking about a release note entry saying the opposite!] /Bruce