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

2017-03-21 Thread Jan Beulich
>>> On 21.03.17 at 14:05, wrote: > On Thu, Mar 16, 2017 at 07:43:57AM -0600, Jan Beulich wrote: >> >>> On 16.03.17 at 14:35, wrote: >> > On Thu, Mar 16, 2017 at 07:12:21AM -0600, Jan Beulich wrote: >> >> >>> On 16.03.17 at 13:12, wrote: >> >> > Everything works. I am not able to reproduce any is

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

2017-03-21 Thread Daniel Kiper
On Thu, Mar 16, 2017 at 07:43:57AM -0600, Jan Beulich wrote: > >>> On 16.03.17 at 14:35, wrote: > > On Thu, Mar 16, 2017 at 07:12:21AM -0600, Jan Beulich wrote: > >> >>> On 16.03.17 at 13:12, wrote: > >> > Everything works. I am not able to reproduce any issues reported by Doug. > >> > Andrew, Ja

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

2017-03-16 Thread Jan Beulich
>>> On 16.03.17 at 14:35, wrote: > On Thu, Mar 16, 2017 at 07:12:21AM -0600, Jan Beulich wrote: >> >>> On 16.03.17 at 13:12, wrote: >> > Everything works. I am not able to reproduce any issues reported by Doug. >> > Andrew, Jan, Doug, are there still any objections to commit the rest of > v16? >

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

2017-03-16 Thread Daniel Kiper
On Thu, Mar 16, 2017 at 07:12:21AM -0600, Jan Beulich wrote: > >>> On 16.03.17 at 13:12, wrote: > > Everything works. I am not able to reproduce any issues reported by Doug. > > Andrew, Jan, Doug, are there still any objections to commit the rest of v16? > > Well, Andrew has been telling me privat

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

2017-03-16 Thread Jan Beulich
>>> On 16.03.17 at 13:12, wrote: > Everything works. I am not able to reproduce any issues reported by Doug. > Andrew, Jan, Doug, are there still any objections to commit the rest of v16? Well, Andrew has been telling me privately that even the part that was already committed regresses XenServer

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

2017-03-16 Thread Daniel Kiper
On Wed, Mar 15, 2017 at 09:27:27AM -0500, Doug Goldstein wrote: > On 3/15/17 6:35 AM, Daniel Kiper wrote: > > On Thu, Mar 09, 2017 at 02:02:49PM -0600, Doug Goldstein wrote: > > > > [...] > > > >> Still missing 'xl info'. > > > > Got Intel NUC5i3MYHE (internally it is NUC5i3MYBE board) into my hand

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

2017-03-15 Thread Daniel Kiper
On Wed, Mar 15, 2017 at 09:42:53AM -0500, Doug Goldstein wrote: > On 3/15/17 9:38 AM, Daniel Kiper wrote: > > On Wed, Mar 15, 2017 at 09:27:27AM -0500, Doug Goldstein wrote: > >> On 3/15/17 6:35 AM, Daniel Kiper wrote: > >>> On Thu, Mar 09, 2017 at 02:02:49PM -0600, Doug Goldstein wrote: > >>> > >>

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

2017-03-15 Thread Doug Goldstein
On 3/15/17 9:38 AM, Daniel Kiper wrote: > On Wed, Mar 15, 2017 at 09:27:27AM -0500, Doug Goldstein wrote: >> On 3/15/17 6:35 AM, Daniel Kiper wrote: >>> On Thu, Mar 09, 2017 at 02:02:49PM -0600, Doug Goldstein wrote: >>> >>> [...] >>> Still missing 'xl info'. >>> >>> Got Intel NUC5i3MYHE (inte

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

2017-03-15 Thread Daniel Kiper
On Wed, Mar 15, 2017 at 09:27:27AM -0500, Doug Goldstein wrote: > On 3/15/17 6:35 AM, Daniel Kiper wrote: > > On Thu, Mar 09, 2017 at 02:02:49PM -0600, Doug Goldstein wrote: > > > > [...] > > > >> Still missing 'xl info'. > > > > Got Intel NUC5i3MYHE (internally it is NUC5i3MYBE board) into my hand

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

2017-03-15 Thread Doug Goldstein
On 3/15/17 6:35 AM, Daniel Kiper wrote: > On Thu, Mar 09, 2017 at 02:02:49PM -0600, Doug Goldstein wrote: > > [...] > >> Still missing 'xl info'. > > Got Intel NUC5i3MYHE (internally it is NUC5i3MYBE board) into my hands. > I have put 8 GiB RAM and 500 GB SATA 3 into it. Updated BIOS/EFI to 0041

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

2017-03-15 Thread Daniel Kiper
On Thu, Mar 09, 2017 at 02:02:49PM -0600, Doug Goldstein wrote: [...] > Still missing 'xl info'. Got Intel NUC5i3MYHE (internally it is NUC5i3MYBE board) into my hands. I have put 8 GiB RAM and 500 GB SATA 3 into it. Updated BIOS/EFI to 0041 version (it is the latest one). Installed latest Debia

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

2017-03-09 Thread Konrad Rzeszutek Wilk
> > All worked fine. > > > > > > > konrad-Super-Server login: [ 188.181526] reboot: Restarting system > > (XEN) Hardware Dom0 shutdown: rebooting machine > > (XEN) APIC error on CPU0: 40(00) > > > > ... reboot. > > So as I've mentioned you have to run 'xl info' and look at nr_cpus to > see t

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

2017-03-09 Thread Doug Goldstein
On 3/7/17 9:44 PM, 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

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

2017-03-09 Thread Jan Beulich
>>> On 09.03.17 at 16:24, wrote: > Though it looks that I found a bug in Xen IOMMU code. If I run Xen from master > in QEMU then QEMU complains and crashes: > > qemu-system-x86_64: /srv/dev/qemu/qemu_20170308/hw/i386/intel_iommu.c:1786: > vtd_mem_write: Assertion `size == 4' failed. > Aborte

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

2017-03-09 Thread Andrew Cooper
On 09/03/17 15:24, Daniel Kiper 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.

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

2017-03-09 Thread Daniel Kiper
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

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

2017-03-08 Thread Konrad Rzeszutek Wilk
..snip.. > > 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. OK, I will

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

2017-03-08 Thread Daniel Kiper
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

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

2017-03-07 Thread Konrad Rzeszutek Wilk
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 > >

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

2017-03-06 Thread Daniel Kiper
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

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

2017-02-22 Thread Doug Goldstein
On 2/22/17 7:34 AM, Daniel Kiper wrote: > On Wed, Feb 22, 2017 at 06:42:40AM -0700, Jan Beulich wrote: > On 21.02.17 at 20:24, wrote: >>> On Tue, Feb 21, 2017 at 08:19:53PM +0100, Daniel Kiper wrote: This way Xen can be loaded on EFI platforms using GRUB2 and other boot loaders which

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

2017-02-22 Thread Daniel Kiper
On Wed, Feb 22, 2017 at 06:42:40AM -0700, Jan Beulich wrote: > >>> On 21.02.17 at 20:24, wrote: > > On Tue, Feb 21, 2017 at 08:19:53PM +0100, Daniel Kiper wrote: > >> This way Xen can be loaded on EFI platforms using GRUB2 and > >> other boot loaders which support multiboot2 protocol. > >> > >> Si

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

2017-02-22 Thread Doug Goldstein
On 2/22/17 5:42 AM, Jan Beulich wrote: On 21.02.17 at 20:24, wrote: >> On Tue, Feb 21, 2017 at 08:19:53PM +0100, Daniel Kiper wrote: >>> This way Xen can be loaded on EFI platforms using GRUB2 and >>> other boot loaders which support multiboot2 protocol. >>> >>> Signed-off-by: Daniel Kiper >

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

2017-02-22 Thread Jan Beulich
>>> On 21.02.17 at 20:24, wrote: > On Tue, Feb 21, 2017 at 08:19:53PM +0100, Daniel Kiper wrote: >> This way Xen can be loaded on EFI platforms using GRUB2 and >> other boot loaders which support multiboot2 protocol. >> >> Signed-off-by: Daniel Kiper >> --- >> v16 - suggestions/fixes: >> - im

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

2017-02-21 Thread Daniel Kiper
On Tue, Feb 21, 2017 at 08:19:53PM +0100, Daniel Kiper wrote: > This way Xen can be loaded on EFI platforms using GRUB2 and > other boot loaders which support multiboot2 protocol. > > Signed-off-by: Daniel Kiper > --- > v16 - suggestions/fixes: > - improve comments in error handling > (s

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

2017-02-21 Thread Daniel Kiper
This way Xen can be loaded on EFI platforms using GRUB2 and other boot loaders which support multiboot2 protocol. Signed-off-by: Daniel Kiper --- v16 - suggestions/fixes: - improve comments in error handling (suggested by Jan Beulich). v15 - suggestions/fixes: - rearrange inline as