Re: [Patch 06/18] fs/logfs/compr.c

On Sunday 03 June 2007, Jörn Engel wrote:
> +#define COMPR_LEVEL 3
> +
> +static DEFINE_MUTEX(compr_mutex);
> +static struct z_stream_s stream;

Is there a particular reason to choose '3' as the only compression
level? Should this perhaps be a per-superblock option instead?

Also, I thought I saw discussion about making the mutex and
stream per-superblock, but don't remember if the idea was discarded.
If it was, you might want to add it to the won't-happen list.

	Arnd <>< 
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

---

• Follow-Ups:
  • Re: [Patch 06/18] fs/logfs/compr.c
    • From: Jörn Engel <[email protected]>

• References:
  • LogFS take four
    • From: Jörn Engel <[email protected]>
  • [Patch 06/18] fs/logfs/compr.c
    • From: Jörn Engel <[email protected]>

• Prev by Date: [PATCH] COBALT: Remove all traces of COBALT from nvram.c
• Next by Date: [PATCH 1/1] Char: stallion, don't fail with less than max panels
• Previous by thread: [Patch 06/18] fs/logfs/compr.c
• Next by thread: Re: [Patch 06/18] fs/logfs/compr.c
• Index(es):
  • Date
  • Thread

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]