Re: fix iounmap and a pageattr memleak (x86 and x86-64)

From: Andrea Arcangeli (andrea_at_novell.com)
Date: 11/05/04

  • Next message: Greg KH: "Re: [RFC] randomized major and minor numbers" 
    Date:	Fri, 5 Nov 2004 09:49:00 +0100
To: Andi Kleen <ak@suse.de>

    On Fri, Nov 05, 2004 at 09:31:02AM +0100, Andi Kleen wrote:
    > At least the NX handling is still broken on i386 (when reverting
    > back it doesn't clear the NX bit for kernel text)

    I'd never use a 32bit kernel on a x86-64 box, so that's sort of low
    interest bug to me, but acked.

    > I still don't like how you remove the reversal handling completely.

    that's to make it fully symmetric. If you're so attached to the old
    API, I'm not going to care if you want it back, but if there are 3
    symmetric users with the third going to work on a different page, my
    code will work the previous code will corrupt the mapping due the
    refcount going down too fast. If you could mention a single case where
    it would make sense not to be symmetric, I would change my mind. I find
    so much simpler to remember that as far as I'm always symmetric the
    refcounting will go right no matter what the other tasks are doing
    around. The below special case just complicates the API for no good
    reason. My problem is that it requires somebody understanding pageattr.c
    and fixing bugs on it daily to remember and to be able to use your below
    API IMHO (I had no idea myself of this undocumented below subtle detail
    until I read the code, infact it was the first thing I've removed after
    I noticed the asymmetry it generated). Bug again no problem, I'll try
    hard to remember it if we're going to keep it ;).

    > > > - if (pte_same(old,standard))
    > > > - get_page(kpte_page);
    -
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

  • Next message: Greg KH: "Re: [RFC] randomized major and minor numbers"

    Relevant Pages