[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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

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.

On Mon, Nov 12, 2018 at 12:50 PM Julien Grall <julien.grall@xxxxxxx <mailto:julien.grall@xxxxxxx>> 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?

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.


Julien Grall

Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.