xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
From: Ian Campbell <ian.campbell@citrix.com>
To: Wei Liu <wei.liu2@citrix.com>, Doug Goldstein <cardoe@cardoe.com>
Cc: Tim Deegan <tim@xen.org>, Keir Fraser <keir@xen.org>,
	Ian Jackson <ian.jackson@eu.citrix.com>,
	Jan Beulich <jbeulich@suse.com>,
	xen-devel@lists.xen.org
Subject: Re: [PATCH] Config.mk: Update SEABIOS_UPSTREAM_TAG to rel-1.9.1
Date: Tue, 16 Feb 2016 13:43:26 +0000	[thread overview]
Message-ID: <1455630206.814.82.camel@citrix.com> (raw)
In-Reply-To: <20160215161354.GA11740@citrix.com>

On Mon, 2016-02-15 at 16:13 +0000, Wei Liu wrote:
> On Mon, Feb 15, 2016 at 08:38:02AM -0600, Doug Goldstein wrote:
> > Switch from tracking a commit post 1.9.0 to the 1.9.1 release.
> > 
> 
> Thanks for this patch.
> 
> We can't apply this patch just yet. We need to wait a bit until the
> change to upstream trickles down to our own tree [0] -- OSSTest will
> work on that shortly. See [1] for an example output.

Our osstest branch only tests seabios.git#master whereas this is tracking
the 1.9.x stable branch.

I don't think we really want/need the overhead of testing multiple branches
of SeaBIOS, so IMHO it is sufficient to observe that all of the commits on
the stable branch have passed our master branch commit gate (even if that
means they did so in the context of other un-backported patches). I haven't
checked but given the timings and http://logs.test-lab.xenproject.org/osste
st/results/all-branch-statuses.txt indicating our seabios branch is up to
date I expect this is the case.

There is some risk to this, but SeaBIOS gets a fairly small number of
changes so the stable branches do not stray too far from master in general
so I think this approach is fine.

IOW it would be fine to push upstreams 1.9-stable to our repo along with
the rel-1.9.1 tag and to then apply this patch.


> 
> Wei.
> 
> [0] git://xenbits.xen.org/osstest/seabios.git
> [1] <osstest-80975-mainreport@xen.org>
> 
> > Signed-off-by: Doug Goldstein <cardoe@cardoe.com>
> > ---
> > 
> > Please make sure to sync git://xenbits.xen.org/seabios.git to
> > upstream prior to pushing this to staging.
> > 
> > ---
> >  Config.mk | 6 +++---
> >  1 file changed, 3 insertions(+), 3 deletions(-)
> > 
> > diff --git a/Config.mk b/Config.mk
> > index 429e460..79eb2bd 100644
> > --- a/Config.mk
> > +++ b/Config.mk
> > @@ -255,9 +255,9 @@ MINIOS_UPSTREAM_REVISION ?=
> > 89268f00b0b0215057cb74edd94e866536a02489
> >  # Fri Jan 15 13:24:03 2016 +0000
> >  # mini-os: Include libxenforeignmemory with libxc
> >  
> > -SEABIOS_UPSTREAM_REVISION ?= 3403ac4313812752be6e6aac35239ca6888a8cab
> > -# Mon Dec 28 13:50:41 2015 +0100
> > -# build: fix typo in buildversion.py
> > +SEABIOS_UPSTREAM_REVISION ?= rel-1.9.1
> > +# Fri Jan 15 15:49:32 2016 +0000
> > +# biostables: Support SMBIOS 2.6+ UUID format
> >  
> >  ETHERBOOT_NICS ?= rtl8139 8086100e
> >  

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

  reply	other threads:[~2016-02-16 13:43 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-02-15 14:38 [PATCH] Config.mk: Update SEABIOS_UPSTREAM_TAG to rel-1.9.1 Doug Goldstein
2016-02-15 16:13 ` Wei Liu
2016-02-16 13:43   ` Ian Campbell [this message]
2016-02-16 13:52     ` Wei Liu
2016-02-24 14:34       ` Doug Goldstein
2016-02-24 14:41         ` Wei Liu
2016-02-24 15:14           ` Jan Beulich
2016-02-24 15:18             ` Wei Liu
2016-02-24 15:44               ` Jan Beulich
2016-02-24 15:50                 ` Wei Liu
2016-02-26 19:02                   ` Konrad Rzeszutek Wilk

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=1455630206.814.82.camel@citrix.com \
    --to=ian.campbell@citrix.com \
    --cc=cardoe@cardoe.com \
    --cc=ian.jackson@eu.citrix.com \
    --cc=jbeulich@suse.com \
    --cc=keir@xen.org \
    --cc=tim@xen.org \
    --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).