Re: [PATCH 00/18] xen/arm64: Suspend to RAM support for Xen

From: Mirela Simonovic <mirela.simonovic@aggios.com>
To: Julien Grall <julien.grall@arm.com>
Cc: Tim Deegan <tim@xen.org>,
	Stefano Stabellini <sstabellini@kernel.org>,
	Xen Devel <xen-devel@lists.xen.org>,
	Davorin Mista <dm@aggios.com>,
	Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>,
	George Dunlap <George.Dunlap@eu.citrix.com>,
	Andrew Cooper <andrew.cooper3@citrix.com>,
	Ian Jackson <ian.jackson@eu.citrix.com>,
	Saeed Nowshadi <saeed.nowshadi@xilinx.com>,
	Wei Liu <wei.liu2@citrix.com>, Jan Beulich <jbeulich@suse.com>,
	xen-devel <xen-devel@lists.xenproject.org>,
	Dario Faggioli <dfaggioli@suse.com>,
	Stefano Stabellini <stefano.stabellini@xilinx.com>
Subject: Re: [PATCH 00/18] xen/arm64: Suspend to RAM support for Xen
Date: Mon, 12 Nov 2018 13:35:51 +0100	[thread overview]
Message-ID: <CAKPH-NjSROq_gFPuj+nez9qkuoUhnyqF=_0nu9etDFhtcBjRDg@mail.gmail.com> (raw)
In-Reply-To: <9db248f8-7c62-d55e-a3a7-b98ece58f28f@arm.com>

Hi Julien,

On Mon, Nov 12, 2018 at 1:08 PM Julien Grall <julien.grall@arm.com> wrote:
>
> On 11/12/18 12:01 PM, Mirela Simonovic wrote:
> > Hi Julien,
>
> Hi Mirela,
>
> Please configure your e-mail client to avoid quoting using "space".
> Otherwise, this is going to make difficult to follow the discussions.
>

Sure, sorry.

> > On Mon, Nov 12, 2018 at 12:50 PM Julien Grall <julien.grall@arm.com
> > <mailto:julien.grall@arm.com>> wrote:
> >
> >     Hi Mirela,
> >
> >     Thank you for posting the series. Could you provide a branch with the
> >     patch applied?
> >
> >
> > I have applied patches on top of upstream/staging.
>
> patches are applied every day to staging. So can you please provide the
> exact commit?
>

Oh yes, the staging already moved. It was this one: 388c55b
tools/misc: fix hard tabs in xen-hvmctx.c

> Also, it is common for big series to provide a branch with the patch
> series applied. This makes easier for people to try your series.
>
> >
> >     On 11/12/18 11:30 AM, Mirela Simonovic wrote:
> >      >
> >     --------------------------------------------------------------------------------
> >      > The series does not include:
> >      > a) UART driver-specific suspend/resume that gets called when
> >     console suspends
> >
> >     I feel it will be difficult to test this series without UART driver
> >     support. Would it be possible to integrate them in that series?
> >
> >
> > Yes that's possible, but you'll need more than a UART driver's
> > suspend/resume to test this. E.g. for testing the series on Xilinx
> > Ultrascale+ MPSoC we need a set of patches that provides the basic
> > platform support needed to boot, which is not yet upstreamed. Luckily,
> > the suspend support is independent (ARM specific).
>
> I am a bit confused. I thought it was possible to boot Xen out-of-box on
> the Zynq MP. What is missing?
>
> > Please let me check with Xilinx people and Stefano how to deal with
> > this, and I'll come back with a real answer.
>
> If that's too difficult to provide in short term. Then I would
> appreciate if testing is done on another platform (e.g Foundation
> Model). So I can at least verify the series before we merge it.
>

No, it's not difficult at all, I just need Xilinx support on this.

> Cheers,
>
> --
> Julien Grall

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel