[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAEwNFnB+5DZQw0CrfkcdP_rd5FBM6fw26=_yS3P283a9TKLNiw@mail.gmail.com>
Date: Fri, 12 Aug 2011 13:34:17 +0900
From: Minchan Kim <minchan.kim@...il.com>
To: Linus Torvalds <torvalds@...ux-foundation.org>
Cc: Avi Kivity <avi@...hat.com>, Marcelo Tosatti <mtosatti@...hat.com>,
Jan Kiszka <jan.kiszka@....de>, qemu-devel@...gnu.org,
agraf@...e.de, kvm <kvm@...r.kernel.org>,
Paul Mackerras <paulus@...ba.org>,
linux-kernel@...r.kernel.org, Hugh Dickins <hughd@...gle.com>,
KOSAKI Motohiro <kosaki.motohiro@...fujitsu.com>,
Andrew Morton <akpm@...ux-foundation.org>,
KAMEZAWA Hiroyuki <kamezawa.hiroyu@...fujitsu.com>,
Mel Gorman <mgorman@...e.de>, Andrew Hastings <abh@...y.com>,
Adam Litke <agl@...ibm.com>
Subject: Re: Fix refcounting in hugetlbfs quota handling
On Fri, Aug 12, 2011 at 9:48 AM, Linus Torvalds
<torvalds@...ux-foundation.org> wrote:
> On Wed, Aug 10, 2011 at 11:40 PM, David Gibson
> <david@...son.dropbear.id.au> wrote:
>>
>> This patch, therefore, stores a pointer to the inode instead of the
>> address_space in the page private data for hugepages. More
>> importantly it correctly adjusts the reference count on the inodes
>> when they're added to the page private data. This ensures that the
>> inode (and therefore the super block) will not be freed before we use
>> it from free_huge_page.
>
> Looks sane, but I *really* want some acks from people who use/know
> hugetlbfs. Who would that be? I'm adding random people who have
> acked/signed-off patches to hugetlbfs recently..
At least, code itself looks good to me but your random choice was failed.
Maybe people you want are as follows.
http://marc.info/?t=126928975800003&r=1&w=2
Ccing right persons.
--
Kind regards,
Minchan Kim
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists