Received-SPF: pass (google.com: domain of linux-kernel+bounces-172962-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
Message-ID: <3b08e1d0-62be-4fae-9dbb-9161992ee067@intel.com>
Date: Wed, 8 May 2024 11:13:21 +0200
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH net-next] netdevice: define and allocate &net_device
 _properly_
To: Jakub Kicinski <kuba@kernel.org>
CC: "David S. Miller" <davem@davemloft.net>, Eric Dumazet
	<edumazet@google.com>, Paolo Abeni <pabeni@redhat.com>, Kees Cook
	<keescook@chromium.org>, "Gustavo A. R. Silva" <gustavoars@kernel.org>,
	"Simon Horman" <horms@kernel.org>,
	<nex.sw.ncis.osdt.itp.upstreaming@intel.com>,
	<linux-hardening@vger.kernel.org>, <netdev@vger.kernel.org>,
	<linux-kernel@vger.kernel.org>
References: <20240507123937.15364-1-aleksander.lobakin@intel.com>
 <20240507111035.5fa9b1eb@kernel.org>
From: Alexander Lobakin <aleksander.lobakin@intel.com>
Content-Language: en-US
In-Reply-To: <20240507111035.5fa9b1eb@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?aXVsc0U5MVhLcUhxTFlRckg1L1J3TUJWaHVBL3F5ZXlDSTNQMERjMHZUSENs?=
 =?utf-8?B?eks4SWNWcE82K2JNT2pnSmU0Zlo0M0RBUTFjR1lLNFJpMTgzR2RWcWZlOHp5?=
 =?utf-8?B?a1BiNlpzUDJreXhnRlkydUVzUjI3aEhNajNlNDNrZUlkUys4L2d2c3VROEZE?=
 =?utf-8?B?QW9BZUkrTnRtVjlMT2JCdG10V3k5aC9FY0M4M1FIbkNwcE9HZmVoR2NtZXFu?=
 =?utf-8?B?MXFjOXhHOVBKZzMzVVVNTmlYbGNTalVyMEpsdkh2RTJQUTBBb1cwMVBrK1ZE?=
 =?utf-8?B?c1Qyc0RxTzdoTmFlckVqMGE5KzVVdEo1QVFCekkwL3FQNWo3VFgzcDJ5Nk0r?=
 =?utf-8?B?WEdaTG9GT3c2KzhqNi9vR0dXVXV6R3BvQ0xTUUt0bXVlcmhDcjNEV092SnJF?=
 =?utf-8?B?b0ZFN2hhSjdRd20zRDVQYUdNSnNGSFkyK1QvOFRkSURqSk9QSlJYWkZHUVBk?=
 =?utf-8?B?YmhERmN3aFRoVGdxdTN2TDJuRE4reWVMeHJzQURNMTRuRlVRVndKWFhNcWNU?=
 =?utf-8?B?bmMxYWQ0TkpOTEY1cGFFQ2dHMmM0cFEwQlBWUWtHYlJCdEdtd0cxRFg5eEdJ?=
 =?utf-8?B?dlpSVjlRS1NScDhPUHhCU3piNkhJVXBSL2FRdm5EVWJpQndaZmVDU0Q4aWpI?=
 =?utf-8?B?NVdYVGJXcTdsb1BDNXc0K1RldzN6RmhZcVk2alNSZThvNVRndWNkbEtoZTk0?=
 =?utf-8?B?SnNVMDRlNXV6dmZZTmJ5SzNEeElqRnZtcFFYOTZqN3cxcW9nY2dPMG0zTUVM?=
 =?utf-8?B?MHY0RDZ0RGt6ajVQaVFzcWRjcm5BWmJnNVRIQk5BVzVhT3E5bTFOa1JZRzJK?=
 =?utf-8?B?QVNjNzJta3RlVzVmaE5TeGw2M1FVcitrQm1nM0tJSHJnTHZmWGlFcVZCeGYy?=
 =?utf-8?B?SHp0eHQ0Z1lxaWVwMjdDcTJKSFJpM3F2ZC8rTUpxVzhSOFZ2UzRpUi9TRzVy?=
 =?utf-8?B?ZUVRak9tWWEySjg4OTZ6U2NvcTRpRlJYY2lNN1hPcGxTQUorQWpzZDUrQUxT?=
 =?utf-8?B?ZnZaQUxzUm9QMTRMT0Iwc3pDTDBxVTJTeVpNakNCeG9ZaS92bnREWFc1WHhI?=
 =?utf-8?B?S3MzV2FpNWdQMzJoNkxrUktPZ2dtbkMvWmNHcWZodHhqc3Q0cHhac0FxNWQ0?=
 =?utf-8?B?NFdZbXliTzhiTG9IOElCbENnRi9LK0UvbjdzdGRwYU90VkxDNG94YUZFZ0Zs?=
 =?utf-8?B?NHVLdStnRzNWNlBncGhjakdkWU9oVDY1Z1EzNTlIQXd4cFlnZlJad1JzNzBJ?=
 =?utf-8?B?MUJQOG9GZXJiM0lGRm5vc3RHcjRmUnltTTd3Z2hnRmtucCtlWjFQRHkvUnNp?=
 =?utf-8?B?TWpEQ1RnUVd0czFuUllBSCswUndFbGlaRzNvOVlhbGZrTHRtSk5rZ1c4THlQ?=
 =?utf-8?B?TW9kNkRTcmdiNTh2WU0ydEszcjlmZXpDVDF5SzNWem1BOExYSE5YaVFidFk2?=
 =?utf-8?B?ZzZsdnFqbjZkRndKN01ZalFwZi90UE9IZzdyUUs0dVZ1dXhqWnJYdHdmNXdF?=
 =?utf-8?B?VjdQZzR0WmNCOVRDK2ZGOHMzTDBLdEI2bjN4cU9UZFo4c2Q1ejVKM2tPcjc0?=
 =?utf-8?B?WkREbGdkMzNSMCtaMmpXSHl0dzN5c0h5czhSQjFuS0x4RzdMNG5BcWFCdVU5?=
 =?utf-8?B?VTRJcVI0R3FBL0FtYjQ3djZhM1NxS1E0bUsrMFQraUVFNXF4LytsVFdaTFpQ?=
 =?utf-8?B?Rjg5TXcyUXlxRmhvdXQ0K04vbWNjQzdSbGFKVTU3QlRlNDNVYi9wa2swQW9n?=
 =?utf-8?B?aENJMjlYNU1QQTcrNXV3WTZGZ0dBUVBJOHllVFhzdVpHRmZKY29uS1FweXUz?=
 =?utf-8?B?czRjUVFnV0hIbHc3ZlQ2ZjJUKzhJYWFyQ1owc0RtNUlCOG5na1FpMWY1VEVk?=
 =?utf-8?B?ejF5RVI2OXQ4YmxoMUNQMzJJb0NyN1dqc3EvWENKc3NyNXh5ajNnREl0aCth?=
 =?utf-8?B?UXV0ME1pc3RsdDhwd1JNczJtK2lMazNpZzJyaFEwRHpxLzRFMHE5a3B6SWdn?=
 =?utf-8?B?dkZ2TkNidmZ1Vlh4NUVYU09Kc3FISCtDVmdXQjZyZmNRTUJJV3ZScjFPRVBp?=
 =?utf-8?B?ZWZZcmhKSWxZSjFsNHZFMkJCb3E3Tk9CaFlwUU11VnpVOGJDK1ppZEZpUUJ3?=
 =?utf-8?B?dkQ2UXNWeUZCV1NIczU2NG5yV0p2bWZ5ckkwWktyQjNMdzVZRDFyQmE5R3FH?=
 =?utf-8?B?NVE9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: cac9dca0-78a3-4372-8323-08dc6f3f3036
X-MS-Exchange-CrossTenant-AuthSource: CYXPR11MB8712.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 08 May 2024 09:13:56.9782
 (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: ROY5jquDCJIQydFJuPpD9J0HJ418odiJsiQFTD4oPQ4FYZspuEkhMk7Y6UppwpEkeRTmnpk+KmbG/QHM6uoczefAvvWEgh7RYmm6Wp0H7zw=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH8PR11MB6904
X-OriginatorOrg: intel.com

From: Jakub Kicinski <kuba@kernel.org>
Date: Tue, 7 May 2024 11:10:35 -0700

> On Tue,  7 May 2024 14:39:37 +0200 Alexander Lobakin wrote:
>> There are several instances of the structure embedded into other
>> structures, but also there's ongoing effort to remove them and we
>> could in the meantime declare &net_device properly.
> 
> Is there a reason you're reposting this before that effort is completed?

To speed up the conversion probably :D

> The warnings this adds come from sparse and you think they should be
> ignored?

For now...

> 
> TBH since Breno is doing the heavy lifting of changing the embedders 
> it'd seem more fair to me if he got to send this at the end. Or at
> least, you know, got a mention or a CC.

I was lazy enough to add tags, sorry. The idea of him sending this at
the end sounds reasonable.

Thanks,
Olek