Re: [PATCH v2 1/8] fs/buffer: simplify block_read_full_folio() with bh_offset()

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

To: Hannes Reinecke <hare@xxxxxxx>
Subject: Re: [PATCH v2 1/8] fs/buffer: simplify block_read_full_folio() with bh_offset()
From: Matthew Wilcox <willy@xxxxxxxxxxxxx>
Date: Wed, 5 Feb 2025 22:03:05 +0000
Cc: Luis Chamberlain <mcgrof@xxxxxxxxxx>, dave@xxxxxxxxxxxx, david@xxxxxxxxxxxxx, djwong@xxxxxxxxxx, kbusch@xxxxxxxxxx, john.g.garry@xxxxxxxxxx, hch@xxxxxx, ritesh.list@xxxxxxxxx, linux-fsdevel@xxxxxxxxxxxxxxx, linux-xfs@xxxxxxxxxxxxxxx, linux-mm@xxxxxxxxx, linux-block@xxxxxxxxxxxxxxx, gost.dev@xxxxxxxxxxx, p.raghav@xxxxxxxxxxx, da.gomez@xxxxxxxxxxx, kernel@xxxxxxxxxxxxxxxx
In-reply-to: <1b211dd3-a45d-4a2e-aa2a-e0d3e302d4ca@suse.de>

On Wed, Feb 05, 2025 at 05:18:20PM +0100, Hannes Reinecke wrote:
> One wonders: shouldn't we use plugging here to make I/O more efficient?

Should we plug at a higher level?

Opposite question: What if getblk() needs to do a read (ie ext2 indirect
block)?

Follow-Ups:
- Re: [PATCH v2 1/8] fs/buffer: simplify block_read_full_folio() with bh_offset()
  - From: Hannes Reinecke

References:
- [PATCH v2 0/8] enable bs > ps for block devices
  - From: Luis Chamberlain
- [PATCH v2 1/8] fs/buffer: simplify block_read_full_folio() with bh_offset()
  - From: Luis Chamberlain
- Re: [PATCH v2 1/8] fs/buffer: simplify block_read_full_folio() with bh_offset()
  - From: Hannes Reinecke

Prev by Date: Re: Immutable vs read-only for Windows compatibility
Next by Date: Re: [PATCH 3/3] fsnotify: disable pre-content and permission events by default
Previous by thread: Re: [PATCH v2 1/8] fs/buffer: simplify block_read_full_folio() with bh_offset()
Next by thread: Re: [PATCH v2 1/8] fs/buffer: simplify block_read_full_folio() with bh_offset()
Index(es):
- Date
- Thread

[Index of Archives] [Linux Ext4 Filesystem] [Union Filesystem] [Filesystem Testing] [Ceph Users] [Ecryptfs] [NTFS 3] [AutoFS] [Kernel Newbies] [Share Photos] [Security] [Netfilter] [Bugtraq] [Yosemite News] [MIPS Linux] [ARM Linux] [Linux Security] [Linux Cachefs] [Reiser Filesystem] [Linux RAID] [NTFS 3] [Samba] [Device Mapper] [CEPH Development]