workqueue: Cleanup subsys attribute registration
authorDan Williams <dan.j.williams@intel.com>
Fri, 8 Mar 2024 05:39:32 +0000 (21:39 -0800)
committerTejun Heo <tj@kernel.org>
Mon, 25 Mar 2024 19:17:51 +0000 (09:17 -1000)
commit79202591a55a365251496162ced3004a0a1fa1cf
treec871213033a3c3df6e0b0bc55cc0e9b62bc67492
parentd70f5d5778e88addcc3e56858d5e9c635c1e420e
workqueue: Cleanup subsys attribute registration

While reviewing users of subsys_virtual_register() I noticed that
wq_sysfs_init() ignores the @groups argument. This looks like a
historical artifact as the original wq_subsys only had one attribute to
register.

On the way to building up an @groups argument to pass to
subsys_virtual_register() a few more cleanups fell out:

* Use DEVICE_ATTR_RO() and DEVICE_ATTR_RW() for
  cpumask_{isolated,requested} and cpumask respectively. Rename the
  @show and @store methods accordingly.

* Co-locate the attribute definition with the methods. This required
  moving wq_unbound_cpumask_show down next to wq_unbound_cpumask_store
  (renamed to cpumask_show() and cpumask_store())

* Use ATTRIBUTE_GROUPS() to skip some boilerplate declarations

Signed-off-by: Dan Williams <dan.j.williams@intel.com>
Reviewed-by: Lai Jiangshan <jiangshanlai@gmail.com>
Signed-off-by: Tejun Heo <tj@kernel.org>
kernel/workqueue.c