From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mx1.redhat.com ([209.132.183.28]) by bombadil.infradead.org with esmtp (Exim 4.80.1 #2 (Red Hat Linux)) id 1WwsQe-0002wc-58 for kexec@lists.infradead.org; Tue, 17 Jun 2014 12:24:00 +0000 Date: Tue, 17 Jun 2014 08:23:34 -0400 From: Vivek Goyal Subject: Re: kexec-save-pg_head_mask-in-vmcoreinfo.patch Message-ID: <20140617122334.GA28340@redhat.com> References: <20140609153153.9766367624fcc723eaac7a15@linux-foundation.org> <20140610153554.GC17337@redhat.com> <20140611132042.1621a230@hananiah.suse.cz> <20140611172422.GG10723@redhat.com> <20140612170436.51e6338d@hananiah.suse.cz> <20140617081001.3e0e1ac9@hananiah.suse.cz> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20140617081001.3e0e1ac9@hananiah.suse.cz> List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "kexec" Errors-To: kexec-bounces+dwmw2=infradead.org@lists.infradead.org To: Petr Tesarik Cc: Haren Myneni , Andrew Morton , kexec@lists.infradead.org, "Eric W. Biederman" On Tue, Jun 17, 2014 at 08:10:01AM +0200, Petr Tesarik wrote: > On Thu, 12 Jun 2014 17:04:36 +0200 > Petr Tesarik wrote: > > > On Wed, 11 Jun 2014 13:24:22 -0400 > > Vivek Goyal wrote: > > > > > On Wed, Jun 11, 2014 at 01:20:42PM +0200, Petr Tesarik wrote: > > > > On Tue, 10 Jun 2014 11:35:54 -0400 > > > > Vivek Goyal wrote: > > > > > > > > > On Mon, Jun 09, 2014 at 03:31:53PM -0700, Andrew Morton wrote: > > > > > > > > > > > > > > > > > > Guys, could you please review this one? > > > > > > > > > > > > Thanks. > > > > > > > > > > > > > > > > > > From: Petr Tesarik > > > > > > Subject: kexec: save PG_head_mask in VMCOREINFO > > > > > > > > > > > > To allow filtering of huge pages, makedumpfile must be able to identify > > > > > > them in the dump. This can be done by checking the appropriate page flag, > > > > > > so communicate its value to makedumpfile through the VMCOREINFO interface. > > > > > > > > > > > > There's only one small catch. Depending on how many page flags are > > > > > > available on a given architecture, this bit can be called PG_head or > > > > > > PG_compound. > > > > > > > > > > > > I sent a similar patch back in 2012, but Eric Biederman did not like using > > > > > > an #ifdef. So, this time I'm adding a common symbol (PG_head_mask) > > > > > > instead. > > > > > > > > > > > > See https://lkml.org/lkml/2012/11/28/91 for the previous version. > > > > > > > > > > > > > > > > Hi Petr, > > > > > > > > > > I went through previous conversation and personally I like that patch > > > > > better. > > > > > > > > > > We already have so many exports in vmcoreinfo which are conditional > > > > > based on config options. Adding one more should be just fine. And as > > > > > you said that it should not bitrot as it will most likely lead > > > > > to complication failures if things shift around. > > > > > > > > > > Does this patch compile with !CONFIG_PAGEFLAGS_EXTENDED. I did not see > > > > > a definition of PG_head_mask in that case. > > > > > > > > With !CONFIG_PAGEFLAGS_EXTENDED, the definition of PG_head_mask is > > > > already there (and has been for quite some time). > > > > > > Oh.., I had not noticed that. Thanks. > > > > > > > > > > > > So I would say post your patch again, I will ack it. CC Eric and hopefully > > > > > he does not have any objections to it. > > Oh, it seems you're still waiting for Eric's opinion, but I didn't > notice that he wasn't in Cc yet. That was in the context of when I said repost the old patch. Later I agreed to existing patch (The one with PG_head_mask) and I have acked it. Thanks Vivek _______________________________________________ kexec mailing list kexec@lists.infradead.org http://lists.infradead.org/mailman/listinfo/kexec