iommu: make kobj_type structure constant

Message ID 20230214-kobj_type-iommu-v1-1-e7392834b9d0@weissschuh.net
State New
Headers
Series iommu: make kobj_type structure constant |

Commit Message

Thomas Weißschuh Feb. 14, 2023, 3:25 a.m. UTC
  Since commit ee6d3dd4ed48 ("driver core: make kobj_type constant.")
the driver core allows the usage of const struct kobj_type.

Take advantage of this to constify the structure definition to prevent
modification at runtime.

Signed-off-by: Thomas Weißschuh <linux@weissschuh.net>
---
 drivers/iommu/iommu.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)


---
base-commit: f6feea56f66d34259c4222fa02e8171c4f2673d1
change-id: 20230214-kobj_type-iommu-9eede43d1075

Best regards,
  

Comments

Joerg Roedel March 22, 2023, 1:20 p.m. UTC | #1
On Tue, Feb 14, 2023 at 03:25:53AM +0000, Thomas Weißschuh wrote:
> Since commit ee6d3dd4ed48 ("driver core: make kobj_type constant.")
> the driver core allows the usage of const struct kobj_type.
> 
> Take advantage of this to constify the structure definition to prevent
> modification at runtime.
> 
> Signed-off-by: Thomas Weißschuh <linux@weissschuh.net>
> ---

Applied, thanks.

Btw, b4 reported DKIM errors checking your patch submission. Can you
check the DKIM setup on you email domain, please?
  
Thomas Weißschuh March 22, 2023, 10:17 p.m. UTC | #2
Hi Joerg,


Mar 22, 2023 08:20:14 Joerg Roedel <joro@8bytes.org>:

> On Tue, Feb 14, 2023 at 03:25:53AM +0000, Thomas Weißschuh wrote:
>> Since commit ee6d3dd4ed48 ("driver core: make kobj_type constant.")
>> the driver core allows the usage of const struct kobj_type.
>>
>> Take advantage of this to constify the structure definition to prevent
>> modification at runtime.
>>
>> Signed-off-by: Thomas Weißschuh <linux@weissschuh.net>
>> ---
>
> Applied, thanks.

Thanks!

> Btw, b4 reported DKIM errors checking your patch submission. Can you
> check the DKIM setup on you email domain, please?

Should be fixed for some time.
(After this patch, though)

Thanks for the hint!

Thomas
  

Patch

diff --git a/drivers/iommu/iommu.c b/drivers/iommu/iommu.c
index 5f6a85aea501..1dd4ec31301d 100644
--- a/drivers/iommu/iommu.c
+++ b/drivers/iommu/iommu.c
@@ -714,7 +714,7 @@  static void iommu_group_release(struct kobject *kobj)
 	kfree(group);
 }
 
-static struct kobj_type iommu_group_ktype = {
+static const struct kobj_type iommu_group_ktype = {
 	.sysfs_ops = &iommu_group_sysfs_ops,
 	.release = iommu_group_release,
 };