All of lore.kernel.org
 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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.