Re: [PATCH RFC v3 01/15] block: cleanup bio_issue

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Yu Kuai <yukuai1@xxxxxxxxxxxxxxx>
Subject: Re: [PATCH RFC v3 01/15] block: cleanup bio_issue
From: Christoph Hellwig <hch@xxxxxxxxxxxxx>
Date: Wed, 3 Sep 2025 06:23:02 -0700
Cc: hch@xxxxxxxxxxxxx, colyli@xxxxxxxxxx, hare@xxxxxxx, dlemoal@xxxxxxxxxx, tieren@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx, axboe@xxxxxxxxx, tj@xxxxxxxxxx, josef@xxxxxxxxxxxxxx, song@xxxxxxxxxx, kmo@xxxxxxxxxxxxx, satyat@xxxxxxxxxx, ebiggers@xxxxxxxxxx, neil@xxxxxxxxxx, akpm@xxxxxxxxxxxxxxxxxxxx, linux-block@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, cgroups@xxxxxxxxxxxxxxx, linux-raid@xxxxxxxxxxxxxxx, yukuai3@xxxxxxxxxx, yi.zhang@xxxxxxxxxx, yangerkun@xxxxxxxxxx, johnny.chenyi@xxxxxxxxxx
In-reply-to: <20250901033220.42982-2-yukuai1@huaweicloud.com>
References: <20250901033220.42982-1-yukuai1@huaweicloud.com> <20250901033220.42982-2-yukuai1@huaweicloud.com>

On Mon, Sep 01, 2025 at 11:32:06AM +0800, Yu Kuai wrote:
> From: Yu Kuai <yukuai3@xxxxxxxxxx>
> 
> Now that bio->bi_issue is only used by io-latency to get bio issue time,
> replace bio_issue with u64 time directly and remove bio_issue to make
> code cleaner.
> 
> Signed-off-by: Yu Kuai <yukuai3@xxxxxxxxxx>

Looks good:

Reviewed-by: Christoph Hellwig <hch@xxxxxx>

References:
- [PATCH RFC v3 00/15] block: fix disordered IO in the case recursive split
  - From: Yu Kuai
- [PATCH RFC v3 01/15] block: cleanup bio_issue
  - From: Yu Kuai

Prev by Date: Re: [PATCH RFC v3 04/15] blk-crypto: fix missing processing for split bio
Next by Date: Re: [PATCH RFC v3 06/15] md/raid0: convert raid0_handle_discard() to use bio_submit_split_bioset()
Previous by thread: [PATCH RFC v3 01/15] block: cleanup bio_issue
Next by thread: Re: [PATCH RFC v3 01/15] block: cleanup bio_issue
Index(es):
- Date
- Thread

[Index of Archives] [Linux RAID Wiki] [ATA RAID] [Linux SCSI Target Infrastructure] [Linux Block] [Linux IDE] [Linux SCSI] [Linux Hams] [Device Mapper] [Device Mapper Cryptographics] [Kernel] [Linux Admin] [Linux Net] [GFS] [RPM] [git] [Yosemite Forum]