Re: [Xen-devel] [PATCH v2 00/41] Add ACPI support for arm64 on Xen

[Julien Grall <julien.grall@xxxxxxxxxx>]

Hi Parth

(You dropped xen-devel, re-cc it)

On 18/05/15 10:59, Parth Dixit wrote:
> yes i tested with linux-next of 15th may and it is working fine with
> it,except mounting of root partiton because kernel is not able to
> detetct the partition
> i did not looked further into it but its presumably configuration of
> right driver.
> Linux mainline will need 3 additional patches
> 1. Adding STAO and XENV
> 2. Hiding of UART
> 3. Enabling xen compilation with ACPI which was disabled by Hanjun.
> I'll wait for comment's on this series and if there are no major changes
> i'll cleanup linux patches and send them for review as well.

I think you can send an RFC now. The STA0 and XENV table are
well-defined and the rest of the patches doesn't seem to be tight with
your Xen series.

> 
> 
>         Few workarounds have been made to get it working, these are as
>         follows
>         1. In Xen interrupts are routed at the boot time with
>         edge/trigger level set to 0
>         because this information is not available at the time of booting.
> 
> 
>     edge/trigger is only a bit. Can't you just avoid to set it rather
>     than using a potentially incorrect value?
> 
> That would require disabling the checks for edge/trigger at multiple
> places for acpi, if you are ok with the approach
> i'll go ahead and send the patch.

IIRC, during the last Linaro connect we agreed to differ edge/trigger
setting for IRQ assigned to a Domain for both ACPI and DT.

> 
>         2. EFI runtime services are disaled in linux but proper solution
>         has to come from linux side.
> 
> 
>     Can you details a bit more? What is missing?
> 
> We only create EFI stub for passing RSDP and memory data in xen, we are
> not implementing full fledged runtime service support
> for EFI. At present linux expects that if EFI interface is present
> runtime services are also available.

Thanks for the explanation. Please add it in the next cover letter. It's
useful for people who weren't present at the meeting or forgot what was
said.

Regards,

-- 
Julien Grall

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