Re: [PATCH RFC] 1/2 central workspace for zlib

From: David Woodhouse (
Date: Mon Jun 02 2003 - 14:36:13 EST

On Mon, 2003-06-02 at 17:37, Jörn Engel wrote:
> On Mon, 2 June 2003 16:59:25 +0100, David Woodhouse wrote:
> > On Mon, 2003-06-02 at 16:53, Jörn Engel wrote:
> > > Maybe lazy allocation. vmalloc() it with the first write(), which
> > > should be never in production use. So the extra overhead doesn't
> > > really matter.
> >
> > Seems reasonable.
> Patch is in CVS.
> Not 100% sure about the correct return code, if the lazy allocation
> fails. Can you check that?

Thanks. The return code doesn't matter. If you look at the caller you
see it's only used as uptodate = !!ret anyway. It can't go any further.

I was concerned briefly that the allocation could happen when we're
trying to evict a page on memory pressure -- but in fact that's
virtually impossible since you'd have to mount the file system and mmap
your file without actually writing anything to the block device.

And anyway, refer to earlier comments about anyone using the naïve
read/erase/modify/writeback mtdblock translation layer for writing in
production wanting to be shot for it :)


- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to More majordomo info at Please read the FAQ at

This archive was generated by hypermail 2b29 : Sat Jun 07 2003 - 22:00:17 EST