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 8731A43D06; Wed, 20 Mar 2024 18:40:37 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 6CAFD40ED3; Wed, 20 Mar 2024 18:40:37 +0100 (CET) Received: from dkmailrelay1.smartsharesystems.com (smartserver.smartsharesystems.com [77.243.40.215]) by mails.dpdk.org (Postfix) with ESMTP id D321E40A6F for ; Wed, 20 Mar 2024 18:40:35 +0100 (CET) Received: from smartserver.smartsharesystems.com (smartserver.smartsharesys.local [192.168.4.10]) by dkmailrelay1.smartsharesystems.com (Postfix) with ESMTP id 82A0D20C80; Wed, 20 Mar 2024 18:40:35 +0100 (CET) Content-class: urn:content-classes:message MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Subject: RE: [PATCH 00/83] move alignment attribute on types Date: Wed, 20 Mar 2024 18:40:34 +0100 X-MimeOLE: Produced By Microsoft Exchange V6.5 Message-ID: <98CBD80474FA8B44BF855DF32C47DC35E9F318@smartserver.smartshare.dk> In-Reply-To: X-MS-Has-Attach: X-MS-TNEF-Correlator: Thread-Topic: [PATCH 00/83] move alignment attribute on types Thread-Index: Adp63npZMyjF6KCKREiEQ+k/0VdUWAADxDEw References: <1710949096-5786-1-git-send-email-roretzla@linux.microsoft.com> From: =?iso-8859-1?Q?Morten_Br=F8rup?= To: "Bruce Richardson" , "Tyler Retzlaff" Cc: , =?iso-8859-1?Q?Mattias_R=F6nnblom?= , "Min Hu (Connor)" , "Abdullah Sevincer" , "Ajit Khaparde" , "Akhil Goyal" , "Alok Prasad" , "Amit Bernstein" , "Anatoly Burakov" , "Andrew Boyer" , "Andrew Rybchenko" , "Ankur Dwivedi" , "Anoob Joseph" , "Ashish Gupta" , "Ashwin Sekhar T K" , "Byron Marohn" , "Chaoyong He" , "Chas Williams" , "Chenbo Xia" , "Chengwen Feng" , "Conor Walsh" , "Cristian Dumitrescu" , "Dariusz Sosnowski" , "David Hunt" , "Devendra Singh Rawat" , "Ed Czeck" , "Evgeny Schemeilin" , "Fan Zhang" , "Gagandeep Singh" , "Guoyang Zhou" , "Harman Kalra" , "Harry van Haaren" , "Hemant Agrawal" , "Honnappa Nagarahalli" , "Hyong Youb Kim" , "Jakub Grajciar" , "Jerin Jacob" , "Jian Wang" , "Jiawen Wu" , "Jie Hai" , "Jingjing Wu" , "John Daley" , "John Miller" , "Joyce Kong" , "Junfeng Guo" , "Kai Ji" , "Kevin Laatz" , "Kiran Kumar K" , "Konstantin Ananyev" , "Lee Daly" , "Liang Ma" , "Liron Himi" , "Long Li" , "Maciej Czekaj" , "Matan Azrad" , "Matt Peters" , "Maxime Coquelin" , "Michael Shamis" , "Nagadheeraj Rottela" , "Nicolas Chautru" , "Nithin Dabilpuram" , "Ori Kam" , "Pablo de Lara" , "Pavan Nikhilesh" , "Peter Mccarthy" , "Radu Nicolau" , "Rahul Lakkireddy" , "Rakesh Kudurumalla" , "Raveendra Padasalagi" , "Reshma Pattan" , "Ron Beider" , "Ruifeng Wang" , "Sachin Saxena" , "Selwin Sebastian" , "Shai Brandes" , "Shepard Siegel" , "Shijith Thotton" , "Sivaprasad Tummala" , "Somnath Kotur" , "Srikanth Yalavarthi" , "Stephen Hemminger" , "Steven Webster" , "Suanming Mou" , "Sunil Kumar Kori" , "Sunil Uttarwar" , "Sunila Sahu" , "Tejasree Kondoj" , "Viacheslav Ovsiienko" , "Vikas Gupta" , "Volodymyr Fialko" , "Wajeeh Atrash" , "Wisam Jaddo" , "Xiaoyun Wang" , "Yipeng Wang" , "Yisen Zhuang" , "Yuying Zhang" , "Zhangfei Gao" , "Zhirun Yan" , "Ziyang Xuan" 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 > From: Bruce Richardson [mailto:bruce.richardson@intel.com] > Sent: Wednesday, 20 March 2024 16.51 >=20 > On Wed, Mar 20, 2024 at 08:36:53AM -0700, Tyler Retzlaff wrote: > > The current location used for __rte_aligned(a) for alignment of = types > > and variables is not compatible with MSVC. There is only a single > > location accepted by both toolchains. > > > > After having established this as the conventional standard for lib/* > > this series is intended to convert the remainder of the source tree = to > > use the same location for __rte_aligned(a) and alignas(a) for > > consistency. > > > No objection to this change. Is there a checkpatch rule available to > enforce this going forward or how can we avoid future introduction of > macros in the wrong place? >=20 > Series-Acked-by: Bruce Richardson Series-Acked-by: Morten Br=F8rup