Re: [PATCH] x86_64: Implement personality ADDR_LIMIT_32BIT

Arjan van de Ven
2008-10-03T12:44:31Z

On Fri, 3 Oct 2008 12:25:52 +0300
"Kirill A. Shutemov" <kirill@shutemov.name> wrote:

> On Fri, Oct 03, 2008 at 10:02:44AM +0200, Ingo Molnar wrote:
> >
> > * Kirill A. Shutemov <kirill@shutemov.name> 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.
>
> Ok, I'll fix it.
>
> >
> > but more generally, we already have ADDR_LIMIT_3GB support on x86.
>
> Does ADDR_LIMIT_3GB really work?

if it's broken we should fix it.... not invent a new one.
Also, traditionally often personalities only start at exec() time iirc.
(but I could be wrong on that)

--
Arjan van de Ven Intel Open Source Technology Centre
For development, discussion and tips for power savings,
visit http://www.lesswatts.org