Re: [Xen-devel] [PATCH v16 4/9] x86: add multiboot2 protocol support for EFI platforms

[Daniel Kiper <daniel.kiper@xxxxxxxxxx>]

On Tue, Mar 07, 2017 at 10:44:15PM -0500, Konrad Rzeszutek Wilk wrote:
> On Tue, Mar 07, 2017 at 12:39:04AM +0100, Daniel Kiper wrote:
> > On Wed, Feb 22, 2017 at 09:04:17AM -0800, Doug Goldstein wrote:
> >
> > [...]
> >
> > > I'm currently at ELC and then on vacation so I don't have access to any
> > > of the machines currently myself. However the machine I most use to test
> > > is a NUC5i5MYHE and a NUC5i3MYHE if you want to ask around if someone
> > > has one internally. But that's why I gave QEMU as an example.
> > >
> > > I was using qemu master from a few weeks ago. I'll have to find the
> > > revision for you. But the command line I use is:
> > >
> > > -enable-kvm -M pc-q35-2.8 -device intel-iommu -cpu host -m 2048 -smp 2
> > > -drive if=pflash,format=raw,file=/tmp/tmp.EiR6ixmYzV -global
> > > isa-debugcon.iobase=0x402 -debugcon file:/tmp/tmp.nuvEXUWfnA -monitor
> > > stdio -chardev socket,host=127.0.0.1,port=25914,id=S0,server,nowait
> > > -device isa-serial,chardev=S0 -device piix3-usb-uhci -device usb-tablet
> > > -netdev id=net0,type=tap -device
> > > virtio-net-pci,netdev=net0,mac=52:54:00:12:34:56 -boot order=n -device
> > > qxl-vga -gdb tcp::14952
> >
> > Sadly, my colleagues and I are not able to reproduce the problem on any of
> > machines available for us (available on the market and some development
> > stuff in our labs). I did tests with QEMU (I am not able to run it with
> > "-device intel-iommu" on my machine; I have to investigate this). Everything
> > works. Joao did some tests on Intel NUC D34010WYK second generation.
> > Everything works. So, Konrad ordered Intel NUC NUC5i3MYHE for me. I am
> > waiting for delivery. Doug, could you tell me what distro, Xen, etc. you
> > have installed on that NUC? I would like to test same config as yours on
> > this machine.
>
> I had a chat with Doug on IRC and:
>  - I had tested earlier on AMD, while he has only Intel boxes,
>  - He was wondering if this was an IOMMU issue.

I had and still have a feeling that it can be related to IOMMU. I will
try to reproduce this on QEMU but first of all I have to check how to
enable this option in my environment (something is broken). Additionally,
there is a chance that the issue spotted by osstest (fixed right now)
played a role here too. So, it will be nice if Doug can do tests with
latest master. Anyway, I will do tests too. Though I am still waiting
for my NUC.

> So to double-check that, I installed Ubuntu 16.10 on my X11SAE
> SuperMicro, which has an Haswell E3-1245 v5 and with IOMMU enabled.
>
> I tested the 'origin/staging' xen.gz build with the upstream grub2
> (I just used the 'master' branch) first and also just booting xen.efi.
>
> Both worked fine.
>
> Then I used v16 of Daniel's patches (this thread). They are also
> now on        git://xenbits.xen.org/people/konradwilk/xen.git mb2.v16
> also the same way - as xen.efi and then using grub.efi and booting it
> (see below)
>
> All worked fine.

Great! Thanks a lot for doing the tests.

> Now in the process I discovered that my patch for grub-mkconfig to
> detect multiboot2 payloads and use those instead of multiboot never
> made it upstream, so I had to modify my grub.cfg by hand (see below).

It will be nice if you post it after GRUB2 2.02 release.

Daniel

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