On Wed, Jun 23, 2021 at 1:53 PM Christoph Hellwig <hch@xxxxxxxxxxxxx> wrote: > > > -static void disk_add_events(struct gendisk *disk) > > +static void disk_add_sysfs(struct gendisk *disk) > > { > > /* FIXME: error handling */ > > - if (sysfs_create_files(&disk_to_dev(disk)->kobj, disk_events_attrs) < 0) > > + if (sysfs_create_files(&disk_to_dev(disk)->kobj, disk_sysfs_attrs) < 0) > > pr_warn("%s: failed to create sysfs files for events\n", > > disk->disk_name); > > +} > > Actually, what we need here is a way how we can setup the ->groups > field of the device to include all attribute groups instead of having > to call sysfs_create_files at all. I don't get this one. You mean in general or in this series? -- per aspera ad upstream