RE: IA64 non-contiguous memory space bugs

From: Chen, Kenneth W
Date: Wed Feb 22 2006 - 11:18:01 EST


David Gibson wrote on Tuesday, February 21, 2006 4:14 PM
> --- working-2.6.orig/include/linux/mm.h 2004-09-07 10:38:00.000000000 +1000
> +++ working-2.6/include/linux/mm.h 2004-09-24 15:02:18.172776168 +1000
> @@ -41,6 +41,13 @@
> #define MM_VM_SIZE(mm) TASK_SIZE
> #endif
>
> +#ifndef REGION_MAX
> +#define REGION_MAX(addr) TASK_SIZE
> +#endif
> +
> +#define GOOD_TASK_VM_RANGE(addr, len) \
> + ( ((addr)+(len) >= (addr)) || ((addr)+(len) <= TASK_SIZE) \
> + || ((addr)+(len) <= REGION_MAX(addr)) )

Looks like the logic is reversed, should be && instead of ||.

- Ken

-
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/