Received-SPF: pass (google.com: domain of linux-kernel+bounces-71761-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1;
Message-ID: <07d32ed5-825c-cee2-7551-e8442177dfc1@intel.com>
Date: Mon, 19 Feb 2024 09:30:59 -0800
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101
 Firefox/91.0 Thunderbird/91.11.0
Subject: Re: [PATCH] dmaengine: idxd: constify the struct device_type usage
Content-Language: en-US
To: "Ricardo B. Marliere" <ricardo@marliere.net>, Dave Jiang
	<dave.jiang@intel.com>, Vinod Koul <vkoul@kernel.org>
CC: <dmaengine@vger.kernel.org>, <linux-kernel@vger.kernel.org>, "Greg
 Kroah-Hartman" <gregkh@linuxfoundation.org>
References: <20240219-device_cleanup-dmaengine-v1-1-9f72f3cf3587@marliere.net>
From: Fenghua Yu <fenghua.yu@intel.com>
In-Reply-To: <20240219-device_cleanup-dmaengine-v1-1-9f72f3cf3587@marliere.net>
Content-Type: text/plain; charset="UTF-8"; format=flowed
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?Tkphamo5U2VsQlFSMWNidFIzNUh5Ny9zMnM0T3FqMlJEVG45SHdSVmdqd0RL?=
 =?utf-8?B?NGRnRVhWQjZtalZ3YzdkT0pEb0pvL1M3VW1pNkhsWUM4bXFnUTRPUnZRV3Jy?=
 =?utf-8?B?ckpPeXZjUWdrUDBDaHJ4RFZ5Syt1OVVLTGhUTXR0VC83c0xQZTVJbE5pRnQ3?=
 =?utf-8?B?WFl6NWVZQWZtUEdKaTNwRjI0WVJRVVBrRUE3OCtZSVVvbFhFdmo5dFZMMXls?=
 =?utf-8?B?UUpWVFNoMkxwTVBYY1AyNFAzWlR4aTFuZ1JzbjZ1dEVieHJScEdSWDBkbk45?=
 =?utf-8?B?QjY1bEI5QWRzQU1hcTV4L29hUG1HamRFSHZvRTgrUFFIL0Q1aU9sY1hUcEND?=
 =?utf-8?B?VnU3amp3b3F3MXBweldZaW5yOXU2c2ZTaVlmbDB5eElObzhXN003Sm5DYits?=
 =?utf-8?B?TUxVTnEvWWMyRlFhN0doeTlNUUVUdXN0OWkwS21aSjRRZ0FCaVdwUW5Uc1FI?=
 =?utf-8?B?alpob3VSeDg1M1ZyeHpjbnpCUDBzRXV5Z2RiSnpKQW9mdmd3eW12cTVQS0s2?=
 =?utf-8?B?UUswaVdiak9XSmRqOEc0K2t4WmE5aWQwL2RXMzhVeXJkRUJsSll5MTFBY3Zh?=
 =?utf-8?B?YWJDQ3VhOWZzQTVDYnhNYnREQ3NERG5mcDY5bGpYZ0JpQ3puMWJvZ2p0ZEtY?=
 =?utf-8?B?NHI2RVhWcGswQkcyN1U5bTZSRUNOU2pXdG1GY3lqUjNSUGVweDRIVlFrYU5y?=
 =?utf-8?B?U3BlWm9tNlBYbkJrckdrUHZFdHFVeUVBZlE0K0pRS2FodWJtdTJnWGtxSHJt?=
 =?utf-8?B?NjFqUGtFNk1mZWJRNXRhL1JocHhXbXN4aWhIWUVTTUZFOER4bGxXTWx1VnVk?=
 =?utf-8?B?UEdlbEloTXVoSEpzT0FJdWcycFdJK0EzSDFRbFNSUUxRbjVXWmpCVUVzUndC?=
 =?utf-8?B?eHVzZ1RJMVY4NXJzWWxmUUhDcmRDUmhoRWw0UGRhZVpJZ1VMTEM5d0I4SzZz?=
 =?utf-8?B?UVhkYVgyTzN1Wm44R3lKRzFlQldib0FuSjRtSm5RdFBTOWJ2ak9yM1JrR0Vk?=
 =?utf-8?B?NTFlUEVFT0xDNkF4alExUFB5QTVLMjI1ZWN6NDlOSDZYcXNFR1BwbFZOYzdT?=
 =?utf-8?B?aWVaeXdGMTJwRmNaK2NxT292ZW5hVmd3QlJ6M21CVU5ra0hNdTVvdEQra1lD?=
 =?utf-8?B?Z0diQjZGMVFYcTVSaU9CQTl3UU5Ud2pKc01GbXBXeS9EcG5VYVNkdjBSa213?=
 =?utf-8?B?bVpxR0pTZTh1RE51bFZvR2lrcVFMeDZoMEMrWWhabFcvanZxTXNnSDQyc00w?=
 =?utf-8?B?cVNzMjBCMC9IUy9zaVhSUlpuVUtsSFl5QkxjVDEzYVcrS1RpdnIwZi9ucUgv?=
 =?utf-8?B?Q1ZKazVaRC92T2NVMTlSWVoyTDl4cS9UT3k5K1RDaU1DdGlTY3VJdUp3eVZs?=
 =?utf-8?B?UVVLZkEyaHFkeXd6Yjd3cFRLNWFNVU1uYTVGWW5qTnM3LzRsZmdNVE5vQzdO?=
 =?utf-8?B?T2ZlNi83YS9NTDZ5Z1B0a0hiMklJMlVLR1JRNzRJbWd3NEphMVQ1WkQ5OWNS?=
 =?utf-8?B?Sk1xUUozdU9vYVV4eHU4djJKbFpnTG9CZ28zbEY5OEhCL2ZmaDNoZVlIQ3ZN?=
 =?utf-8?B?UW1lamY1ejlZOWM4QUY5aEc2U0xBUXVEaHBUaDI0MWFxSkg2RG5Wb1BQai83?=
 =?utf-8?B?cDFUVmFiazZ5djRmUW1vUzRPMFZPdERmcUdvMVF5V2JJczJCb3daSlhpbm9m?=
 =?utf-8?B?QmMzOE5FM2NTK2VTNGIwKzZNeDBiUklJNlJ5NDJKRHNKdEt0dHpUOVZVTmRS?=
 =?utf-8?B?UVFBcS9JZEVGNEk1KzlxVTU3enJCUjdiUGp5UkpoTlVIcjhOOHN3MDBFT0cv?=
 =?utf-8?B?b2NDMldYRmpSYUdscllMVEdUYjhhM2NMOTVTQXVZNU5TYjRkbWRIa3Q1VVlR?=
 =?utf-8?B?YUJjV2FsSm14Qll2Ykp6U0NyMDlGU1ovQVM5MXZEMzNXYlRXaXIxWXR3NU9P?=
 =?utf-8?B?MWlXcGVPUUdXcm5zM0VTbUZYZWFTcnl5VEFpazRGMldNSitSWHhVT2kxaGN4?=
 =?utf-8?B?SlhOeEdoczJESDRINHFSNzhxdG5jZU9GTEE3cFY3aFc0RmFTTFVBMlFsL3Uy?=
 =?utf-8?B?R1VScWZObjF5N2VMUTluVmlIOG9razFKdEwzdG83RTBSNXBXaGU5QVFnSFdY?=
 =?utf-8?Q?9amnyzf4wa04qkq/hQ9gqFF/L?=
X-MS-Exchange-CrossTenant-Network-Message-Id: feb149a5-ebc3-447a-2b51-08dc31709a26
X-MS-Exchange-CrossTenant-AuthSource: IA1PR11MB6097.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 19 Feb 2024 17:31:27.3814
 (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: G04BeuqrFHjLRZa81G5HfU/GgsW0Gq5Lq9T6WJ4eMG5gjF2KoYSXyzLvSq7lUv/fGbHnpYAeZDSu3IdcW6pwvg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CH0PR11MB5690
X-OriginatorOrg: intel.com


On 2/19/24 03:46, Ricardo B. Marliere wrote:
> Since commit aed65af1cc2f ("drivers: make device_type const"), the driver
> core can properly handle constant struct device_type. Move the
> dsa_device_type, iax_device_type, idxd_wq_device_type, idxd_cdev_file_type,
> idxd_cdev_device_type and idxd_group_device_type variables to be constant
> structures as well, placing it into read-only memory which can not be
> modified at runtime.
> 
> Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> Signed-off-by: Ricardo B. Marliere <ricardo@marliere.net>

Reviewed-by: Fenghua Yu <fenghua.yu@intel.com>

Thanks.

-Fenghua