* [PATCH v2] block: remove more NULL checks after bdev_get_queue() @ 2023-02-03 2:40 ` Juhyung Park 2023-02-19 20:42 ` Juhyung Park ` (2 more replies) 0 siblings, 3 replies; 4+ messages in thread From: Juhyung Park @ 2023-02-03 2:40 UTC (permalink / raw) To: linux-block; +Cc: axboe, Juhyung Park bdev_get_queue() never returns NULL. Several commits [1][2] have been made before to remove such superfluous checks, but some still remained. For places where bdev_get_queue() is called solely for NULL checks, it is removed entirely. [1] commit ec9fd2a13d74 ("blk-lib: don't check bdev_get_queue() NULL check") [2] commit fea127b36c93 ("block: remove superfluous check for request queue in bdev_is_zoned()") Signed-off-by: Juhyung Park <qkrwngud825@gmail.com> --- block/blk-zoned.c | 10 ---------- include/linux/blkdev.h | 7 +------ kernel/trace/blktrace.c | 6 +----- 3 files changed, 2 insertions(+), 21 deletions(-) diff --git a/block/blk-zoned.c b/block/blk-zoned.c index 614b575be899..fce9082384d6 100644 --- a/block/blk-zoned.c +++ b/block/blk-zoned.c @@ -334,17 +334,12 @@ int blkdev_report_zones_ioctl(struct block_device *bdev, fmode_t mode, { void __user *argp = (void __user *)arg; struct zone_report_args args; - struct request_queue *q; struct blk_zone_report rep; int ret; if (!argp) return -EINVAL; - q = bdev_get_queue(bdev); - if (!q) - return -ENXIO; - if (!bdev_is_zoned(bdev)) return -ENOTTY; @@ -391,7 +386,6 @@ int blkdev_zone_mgmt_ioctl(struct block_device *bdev, fmode_t mode, unsigned int cmd, unsigned long arg) { void __user *argp = (void __user *)arg; - struct request_queue *q; struct blk_zone_range zrange; enum req_op op; int ret; @@ -399,10 +393,6 @@ int blkdev_zone_mgmt_ioctl(struct block_device *bdev, fmode_t mode, if (!argp) return -EINVAL; - q = bdev_get_queue(bdev); - if (!q) - return -ENXIO; - if (!bdev_is_zoned(bdev)) return -ENOTTY; diff --git a/include/linux/blkdev.h b/include/linux/blkdev.h index b9637d63e6f0..89dd9b02b45b 100644 --- a/include/linux/blkdev.h +++ b/include/linux/blkdev.h @@ -1276,12 +1276,7 @@ static inline bool bdev_nowait(struct block_device *bdev) static inline enum blk_zoned_model bdev_zoned_model(struct block_device *bdev) { - struct request_queue *q = bdev_get_queue(bdev); - - if (q) - return blk_queue_zoned_model(q); - - return BLK_ZONED_NONE; + return blk_queue_zoned_model(bdev_get_queue(bdev)); } static inline bool bdev_is_zoned(struct block_device *bdev) diff --git a/kernel/trace/blktrace.c b/kernel/trace/blktrace.c index 918a7d12df8f..e5e8963d6808 100644 --- a/kernel/trace/blktrace.c +++ b/kernel/trace/blktrace.c @@ -729,14 +729,10 @@ EXPORT_SYMBOL_GPL(blk_trace_startstop); **/ int blk_trace_ioctl(struct block_device *bdev, unsigned cmd, char __user *arg) { - struct request_queue *q; + struct request_queue *q = bdev_get_queue(bdev); int ret, start = 0; char b[BDEVNAME_SIZE]; - q = bdev_get_queue(bdev); - if (!q) - return -ENXIO; - mutex_lock(&q->debugfs_mutex); switch (cmd) { -- 2.39.1 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH v2] block: remove more NULL checks after bdev_get_queue() 2023-02-03 2:40 ` [PATCH v2] block: remove more NULL checks after bdev_get_queue() Juhyung Park @ 2023-02-19 20:42 ` Juhyung Park 2023-02-20 11:59 ` Pankaj Raghav 2023-02-22 16:54 ` Jens Axboe 2 siblings, 0 replies; 4+ messages in thread From: Juhyung Park @ 2023-02-19 20:42 UTC (permalink / raw) To: linux-block; +Cc: axboe, johannes.thumshirn, p.raghav, kch Hi everyone, A friendly bump here. 6.3's merge window is about to begin. Thanks, regards On Fri, Feb 3, 2023 at 11:41 AM Juhyung Park <qkrwngud825@gmail.com> wrote: > > bdev_get_queue() never returns NULL. Several commits [1][2] have been made > before to remove such superfluous checks, but some still remained. > > For places where bdev_get_queue() is called solely for NULL checks, it is > removed entirely. > > [1] commit ec9fd2a13d74 ("blk-lib: don't check bdev_get_queue() NULL check") > [2] commit fea127b36c93 ("block: remove superfluous check for request queue in bdev_is_zoned()") > > Signed-off-by: Juhyung Park <qkrwngud825@gmail.com> > --- > block/blk-zoned.c | 10 ---------- > include/linux/blkdev.h | 7 +------ > kernel/trace/blktrace.c | 6 +----- > 3 files changed, 2 insertions(+), 21 deletions(-) > > diff --git a/block/blk-zoned.c b/block/blk-zoned.c > index 614b575be899..fce9082384d6 100644 > --- a/block/blk-zoned.c > +++ b/block/blk-zoned.c > @@ -334,17 +334,12 @@ int blkdev_report_zones_ioctl(struct block_device *bdev, fmode_t mode, > { > void __user *argp = (void __user *)arg; > struct zone_report_args args; > - struct request_queue *q; > struct blk_zone_report rep; > int ret; > > if (!argp) > return -EINVAL; > > - q = bdev_get_queue(bdev); > - if (!q) > - return -ENXIO; > - > if (!bdev_is_zoned(bdev)) > return -ENOTTY; > > @@ -391,7 +386,6 @@ int blkdev_zone_mgmt_ioctl(struct block_device *bdev, fmode_t mode, > unsigned int cmd, unsigned long arg) > { > void __user *argp = (void __user *)arg; > - struct request_queue *q; > struct blk_zone_range zrange; > enum req_op op; > int ret; > @@ -399,10 +393,6 @@ int blkdev_zone_mgmt_ioctl(struct block_device *bdev, fmode_t mode, > if (!argp) > return -EINVAL; > > - q = bdev_get_queue(bdev); > - if (!q) > - return -ENXIO; > - > if (!bdev_is_zoned(bdev)) > return -ENOTTY; > > diff --git a/include/linux/blkdev.h b/include/linux/blkdev.h > index b9637d63e6f0..89dd9b02b45b 100644 > --- a/include/linux/blkdev.h > +++ b/include/linux/blkdev.h > @@ -1276,12 +1276,7 @@ static inline bool bdev_nowait(struct block_device *bdev) > > static inline enum blk_zoned_model bdev_zoned_model(struct block_device *bdev) > { > - struct request_queue *q = bdev_get_queue(bdev); > - > - if (q) > - return blk_queue_zoned_model(q); > - > - return BLK_ZONED_NONE; > + return blk_queue_zoned_model(bdev_get_queue(bdev)); > } > > static inline bool bdev_is_zoned(struct block_device *bdev) > diff --git a/kernel/trace/blktrace.c b/kernel/trace/blktrace.c > index 918a7d12df8f..e5e8963d6808 100644 > --- a/kernel/trace/blktrace.c > +++ b/kernel/trace/blktrace.c > @@ -729,14 +729,10 @@ EXPORT_SYMBOL_GPL(blk_trace_startstop); > **/ > int blk_trace_ioctl(struct block_device *bdev, unsigned cmd, char __user *arg) > { > - struct request_queue *q; > + struct request_queue *q = bdev_get_queue(bdev); > int ret, start = 0; > char b[BDEVNAME_SIZE]; > > - q = bdev_get_queue(bdev); > - if (!q) > - return -ENXIO; > - > mutex_lock(&q->debugfs_mutex); > > switch (cmd) { > -- > 2.39.1 > ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v2] block: remove more NULL checks after bdev_get_queue() 2023-02-03 2:40 ` [PATCH v2] block: remove more NULL checks after bdev_get_queue() Juhyung Park 2023-02-19 20:42 ` Juhyung Park @ 2023-02-20 11:59 ` Pankaj Raghav 2023-02-22 16:54 ` Jens Axboe 2 siblings, 0 replies; 4+ messages in thread From: Pankaj Raghav @ 2023-02-20 11:59 UTC (permalink / raw) To: Juhyung Park; +Cc: linux-block, axboe [-- Attachment #1: Type: text/plain, Size: 608 bytes --] On Fri, Feb 03, 2023 at 11:40:29AM +0900, Juhyung Park wrote: > bdev_get_queue() never returns NULL. Several commits [1][2] have been made > before to remove such superfluous checks, but some still remained. > > For places where bdev_get_queue() is called solely for NULL checks, it is > removed entirely. > > [1] commit ec9fd2a13d74 ("blk-lib: don't check bdev_get_queue() NULL check") > [2] commit fea127b36c93 ("block: remove superfluous check for request queue in bdev_is_zoned()") > > Signed-off-by: Juhyung Park <qkrwngud825@gmail.com> Looks good, Reviewed-by: Pankaj Raghav <p.raghav@samsung.com> [-- Attachment #2: Type: text/plain, Size: 0 bytes --] ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v2] block: remove more NULL checks after bdev_get_queue() 2023-02-03 2:40 ` [PATCH v2] block: remove more NULL checks after bdev_get_queue() Juhyung Park 2023-02-19 20:42 ` Juhyung Park 2023-02-20 11:59 ` Pankaj Raghav @ 2023-02-22 16:54 ` Jens Axboe 2 siblings, 0 replies; 4+ messages in thread From: Jens Axboe @ 2023-02-22 16:54 UTC (permalink / raw) To: linux-block, Juhyung Park On Fri, 03 Feb 2023 11:40:29 +0900, Juhyung Park wrote: > bdev_get_queue() never returns NULL. Several commits [1][2] have been made > before to remove such superfluous checks, but some still remained. > > For places where bdev_get_queue() is called solely for NULL checks, it is > removed entirely. > > [1] commit ec9fd2a13d74 ("blk-lib: don't check bdev_get_queue() NULL check") > [2] commit fea127b36c93 ("block: remove superfluous check for request queue in bdev_is_zoned()") > > [...] Applied, thanks! [1/1] block: remove more NULL checks after bdev_get_queue() (no commit info) Best regards, -- Jens Axboe ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2023-02-22 16:54 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <CGME20230220115915eucas1p164d1b65b3cb0b140468af33e0f966807@eucas1p1.samsung.com>
2023-02-03 2:40 ` [PATCH v2] block: remove more NULL checks after bdev_get_queue() Juhyung Park
2023-02-19 20:42 ` Juhyung Park
2023-02-20 11:59 ` Pankaj Raghav
2023-02-22 16:54 ` Jens Axboe
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox