Re: [PATCH] x86_64: Implement personality ADDR_LIMIT_32BIT

From: Ingo Molnar
Date: Fri Oct 03 2008 - 04:03:31 EST



* Kirill A. Shutemov <kirill@xxxxxxxxxxxxx> wrote:

> - /* for MAP_32BIT mappings we force the legact mmap base */
> - if (!test_thread_flag(TIF_IA32) && (flags & MAP_32BIT))
> + /* for MAP_32BIT mappings and ADDR_LIMIT_32BIT personality we force the
> + * legact mmap base
> + */

please use the customary multi-line comment style:

/*
* Comment .....
* ...... goes here:
*/

and you might use the opportunity to fix the s/legact/legacy typo as
well.

but more generally, we already have ADDR_LIMIT_3GB support on x86. Why
should support for ADDR_LIMIT_32BIT be added?

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