* [PATCH] kobject: remove kset from struct kset_uevent_ops callbacks
@ 2021-12-27 16:39 Greg Kroah-Hartman
2021-12-27 16:43 ` Rafael J. Wysocki
2021-12-27 20:41 ` Wedson Almeida Filho
0 siblings, 2 replies; 4+ messages in thread
From: Greg Kroah-Hartman @ 2021-12-27 16:39 UTC (permalink / raw)
To: linux-kernel; +Cc: Greg Kroah-Hartman, Rafael J . Wysocki, Wedson Almeida Filho
There is no need to pass the pointer to the kset in the struct
kset_uevent_ops callbacks as no one uses it, so just remove that pointer
entirely.
Cc: Rafael J. Wysocki <rafael@kernel.org>
Cc: Wedson Almeida Filho <wedsonaf@google.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
---
Documentation/core-api/kobject.rst | 7 +++----
Documentation/translations/zh_CN/core-api/kobject.rst | 7 +++----
drivers/base/bus.c | 2 +-
drivers/base/core.c | 11 +++++------
drivers/dma-buf/dma-buf-sysfs-stats.c | 2 +-
fs/dlm/lockspace.c | 3 +--
fs/gfs2/sys.c | 3 +--
include/linux/kobject.h | 7 +++----
kernel/params.c | 2 +-
lib/kobject_uevent.c | 6 +++---
10 files changed, 22 insertions(+), 28 deletions(-)
diff --git a/Documentation/core-api/kobject.rst b/Documentation/core-api/kobject.rst
index d3b5bf9f643a..3d6e3107315d 100644
--- a/Documentation/core-api/kobject.rst
+++ b/Documentation/core-api/kobject.rst
@@ -373,10 +373,9 @@ If a kset wishes to control the uevent operations of the kobjects
associated with it, it can use the struct kset_uevent_ops to handle it::
struct kset_uevent_ops {
- int (* const filter)(struct kset *kset, struct kobject *kobj);
- const char *(* const name)(struct kset *kset, struct kobject *kobj);
- int (* const uevent)(struct kset *kset, struct kobject *kobj,
- struct kobj_uevent_env *env);
+ int (* const filter)(struct kobject *kobj);
+ const char *(* const name)(struct kobject *kobj);
+ int (* const uevent)(struct kobject *kobj, struct kobj_uevent_env *env);
};
diff --git a/Documentation/translations/zh_CN/core-api/kobject.rst b/Documentation/translations/zh_CN/core-api/kobject.rst
index b7c37794cc7f..95634083dca0 100644
--- a/Documentation/translations/zh_CN/core-api/kobject.rst
+++ b/Documentation/translations/zh_CN/core-api/kobject.rst
@@ -325,10 +325,9 @@ ksets
结构体kset_uevent_ops来处理它::
struct kset_uevent_ops {
- int (* const filter)(struct kset *kset, struct kobject *kobj);
- const char *(* const name)(struct kset *kset, struct kobject *kobj);
- int (* const uevent)(struct kset *kset, struct kobject *kobj,
- struct kobj_uevent_env *env);
+ int (* const filter)(struct kobject *kobj);
+ const char *(* const name)(struct kobject *kobj);
+ int (* const uevent)(struct kobject *kobj, struct kobj_uevent_env *env);
};
diff --git a/drivers/base/bus.c b/drivers/base/bus.c
index a64454f5f8c0..97936ec49bde 100644
--- a/drivers/base/bus.c
+++ b/drivers/base/bus.c
@@ -163,7 +163,7 @@ static struct kobj_type bus_ktype = {
.release = bus_release,
};
-static int bus_uevent_filter(struct kset *kset, struct kobject *kobj)
+static int bus_uevent_filter(struct kobject *kobj)
{
const struct kobj_type *ktype = get_ktype(kobj);
diff --git a/drivers/base/core.c b/drivers/base/core.c
index d712ea11066b..60d703ebd123 100644
--- a/drivers/base/core.c
+++ b/drivers/base/core.c
@@ -2261,7 +2261,7 @@ static struct kobj_type device_ktype = {
};
-static int dev_uevent_filter(struct kset *kset, struct kobject *kobj)
+static int dev_uevent_filter(struct kobject *kobj)
{
const struct kobj_type *ktype = get_ktype(kobj);
@@ -2275,7 +2275,7 @@ static int dev_uevent_filter(struct kset *kset, struct kobject *kobj)
return 0;
}
-static const char *dev_uevent_name(struct kset *kset, struct kobject *kobj)
+static const char *dev_uevent_name(struct kobject *kobj)
{
struct device *dev = kobj_to_dev(kobj);
@@ -2286,8 +2286,7 @@ static const char *dev_uevent_name(struct kset *kset, struct kobject *kobj)
return NULL;
}
-static int dev_uevent(struct kset *kset, struct kobject *kobj,
- struct kobj_uevent_env *env)
+static int dev_uevent(struct kobject *kobj, struct kobj_uevent_env *env)
{
struct device *dev = kobj_to_dev(kobj);
int retval = 0;
@@ -2382,7 +2381,7 @@ static ssize_t uevent_show(struct device *dev, struct device_attribute *attr,
/* respect filter */
if (kset->uevent_ops && kset->uevent_ops->filter)
- if (!kset->uevent_ops->filter(kset, &dev->kobj))
+ if (!kset->uevent_ops->filter(&dev->kobj))
goto out;
env = kzalloc(sizeof(struct kobj_uevent_env), GFP_KERNEL);
@@ -2390,7 +2389,7 @@ static ssize_t uevent_show(struct device *dev, struct device_attribute *attr,
return -ENOMEM;
/* let the kset specific function add its keys */
- retval = kset->uevent_ops->uevent(kset, &dev->kobj, env);
+ retval = kset->uevent_ops->uevent(&dev->kobj, env);
if (retval)
goto out;
diff --git a/drivers/dma-buf/dma-buf-sysfs-stats.c b/drivers/dma-buf/dma-buf-sysfs-stats.c
index 053baadcada9..2bba0babcb62 100644
--- a/drivers/dma-buf/dma-buf-sysfs-stats.c
+++ b/drivers/dma-buf/dma-buf-sysfs-stats.c
@@ -132,7 +132,7 @@ void dma_buf_stats_teardown(struct dma_buf *dmabuf)
/* Statistics files do not need to send uevents. */
-static int dmabuf_sysfs_uevent_filter(struct kset *kset, struct kobject *kobj)
+static int dmabuf_sysfs_uevent_filter(struct kobject *kobj)
{
return 0;
}
diff --git a/fs/dlm/lockspace.c b/fs/dlm/lockspace.c
index 10eddfa6c3d7..0bbb346cb892 100644
--- a/fs/dlm/lockspace.c
+++ b/fs/dlm/lockspace.c
@@ -216,8 +216,7 @@ static int do_uevent(struct dlm_ls *ls, int in)
return ls->ls_uevent_result;
}
-static int dlm_uevent(struct kset *kset, struct kobject *kobj,
- struct kobj_uevent_env *env)
+static int dlm_uevent(struct kobject *kobj, struct kobj_uevent_env *env)
{
struct dlm_ls *ls = container_of(kobj, struct dlm_ls, ls_kobj);
diff --git a/fs/gfs2/sys.c b/fs/gfs2/sys.c
index c0a34d9ddee4..a6002b2d146d 100644
--- a/fs/gfs2/sys.c
+++ b/fs/gfs2/sys.c
@@ -767,8 +767,7 @@ void gfs2_sys_fs_del(struct gfs2_sbd *sdp)
wait_for_completion(&sdp->sd_kobj_unregister);
}
-static int gfs2_uevent(struct kset *kset, struct kobject *kobj,
- struct kobj_uevent_env *env)
+static int gfs2_uevent(struct kobject *kobj, struct kobj_uevent_env *env)
{
struct gfs2_sbd *sdp = container_of(kobj, struct gfs2_sbd, sd_kobj);
struct super_block *s = sdp->sd_vfs;
diff --git a/include/linux/kobject.h b/include/linux/kobject.h
index 683172b2e094..ad90b49824dc 100644
--- a/include/linux/kobject.h
+++ b/include/linux/kobject.h
@@ -153,10 +153,9 @@ struct kobj_uevent_env {
};
struct kset_uevent_ops {
- int (* const filter)(struct kset *kset, struct kobject *kobj);
- const char *(* const name)(struct kset *kset, struct kobject *kobj);
- int (* const uevent)(struct kset *kset, struct kobject *kobj,
- struct kobj_uevent_env *env);
+ int (* const filter)(struct kobject *kobj);
+ const char *(* const name)(struct kobject *kobj);
+ int (* const uevent)(struct kobject *kobj, struct kobj_uevent_env *env);
};
struct kobj_attribute {
diff --git a/kernel/params.c b/kernel/params.c
index 9b90e3c4d3c0..5b92310425c5 100644
--- a/kernel/params.c
+++ b/kernel/params.c
@@ -926,7 +926,7 @@ static const struct sysfs_ops module_sysfs_ops = {
.store = module_attr_store,
};
-static int uevent_filter(struct kset *kset, struct kobject *kobj)
+static int uevent_filter(struct kobject *kobj)
{
const struct kobj_type *ktype = get_ktype(kobj);
diff --git a/lib/kobject_uevent.c b/lib/kobject_uevent.c
index c87d5b6a8a55..7c44b7ae4c5c 100644
--- a/lib/kobject_uevent.c
+++ b/lib/kobject_uevent.c
@@ -501,7 +501,7 @@ int kobject_uevent_env(struct kobject *kobj, enum kobject_action action,
}
/* skip the event, if the filter returns zero. */
if (uevent_ops && uevent_ops->filter)
- if (!uevent_ops->filter(kset, kobj)) {
+ if (!uevent_ops->filter(kobj)) {
pr_debug("kobject: '%s' (%p): %s: filter function "
"caused the event to drop!\n",
kobject_name(kobj), kobj, __func__);
@@ -510,7 +510,7 @@ int kobject_uevent_env(struct kobject *kobj, enum kobject_action action,
/* originating subsystem */
if (uevent_ops && uevent_ops->name)
- subsystem = uevent_ops->name(kset, kobj);
+ subsystem = uevent_ops->name(kobj);
else
subsystem = kobject_name(&kset->kobj);
if (!subsystem) {
@@ -554,7 +554,7 @@ int kobject_uevent_env(struct kobject *kobj, enum kobject_action action,
/* let the kset specific function add its stuff */
if (uevent_ops && uevent_ops->uevent) {
- retval = uevent_ops->uevent(kset, kobj, env);
+ retval = uevent_ops->uevent(kobj, env);
if (retval) {
pr_debug("kobject: '%s' (%p): %s: uevent() returned "
"%d\n", kobject_name(kobj), kobj,
--
2.34.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] kobject: remove kset from struct kset_uevent_ops callbacks
2021-12-27 16:39 [PATCH] kobject: remove kset from struct kset_uevent_ops callbacks Greg Kroah-Hartman
@ 2021-12-27 16:43 ` Rafael J. Wysocki
2021-12-27 20:41 ` Wedson Almeida Filho
1 sibling, 0 replies; 4+ messages in thread
From: Rafael J. Wysocki @ 2021-12-27 16:43 UTC (permalink / raw)
To: Greg Kroah-Hartman
Cc: Linux Kernel Mailing List, Rafael J . Wysocki, Wedson Almeida Filho
On Mon, Dec 27, 2021 at 5:39 PM Greg Kroah-Hartman
<gregkh@linuxfoundation.org> wrote:
>
> There is no need to pass the pointer to the kset in the struct
> kset_uevent_ops callbacks as no one uses it, so just remove that pointer
> entirely.
>
> Cc: Rafael J. Wysocki <rafael@kernel.org>
> Cc: Wedson Almeida Filho <wedsonaf@google.com>
> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Reviewed-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
> ---
> Documentation/core-api/kobject.rst | 7 +++----
> Documentation/translations/zh_CN/core-api/kobject.rst | 7 +++----
> drivers/base/bus.c | 2 +-
> drivers/base/core.c | 11 +++++------
> drivers/dma-buf/dma-buf-sysfs-stats.c | 2 +-
> fs/dlm/lockspace.c | 3 +--
> fs/gfs2/sys.c | 3 +--
> include/linux/kobject.h | 7 +++----
> kernel/params.c | 2 +-
> lib/kobject_uevent.c | 6 +++---
> 10 files changed, 22 insertions(+), 28 deletions(-)
>
> diff --git a/Documentation/core-api/kobject.rst b/Documentation/core-api/kobject.rst
> index d3b5bf9f643a..3d6e3107315d 100644
> --- a/Documentation/core-api/kobject.rst
> +++ b/Documentation/core-api/kobject.rst
> @@ -373,10 +373,9 @@ If a kset wishes to control the uevent operations of the kobjects
> associated with it, it can use the struct kset_uevent_ops to handle it::
>
> struct kset_uevent_ops {
> - int (* const filter)(struct kset *kset, struct kobject *kobj);
> - const char *(* const name)(struct kset *kset, struct kobject *kobj);
> - int (* const uevent)(struct kset *kset, struct kobject *kobj,
> - struct kobj_uevent_env *env);
> + int (* const filter)(struct kobject *kobj);
> + const char *(* const name)(struct kobject *kobj);
> + int (* const uevent)(struct kobject *kobj, struct kobj_uevent_env *env);
> };
>
>
> diff --git a/Documentation/translations/zh_CN/core-api/kobject.rst b/Documentation/translations/zh_CN/core-api/kobject.rst
> index b7c37794cc7f..95634083dca0 100644
> --- a/Documentation/translations/zh_CN/core-api/kobject.rst
> +++ b/Documentation/translations/zh_CN/core-api/kobject.rst
> @@ -325,10 +325,9 @@ ksets
> 结构体kset_uevent_ops来处理它::
>
> struct kset_uevent_ops {
> - int (* const filter)(struct kset *kset, struct kobject *kobj);
> - const char *(* const name)(struct kset *kset, struct kobject *kobj);
> - int (* const uevent)(struct kset *kset, struct kobject *kobj,
> - struct kobj_uevent_env *env);
> + int (* const filter)(struct kobject *kobj);
> + const char *(* const name)(struct kobject *kobj);
> + int (* const uevent)(struct kobject *kobj, struct kobj_uevent_env *env);
> };
>
>
> diff --git a/drivers/base/bus.c b/drivers/base/bus.c
> index a64454f5f8c0..97936ec49bde 100644
> --- a/drivers/base/bus.c
> +++ b/drivers/base/bus.c
> @@ -163,7 +163,7 @@ static struct kobj_type bus_ktype = {
> .release = bus_release,
> };
>
> -static int bus_uevent_filter(struct kset *kset, struct kobject *kobj)
> +static int bus_uevent_filter(struct kobject *kobj)
> {
> const struct kobj_type *ktype = get_ktype(kobj);
>
> diff --git a/drivers/base/core.c b/drivers/base/core.c
> index d712ea11066b..60d703ebd123 100644
> --- a/drivers/base/core.c
> +++ b/drivers/base/core.c
> @@ -2261,7 +2261,7 @@ static struct kobj_type device_ktype = {
> };
>
>
> -static int dev_uevent_filter(struct kset *kset, struct kobject *kobj)
> +static int dev_uevent_filter(struct kobject *kobj)
> {
> const struct kobj_type *ktype = get_ktype(kobj);
>
> @@ -2275,7 +2275,7 @@ static int dev_uevent_filter(struct kset *kset, struct kobject *kobj)
> return 0;
> }
>
> -static const char *dev_uevent_name(struct kset *kset, struct kobject *kobj)
> +static const char *dev_uevent_name(struct kobject *kobj)
> {
> struct device *dev = kobj_to_dev(kobj);
>
> @@ -2286,8 +2286,7 @@ static const char *dev_uevent_name(struct kset *kset, struct kobject *kobj)
> return NULL;
> }
>
> -static int dev_uevent(struct kset *kset, struct kobject *kobj,
> - struct kobj_uevent_env *env)
> +static int dev_uevent(struct kobject *kobj, struct kobj_uevent_env *env)
> {
> struct device *dev = kobj_to_dev(kobj);
> int retval = 0;
> @@ -2382,7 +2381,7 @@ static ssize_t uevent_show(struct device *dev, struct device_attribute *attr,
>
> /* respect filter */
> if (kset->uevent_ops && kset->uevent_ops->filter)
> - if (!kset->uevent_ops->filter(kset, &dev->kobj))
> + if (!kset->uevent_ops->filter(&dev->kobj))
> goto out;
>
> env = kzalloc(sizeof(struct kobj_uevent_env), GFP_KERNEL);
> @@ -2390,7 +2389,7 @@ static ssize_t uevent_show(struct device *dev, struct device_attribute *attr,
> return -ENOMEM;
>
> /* let the kset specific function add its keys */
> - retval = kset->uevent_ops->uevent(kset, &dev->kobj, env);
> + retval = kset->uevent_ops->uevent(&dev->kobj, env);
> if (retval)
> goto out;
>
> diff --git a/drivers/dma-buf/dma-buf-sysfs-stats.c b/drivers/dma-buf/dma-buf-sysfs-stats.c
> index 053baadcada9..2bba0babcb62 100644
> --- a/drivers/dma-buf/dma-buf-sysfs-stats.c
> +++ b/drivers/dma-buf/dma-buf-sysfs-stats.c
> @@ -132,7 +132,7 @@ void dma_buf_stats_teardown(struct dma_buf *dmabuf)
>
>
> /* Statistics files do not need to send uevents. */
> -static int dmabuf_sysfs_uevent_filter(struct kset *kset, struct kobject *kobj)
> +static int dmabuf_sysfs_uevent_filter(struct kobject *kobj)
> {
> return 0;
> }
> diff --git a/fs/dlm/lockspace.c b/fs/dlm/lockspace.c
> index 10eddfa6c3d7..0bbb346cb892 100644
> --- a/fs/dlm/lockspace.c
> +++ b/fs/dlm/lockspace.c
> @@ -216,8 +216,7 @@ static int do_uevent(struct dlm_ls *ls, int in)
> return ls->ls_uevent_result;
> }
>
> -static int dlm_uevent(struct kset *kset, struct kobject *kobj,
> - struct kobj_uevent_env *env)
> +static int dlm_uevent(struct kobject *kobj, struct kobj_uevent_env *env)
> {
> struct dlm_ls *ls = container_of(kobj, struct dlm_ls, ls_kobj);
>
> diff --git a/fs/gfs2/sys.c b/fs/gfs2/sys.c
> index c0a34d9ddee4..a6002b2d146d 100644
> --- a/fs/gfs2/sys.c
> +++ b/fs/gfs2/sys.c
> @@ -767,8 +767,7 @@ void gfs2_sys_fs_del(struct gfs2_sbd *sdp)
> wait_for_completion(&sdp->sd_kobj_unregister);
> }
>
> -static int gfs2_uevent(struct kset *kset, struct kobject *kobj,
> - struct kobj_uevent_env *env)
> +static int gfs2_uevent(struct kobject *kobj, struct kobj_uevent_env *env)
> {
> struct gfs2_sbd *sdp = container_of(kobj, struct gfs2_sbd, sd_kobj);
> struct super_block *s = sdp->sd_vfs;
> diff --git a/include/linux/kobject.h b/include/linux/kobject.h
> index 683172b2e094..ad90b49824dc 100644
> --- a/include/linux/kobject.h
> +++ b/include/linux/kobject.h
> @@ -153,10 +153,9 @@ struct kobj_uevent_env {
> };
>
> struct kset_uevent_ops {
> - int (* const filter)(struct kset *kset, struct kobject *kobj);
> - const char *(* const name)(struct kset *kset, struct kobject *kobj);
> - int (* const uevent)(struct kset *kset, struct kobject *kobj,
> - struct kobj_uevent_env *env);
> + int (* const filter)(struct kobject *kobj);
> + const char *(* const name)(struct kobject *kobj);
> + int (* const uevent)(struct kobject *kobj, struct kobj_uevent_env *env);
> };
>
> struct kobj_attribute {
> diff --git a/kernel/params.c b/kernel/params.c
> index 9b90e3c4d3c0..5b92310425c5 100644
> --- a/kernel/params.c
> +++ b/kernel/params.c
> @@ -926,7 +926,7 @@ static const struct sysfs_ops module_sysfs_ops = {
> .store = module_attr_store,
> };
>
> -static int uevent_filter(struct kset *kset, struct kobject *kobj)
> +static int uevent_filter(struct kobject *kobj)
> {
> const struct kobj_type *ktype = get_ktype(kobj);
>
> diff --git a/lib/kobject_uevent.c b/lib/kobject_uevent.c
> index c87d5b6a8a55..7c44b7ae4c5c 100644
> --- a/lib/kobject_uevent.c
> +++ b/lib/kobject_uevent.c
> @@ -501,7 +501,7 @@ int kobject_uevent_env(struct kobject *kobj, enum kobject_action action,
> }
> /* skip the event, if the filter returns zero. */
> if (uevent_ops && uevent_ops->filter)
> - if (!uevent_ops->filter(kset, kobj)) {
> + if (!uevent_ops->filter(kobj)) {
> pr_debug("kobject: '%s' (%p): %s: filter function "
> "caused the event to drop!\n",
> kobject_name(kobj), kobj, __func__);
> @@ -510,7 +510,7 @@ int kobject_uevent_env(struct kobject *kobj, enum kobject_action action,
>
> /* originating subsystem */
> if (uevent_ops && uevent_ops->name)
> - subsystem = uevent_ops->name(kset, kobj);
> + subsystem = uevent_ops->name(kobj);
> else
> subsystem = kobject_name(&kset->kobj);
> if (!subsystem) {
> @@ -554,7 +554,7 @@ int kobject_uevent_env(struct kobject *kobj, enum kobject_action action,
>
> /* let the kset specific function add its stuff */
> if (uevent_ops && uevent_ops->uevent) {
> - retval = uevent_ops->uevent(kset, kobj, env);
> + retval = uevent_ops->uevent(kobj, env);
> if (retval) {
> pr_debug("kobject: '%s' (%p): %s: uevent() returned "
> "%d\n", kobject_name(kobj), kobj,
> --
> 2.34.1
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] kobject: remove kset from struct kset_uevent_ops callbacks
2021-12-27 16:39 [PATCH] kobject: remove kset from struct kset_uevent_ops callbacks Greg Kroah-Hartman
2021-12-27 16:43 ` Rafael J. Wysocki
@ 2021-12-27 20:41 ` Wedson Almeida Filho
2021-12-28 10:08 ` Greg Kroah-Hartman
1 sibling, 1 reply; 4+ messages in thread
From: Wedson Almeida Filho @ 2021-12-27 20:41 UTC (permalink / raw)
To: Greg Kroah-Hartman; +Cc: linux-kernel, Rafael J . Wysocki
On Mon, Dec 27, 2021 at 05:39:24PM +0100, Greg Kroah-Hartman wrote:
> There is no need to pass the pointer to the kset in the struct
> kset_uevent_ops callbacks as no one uses it, so just remove that pointer
> entirely.
In the future, if someone needs the kset they can always find it by looking for
the first non-null kset in the kobject or its parent chain, right? IOW, we're
not losing any information by removing this.
>
> Cc: Rafael J. Wysocki <rafael@kernel.org>
> Cc: Wedson Almeida Filho <wedsonaf@google.com>
> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Reviewed-by: Wedson Almeida Filho <wedsonaf@google.com>
> ---
> Documentation/core-api/kobject.rst | 7 +++----
> Documentation/translations/zh_CN/core-api/kobject.rst | 7 +++----
> drivers/base/bus.c | 2 +-
> drivers/base/core.c | 11 +++++------
> drivers/dma-buf/dma-buf-sysfs-stats.c | 2 +-
> fs/dlm/lockspace.c | 3 +--
> fs/gfs2/sys.c | 3 +--
> include/linux/kobject.h | 7 +++----
> kernel/params.c | 2 +-
> lib/kobject_uevent.c | 6 +++---
> 10 files changed, 22 insertions(+), 28 deletions(-)
>
> diff --git a/Documentation/core-api/kobject.rst b/Documentation/core-api/kobject.rst
> index d3b5bf9f643a..3d6e3107315d 100644
> --- a/Documentation/core-api/kobject.rst
> +++ b/Documentation/core-api/kobject.rst
> @@ -373,10 +373,9 @@ If a kset wishes to control the uevent operations of the kobjects
> associated with it, it can use the struct kset_uevent_ops to handle it::
>
> struct kset_uevent_ops {
> - int (* const filter)(struct kset *kset, struct kobject *kobj);
> - const char *(* const name)(struct kset *kset, struct kobject *kobj);
> - int (* const uevent)(struct kset *kset, struct kobject *kobj,
> - struct kobj_uevent_env *env);
> + int (* const filter)(struct kobject *kobj);
> + const char *(* const name)(struct kobject *kobj);
> + int (* const uevent)(struct kobject *kobj, struct kobj_uevent_env *env);
> };
>
>
> diff --git a/Documentation/translations/zh_CN/core-api/kobject.rst b/Documentation/translations/zh_CN/core-api/kobject.rst
> index b7c37794cc7f..95634083dca0 100644
> --- a/Documentation/translations/zh_CN/core-api/kobject.rst
> +++ b/Documentation/translations/zh_CN/core-api/kobject.rst
> @@ -325,10 +325,9 @@ ksets
> 结构体kset_uevent_ops来处理它::
>
> struct kset_uevent_ops {
> - int (* const filter)(struct kset *kset, struct kobject *kobj);
> - const char *(* const name)(struct kset *kset, struct kobject *kobj);
> - int (* const uevent)(struct kset *kset, struct kobject *kobj,
> - struct kobj_uevent_env *env);
> + int (* const filter)(struct kobject *kobj);
> + const char *(* const name)(struct kobject *kobj);
> + int (* const uevent)(struct kobject *kobj, struct kobj_uevent_env *env);
> };
>
>
> diff --git a/drivers/base/bus.c b/drivers/base/bus.c
> index a64454f5f8c0..97936ec49bde 100644
> --- a/drivers/base/bus.c
> +++ b/drivers/base/bus.c
> @@ -163,7 +163,7 @@ static struct kobj_type bus_ktype = {
> .release = bus_release,
> };
>
> -static int bus_uevent_filter(struct kset *kset, struct kobject *kobj)
> +static int bus_uevent_filter(struct kobject *kobj)
> {
> const struct kobj_type *ktype = get_ktype(kobj);
>
> diff --git a/drivers/base/core.c b/drivers/base/core.c
> index d712ea11066b..60d703ebd123 100644
> --- a/drivers/base/core.c
> +++ b/drivers/base/core.c
> @@ -2261,7 +2261,7 @@ static struct kobj_type device_ktype = {
> };
>
>
> -static int dev_uevent_filter(struct kset *kset, struct kobject *kobj)
> +static int dev_uevent_filter(struct kobject *kobj)
> {
> const struct kobj_type *ktype = get_ktype(kobj);
>
> @@ -2275,7 +2275,7 @@ static int dev_uevent_filter(struct kset *kset, struct kobject *kobj)
> return 0;
> }
>
> -static const char *dev_uevent_name(struct kset *kset, struct kobject *kobj)
> +static const char *dev_uevent_name(struct kobject *kobj)
> {
> struct device *dev = kobj_to_dev(kobj);
>
> @@ -2286,8 +2286,7 @@ static const char *dev_uevent_name(struct kset *kset, struct kobject *kobj)
> return NULL;
> }
>
> -static int dev_uevent(struct kset *kset, struct kobject *kobj,
> - struct kobj_uevent_env *env)
> +static int dev_uevent(struct kobject *kobj, struct kobj_uevent_env *env)
> {
> struct device *dev = kobj_to_dev(kobj);
> int retval = 0;
> @@ -2382,7 +2381,7 @@ static ssize_t uevent_show(struct device *dev, struct device_attribute *attr,
>
> /* respect filter */
> if (kset->uevent_ops && kset->uevent_ops->filter)
> - if (!kset->uevent_ops->filter(kset, &dev->kobj))
> + if (!kset->uevent_ops->filter(&dev->kobj))
> goto out;
>
> env = kzalloc(sizeof(struct kobj_uevent_env), GFP_KERNEL);
> @@ -2390,7 +2389,7 @@ static ssize_t uevent_show(struct device *dev, struct device_attribute *attr,
> return -ENOMEM;
>
> /* let the kset specific function add its keys */
> - retval = kset->uevent_ops->uevent(kset, &dev->kobj, env);
> + retval = kset->uevent_ops->uevent(&dev->kobj, env);
> if (retval)
> goto out;
>
> diff --git a/drivers/dma-buf/dma-buf-sysfs-stats.c b/drivers/dma-buf/dma-buf-sysfs-stats.c
> index 053baadcada9..2bba0babcb62 100644
> --- a/drivers/dma-buf/dma-buf-sysfs-stats.c
> +++ b/drivers/dma-buf/dma-buf-sysfs-stats.c
> @@ -132,7 +132,7 @@ void dma_buf_stats_teardown(struct dma_buf *dmabuf)
>
>
> /* Statistics files do not need to send uevents. */
> -static int dmabuf_sysfs_uevent_filter(struct kset *kset, struct kobject *kobj)
> +static int dmabuf_sysfs_uevent_filter(struct kobject *kobj)
> {
> return 0;
> }
> diff --git a/fs/dlm/lockspace.c b/fs/dlm/lockspace.c
> index 10eddfa6c3d7..0bbb346cb892 100644
> --- a/fs/dlm/lockspace.c
> +++ b/fs/dlm/lockspace.c
> @@ -216,8 +216,7 @@ static int do_uevent(struct dlm_ls *ls, int in)
> return ls->ls_uevent_result;
> }
>
> -static int dlm_uevent(struct kset *kset, struct kobject *kobj,
> - struct kobj_uevent_env *env)
> +static int dlm_uevent(struct kobject *kobj, struct kobj_uevent_env *env)
> {
> struct dlm_ls *ls = container_of(kobj, struct dlm_ls, ls_kobj);
>
> diff --git a/fs/gfs2/sys.c b/fs/gfs2/sys.c
> index c0a34d9ddee4..a6002b2d146d 100644
> --- a/fs/gfs2/sys.c
> +++ b/fs/gfs2/sys.c
> @@ -767,8 +767,7 @@ void gfs2_sys_fs_del(struct gfs2_sbd *sdp)
> wait_for_completion(&sdp->sd_kobj_unregister);
> }
>
> -static int gfs2_uevent(struct kset *kset, struct kobject *kobj,
> - struct kobj_uevent_env *env)
> +static int gfs2_uevent(struct kobject *kobj, struct kobj_uevent_env *env)
> {
> struct gfs2_sbd *sdp = container_of(kobj, struct gfs2_sbd, sd_kobj);
> struct super_block *s = sdp->sd_vfs;
> diff --git a/include/linux/kobject.h b/include/linux/kobject.h
> index 683172b2e094..ad90b49824dc 100644
> --- a/include/linux/kobject.h
> +++ b/include/linux/kobject.h
> @@ -153,10 +153,9 @@ struct kobj_uevent_env {
> };
>
> struct kset_uevent_ops {
> - int (* const filter)(struct kset *kset, struct kobject *kobj);
> - const char *(* const name)(struct kset *kset, struct kobject *kobj);
> - int (* const uevent)(struct kset *kset, struct kobject *kobj,
> - struct kobj_uevent_env *env);
> + int (* const filter)(struct kobject *kobj);
> + const char *(* const name)(struct kobject *kobj);
> + int (* const uevent)(struct kobject *kobj, struct kobj_uevent_env *env);
> };
>
> struct kobj_attribute {
> diff --git a/kernel/params.c b/kernel/params.c
> index 9b90e3c4d3c0..5b92310425c5 100644
> --- a/kernel/params.c
> +++ b/kernel/params.c
> @@ -926,7 +926,7 @@ static const struct sysfs_ops module_sysfs_ops = {
> .store = module_attr_store,
> };
>
> -static int uevent_filter(struct kset *kset, struct kobject *kobj)
> +static int uevent_filter(struct kobject *kobj)
> {
> const struct kobj_type *ktype = get_ktype(kobj);
>
> diff --git a/lib/kobject_uevent.c b/lib/kobject_uevent.c
> index c87d5b6a8a55..7c44b7ae4c5c 100644
> --- a/lib/kobject_uevent.c
> +++ b/lib/kobject_uevent.c
> @@ -501,7 +501,7 @@ int kobject_uevent_env(struct kobject *kobj, enum kobject_action action,
> }
> /* skip the event, if the filter returns zero. */
> if (uevent_ops && uevent_ops->filter)
> - if (!uevent_ops->filter(kset, kobj)) {
> + if (!uevent_ops->filter(kobj)) {
> pr_debug("kobject: '%s' (%p): %s: filter function "
> "caused the event to drop!\n",
> kobject_name(kobj), kobj, __func__);
> @@ -510,7 +510,7 @@ int kobject_uevent_env(struct kobject *kobj, enum kobject_action action,
>
> /* originating subsystem */
> if (uevent_ops && uevent_ops->name)
> - subsystem = uevent_ops->name(kset, kobj);
> + subsystem = uevent_ops->name(kobj);
> else
> subsystem = kobject_name(&kset->kobj);
> if (!subsystem) {
> @@ -554,7 +554,7 @@ int kobject_uevent_env(struct kobject *kobj, enum kobject_action action,
>
> /* let the kset specific function add its stuff */
> if (uevent_ops && uevent_ops->uevent) {
> - retval = uevent_ops->uevent(kset, kobj, env);
> + retval = uevent_ops->uevent(kobj, env);
> if (retval) {
> pr_debug("kobject: '%s' (%p): %s: uevent() returned "
> "%d\n", kobject_name(kobj), kobj,
> --
> 2.34.1
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] kobject: remove kset from struct kset_uevent_ops callbacks
2021-12-27 20:41 ` Wedson Almeida Filho
@ 2021-12-28 10:08 ` Greg Kroah-Hartman
0 siblings, 0 replies; 4+ messages in thread
From: Greg Kroah-Hartman @ 2021-12-28 10:08 UTC (permalink / raw)
To: Wedson Almeida Filho; +Cc: linux-kernel, Rafael J . Wysocki
On Mon, Dec 27, 2021 at 08:41:36PM +0000, Wedson Almeida Filho wrote:
> On Mon, Dec 27, 2021 at 05:39:24PM +0100, Greg Kroah-Hartman wrote:
> > There is no need to pass the pointer to the kset in the struct
> > kset_uevent_ops callbacks as no one uses it, so just remove that pointer
> > entirely.
>
> In the future, if someone needs the kset they can always find it by looking for
> the first non-null kset in the kobject or its parent chain, right? IOW, we're
> not losing any information by removing this.
Or even easier, we can add it back if needed :)
> >
> > Cc: Rafael J. Wysocki <rafael@kernel.org>
> > Cc: Wedson Almeida Filho <wedsonaf@google.com>
> > Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
>
> Reviewed-by: Wedson Almeida Filho <wedsonaf@google.com>
Thanks for the review.
greg k-h
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2021-12-28 10:08 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-12-27 16:39 [PATCH] kobject: remove kset from struct kset_uevent_ops callbacks Greg Kroah-Hartman
2021-12-27 16:43 ` Rafael J. Wysocki
2021-12-27 20:41 ` Wedson Almeida Filho
2021-12-28 10:08 ` Greg Kroah-Hartman
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.