Re: [PATCH -mm] relayfs: support larger relay buffer take 3

From: Tom Zanussi
Date: Thu Apr 17 2008 - 00:06:19 EST



On Wed, 2008-04-16 at 15:51 -0400, Masami Hiramatsu wrote:
> Use vmalloc() and memset() instead of kcalloc() to allocate a page* array
> when the array size is bigger than one page. This enables relayfs to support
> bigger relay buffers than 64MB on 4k-page system, 512MB on 16k-page system.
>
> Signed-off-by: Masami Hiramatsu <mhiramat@xxxxxxxxxx>
> ---
> Changes from take2 to take3:
> - Use struct page ** instead of struct page *.
> - move functions to the place before relay_mmap_buf.
> - add comments.
>
> This is useful for a 64-bit system which has a plenty of memory (tens of
> giga bytes) and a large kernel memory space.
>
> I tested it on x86-64 and ia64.
>

Hi,

Looks fine to me.

Reviewed-by: Tom Zanussi <tzanussi@xxxxxxxxx>



--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/