On Mon, Jan 13, 2020 at 07:22:42PM +0100, Daniel Wagner wrote: > > + int (*readahead)(struct file *, struct address_space *, > > + struct pagevec *, pgoff_t index); > > Shouldn't this be no return type? Just trying to map your commit > message to the code. Yes, I'll fix it. I'm not really impressed with the filesystem documentation at this point; I feel like more could be automated and there's too much duplication. > Maybe I just miss the point, in the case, sorry for the noise. Well, mostly I'm looking for architectural and design feedback. I have compiled this code, but not run it.