[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 27/27] direct-io: remove random prefetches
 
- To: Christoph Hellwig <hch@xxxxxx>
 
- From: David Sterba <dsterba@xxxxxxx>
 
- Date: Tue, 19 Apr 2022 14:55:17 +0200
 
- Cc: Jens Axboe <axboe@xxxxxxxxx>, dm-devel@xxxxxxxxxx,	linux-xfs@xxxxxxxxxxxxxxx, linux-fsdevel@xxxxxxxxxxxxxxx,	linux-um@xxxxxxxxxxxxxxxxxxx, linux-block@xxxxxxxxxxxxxxx,	drbd-dev@xxxxxxxxxxxxxxxx, nbd@xxxxxxxxxxxxxxxx,	ceph-devel@xxxxxxxxxxxxxxx,	virtualization@xxxxxxxxxxxxxxxxxxxxxxxxxx,	xen-devel@xxxxxxxxxxxxxxxxxxxx, linux-bcache@xxxxxxxxxxxxxxx,	linux-raid@xxxxxxxxxxxxxxx, linux-mmc@xxxxxxxxxxxxxxx,	linux-mtd@xxxxxxxxxxxxxxxxxxx, linux-nvme@xxxxxxxxxxxxxxxxxxx,	linux-s390@xxxxxxxxxxxxxxx, linux-scsi@xxxxxxxxxxxxxxx,	target-devel@xxxxxxxxxxxxxxx, linux-btrfs@xxxxxxxxxxxxxxx,	linux-ext4@xxxxxxxxxxxxxxx, linux-f2fs-devel@xxxxxxxxxxxxxxxxxxxxx,	cluster-devel@xxxxxxxxxx, jfs-discussion@xxxxxxxxxxxxxxxxxxxxx,	linux-nilfs@xxxxxxxxxxxxxxx, ntfs3@xxxxxxxxxxxxxxx,	ocfs2-devel@xxxxxxxxxxxxxx, linux-mm@xxxxxxxxx
 
- Delivery-date: Tue, 19 Apr 2022 12:59:32 +0000
 
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
 
- Mail-followup-to: dsterba@xxxxxxx, Christoph Hellwig <hch@xxxxxx>,	Jens Axboe <axboe@xxxxxxxxx>, dm-devel@xxxxxxxxxx,	linux-xfs@xxxxxxxxxxxxxxx, linux-fsdevel@xxxxxxxxxxxxxxx,	linux-um@xxxxxxxxxxxxxxxxxxx, linux-block@xxxxxxxxxxxxxxx,	drbd-dev@xxxxxxxxxxxxxxxx, nbd@xxxxxxxxxxxxxxxx,	ceph-devel@xxxxxxxxxxxxxxx,	virtualization@xxxxxxxxxxxxxxxxxxxxxxxxxx,	xen-devel@xxxxxxxxxxxxxxxxxxxx, linux-bcache@xxxxxxxxxxxxxxx,	linux-raid@xxxxxxxxxxxxxxx, linux-mmc@xxxxxxxxxxxxxxx,	linux-mtd@xxxxxxxxxxxxxxxxxxx, linux-nvme@xxxxxxxxxxxxxxxxxxx,	linux-s390@xxxxxxxxxxxxxxx, linux-scsi@xxxxxxxxxxxxxxx,	target-devel@xxxxxxxxxxxxxxx, linux-btrfs@xxxxxxxxxxxxxxx,	linux-ext4@xxxxxxxxxxxxxxx, linux-f2fs-devel@xxxxxxxxxxxxxxxxxxxxx,	cluster-devel@xxxxxxxxxx, jfs-discussion@xxxxxxxxxxxxxxxxxxxxx,	linux-nilfs@xxxxxxxxxxxxxxx, ntfs3@xxxxxxxxxxxxxxx,	ocfs2-devel@xxxxxxxxxxxxxx, linux-mm@xxxxxxxxx
 
 
 
On Fri, Apr 15, 2022 at 06:52:58AM +0200, Christoph Hellwig wrote:
> Randomly poking into block device internals for manual prefetches isn't
> exactly a very maintainable thing to do.  And none of the performance
> criticil direct I/O implementations still use this library function
> anyway, so just drop it.
> 
> Signed-off-by: Christoph Hellwig <hch@xxxxxx>
That the direct io function needed a valid bdev just for the prefetch
but nothing else was one of the reasons we had to keep the latest_bdev
in btrfs, so good riddance.
You may want to add the reference to the patch that added the prefetch,
65dd2aa90aa1 ("dio: optimize cache misses in the submission path") and
also remove #include <linux/prefetch.h> as there are no more uses of
prefetch in the file.
With that
Reviewed-by: David Sterba <dsterba@xxxxxxxx>
 
    
     |