linux-kernel - RE: zcache - trivial tweak to build-time error message wording

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Thu, 1 Sep 2011 07:41:12 -0700 (PDT)
From:	Dan Magenheimer <dan.magenheimer@...cle.com>
To:	Valdis.Kletnieks@...edu
Cc:	linux-kernel@...r.kernel.org
Subject: RE: zcache - trivial tweak to build-time error message wording

> From: Valdis.Kletnieks@...edu [mailto:Valdis.Kletnieks@...edu]
> Sent: Wednesday, August 31, 2011 2:23 PM
> To: Dan Magenheimer
> Cc: linux-kernel@...r.kernel.org
> Subject: zcache - trivial tweak to build-time error message wording
> 
> Clarify that the requirement is "at least one of" - the error message could
> potentially be read as both being required.
> 
> Yes, the line is over 80 chars.  You can't get 79 chars of text and a #error
> into 80 chars :)
> 
> Patch against linux-next-20110831
> 
> Signed-off-by: Valdis Kletnieks <valdis.kletnieks@...edu>
> 
> --- linux-next/drivers/staging/zcache/zcache-main.c.dist	2011-08-31 13:31:27.054696837 -0400
> +++ linux-next/drivers/staging/zcache/zcache-main.c	2011-08-31 16:16:46.055659964 -0400
> @@ -34,7 +34,7 @@
>  #include "../zram/xvmalloc.h" /* if built in drivers/staging */
> 
>  #if (!defined(CONFIG_CLEANCACHE) && !defined(CONFIG_FRONTSWAP))
> -#error "zcache is useless without CONFIG_CLEANCACHE or CONFIG_FRONTSWAP"
> +#error "zcache is useless without at least one of CONFIG_CLEANCACHE or CONFIG_FRONTSWAP"

Actually, if the reader interprets that both are required,
that's fine with me.  Cleancache and frontswap were split
up for upstreaming purposes but, except in unusual circumstances,
should always be used together.  That is difficult, of course,
if frontswap is not yet merged in your build tree, but after
frontswap is merged in Linus's tree, I might be inclined to
require both CONFIG_CLEANCACHE *and* CONFIG_FRONTSWAP in
the future Kconfig for zcache.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/