Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753539AbbGaPGJ (ORCPT ); Fri, 31 Jul 2015 11:06:09 -0400 Received: from mx1.redhat.com ([209.132.183.28]:48316 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753199AbbGaPGG (ORCPT ); Fri, 31 Jul 2015 11:06:06 -0400 Message-ID: <55BB8ED8.7090803@redhat.com> Date: Fri, 31 Jul 2015 17:06:00 +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 28/36] mm, numa: skip PTE-mapped THP on numa fault References: <1437402069-105900-1-git-send-email-kirill.shutemov@linux.intel.com> <1437402069-105900-29-git-send-email-kirill.shutemov@linux.intel.com> In-Reply-To: <1437402069-105900-29-git-send-email-kirill.shutemov@linux.intel.com> Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="AXweow82f5MAWuNrXSD9tdLGEwQm0wnHO" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2167 Lines: 67 This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --AXweow82f5MAWuNrXSD9tdLGEwQm0wnHO Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: quoted-printable On 07/20/2015 04:21 PM, Kirill A. Shutemov wrote: > We're going to have THP mapped with PTEs. It will confuse numabalancing= =2E > Let's skip them for now. Fair enough. Acked-by: Jerome Marchand >=20 > Signed-off-by: Kirill A. Shutemov > Tested-by: Sasha Levin > Tested-by: Aneesh Kumar K.V > --- > mm/memory.c | 6 ++++++ > 1 file changed, 6 insertions(+) >=20 > diff --git a/mm/memory.c b/mm/memory.c > index 074edab89b52..52f6fa02c099 100644 > --- a/mm/memory.c > +++ b/mm/memory.c > @@ -3186,6 +3186,12 @@ static int do_numa_page(struct mm_struct *mm, st= ruct vm_area_struct *vma, > return 0; > } > =20 > + /* TODO: handle PTE-mapped THP */ > + if (PageCompound(page)) { > + pte_unmap_unlock(ptep, ptl); > + return 0; > + } > + > /* > * Avoid grouping on RO pages in general. RO pages shouldn't hurt as > * much anyway since they can be in shared cache state. This misses >=20 --AXweow82f5MAWuNrXSD9tdLGEwQm0wnHO 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 iQEcBAEBCAAGBQJVu47YAAoJEHTzHJCtsuoCbasIAJgqsduVoqeOUNlUaQ1WP+tp HirYtRMig2zvYlx3TPq9M5xribvKtTKRMvsyhSZb3t6u1y5lCbCgr+K+cBtMTL5Z KF744MQuVxRpL8FDfTbwT+eYW/4uLIemMx8Fao0fGZibB7j7rZeB/aKy9dC3t0+V tYw6OHkeiVoSrrMLxe1B9VLWBI01iHFRcy4CcqxuTHlkW15RgKfm6GY/5dUf0wj3 js764sshaBdYBbYmZpzadji3362kpGFtDy/BMib+WtEN/kgISEb1a/WLKKHhRjTX KfyRfa8yQ4yl3hGsiOG3u2ylVW+uiWy/SdlBp3RUAtLXn2uHcPHeJ2VsVFXAZR8= =0iGW -----END PGP SIGNATURE----- --AXweow82f5MAWuNrXSD9tdLGEwQm0wnHO-- -- 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/