vmalloc for zlib workspace

matsunaga matsunaga_kazuhisa at yahoo.co.jp
Thu May 22 11:40:37 EDT 2003


Hi

In the jffs2_zlib_init() function of fs/jffs2/comr_zlib.c. 
workspace is aquired by vmalloc.

But it could be allocated by static variable, like;

static char dbuf[WORKSPACESIZE];

int __init jffs2_zlib_init(void)
{
        def_strm.workspace = dbuf;

Care should be taken as to the workspace size, but it is old refined library and it would hardly be changed.
It would reduce TLB load and improve performance (a little).

BR.

Matsunaga.

P.S.
GC thread BUG() (inode wrap around) seems to be fixed. Thanks.

P.S.2
fs/jffs2/gc.c
line 139
     struct jffs2_inode_cache *ic;
seems not necessary or the name should be changed.
__________________________________________________
Do You Yahoo!?
Yahoo! BB is Broadband by Yahoo!  http://bb.yahoo.co.jp/




More information about the linux-mtd mailing list