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 E623845E4C; Fri, 13 Dec 2024 12:27:22 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id D3C8C402BB; Fri, 13 Dec 2024 12:27:22 +0100 (CET) Received: from mgamail.intel.com (mgamail.intel.com [192.198.163.7]) by mails.dpdk.org (Postfix) with ESMTP id 9F92E40269 for ; Fri, 13 Dec 2024 12:27:20 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1734089241; x=1765625241; h=date:from:to:cc:subject:message-id:references: in-reply-to:mime-version; bh=S5N1+etFovskqvzyFv6qfQAP3jzSrOzjS3xUJWvjpl0=; b=MSYD2p10Ew+aeOIv3n3liwvyLMZK6ipzgS2wKixUhY/WYuABTCUZzBZz vCOjfBCxaaeeS4+nWt14pMwFXeZ8PX/tTza/LYMidaaSPr0NU9BzH2biN BycK7UYPyuhK8100pY9AB+hii8tH5ccejJXVBVPxvCOrnRQ32gqZkpY41 ZwiExHMYeb5F0C3q1r1AcDxf6DjxxX9XD7dojCf/Rqurx5M4T6TkcfXMk /ypHzKFacDKMXcslkKKnqP7xRTx5av4E0eVTy1J5jO6i7IFEPBUkjfJ5C 8kbbUGcJS/hJdnKNoGVJgf8DOF8wx6BrqZhqJqiX6tULK4CIxaYP15bx5 g==; X-CSE-ConnectionGUID: f7XvCF3eSBKx1k9sZTSocA== X-CSE-MsgGUID: eP4BIPeuRuWv7Pr3DPERxw== X-IronPort-AV: E=McAfee;i="6700,10204,11284"; a="59934635" X-IronPort-AV: E=Sophos;i="6.12,231,1728975600"; d="scan'208";a="59934635" Received: from fmviesa009.fm.intel.com ([10.60.135.149]) by fmvoesa101.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 13 Dec 2024 03:27:20 -0800 X-CSE-ConnectionGUID: B2z385DdTGuspLKuCrfTPw== X-CSE-MsgGUID: IP/eehziQlyndFfQ+EYUrA== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.12,231,1728975600"; d="scan'208";a="97071421" Received: from orsmsx602.amr.corp.intel.com ([10.22.229.15]) by fmviesa009.fm.intel.com with ESMTP/TLS/AES256-GCM-SHA384; 13 Dec 2024 03:27:19 -0800 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.44; Fri, 13 Dec 2024 03:27:19 -0800 Received: from orsedg603.ED.cps.intel.com (10.7.248.4) 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.44 via Frontend Transport; Fri, 13 Dec 2024 03:27:19 -0800 Received: from NAM10-DM6-obe.outbound.protection.outlook.com (104.47.58.45) by edgegateway.intel.com (134.134.137.100) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2507.44; Fri, 13 Dec 2024 03:27:18 -0800 ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=hXFCD6V87XIDOSJStB9yTZEKsgGuLqwXBG6evsTGGX1oirAHLne9/syygYDZUhtgNNqit9QiRC6zG9jQZGh/Ggvoim8gdENwDEPA/OulLaqJ703XmO/+XTHCG8PEajM7T0LVMJV3l+a1+kbHDjYM1FyIiBcDnU5/Vi29x60eX4OwltQRlaTho2O8EBqAKxbA/a1hAl9iFZS9kN/IR6NY7Vcb+iZgRddZkS5JSEehfzbAW3xW/GbDlIlBWAftwclA8HfDH45/OlzvSv15W1w7+5A9YVtB4csis0XQ6cINtTah9Tr8LT6kWK806S2liPBV9YtLNChV0/blwruC5/Qizw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; 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=d1TsH1PxZdJcmEXPlui+B9VJDXnpaZG2COnGKp0tsvc=; b=Tb1JcmY0fKkhUhs9TfZGeGO24w+kg/puIDgeDZ9/aMe/n5D9IimEtlwVJuxcW3uDZoB8dsYyfifK/Nwwqd90Q03A2LwQ+ct/kvWac5oquQEC2dfNBnRyjRwvJ2jpMua1gwxiHBQM0iGn4py51J0gGv9YvFvqP/ngj6u5lZ5WTKCe1mKb2j/luVWW9xglddKhwDAUXjKQDZ8kXeJ+AngXPqI4BFuvR1g7sZ1bqfOdFrEsMF7795uSYahP+ds8LicVD+nnePFTXaQHj1Q9AJKIrnCoclYNeACT6uRojQMHMhs5yJ/80xK0FGe3W6LQnH9WVJQWr/0TJNi31fkBhFvUrQ== 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 SA1PR11MB8521.namprd11.prod.outlook.com (2603:10b6:806:3ab::22) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8251.17; Fri, 13 Dec 2024 11:27:16 +0000 Received: from DS0PR11MB7309.namprd11.prod.outlook.com ([fe80::f120:cc1f:d78d:ae9b]) by DS0PR11MB7309.namprd11.prod.outlook.com ([fe80::f120:cc1f:d78d:ae9b%4]) with mapi id 15.20.8093.018; Fri, 13 Dec 2024 11:27:16 +0000 Date: Fri, 13 Dec 2024 11:27:11 +0000 From: Bruce Richardson To: David Marchand CC: , Subject: Re: [PATCH v2 0/6] Add a stricter headers check Message-ID: References: <20241127112617.1331125-1-david.marchand@redhat.com> <20241213105010.1527683-1-david.marchand@redhat.com> Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <20241213105010.1527683-1-david.marchand@redhat.com> X-ClientProxiedBy: DB8PR06CA0013.eurprd06.prod.outlook.com (2603:10a6:10:100::26) To DS0PR11MB7309.namprd11.prod.outlook.com (2603:10b6:8:13e::17) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: DS0PR11MB7309:EE_|SA1PR11MB8521:EE_ X-MS-Office365-Filtering-Correlation-Id: af124a85-e7d4-4064-0b7d-08dd1b691920 X-LD-Processed: 46c98d88-e344-4ed4-8496-4ed7712e255d,ExtAddr X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|366016|376014|1800799024; X-Microsoft-Antispam-Message-Info: =?us-ascii?Q?sHdY86TOS/BAGfTJaajjDecA39HtibNsgTrd/4eWinrp6ITJT9LmVW4HmPU6?= =?us-ascii?Q?2TPd572DKvN+9eG40JnzMxF95qgEH4ZvLDK7fwze+trlKxBbbN5OIOnyxShG?= =?us-ascii?Q?UEiLHVGLk61cDSzvvc+EazuSBdrGprc2F3Tw8bQsLuu3jq1S2NJKAj41vaqx?= =?us-ascii?Q?RWlu2Mr7sUp92hWVqbunTmSYuuFh0PwsWI8z/rcyqQj2ckXCUt/i82g4l9qq?= =?us-ascii?Q?4XQbTFMcKhrkIauUsvgFi/2u7Eo1Ja+PvU2gBNZ3OEo04m1hgvD0GUPFzI/3?= =?us-ascii?Q?EDKj6oQ7d//4ZGCmIsIYASuq1ldP0uJqWpHVofEud4DRYkJHULmJCGAzm8y7?= =?us-ascii?Q?pq/RvMWDb96CS7JlwwQfcKI5E5js8pRN3P50b3OQdZRMwO1g2A/b72qbBcGf?= =?us-ascii?Q?e3q31RdbTkC+AgNSZeyXD6E44nB6h22xfy2IDXTZqZnXSBAc6t0GAifV2y3B?= =?us-ascii?Q?tTV0Ws+B1NEeHKd66ytLJM5fPQXo9S8ayOXiYQwW8FYPF8P6IARypqtDxPpp?= =?us-ascii?Q?k113A3YgNFKSLUK4iTOjYE0slS9iBF/mMN4dbtIrEBgYZzjCQPPYTr9b1Rpw?= =?us-ascii?Q?PJmnt7jQvJo5DU4k+j+3lDGJW8iZw0ye3h7F9AtmYhdE2ccXjldSpFbjksyn?= =?us-ascii?Q?f5X/YmLGe/+oNM7MGBVRvHGtfOMhWWt2joRGw0oHyS6EgUug99AbN/z8c4DM?= =?us-ascii?Q?9w6S0+v5IR1uibfSKvI2QW45w5V5H2fJEUkxklw1pjveoz+K21VPVMARqiNX?= =?us-ascii?Q?lOgOZ5cX+DZFAlkG3+yAoMI6+AWpO/5SLVyBa/zwDCEi3FoLoJ4Ocz4MgmeA?= =?us-ascii?Q?uvCvgGZkvDRhRYWR0wYqMT+0lWEdwSjXVwVIr6/TJGfDx+3XYKZ4u9eUGU+O?= =?us-ascii?Q?zPMnoZ4R+QgVGstYxUIhHIDC7pTqNnMvVuMuSPDgV6OLmwvu2KM0gddaIw7j?= =?us-ascii?Q?j+lpLrcIqoiMO3EJyXNNLrhCzHa+jHG0RXVPkMMYY0B9aPgQN07rmm8MdwOE?= =?us-ascii?Q?oagAqd7DoVjL2Jykqv6Rqj0Is8UrBD/1Z90PcytDKZKEz3cWpQbrZQqQn4Vs?= =?us-ascii?Q?Dr+tXYDWOIQg2Lpm8G/YX0pj+M1O1BHJhTbKJMb8j1ucFtR5g0qJHBufB/5M?= =?us-ascii?Q?0la0Cbdx4SqXpsxQJqfPXeLrhpo3U9aadce2aG8wdNKw6m0XVSfMaS2K0dh3?= =?us-ascii?Q?T38ji/Q7DepFzz1htVOg1RnBQdbPA0QemJF7iayM9+E4dfv6cfzB5qGVwRfZ?= =?us-ascii?Q?SknnyBh+WNejQ+zdeyimTnsxC6I52UtGEp2NuPww8nQrGLpskiRnZAbryha2?= =?us-ascii?Q?WnwXciscUluNQWw4rRRRsbt6U4mSRIjXdKSidwn9tuhTOkkxgdGiSoTHUhEz?= =?us-ascii?Q?SZZgGw+MwzKPKgYac1ISU9+1huJk?= 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:(13230040)(366016)(376014)(1800799024); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?CSN2aGbfsrY+4fhazAg42j7vfk7/6DxzW0SuMYnW33KlD7i3gUdTGOs4jWCe?= =?us-ascii?Q?GQz62vcYvtbArUfbqLQHv8zof62k84k3VDp/arpt52WS24Lu43NHV+xW2zUA?= =?us-ascii?Q?0ClLmlVFu/jtyLuChlyaxZN/3qEZTQrD4+iWfaf8w4ccpXbkz+oazeu5eURA?= =?us-ascii?Q?OopGlB8YYxfb4nztWHP2AxprrirRh3RdDhubNUlucxFMwqneJVhstovVYZhB?= =?us-ascii?Q?J4ixwQeHm0oAtTfZ0QaOObsYtx2RtdimrBdI6TfuO3rzGqUq/G2dKX5kA/1+?= =?us-ascii?Q?SPP2lzR0wNcdsEnTFNoaBKJflE93A8Tbhg6Z2m6Q+rlfdsYHZFLr/uHq1MTN?= =?us-ascii?Q?8LmEFD7nrQn0XeOAgXEABpVDxckYHTxdFR7GLBKdsQ+BrSD74x0QGh+uGDkN?= =?us-ascii?Q?whgGGndPDh2ry/mwbwEu6szsd11RS3kx2W+pCSyHeD1VYk8GhXLyrdse+6Ng?= =?us-ascii?Q?GrDIWphvd6KkfzrZYDRknz/FJzXojmN6FY3/bj/2+ZXS5gYlRqQ78SFF511T?= =?us-ascii?Q?si+gA74VvnklIYEPPyXihC4dK3ACXdyOcWmXOU8AypQVDt73XGc+zSCmvvgp?= =?us-ascii?Q?92brwqJUjQrLOl0joB7+10xufRCKGs2HnYfUGS2zDJUrmckNiGbU+ocr6MNI?= =?us-ascii?Q?YHu/I1iZXAwISWGl36akmfG4MxaqSOA2vdcUR0w/72wNcADMr+v3ekIhOa6h?= =?us-ascii?Q?4niD3qLBmmiHMwmtcgEffaoJOErUC4PwngYZpZGZ7VEUZ5a8RxM8XQvGEo6H?= =?us-ascii?Q?tiGZ5mx6nZIYSPRlTfrUqDsoh6nc2X+gIKfVZzGWiXQ43v/iezC1Ik/A6OfM?= =?us-ascii?Q?fttIMURCA+l0AaEAfP5C2EiH9ayWKG7SvlHuPpQGDAS+09KDW0Zf87DQ3ri/?= =?us-ascii?Q?+sN+KB2KlgLjRiw4en16KE4xSDSJuZcEYtoUidEiK41TKEFMPGscb96lSNm9?= =?us-ascii?Q?WHJ6KSSsNMCtYUpExyxycfvsFQ8uIqFZsgvJEqv0ELDyLIYbf5YwPsLGGXRe?= =?us-ascii?Q?Hb18BDaTJndGzipW43XxWUpUVo9Uv1lIQDCrfG0SXWZ46n/pttyV/5yOO0zw?= =?us-ascii?Q?jkeZ2w2YUgTcDJrWwktQxUOrudBXUG3KIqS7tg/p6gLiWV6ATogc3V4T2vkK?= =?us-ascii?Q?9oiNYAFex2LfKmCv26W0IKYYA1IQaPkXwk/BXWNWwpGb69rwuGUO9/wTIrv9?= =?us-ascii?Q?wp2sjQcstCyZBcw52jAO0M59Papy0BzN0zU4/zw4IKTeoyMvqyI/jy37Zeae?= =?us-ascii?Q?YXzpo5HQUknn05T8amiavUGPCxl8U/R7CRl+lGS4NkVfrK6gUvLQwmLWOpSW?= =?us-ascii?Q?XO+f5W7vzJc8g/8VQXS6GGn/w6P2WlTZqQ9wC2OZmETM+tXyMnrbQWc7V2cd?= =?us-ascii?Q?1KRoXUCXARAQQxwCqVXx5O82kvmfn69ANsSi5X7DRP9obz+dG8HeaWn486f6?= =?us-ascii?Q?4BneRtBUXcl1+5fF4MkZl0i05TXxlI2DTZupWHny5CFTjLZJBYDk0Ossxr1p?= =?us-ascii?Q?SSARFSBPVSUEWkhUZw5Aa10K6WStAuc1iYjuX5iArDt8S4TGpQ4K9XiCfS5O?= =?us-ascii?Q?hXzvqHEQd0SOoD/YYqsfk0Th4MBOAOlQPdgetFbMGEe2YhaX1kz8xIb7KZzb?= =?us-ascii?Q?pg=3D=3D?= X-MS-Exchange-CrossTenant-Network-Message-Id: af124a85-e7d4-4064-0b7d-08dd1b691920 X-MS-Exchange-CrossTenant-AuthSource: DS0PR11MB7309.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 13 Dec 2024 11:27:16.4992 (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: 39iIhaQJ1OOb6vC79u6I6iIcTY7OK/u85RyN9BFiBk4fk9M3PZN1kPBdIG1KPaEsRcicfVtp6SRVdQVJZUVVSgQ6sFIEEnwSVAZerk45pTQ= X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA1PR11MB8521 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 Fri, Dec 13, 2024 at 11:50:04AM +0100, David Marchand wrote: > As explained in patch 6, the current headers check can not catch > issues when a public header includes an internal header. > Fixing this from meson does not seem an easy task. > > This series approach is to reimplement the check in a Makefile invoked > out of DPDK (like what is done for external compilation of examples). > This has the advantage of being simple, and avoiding any (non intentional) > implicit include path coming from the meson framework. > > As there was no easy way to distinguish "indirect" headers in an > installed DPDK, I chose to move those headers in a new sub directory > (patch 5). > > Patch 1-4 fixes have not been marked as backport material as those bugs > seems minor/easy to fix externally (by either including missing headers, > or enabling enable_driver_sdk option). > > For now, I left the check_includes= option untouched, as there may be > users of this check and this check still catches issues without > requiring to install DPDK. > For patches 5 & 6, I wonder if we can find a slightly different way to do this. I like the idea of using make to build chkincs free from possible environmental contamination from meson, but I really don't like the complexity of the resulting makefile! Rather than having to move the indirect headers to a subdirectory, and then have a makefile run a scan from the headers directory, how about instead generating the makefile (or possibly a build.ninja file) directly from meson itself? This means the makefile can already have the list of headers and C files necessary - no need for an extra subdirectory - and no need for large amounts of wildcard matching and replacements. The downside is that the makefile is no longer with the source, but in the build directory. However, since the most likely use for this makefile is from the test-meson-builds script and from automated CIs, I don't see this being a major issue. WDYT? /Bruce