Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754171AbbGaOvw (ORCPT ); Fri, 31 Jul 2015 10:51:52 -0400 Received: from mx1.redhat.com ([209.132.183.28]:57313 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753565AbbGaOvv (ORCPT ); Fri, 31 Jul 2015 10:51:51 -0400 Message-ID: <55BB8B81.5030904@redhat.com> Date: Fri, 31 Jul 2015 16:51:45 +0200 From: Jerome Marchand User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.7.0 MIME-Version: 1.0 To: "Kirill A. Shutemov" , Andrew Morton , Andrea Arcangeli , Hugh Dickins CC: Dave Hansen , Mel Gorman , Rik van Riel , Vlastimil Babka , Christoph Lameter , Naoya Horiguchi , Steve Capper , "Aneesh Kumar K.V" , Johannes Weiner , Michal Hocko , Sasha Levin , linux-kernel@vger.kernel.org, linux-mm@kvack.org Subject: Re: [PATCHv9 13/36] mm: drop tail page refcounting References: <1437402069-105900-1-git-send-email-kirill.shutemov@linux.intel.com> <1437402069-105900-14-git-send-email-kirill.shutemov@linux.intel.com> In-Reply-To: <1437402069-105900-14-git-send-email-kirill.shutemov@linux.intel.com> Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="xx3CWnb6KwdlmaRsBXgsoBIagJpT3B1cd" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2877 Lines: 74 This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --xx3CWnb6KwdlmaRsBXgsoBIagJpT3B1cd Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: quoted-printable On 07/20/2015 04:20 PM, Kirill A. Shutemov wrote: > Tail page refcounting is utterly complicated and painful to support. >=20 > It uses ->_mapcount on tail pages to store how many times this page is > pinned. get_page() bumps ->_mapcount on tail page in addition to > ->_count on head. This information is required by split_huge_page() to > be able to distribute pins from head of compound page to tails during > the split. >=20 > We will need ->_mapcount to account PTE mappings of subpages of the > compound page. We eliminate need in current meaning of ->_mapcount in > tail pages by forbidding split entirely if the page is pinned. >=20 > The only user of tail page refcounting is THP which is marked BROKEN fo= r > now. >=20 > Let's drop all this mess. It makes get_page() and put_page() much > simpler. >=20 > Signed-off-by: Kirill A. Shutemov > Tested-by: Sasha Levin > Tested-by: Aneesh Kumar K.V > Acked-by: Vlastimil Babka Acked-by: Jerome Marchand > --- > arch/mips/mm/gup.c | 4 - > arch/powerpc/mm/hugetlbpage.c | 13 +- > arch/s390/mm/gup.c | 13 +- > arch/sparc/mm/gup.c | 14 +-- > arch/x86/mm/gup.c | 4 - > include/linux/mm.h | 47 ++------ > include/linux/mm_types.h | 17 +-- > mm/gup.c | 34 +----- > mm/huge_memory.c | 41 +------ > mm/hugetlb.c | 2 +- > mm/internal.h | 44 ------- > mm/swap.c | 273 +++-------------------------------= -------- > 12 files changed, 40 insertions(+), 466 deletions(-) >=20 --xx3CWnb6KwdlmaRsBXgsoBIagJpT3B1cd Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQEcBAEBCAAGBQJVu4uBAAoJEHTzHJCtsuoCoSMH/RC/90otzHeGxEJbQVU91ffO ESX1zg2XbiTYFiTlYlwikb/s5AjMg9WvF6UI8THl1eY04b2ybIvzJjAK3QkdT0YA RixbpOejf9HamG4GtSx5HjUEi/q7n95GPA/o6yvdNLKu+nYcAQXkysvuqy191B8r qD+vKamNg88XkE9NG1uFloPko/zu7PawMzKluVyL7EQpfKGD5oEKSMzsEv3X/n6I N7SqdrW4E+XLMIpv1JwSbNlnEySzMHxgVcDA1sXlHFy+PEVQSAvgpghcptveXtO2 wc3JdH+YXKNpbyWqG7mZGKnnkDxf1BVVqK3ujdc9wlzaYsvpRQ1zW+MtprUMuqY= =ABSG -----END PGP SIGNATURE----- --xx3CWnb6KwdlmaRsBXgsoBIagJpT3B1cd-- -- 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/