Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754639AbbGXReX (ORCPT ); Fri, 24 Jul 2015 13:34:23 -0400 Received: from mail-la0-f54.google.com ([209.85.215.54]:32776 "EHLO mail-la0-f54.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753770AbbGXReV (ORCPT ); Fri, 24 Jul 2015 13:34:21 -0400 MIME-Version: 1.0 In-Reply-To: <20150714152516.29844.69929.stgit@buzz> References: <20150714152516.29844.69929.stgit@buzz> Date: Fri, 24 Jul 2015 18:34:19 +0100 Message-ID: Subject: Re: [PATCHSET v4 0/5] pagemap: make useable for non-privilege users From: Mark Williamson To: Konstantin Khlebnikov Cc: linux-mm@kvack.org, Andrew Morton , Naoya Horiguchi , "Kirill A. Shutemov" , kernel list , Linux API Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1996 Lines: 53 Hi Konstantin, Thank you for the further update - I tested this patchset against our code and it allows our software to work correctly (with minor userland changes, as before). I'll follow up with review messages but there aren't really any concerns that I can see. Cheers, Mark On Tue, Jul 14, 2015 at 4:37 PM, Konstantin Khlebnikov wrote: > This patchset makes pagemap useable again in the safe way (after row hammer > bug it was made CAP_SYS_ADMIN-only). This patchset restores access for > non-privileged users but hides PFNs from them. > > Also it adds bit 'map-exlusive' which is set if page is mapped only here: > it helps in estimation of working set without exposing pfns and allows to > distinguish CoWed and non-CoWed private anonymous pages. > > Second patch removes page-shift bits and completes migration to the new > pagemap format: flags soft-dirty and mmap-exlusive are available only > in the new format. > > Changes since v3: > * patches reordered: cleanup now in second patch > * update pagemap for hugetlb, add missing 'FILE' bit > * fix PM_PFRAME_BITS: its 55 not 54 as was in previous versions > > --- > > Konstantin Khlebnikov (5): > pagemap: check permissions and capabilities at open time > pagemap: switch to the new format and do some cleanup > pagemap: rework hugetlb and thp report > pagemap: hide physical addresses from non-privileged users > pagemap: add mmap-exclusive bit for marking pages mapped only here > > > Documentation/vm/pagemap.txt | 3 > fs/proc/task_mmu.c | 267 ++++++++++++++++++------------------------ > tools/vm/page-types.c | 35 +++--- > 3 files changed, 137 insertions(+), 168 deletions(-) > > -- > Konstantin -- 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/