Skip to content

Commit

Permalink
driver core: class: use lock_class_key already present in struct subs…
Browse files Browse the repository at this point in the history
…ys_private

In commit 37e98d9 ("driver core: bus: move lock_class_key into
dynamic structure"), we moved the lock_class_key into the internal
structure shared by busses and classes, but only used it for buses.

Move the class code to use this structure as it is already present and
being allocated, instead of the statically allocated on-the-stack
variable that class_create() was using as part of a macro wrapper around
the core function call.

Reviewed-by: Rafael J. Wysocki <rafael@kernel.org>
Link: https://lore.kernel.org/r/20230324100132.1633647-1-gregkh@linuxfoundation.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
  • Loading branch information
gregkh committed Mar 24, 2023
1 parent 5b9ff0b commit dcfbb67
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 40 deletions.
15 changes: 9 additions & 6 deletions drivers/base/class.c
Original file line number Diff line number Diff line change
Expand Up @@ -154,9 +154,10 @@ static void class_remove_groups(struct class *cls,
return sysfs_remove_groups(&cls->p->subsys.kobj, groups);
}

int __class_register(struct class *cls, struct lock_class_key *key)
int class_register(struct class *cls)
{
struct subsys_private *cp;
struct lock_class_key *key;
int error;

pr_debug("device class '%s': registering\n", cls->name);
Expand All @@ -167,6 +168,8 @@ int __class_register(struct class *cls, struct lock_class_key *key)
klist_init(&cp->klist_devices, klist_class_dev_get, klist_class_dev_put);
INIT_LIST_HEAD(&cp->interfaces);
kset_init(&cp->glue_dirs);
key = &cp->lock_key;
lockdep_register_key(key);
__mutex_init(&cp->mutex, "subsys mutex", key);
error = kobject_set_name(&cp->subsys.kobj, "%s", cls->name);
if (error) {
Expand Down Expand Up @@ -201,7 +204,7 @@ int __class_register(struct class *cls, struct lock_class_key *key)
cls->p = NULL;
return error;
}
EXPORT_SYMBOL_GPL(__class_register);
EXPORT_SYMBOL_GPL(class_register);

void class_unregister(struct class *cls)
{
Expand All @@ -218,7 +221,7 @@ static void class_create_release(struct class *cls)
}

/**
* __class_create - create a struct class structure
* class_create - create a struct class structure
* @name: pointer to a string for the name of this class.
* @key: the lock_class_key for this class; used by mutex lock debugging
*
Expand All @@ -230,7 +233,7 @@ static void class_create_release(struct class *cls)
* Note, the pointer created here is to be destroyed when finished by
* making a call to class_destroy().
*/
struct class *__class_create(const char *name, struct lock_class_key *key)
struct class *class_create(const char *name)
{
struct class *cls;
int retval;
Expand All @@ -244,7 +247,7 @@ struct class *__class_create(const char *name, struct lock_class_key *key)
cls->name = name;
cls->class_release = class_create_release;

retval = __class_register(cls, key);
retval = class_register(cls);
if (retval)
goto error;

Expand All @@ -254,7 +257,7 @@ struct class *__class_create(const char *name, struct lock_class_key *key)
kfree(cls);
return ERR_PTR(retval);
}
EXPORT_SYMBOL_GPL(__class_create);
EXPORT_SYMBOL_GPL(class_create);

/**
* class_destroy - destroys a struct class structure
Expand Down
36 changes: 2 additions & 34 deletions include/linux/device/class.h
Original file line number Diff line number Diff line change
Expand Up @@ -82,18 +82,9 @@ struct class_dev_iter {

extern struct kobject *sysfs_dev_block_kobj;
extern struct kobject *sysfs_dev_char_kobj;
extern int __must_check __class_register(struct class *class,
struct lock_class_key *key);
extern int __must_check class_register(struct class *class);
extern void class_unregister(struct class *class);

/* This is a #define to keep the compiler from merging different
* instances of the __key variable */
#define class_register(class) \
({ \
static struct lock_class_key __key; \
__class_register(class, &__key); \
})

struct class_compat;
struct class_compat *class_compat_register(const char *name);
void class_compat_unregister(struct class_compat *cls);
Expand Down Expand Up @@ -246,30 +237,7 @@ struct class_interface {
extern int __must_check class_interface_register(struct class_interface *);
extern void class_interface_unregister(struct class_interface *);

extern struct class * __must_check __class_create(const char *name,
struct lock_class_key *key);
extern struct class * __must_check class_create(const char *name);
extern void class_destroy(struct class *cls);

/* This is a #define to keep the compiler from merging different
* instances of the __key variable */

/**
* class_create - create a struct class structure
* @name: pointer to a string for the name of this class.
*
* This is used to create a struct class pointer that can then be used
* in calls to device_create().
*
* Returns &struct class pointer on success, or ERR_PTR() on error.
*
* Note, the pointer created here is to be destroyed when finished by
* making a call to class_destroy().
*/
#define class_create(name) \
({ \
static struct lock_class_key __key; \
__class_create(name, &__key); \
})


#endif /* _DEVICE_CLASS_H_ */

0 comments on commit dcfbb67

Please sign in to comment.