| Message ID | 20210702152714.7978-1-penguin-kernel@I-love.SAKURA.ne.jp (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | loop: remove unused variable in loop_set_status() | expand |
On 7/2/21 9:27 AM, Tetsuo Handa wrote: > Commit 0384264ea8a39bd9 ("block: pass a gendisk to bdev_disk_changed") > changed to pass lo->lo_disk instead of lo->lo_device. Applied, thanks.
On Fri, Jul 02, 2021 at 09:29:26AM -0600, Jens Axboe wrote: > On 7/2/21 9:27 AM, Tetsuo Handa wrote: > > Commit 0384264ea8a39bd9 ("block: pass a gendisk to bdev_disk_changed") > > changed to pass lo->lo_disk instead of lo->lo_device. > > Applied, thanks. FYI, I also sent a patch on June 25th for this..
On 7/2/21 9:31 AM, Christoph Hellwig wrote: > On Fri, Jul 02, 2021 at 09:29:26AM -0600, Jens Axboe wrote: >> On 7/2/21 9:27 AM, Tetsuo Handa wrote: >>> Commit 0384264ea8a39bd9 ("block: pass a gendisk to bdev_disk_changed") >>> changed to pass lo->lo_disk instead of lo->lo_device. >> >> Applied, thanks. > > FYI, I also sent a patch on June 25th for this.. Oops, looks like I missed that one.
diff --git a/drivers/block/loop.c b/drivers/block/loop.c index cc0e8c39a48b..9f5a93688164 100644 --- a/drivers/block/loop.c +++ b/drivers/block/loop.c @@ -1434,7 +1434,6 @@ static int loop_set_status(struct loop_device *lo, const struct loop_info64 *info) { int err; - struct block_device *bdev; kuid_t uid = current_uid(); int prev_lo_flags; bool partscan = false; @@ -1503,7 +1502,6 @@ loop_set_status(struct loop_device *lo, const struct loop_info64 *info) if (!err && (lo->lo_flags & LO_FLAGS_PARTSCAN) && !(prev_lo_flags & LO_FLAGS_PARTSCAN)) { lo->lo_disk->flags &= ~GENHD_FL_NO_PART_SCAN; - bdev = lo->lo_device; partscan = true; } out_unlock:
Commit 0384264ea8a39bd9 ("block: pass a gendisk to bdev_disk_changed") changed to pass lo->lo_disk instead of lo->lo_device. Signed-off-by: Tetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp> --- drivers/block/loop.c | 2 -- 1 file changed, 2 deletions(-)