From mboxrd@z Thu Jan  1 00:00:00 1970
From: Ian Campbell <ian.campbell@citrix.com>
Subject: Re: pvgrub regression in xen.git 123c77937975
Date: Sun, 10 May 2015 14:32:44 +0100
Message-ID: <1431264764.16382.5.camel@citrix.com>
References: <1431101809.2660.536.camel@citrix.com>
	<1431238749.15640.316.camel@citrix.com>
	<20150510122018.GA6996@zion.uk.xensource.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xen.org>
In-Reply-To: <20150510122018.GA6996@zion.uk.xensource.com>
List-Unsubscribe: <http://lists.xen.org/cgi-bin/mailman/options/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xen.org>
List-Help: <mailto:xen-devel-request@lists.xen.org?subject=help>
List-Subscribe: <http://lists.xen.org/cgi-bin/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=subscribe>
Sender: xen-devel-bounces@lists.xen.org
Errors-To: xen-devel-bounces@lists.xen.org
To: Wei Liu <wei.liu2@citrix.com>
Cc: Ian Jackson <Ian.Jackson@eu.citrix.com>, xen-devel <xen-devel@lists.xen.org>
List-Id: xen-devel@lists.xenproject.org

On Sun, 2015-05-10 at 13:20 +0100, Wei Liu wrote:
> On Sun, May 10, 2015 at 07:19:09AM +0100, Ian Campbell wrote:
> > On Fri, 2015-05-08 at 17:16 +0100, Ian Campbell wrote:
> > > There seems to be a pvgrub regression somewhere in the range
> > > 3a28f760508f..123c77937975
> > 
> > The bisector has fingered:
> > 
> >         commit 840837907c6186307c19abbec926852ba448facd
> >           Author: Wei Liu <wei.liu2@citrix.com>
> >           Date:   Mon Mar 16 09:52:22 2015 +0000
> >           
> >               libxc: add p2m_size to xc_dom_image
> >         
> > Full report below. I've put a copy of the graph at:
> > http://xenbits.xen.org/people/ianc/tmp/201505/bisect.pvgrub.html
> > 
> > Wei, you can see an instance of the failure at
> > http://osstest.xs.citrite.net/~osstest/testlogs/logs/37344/
> 
> Unfortunately Citrix access gateway doesn't work for me at the moment so
> I couldn't see the log.

FWIW http://xenbits.xen.org/people/ianc/tmp/201505/37344/ (no build-*
logs, just test-*), but it looks like you don't actually need them.

> But I've looked at the code. PV grub's kexec function doesn't update the
> newly added field. I will try to reproduce on my local test box.

I've seen your patch, will take a proper look tomorrow.

Thanks for the quick turnaround!

Ian.