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

Jörn Engel (joern@wohnheim.fh-wedel.de)
Sat, 31 May 2003 13:07:43 +0200


On Sat, 31 May 2003 20:51:04 +1000, James Morris wrote:
> On Fri, 30 May 2003, David S. Miller wrote:
>
> > My understanding is that these are just scratchpads. The contents
> > while a decompress/compress operation is not occuring does not
> > matter.
>
> It depends on how the zlib library is used. The filesystems and crypto
> code use it so that each operation is distinct, although it is possible to
> maintain compression history between operations: PPP does this via a
> sliding compression window, and there are other potential users such as
> ROHC.
>
> One way of addressing this would to allow the user to supply their own
> workspace if compression history needs to be maintained.

Agreed. How much memory is needed for the history? Most of the
workspace or substancially less?

> > So if we have 2 such scratchpads per cpu, one for normal and one for
> > BH context, his idea truly can work and be useful to everyone.
> > It would also be lockless on SMP.
>
> And perhaps implement with a lazy allocation scheme so that these
> scratchpads are only allocated if needed (i.e. a caller does not provide
> its own workspace).

Disagreed. Two scratchpads per cpu won't hurt in the mean case much
and lazy allocation wouldn't improve the worst case either. Keep them
static and simple.

Jörn

-- 
With a PC, I always felt limited by the software available. On Unix, 
I am limited only by my knowledge.
-- Peter J. Schoenster
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/