Re: [PATCH v2] block/mq-deadline: Remove the redundant rb_entry_rq in the deadline_from_pos().

[Li Nan <linan666@huaweicloud.com>]

在 2025/8/29 11:45, chengkaitao 写道:
> From: chengkaitao <chengkaitao@kylinos.cn>
> 
> In commit(fde02699c242), the "if (blk_rq_is_seq_zoned_write(rq))"
> was removed, but the "rb_entry_rq(node)" and some other code were
> inadvertently left behind. This patch fixed it.
> 
> Signed-off-by: chengkaitao <chengkaitao@kylinos.cn>
> ---
>   block/mq-deadline.c | 4 ----
>   1 file changed, 4 deletions(-)
> 
> diff --git a/block/mq-deadline.c b/block/mq-deadline.c
> index 1a031922c447..63145cc9825f 100644
> --- a/block/mq-deadline.c
> +++ b/block/mq-deadline.c
> @@ -133,10 +133,6 @@ static inline struct request *deadline_from_pos(struct dd_per_prio *per_prio,
>   	struct rb_node *node = per_prio->sort_list[data_dir].rb_node;
>   	struct request *rq, *res = NULL;
>   
> -	if (!node)
> -		return NULL;
> -
> -	rq = rb_entry_rq(node);
>   	while (node) {
>   		rq = rb_entry_rq(node);
>   		if (blk_rq_pos(rq) >= pos) {

LGTM

Reviewed-by: Li Nan <linan122@huawei.com>

-- 
Thanks,
Nan