xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
From: Ian Campbell <ian.campbell@citrix.com>
To: Ian Jackson <Ian.Jackson@eu.citrix.com>, Jan Beulich <JBeulich@suse.com>
Cc: xen-devel <xen-devel@lists.xen.org>,
	Wei Liu <wei.liu2@citrix.com>,
	Stefano Stabellini <Stefano.Stabellini@eu.citrix.com>
Subject: Re: OVMF related osstest failures on multiple branches
Date: Wed, 6 Jan 2016 17:08:21 +0000	[thread overview]
Message-ID: <1452100101.21055.123.camel@citrix.com> (raw)
In-Reply-To: <22157.16570.529170.710200@mariner.uk.xensource.com>

On Wed, 2016-01-06 at 16:28 +0000, Ian Jackson wrote:
> Jan Beulich writes ("Re: [Xen-devel] OVMF related osstest failures on
> multiple branches"):
> > On 06.01.16 at 16:28, <ian.campbell@citrix.com> wrote:
> > > Running xen-4.6-testing with ovmf.git 52a99493cce8 instead of
> > > cb9a7ebabcd6
> > > does seem to have worked (i.e. the flight hasn't actually finished
> > > yet but
> > > it has passed the debian-hvm-install step).
> 
> Great.  So AFAICT we would conclude that Debian jessie does not work
> with the older OVMF.

Strictly speaking Jessie's compiler cannot build a working OVMF binary.

Jessie as a guest appears to work just fine.

> > > If we want to follow [1] then the plan of attack is:
> ...
> > >  * I need to identify the patch(es) which actually fix this issue and
> > >    cherrypick it into new stable branches in ovmf.git for 4.4, 4.5
> > > and 4.6.
> > >  * Ian J to update OVMF_UPSTREAM_REVISION in the corresponding
> > > xen.git
> > >    stable branches to point to all those commits (either branch name
> > > or
> > >    SHA, not sure).
> > >  * The release checklist needs updating to include tagging this new
> > > tree
> > >    and updating OVMF_UPSTREAM_REVISION to point to the tag instead of
> > > the
> > >    commit (I think this is strictly speaking option, but we should do
> > > it).
> > >  * We might want to consider retroactively tagging the versions of
> > > ovmf
> > >    used in 4.4.[01234], 4.5.[012] and 4.6.0 in ovmf.git, which would
> > > be
> > >    helpful for people using gitk etc to look at the history I suppose
> > > 
> > > That assumes a seabios/qemut style model to updating ovmf, i.e.
> > > ungated
> > > manual Config.mk update, if we were to switch to a gate it would be
> > > different but regardless of the merits of doing things that way it
> > > does't
> > > seem like a thing to do on a stable branch.
> 
> This is quite a lot of work.  Doing this work would perhaps make sense
> if we had a reasonable idea what was going on in ovmf.git, which
> changes to cherry pick to stable branches, etc.  But AFAICT we don't.

FWIW adding just 39ef30bb47b6 to cb9a7ebabcd6 seems to have worked, but
finding that was based on a search for "GCC" in the commit log and a wild
hunch ;-).

> So I would be tempted to just update the Config.mk reference in stable
> trees.

That's my inclination too.

Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

  reply	other threads:[~2016-01-06 17:08 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-01-06 12:35 OVMF related osstest failures on multiple branches Ian Campbell
2016-01-06 14:27 ` Ian Campbell
2016-01-06 15:28   ` Ian Campbell
2016-01-06 16:20     ` Jan Beulich
2016-01-06 16:28       ` Ian Jackson
2016-01-06 17:08         ` Ian Campbell [this message]
2016-01-06 17:14           ` Ian Jackson
2016-01-07 15:36             ` Ian Jackson
2016-01-07 15:42               ` Ian Campbell
2016-01-07 15:47                 ` Ian Jackson
2016-01-07 15:53                   ` Ian Jackson
2016-01-06 16:50     ` Wei Liu

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1452100101.21055.123.camel@citrix.com \
    --to=ian.campbell@citrix.com \
    --cc=Ian.Jackson@eu.citrix.com \
    --cc=JBeulich@suse.com \
    --cc=Stefano.Stabellini@eu.citrix.com \
    --cc=wei.liu2@citrix.com \
    --cc=xen-devel@lists.xen.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).