xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
From: Wei Liu <wei.liu2@citrix.com>
To: Ian Campbell <ian.campbell@citrix.com>
Cc: xen-devel@lists.xenproject.org, Wei Liu <wei.liu2@citrix.com>,
	Mark Pryor <pryorm09@gmail.com>, Jan Beulich <JBeulich@suse.com>,
	"Luis R. Rodriguez" <mcgrof@do-not-panic.com>
Subject: Re: [PATCH seabios.git rel-1.7.5] fix release-1.7.5 for gcc5
Date: Tue, 9 Feb 2016 16:30:48 +0000	[thread overview]
Message-ID: <20160209163048.GA18470@citrix.com> (raw)
In-Reply-To: <1455029873.19857.86.camel@citrix.com>

On Tue, Feb 09, 2016 at 02:57:53PM +0000, Ian Campbell wrote:
> On Tue, 2016-02-09 at 09:58 +0000, Wei Liu wrote:
> > On Fri, Feb 05, 2016 at 05:39:39PM -0800, Luis R. Rodriguez wrote:
> > > On Thu, Nov 19, 2015 at 3:06 PM, Luis R. Rodriguez
> > > <mcgrof@do-not-panic.com> wrote:
> > > > On Fri, Oct 9, 2015 at 12:34 AM, Jan Beulich <JBeulich@suse.com>
> > > > wrote:
> > > > > > > > On 08.10.15 at 21:36, <pryorm09@gmail.com> wrote:
> > > > > > Signed-off-by: Mark Pryor <pryorm09@gmail.com>
> > > > > 
> > > > > Without any description I cannot see what is being fixed here, or
> > > > > why
> > > > > there are _different_ comment changes on the inclusion of the same
> > > > > comment. Since I assume that whatever issue there was has been
> > > > > taken care of upstream, I'd suggest - just like for the other patch
> > > > > - to
> > > > > instead request inclusion of the respective upstream commit (again
> > > > > obeying ./MAINTAINERS of the affected Xen branch).
> > > > 
> > > > I'll submit a proper replacement patch with valid description.
> > > 
> > > I submitted a replacement a while ago...
> > > 
> > 
> > Can you point me to the message-id of you email to xen-devel?
> 
>     <    1448041672-3986-2-git-send-email-mcgrof@do-not-panic.com    >
> 
> > I can't seem to be able to find it in my archive.
> > 
> > FWIW I use "~f Luis ~h seabios" pattern in mutt and only found this
> > thread.
> 
> It was:
> 	[PATCH 1/9] Revert "Use the extra stack for 16bit USB and PS2 keyboard/mouse commands."
> 
> which is something Jan just gave some feedback on this morning in
>     <    56B9AC7D02000078000CFDE9@prv-mh.provo.novell.com>
> 
> I think that commit is in the version of SeaBIOS referenced by 4.6.0
> already, since xen 4.6.0 points to seabios rel-1.8.2 and 
> 
> $ git describe --contains 12085439561f44107c7d3bea5a333a90ea987643
> rel-1.8.0~94
> 

OK. Nothing for me to worry about then. I just checked, it already is
present in our tree.

Wei.

> Ian.

      reply	other threads:[~2016-02-09 16:30 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-10-08 19:36 [PATCH seabios.git rel-1.7.5] fix release-1.7.5 for gcc5 Mark Pryor
2015-10-09  7:34 ` Jan Beulich
2015-11-19 23:06   ` Luis R. Rodriguez
2016-02-06  1:39     ` Luis R. Rodriguez
2016-02-09  9:58       ` Wei Liu
2016-02-09 14:57         ` Ian Campbell
2016-02-09 16:30           ` Wei Liu [this message]

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=20160209163048.GA18470@citrix.com \
    --to=wei.liu2@citrix.com \
    --cc=JBeulich@suse.com \
    --cc=ian.campbell@citrix.com \
    --cc=mcgrof@do-not-panic.com \
    --cc=pryorm09@gmail.com \
    --cc=xen-devel@lists.xenproject.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).