From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-ea0-f172.google.com (mail-ea0-f172.google.com [209.85.215.172]) by kanga.kvack.org (Postfix) with ESMTP id 2630F6B0031 for ; Thu, 21 Nov 2013 09:38:26 -0500 (EST) Received: by mail-ea0-f172.google.com with SMTP id q10so2417747ead.17 for ; Thu, 21 Nov 2013 06:38:25 -0800 (PST) Received: from mx1.redhat.com (mx1.redhat.com. [209.132.183.28]) by mx.google.com with ESMTP id s8si20119321eeh.17.2013.11.21.06.38.22 for ; Thu, 21 Nov 2013 06:38:22 -0800 (PST) Date: Thu, 21 Nov 2013 09:38:17 -0500 From: Naoya Horiguchi Message-ID: <1385044697-rn5og2ir-mutt-n-horiguchi@ah.jp.nec.com> In-Reply-To: <1385038810-15513-1-git-send-email-kirill.shutemov@linux.intel.com> References: <1385038810-15513-1-git-send-email-kirill.shutemov@linux.intel.com> Subject: Re: [PATCH] mm: place page->pmd_huge_pte to right union Mime-Version: 1.0 Content-Type: text/plain; charset=iso-2022-jp Content-Transfer-Encoding: 7bit Content-Disposition: inline Sender: owner-linux-mm@kvack.org List-ID: To: "Kirill A. Shutemov" Cc: Andrew Morton , linux-mm@kvack.org, linux-kernel@vger.kernel.org On Thu, Nov 21, 2013 at 03:00:10PM +0200, Kirill A. Shutemov wrote: > I don't know what went wrong, mis-merge or something, but ->pmd_huge_pte > placed in wrong union within struct page. > > In original patch[1] it's placed to union with ->lru and ->slab, but in > commit e009bb30c8df it's in union with ->index and ->freelist. > > That union seems also unused for pages with table tables and safe to > re-use, but it's not what I've tested. > > Let's move it to original place. It fixes indentation at least. :) > > [1] https://lkml.org/lkml/2013/10/7/288 > > Signed-off-by: Kirill A. Shutemov Reviewed-by: Naoya Horiguchi > --- > include/linux/mm_types.h | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/include/linux/mm_types.h b/include/linux/mm_types.h > index 10f5a7272b80..011eb85d7b0f 100644 > --- a/include/linux/mm_types.h > +++ b/include/linux/mm_types.h > @@ -65,9 +65,6 @@ struct page { > * this page is only used to > * free other pages. > */ > -#if defined(CONFIG_TRANSPARENT_HUGEPAGE) && USE_SPLIT_PMD_PTLOCKS > - pgtable_t pmd_huge_pte; /* protected by page->ptl */ > -#endif > }; > > union { > @@ -135,6 +132,9 @@ struct page { > > struct list_head list; /* slobs list of pages */ > struct slab *slab_page; /* slab fields */ > +#if defined(CONFIG_TRANSPARENT_HUGEPAGE) && USE_SPLIT_PMD_PTLOCKS > + pgtable_t pmd_huge_pte; /* protected by page->ptl */ > +#endif > }; > > /* Remainder is not double word aligned */ > -- > 1.8.4.3 > > -- > To unsubscribe, send a message with 'unsubscribe linux-mm' in > the body to majordomo@kvack.org. For more info on Linux MM, > see: http://www.linux-mm.org/ . > Don't email: email@kvack.org > -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org