This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
Home Products Support Community News


Re: Q: Clarification about extra option ..Re: [Xen-devel] Re: [PATCH] pv

To: Shriram Rajagopalan <rshriram@xxxxxxxxx>
Subject: Re: Q: Clarification about extra option ..Re: [Xen-devel] Re: [PATCH] pvops: Make suspend work when CONFIG_SUSPEND=n
From: Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx>
Date: Mon, 7 Mar 2011 12:53:25 -0500
Cc: Ian Campbell <Ian.Campbell@xxxxxxxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>, Frank Pan <frankpzh@xxxxxxxxx>, Jeremy Fitzhardinge <Jeremy.Fitzhardinge@xxxxxxxxxx>, Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx>
Delivery-date: Mon, 07 Mar 2011 09:54:52 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <AANLkTimL2srKo2n0jEfWzyexByfaCRg34UkVSNPRRqXL@xxxxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <AANLkTikcbVEXdRsS1WYCNTinuYLcmiJ1dz2ReS18_A+3@xxxxxxxxxxxxxx> <201103042107.55975.rjw@xxxxxxx> <AANLkTine64a2=uvx==jcgJY3ssOshGjm64OVXT4m9E=r@xxxxxxxxxxxxxx> <201103062312.07555.rjw@xxxxxxx> <alpine.DEB.2.00.1103071243290.2968@kaball-desktop> <AANLkTimL2srKo2n0jEfWzyexByfaCRg34UkVSNPRRqXL@xxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mutt/1.5.20 (2009-06-14)
On Mon, Mar 07, 2011 at 08:33:25AM -0800, Shriram Rajagopalan wrote:
> On Mon, Mar 7, 2011 at 4:44 AM, Stefano Stabellini <
> stefano.stabellini@xxxxxxxxxxxxx> wrote:
> > On Sun, 6 Mar 2011, Rafael J. Wysocki wrote:
> > > > The only issue is that I cannot completely "test" these two patches
> > > > against Rafael's tree
> > > >  - I have verified that the kernel config file generated is as
> > expected.
> > > >  - I cannot verify any other xen save/restore functionality as my xen
> > > >    suspend freeze-thaw patches dont apply cleanly on Rafael's tree
> > > >   (it does not have xen suspend refactoring patches
> > > >   ceb180294790c8a6a437533488616f6b591b49d0, that my patches depend on.
> > > >   They are present only in Stefano's tree).
> > >
> > > In that case, I'm afraid, it's better to wait until both trees are merged
> > > and push your patches at that time.

<nods> But we need to review/test/etc his patches before the merge window..
> >
> > I agree even if it probably means loosing the next merge window.
> >
> > I believe the original intention of the freeze-thaw patches was to fix the
> bug that caused
> Guest VM hangs, while taking a checkpoint.
> Anyway, is there any chance of pushing these patches into the pvops tree
> atleast?
> I can rebase them to pvops.

so Shriram could create a branch that has both Stefano's (and also Ian's 
and Rafeal's tree and you can use that. And when the merge window happens
we would let Stefano's and Rafeal's tree go in and after that ask Linus to pull
this complicated tree.

That means you are going to have to create this hybrid tree. You can create a 
git branch
from Stefano's tree, merge in Rafael's tree, and test it (baremetal and also 
Once you are satisfied that it works and it is bug-free:

 a). send the tar ball to me (or just a link).
 b). I am going to extract that branch in my tree
 c). You are going to post the patches that you had to develop so we can review
 d). If everybody is happy and testing on our test-beds is OK, I can push this 
     later in the merge window cycle.

It is a bit of dance but it ought to work out?

Xen-devel mailing list

<Prev in Thread] Current Thread [Next in Thread>