USB: make usb class a const structure

Now that the driver core allows for struct class to be in read-only
memory, remove the usb_class structure and create the usbmisc_class
const class structure declared at build time which places it into
read-only memory, instead of having it to be dynamically allocated
at load time.

Additionally, now we register usb class at startup and unregister it
when shutting down, so we don't have to count uses of the class.
Therefore we don't need the 'usb_class' structure anymore. Due to this
fact, remove all static functions related to class initialization and
deinitialization. We can't use them in 'usb.c' since they are static
and we don't really need them anymore.

Since we have to register the class in usb_init function in 'usb.c'
and use it in 'file.c' as well, declare the usbmisc_class structure
as 'export' in the 'usb.h' file.

Debatable moment: the class registration and unregistration functions
could be extracted to the 'file.c'. I think we don't want to do this
since it would be one-line functions. They would make the code paths
more confusing and add calling overhead.

Suggested-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Ivan Orlov <ivan.orlov0322@gmail.com>
Link: https://lore.kernel.org/r/20230621202514.1223670-1-ivan.orlov0322@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

diff --git a/drivers/usb/core/file.c b/drivers/usb/core/file.c
index c4ed3310e06965cb789e173b397143e8ec5af7c1..a88ced93b5e71b6826d53cf20c246d23f3f95359 100644 (file)
--- a/drivers/usb/core/file.c
+++ b/drivers/usb/core/file.c
@@ -29,7 +29,6 @@
 #define MAX_USB_MINORS 256
 static const struct file_operations *usb_minors[MAX_USB_MINORS];
 static DECLARE_RWSEM(minor_rwsem);
-static DEFINE_MUTEX(init_usb_class_mutex);
 
 static int usb_open(struct inode *inode, struct file *file)
 {
@@ -57,11 +56,6 @@ static const struct file_operations usb_fops = {
        .llseek =       noop_llseek,
 };
 
-static struct usb_class {
-       struct kref kref;
-       struct class *class;
-} *usb_class;
-
 static char *usb_devnode(const struct device *dev, umode_t *mode)
 {
        struct usb_class_driver *drv;
@@ -72,50 +66,10 @@ static char *usb_devnode(const struct device *dev, umode_t *mode)
        return drv->devnode(dev, mode);
 }
 
-static int init_usb_class(void)
-{
-       int result = 0;
-
-       if (usb_class != NULL) {
-               kref_get(&usb_class->kref);
-               goto exit;
-       }
-
-       usb_class = kmalloc(sizeof(*usb_class), GFP_KERNEL);
-       if (!usb_class) {
-               result = -ENOMEM;
-               goto exit;
-       }
-
-       kref_init(&usb_class->kref);
-       usb_class->class = class_create("usbmisc");
-       if (IS_ERR(usb_class->class)) {
-               result = PTR_ERR(usb_class->class);
-               printk(KERN_ERR "class_create failed for usb devices\n");
-               kfree(usb_class);
-               usb_class = NULL;
-               goto exit;
-       }
-       usb_class->class->devnode = usb_devnode;
-
-exit:
-       return result;
-}
-
-static void release_usb_class(struct kref *kref)
-{
-       /* Ok, we cheat as we know we only have one usb_class */
-       class_destroy(usb_class->class);
-       kfree(usb_class);
-       usb_class = NULL;
-}
-
-static void destroy_usb_class(void)
-{
-       mutex_lock(&init_usb_class_mutex);
-       kref_put(&usb_class->kref, release_usb_class);
-       mutex_unlock(&init_usb_class_mutex);
-}
+const struct class usbmisc_class = {
+       .name           = "usbmisc",
+       .devnode        = usb_devnode,
+};
 
 int usb_major_init(void)
 {
@@ -156,7 +110,7 @@ void usb_major_cleanup(void)
 int usb_register_dev(struct usb_interface *intf,
                     struct usb_class_driver *class_driver)
 {
-       int retval;
+       int retval = 0;
        int minor_base = class_driver->minor_base;
        int minor;
        char name[20];
@@ -175,13 +129,6 @@ int usb_register_dev(struct usb_interface *intf,
        if (intf->minor >= 0)
                return -EADDRINUSE;
 
-       mutex_lock(&init_usb_class_mutex);
-       retval = init_usb_class();
-       mutex_unlock(&init_usb_class_mutex);
-
-       if (retval)
-               return retval;
-
        dev_dbg(&intf->dev, "looking for a minor, starting at %d\n", minor_base);
 
        down_write(&minor_rwsem);
@@ -200,7 +147,7 @@ int usb_register_dev(struct usb_interface *intf,
 
        /* create a usb class device for this usb interface */
        snprintf(name, sizeof(name), class_driver->name, minor - minor_base);
-       intf->usb_dev = device_create(usb_class->class, &intf->dev,
+       intf->usb_dev = device_create(&usbmisc_class, &intf->dev,
                                      MKDEV(USB_MAJOR, minor), class_driver,
                                      "%s", kbasename(name));
        if (IS_ERR(intf->usb_dev)) {
@@ -234,7 +181,7 @@ void usb_deregister_dev(struct usb_interface *intf,
                return;
 
        dev_dbg(&intf->dev, "removing %d minor\n", intf->minor);
-       device_destroy(usb_class->class, MKDEV(USB_MAJOR, intf->minor));
+       device_destroy(&usbmisc_class, MKDEV(USB_MAJOR, intf->minor));
 
        down_write(&minor_rwsem);
        usb_minors[intf->minor] = NULL;
@@ -242,6 +189,5 @@ void usb_deregister_dev(struct usb_interface *intf,
 
        intf->usb_dev = NULL;
        intf->minor = -1;
-       destroy_usb_class();
 }
 EXPORT_SYMBOL_GPL(usb_deregister_dev);

diff --git a/drivers/usb/core/usb.c b/drivers/usb/core/usb.c
index 901ec732321c5eccd3133b9c5ee55579110a85e4..39bdf9589b06b307cab79e36709a428c5574cc21 100644 (file)
--- a/drivers/usb/core/usb.c
+++ b/drivers/usb/core/usb.c
@@ -1101,6 +1101,9 @@ static int __init usb_init(void)
        retval = usb_major_init();
        if (retval)
                goto major_init_failed;
+       retval = class_register(&usbmisc_class);
+       if (retval)
+               goto class_register_failed;
        retval = usb_register(&usbfs_driver);
        if (retval)
                goto driver_register_failed;
@@ -1120,6 +1123,8 @@ hub_init_failed:
 usb_devio_init_failed:
        usb_deregister(&usbfs_driver);
 driver_register_failed:
+       class_unregister(&usbmisc_class);
+class_register_failed:
        usb_major_cleanup();
 major_init_failed:
        bus_unregister_notifier(&usb_bus_type, &usb_bus_nb);
@@ -1147,6 +1152,7 @@ static void __exit usb_exit(void)
        usb_deregister(&usbfs_driver);
        usb_devio_cleanup();
        usb_hub_cleanup();
+       class_unregister(&usbmisc_class);
        bus_unregister_notifier(&usb_bus_type, &usb_bus_nb);
        bus_unregister(&usb_bus_type);
        usb_acpi_unregister();

diff --git a/drivers/usb/core/usb.h b/drivers/usb/core/usb.h
index ffe3f6818e9cfa3b1202d42c2074fa2d9cf564e0..69ca59841083b360a12243d15e2680b4d510b491 100644 (file)
--- a/drivers/usb/core/usb.h
+++ b/drivers/usb/core/usb.h
@@ -141,6 +141,7 @@ static inline int usb_disable_usb2_hardware_lpm(struct usb_device *udev)
 
 #endif
 
+extern const struct class usbmisc_class;
 extern const struct bus_type usb_bus_type;
 extern struct mutex usb_port_peer_mutex;
 extern struct device_type usb_device_type;