[PATCH v7 1/2] nvme: add per-queue sysfs directories

Saravanan D saravanand at crusoe.ai
Thu Nov 5 21:23:42 PST 2026


On Wed, Nov 04, 2026 at 06:30:56PM +0100, Daniel Wagner wrote:
> > +	ret = sysfs_create_groups(&qinfo->kobj, ctrl->ops->queue_attr_groups);
> > +	if (ret)
> > +		goto out_put;
>
> Don't you have to call kobject_del(&qinfo->kobj); here?

Yes. Sashiko flagged the same error path.

I will send a v8 that adds an out_del label calling kobject_del()
before kobject_put(), so a failed sysfs_create_groups() unlinks the
<qid> directory before nvme_add_queue_info() returns.

Thanks for the review.

Saravanan D.



More information about the Linux-nvme mailing list