Re: [PATCH 04/14] bdi: initialize ->ra_pages in bdi_init

From: Christoph Hellwig
Date: Tue Jul 21 2020 - 01:00:54 EST


On Mon, Jul 20, 2020 at 11:27:57PM +0200, Richard Weinberger wrote:
> On Mon, Jul 20, 2020 at 2:37 PM Artem Bityutskiy <dedekind1@xxxxxxxxx> wrote:
> >
> > On Mon, 2020-07-20 at 14:07 +0200, Christoph Hellwig wrote:
> > > What about jffs2 and blk2mtd raw block devices?
>
> I don't worry much about blk2mtd.
>
> > If my memory serves me correctly JFFS2 did not mind readahead.
>
> This covers my knowledge too.
> I fear enabling readahead on JFFS2 will cause performance issues, this
> filesystem
> is mostly used on small and slow NOR devices.

I'm going to wait for Hans for feedback on vboxsf, but in doubt I'll
ust add a prep patch or fold for this one to explicit set ra_pages to 0
with a comment then.