On 3/25/24 20:19, Matthew Wilcox wrote:
On Wed, Mar 13, 2024 at 06:02:42PM +0100, Pankaj Raghav (Samsung) wrote:
This is the third version of the series that enables block size > page size
(Large Block Size) in XFS. The context and motivation can be seen in cover
letter of the RFC v1[1]. We also recorded a talk about this effort at LPC [3],
if someone would like more context on this effort.
Thank you. This is a lot better.
I'm still trying to understand your opinion on the contents of the
file_ra_state. Is it supposed to be properly aligned at all times, or
do we work with it in the terms of "desired number of pages" and then
force it to conform to the minimum-block-size reality right at the end?
Because you seem to be doing both at various points.
Guess what, that's what I had been pondering, too.
Each way has its benefits, I guess.
Question really is do we keep the readahead iterator in units of pages,
and convert the result, or do we modify the readahead iterator to work
on folios, and convert the inputs.
Doesn't really matter much, but we need to decide. The former is
probably easier on the caller, and the latter is easier on the consumer.
Take your pick; I really don't mind.
But we should document the result :-)
Cheers,
Hannes
