Re: [PATCH,RFC] ext3: Add support for non-native signed/unsignedhtree hash algorithms

From: Andrew Morton
Date: Wed Oct 22 2008 - 20:23:10 EST


On Mon, 20 Oct 2008 23:43:44 -0400
"Theodore Ts'o" <tytso@xxxxxxx> wrote:

> + i = le32_to_cpu(es->s_flags);
> + if (i & EXT2_FLAGS_UNSIGNED_HASH)
> + sbi->s_hash_unsigned = 3;
> + else if ((i & EXT2_FLAGS_SIGNED_HASH) == 0) {
> + char c;
> +
> + c = (char) 255;
> + if (((int) c) == -1) {

arm says

fs/ext3/super.c: In function `ext3_fill_super':
fs/ext3/super.c:1750: warning: comparison is always false due to limited range of data type

Also, is there any way in which this new code can be, umm, cleaned up?
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/