Re: [PATCH 3/3] null_blk: register configfs subsystem after creating default devices

From: Zizhi Wo

Date: Mon Jul 06 2026 - 09:52:40 EST




在 2026/7/6 21:17, Bart Van Assche 写道:
On 7/6/26 5:35 AM, Zizhi Wo wrote:
@@ -2187,8 +2185,6 @@ static int __init null_init(void)
          null_destroy_dev(nullb);
      }
      unregister_blkdev(null_major, "nullb");
-err_conf:
-    configfs_unregister_subsystem(&nullb_subsys);
      return ret;
  }

Please make null_exit() consistent with the cleanup path in null_init().
null_exit() calls unregister_blkdev() before it destroys the null_blk
instances. That doesn't look right to me.

Thanks,

Bart.



Thanks for pointing this out, I hadn't noticed that. I'll add this
change in the next version.

Thanks,
Zizhi Wo