Re: [PATCH] Add seq_vprintf and use in gfs2 (was Re: GFS2: Cachelast hash bucket for glock seq_files)

From: Steven Whitehouse
Date: Mon Jun 11 2012 - 09:33:47 EST


Hi,

On Mon, 2012-06-11 at 13:12 +0200, Eric Dumazet wrote:
> On Mon, 2012-06-11 at 11:21 +0100, Steven Whitehouse wrote:
>
> > Thats true, but are there any arches with a 64k page size? In any case
> > I'll follow up with another patch for that rather than try to combine it
> > with this one...
>
> Some arches have page size from 16K to 1MB in size
>
> sh, frv, hexagon, tile, ia64, mips, microblaze, sparc64, ppc64...
>
>
>
I can't imagine running GFS2 on many of those, sparc64 and/or ppc64
maybe, but rather unlikely on the others. Nevertheless, here is a patch
to ensure that we don't land up allocating too much memory. No
performance impact on x86_64 since the buffer size hasn't changed in
that case.

Hopefully this should be the last bit for this set of patches. Many
thanks for taking the time to look at this - things should be much
improved from my initial idea and I think that our QE and support teams
will be happy with the result,

Steve.