| Message ID | 20230725182707.948427-3-mfo@canonical.com |
|---|---|
| State | New |
| Headers | show |
| Series | loop: fix regression from max_loop default value change | expand |
diff --git a/drivers/block/loop.c b/drivers/block/loop.c index 4916fe78ab8f..c44a04217eb5 100644 --- a/drivers/block/loop.c +++ b/drivers/block/loop.c @@ -2089,6 +2089,7 @@ static void loop_remove(struct loop_device *lo) put_disk(lo->lo_disk); } +#ifdef CONFIG_BLOCK_LEGACY_AUTOLOAD static void loop_probe(dev_t dev) { int idx = MINOR(dev) >> part_shift; @@ -2097,6 +2098,9 @@ static void loop_probe(dev_t dev) return; loop_add(idx); } +#else +#define loop_probe NULL +#endif /* !CONFIG_BLOCK_LEGACY_AUTOLOAD */ static int loop_control_remove(int idx) {
BugLink: https://bugs.launchpad.net/bugs/2015400 The 'probe' callback in __register_blkdev() is only used under the CONFIG_BLOCK_LEGACY_AUTOLOAD deprecation guard. The loop_probe() function is only used for that callback, so guard it too, accordingly. See commit fbdee71bb5d8 ("block: deprecate autoloading based on dev_t"). Signed-off-by: Mauricio Faria de Oliveira <mfo@canonical.com> Reviewed-by: Christoph Hellwig <hch@lst.de> Link: https://lore.kernel.org/r/20230720143033.841001-2-mfo@canonical.com Signed-off-by: Jens Axboe <axboe@kernel.dk> (cherry picked from commit 23881aec85f3219e8462e87c708815ee2cd82358) Signed-off-by: Mauricio Faria de Oliveira <mfo@canonical.com> --- drivers/block/loop.c | 4 ++++ 1 file changed, 4 insertions(+)