Received-SPF: pass (google.com: domain of linux-kernel+bounces-121413-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249;
Message-ID: <2aeb41a7-912a-4176-a73c-841b46334709@intel.com>
Date: Wed, 27 Mar 2024 15:17:41 +0100
User-Agent: Mozilla Thunderbird
Subject: Re: [Intel-wired-lan] [PATCH net-next 2/3] idpf: make virtchnl2.h
 self-contained
To: Jakub Kicinski <kuba@kernel.org>
CC: Kees Cook <keescook@chromium.org>, <netdev@vger.kernel.org>, "Gustavo A.
 R. Silva" <gustavoars@kernel.org>, <linux-kernel@vger.kernel.org>, "Nathan
 Chancellor" <nathan@kernel.org>, Eric Dumazet <edumazet@google.com>,
	<intel-wired-lan@lists.osuosl.org>, <linux-hardening@vger.kernel.org>, "Simon
 Horman" <horms@kernel.org>, <nex.sw.ncis.osdt.itp.upstreaming@intel.com>,
	Paolo Abeni <pabeni@redhat.com>, "David S. Miller" <davem@davemloft.net>
References: <20240326164116.645718-1-aleksander.lobakin@intel.com>
 <20240326164116.645718-3-aleksander.lobakin@intel.com>
 <20240326210412.1bfce311@kernel.org>
From: Alexander Lobakin <aleksander.lobakin@intel.com>
Content-Language: en-US
In-Reply-To: <20240326210412.1bfce311@kernel.org>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?bWNpejJTTVRrdGNSS21XQkFtK0FoVWd3T1RZL2FGTlFtZ0lXczNjRko2VU1p?=
 =?utf-8?B?MUJOWDBIUXlUeklWU3BhZEMxM3ZZT05DTWJwb25QWTIxemsvcFBxWHRpNXE0?=
 =?utf-8?B?WWd6RS8wQjlRaFhVVzFmaS9CRkc4U1ZuL3NXTThpdVdER0FERmF5SEpKNnVR?=
 =?utf-8?B?Q2NRY3JxQVRRa1JQM3JqYUJORU5pUDVpSjVvVVQ2VE1vWTJkZmpmZ3Z3K2NQ?=
 =?utf-8?B?amJqZ1ByYThnZTVvd2xxSm9XbUZJMC90N29NYk1kRDZZWDJ4STVkaHdOamJ3?=
 =?utf-8?B?NVYzN0RrS09JMk5PeDBHekl0Q2NsMDdCNTlDYUFjdkZCeFRYZTJGMTNDU1Jz?=
 =?utf-8?B?UU5SR083R1p2bDBxWFRSRWlRYjJKck9QTXdHVEJuMlpLc0NIblhZcnFxL3hN?=
 =?utf-8?B?WXRRNm5DUGFhNk1JdTJUQjF4cnRCL0RGMHVkWUtKYzVrdnpnT3ZKQW5YbFRC?=
 =?utf-8?B?MkFXZkRJbCs4TEtvb2tBREkreTRWMnBkQ3haaEpCRjhjeWJCaFM4LzR4b202?=
 =?utf-8?B?K0FsTXo2ODh5b2FmQnd3YXdYTDZaaG5DSnJVSTRkM1dFdWRhNXpLSU54Z0F2?=
 =?utf-8?B?KzhnWm1RamhZYTZqNnJwblpNcVF4OHRGTnlxM214UTFSTWdycDlWVDdJVVZj?=
 =?utf-8?B?VTRLWmZsaVJ4NSt4ME5NWmtRTnNWV3dxcmFFRFlDZ1U1Z1dMQWMxQ0RuOUsv?=
 =?utf-8?B?SjI0eHNZZloxZWMraEtOK0NES3lOaHpEUmhMczJrN1RhSTM4QUdxSVp5UzZ3?=
 =?utf-8?B?cWJ0S2c2QVhFTDVHR0lGWndxSXRCL2RYWnNaNG9IQWIvaGI4TzhjbW1uWmcy?=
 =?utf-8?B?UUk2VlgzTlc5OFZkeElWSUlxVWxwa1FyVWVYRWExOTJhZnduUkxXMm10ZTlr?=
 =?utf-8?B?UEpJNWx5anZoUkRzcUl2SjNHSGF2N0tjRkR1OG43WnNnaHFEbU1qRTB4aEYv?=
 =?utf-8?B?VmNZQUtzdXBLdGlrRS9IQnJpc3B4Uzl6bU5VeUN1OGc3WWlKeGNJdU05ckx4?=
 =?utf-8?B?bjJhREV6cWlsK0VBWVJtWjhPNXdaSnFGUURQSFRleitTVU10M1dZRUNuSUlN?=
 =?utf-8?B?UnJLSkFlTHppazUydTlidXpMdW9VZ2hEa3VIQjJEanFFSUxySzZwdm5qMWtN?=
 =?utf-8?B?QVNOWS9PZHQvS1hXTHNmbHFsNm12UDR4aWFkUXFhNFphL2hFT2RYS25CblhW?=
 =?utf-8?B?OVVvd2JjdDl2d2I4RHdIMHZXb0c5cDFoWndmZnlFLzcxL3IySmRmY0xkTTNT?=
 =?utf-8?B?bXFWSkJVdnpBOVNKZHhkTEV6bWpvUEc5eVd1UlgzLzc2SzZKMXlSWFRVcER5?=
 =?utf-8?B?UldTRFpRdnZMbTg3d2RZaFVMM0pYejJXVCtxdVNsamFzcHpKM2hnZkRpYlBB?=
 =?utf-8?B?cEpVSldnT284WjArVXVpN1g4UTNiYnkrRklTMDVFZkhnaGp5THBLK050a3hj?=
 =?utf-8?B?aUVhTW9LMGF4MkdxekZCajZ6OHkzcjY3V3J2RDdFRGtGQXo2N1VJbUNBdjVi?=
 =?utf-8?B?WXhRSWxqZjVuc2MyeHU3TEhYM0VXMStVeXZQcjFQdjMzYUJhQWJIQXZiZUha?=
 =?utf-8?B?ckVKY3RnSi9YUktJMjFlOW15NWFGdnBvZGhkK1RLQnZFY3NXMVgzbHRSb0Nr?=
 =?utf-8?B?UnF2cHhTbElQdTNMWG8zU1dGMkcxbWU2WFJLMThtTHlCRDlOb2hQMFY5WVQw?=
 =?utf-8?B?VzZHY3JyL3pKVDIvRTRQRlIvM281S2QyVU5ac09nenZaa1pkaTB3dzVGbW5s?=
 =?utf-8?B?MEZZREhnbTA1eWdmS0ZHdE0za0ZuMVd5dS96bVRhV1dPTmdDdmF0bTR5Y3Rq?=
 =?utf-8?B?bkFpL2RrK3pZa1ZWTEllWFJQdkRKbFkwWGNEVUhFR29BVDVNcnN2akFMdmVH?=
 =?utf-8?B?M0tBdEU0R3hHbFdsK1JMZ0phM0pYMTAvbVd0WHZzOTRBOXkxYjNtS3g5eTRp?=
 =?utf-8?B?UUMvUjZ5aThEVmNYakFpL09UWXZKdnRnQS85c0hnaXNpbVJUb0xZSW9YaFZT?=
 =?utf-8?B?MVBDUmNYWkczYlJmMWdrcXYyTlcvRGFrMWxmZEdpVkhJNHNXNm0yeitIbFk5?=
 =?utf-8?B?ZWord1R1QWY5KzFjaTlOMGplMVBGMy8vYkhWZTI4L0pJVzJIRHZMWWttWXgz?=
 =?utf-8?B?b20wY1FySEViN0ZleVBuNmgycStGZERBWXQxdTdqaG1CTnhWcDFrWXhDRjBn?=
 =?utf-8?B?NlE9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 72f5cffc-b919-461b-9f3a-08dc4e68b2dd
X-MS-Exchange-CrossTenant-AuthSource: DS0PR11MB8718.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 27 Mar 2024 14:17:56.6543
 (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: b42yrOw5RwYIFD236iIJ/WzAuexI8VPBX0KT06aTdKUD2dziULqof7bq72kMqSQoVom48myainl5zS/AzqkacRTB96go52bvI/jUvpXRhtU=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ2PR11MB8323
X-OriginatorOrg: intel.com

From: Jakub Kicinski <kuba@kernel.org>
Date: Tue, 26 Mar 2024 21:04:12 -0700

> On Tue, 26 Mar 2024 17:41:15 +0100 Alexander Lobakin wrote:
>> To ease maintaining of virtchnl2.h, which already is messy enough,
>> make it self-contained by adding missing if_ether.h include due to
>> %ETH_ALEN usage.
>> At the same time, virtchnl2_lan_desc.h is not used anywhere in the
>> file, so remove this include to speed up C preprocessing.
> 
> Breaks allmodconfig, it seems..

Oops, forgot one line and my night CI didn't notice anything...
Sending v2.

Thanks,
Olek