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

Re: [Xen-devel] [PATCH v12 00/10] x86: multiboot2 protocol support



On Fri, Jan 20, 2017 at 02:42:30PM -0500, Doug Goldstein wrote:
> On 1/19/17 8:34 PM, Daniel Kiper wrote:
> > Hi,
> >
> > I am sending twelfth version of multiboot2 protocol support for
> > legacy BIOS and EFI platforms. This patch series release contains
> > fixes for all known/confirmed issues.
>
> With my fix to efi_multiboot2() in 5/10 and the entire series applied, I
> get the following on some of the systems I have access to:
>
> (XEN) [    2.000533] HVM: HAP page sizes: 4kB, 2MB, 1GB
> (XEN) [    7.012109] Stuck ??
> (XEN) [    7.012129] Failed to bring up CPU 1 (error -5)
> (XEN) [   12.023606] Stuck ??
> (XEN) [   12.023622] Failed to bring up CPU 2 (error -5)
> (XEN) [   17.035099] Stuck ??
> (XEN) [   17.035115] Failed to bring up CPU 3 (error -5)
> (XEN) [   17.035116] Brought up 1 CPUs
>
> On other machines they reset when setting PAGING into cr0 (actually the
> jmp following it) on line 124 of trampoline.S

Thanks! I will take a look.

> If I drop the series to just 2-5 against staging (since patch 1 has
> already gone in) and apply the fix to efi_multiboot2() then all the
> machines I presently have access to boot.

Great!

> Effectively the fix to efi_multiboot2() gets us back to the same level
> of hardware support that v11 + my v5 was at for 1-5. So I will extend my:
>
> Reviewed-by: Doug Goldstein <cardoe@xxxxxxxxxx>
> Tested-by: Doug Goldstein <cardoe@xxxxxxxxxx>

Thanks!

> on the condition that the fix is applied to 5/10 prior to commit.

Will do.

By the way, I have asked my team colleagues to do more tests of this series.
I will come back to you if I have something in hand.

Thank you for your help.

Have a nice weekend,

Daniel

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

 


Rackspace

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