Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934066AbbEMLkS (ORCPT ); Wed, 13 May 2015 07:40:18 -0400 Received: from forward-corp1m.cmail.yandex.net ([5.255.216.100]:35320 "EHLO forward-corp1m.cmail.yandex.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933852AbbEMLkC (ORCPT ); Wed, 13 May 2015 07:40:02 -0400 Authentication-Results: smtpcorp1m.mail.yandex.net; dkim=pass header.i=@yandex-team.ru Message-ID: <5553380A.1040605@yandex-team.ru> Date: Wed, 13 May 2015 14:39:54 +0300 From: Konstantin Khlebnikov User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0 MIME-Version: 1.0 To: "Kirill A. Shutemov" CC: linux-mm@kvack.org, Naoya Horiguchi , linux-kernel@vger.kernel.org, Andrew Morton , Mark Williamson , Pavel Emelyanov , linux-api@vger.kernel.org, Andy Lutomirski , Vlastimil Babka , Pavel Machek , Mark Seaborn , Linus Torvalds , Daniel James , Finn Grimwood Subject: Re: [PATCH v2 3/3] pagemap: switch to the new format and do some cleanup References: <20150512090156.24768.2521.stgit@buzz> <20150512094306.24768.51325.stgit@buzz> <20150512105431.GD18365@node.dhcp.inet.fi> In-Reply-To: <20150512105431.GD18365@node.dhcp.inet.fi> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2253 Lines: 51 On 12.05.2015 13:54, Kirill A. Shutemov wrote: > On Tue, May 12, 2015 at 12:43:06PM +0300, Konstantin Khlebnikov wrote: >> This patch removes page-shift bits (scheduled to remove since 3.11) and >> completes migration to the new bit layout. Also it cleans messy macro. >> >> Signed-off-by: Konstantin Khlebnikov >> --- >> fs/proc/task_mmu.c | 152 ++++++++++++++++--------------------------------- >> tools/vm/page-types.c | 29 +++------ >> 2 files changed, 58 insertions(+), 123 deletions(-) >> >> diff --git a/fs/proc/task_mmu.c b/fs/proc/task_mmu.c >> index 0b7a8ffec95f..66bc7207ce90 100644 >> --- a/fs/proc/task_mmu.c >> +++ b/fs/proc/task_mmu.c >> @@ -710,23 +710,6 @@ const struct file_operations proc_tid_smaps_operations = { >> .release = proc_map_release, >> }; >> >> -/* >> - * We do not want to have constant page-shift bits sitting in >> - * pagemap entries and are about to reuse them some time soon. >> - * >> - * Here's the "migration strategy": >> - * 1. when the system boots these bits remain what they are, >> - * but a warning about future change is printed in log; >> - * 2. once anyone clears soft-dirty bits via clear_refs file, >> - * these flag is set to denote, that user is aware of the >> - * new API and those page-shift bits change their meaning. >> - * The respective warning is printed in dmesg; >> - * 3. In a couple of releases we will remove all the mentions >> - * of page-shift in pagemap entries. >> - */ > > Wouldn't it be better to just have v2=1 by default for couple releases to > see if anything breaks? This way we can revert easily if regression reported. > I guess someone could miss this change coming if he didn't touch clear_refs. > I don't believe that constant PAGE_SHIFT bits are used by anybody. Recent change of permissions was much more destructive and there is just one report about that. Kernel prints message at first pagemap open for ten releases. I think that's enough. -- 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/