Re: [Xen-devel] [PATCH v2 4/4] arm64: update the introduction of xen boot commands in docs/grub.texi

2016-02-26 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 26.02.2016 12:15, Fu Wei wrote: > Hi Andrei, > > On 26 February 2016 at 18:50, Andrei Borzenkov wrote: >> On Fri, Feb 26, 2016 at 8:59 AM, Fu Wei wrote: >>> +@subsection xen_module >>> >>> -@deffn Command xen_linux file [arguments] >>>

Re: [Xen-devel] [PATCH v2 4/6] xen: add capability to load initrd outside of initial mapping

2016-02-12 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 11.02.2016 15:13, Juergen Gross wrote: > On 11/02/16 13:33, Daniel Kiper wrote: >> On Thu, Feb 11, 2016 at 08:53:24AM +0100, Juergen Gross wrote: >>> Modern pvops linux kernels support an initrd not covered by the initial >>> mapping. This capability is flagged by an elf-note. >>> >>> In case

Re: [Xen-devel] [PATCH v2 3/6] xen: factor out allocation of page tables into separate function

2016-02-12 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 11.02.2016 13:53, Juergen Gross wrote: > On 11/02/16 13:27, Daniel Kiper wrote: >> On Thu, Feb 11, 2016 at 08:53:23AM +0100, Juergen Gross wrote: >>> Do the allocation of page tables in a separate function. This will >>> allow to do the allocation at different times of the boot preparations >>>

Re: [Xen-devel] [PATCH v2 2/6] relocator: Do not use memory region if its starta is smaller than size

2016-02-12 Thread Vladimir 'φ-coder/phcoder' Serbinenko
Applied, thanks On 20.07.2015 16:35, Daniel Kiper wrote: > malloc_in_range() should not use memory region if its starta is smaller > than size. Otherwise target wraps around and points to region which is > usually not a RAM, e.g.: > > loader/multiboot.c:93: segment 0: paddr=0x80,

Re: [Xen-devel] Uniform commands for booting xen

2016-01-22 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 18.01.2016 11:28, Ian Campbell wrote: > On Mon, 2016-01-11 at 15:06 +0100, Vladimir 'φ-coder/phcoder' Serbinenko > wrote: >> On 13.11.2015 10:50, Ian Campbell wrote: >>> On Fri, 2015-11-13 at 12:04 +0300, Andrei Borzenkov wrote: >>>>> How do you expr

Re: [Xen-devel] [BUG] XEN domU crash when PV grub chainloads 32-bit domU grub

2016-01-22 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 22.09.2015 10:53, Ian Campbell wrote: > Hi Vladimir & grub-devel, > > Do you have any thoughts on this issue with i386 pv-grub2? > Is it still an issue? If so I'll try to replicate it. From stack dump I see that it has jumped to NULL. GRUB has no threads so it's not a race condition with

Re: [Xen-devel] [BUG] XEN domU crash when PV grub chainloads 32-bit domU grub

2016-01-22 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 22.01.2016 14:01, Andrew Cooper wrote: > On 22/01/16 12:56, Vladimir 'φ-coder/phcoder' Serbinenko wrote: >> On 22.09.2015 10:53, Ian Campbell wrote: >>> Hi Vladimir & grub-devel, >>> >>> Do you have any thoughts on this issue with i386 pv-grub2? >

Re: [Xen-devel] Uniform commands for booting xen

2016-01-11 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 13.11.2015 10:50, Ian Campbell wrote: > On Fri, 2015-11-13 at 12:04 +0300, Andrei Borzenkov wrote: >>> How do you express modules other than kernel+initrd in that >>> scheme, without grub needing to be aware of any new addition we >>> may find necessary going forward? >>> >> >> Are modules used

Re: [Xen-devel] Uniform commands for booting xen

2016-01-11 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 11.01.2016 15:32, Jan Beulich wrote: On 11.01.16 at 15:06, wrote: >> On 13.11.2015 10:50, Ian Campbell wrote: >>> On Fri, 2015-11-13 at 12:04 +0300, Andrei Borzenkov wrote: > How do you express modules other than kernel+initrd in that > scheme, without grub

[Xen-devel] [PATCH GRUB] Allow initrd concatenation on ARM64

2015-11-12 Thread Vladimir 'φ-coder/phcoder' Serbinenko
While on it also change "xen_linux" to "xen_kernel" to be vendor-neutral Could someone test please? I only compile-tested it diff --git a/docs/grub.texi b/docs/grub.texi index 1df3db2..112b42b 100644 --- a/docs/grub.texi +++ b/docs/grub.texi @@ -3859,7 +3859,7 @@ you forget a command, you can run

[Xen-devel] Uniform commands for booting xen

2015-11-12 Thread Vladimir 'φ-coder/phcoder' Serbinenko
Hello, all. I'd like to have set of commands that would boot xen on all platforms. I thought of following set: xen_hypervisor FILE XEN_OPTIONS xen_kernel FILE KERNEL_OPTIONS xen_initrd INITRD INITRD INITRD all initrds are concatenated. xen_xsm ??? On arm64 it would use the arm64 xen FDT protocol

Re: [Xen-devel] [PATCH v2 5/6] multiboot2: Add support for relocatable images

2015-11-12 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 12.11.2015 14:15, Daniel Kiper wrote: > On Tue, Nov 10, 2015 at 04:23:46PM +0100, Vladimir 'phcoder' Serbinenko wrote: >> Le 10 nov. 2015 3:52 PM, "Daniel Kiper" <daniel.ki...@oracle.com> a écrit : >>> On Mon, Nov 09, 2015 at 09:08:35PM +0100, Vladimir 'φ-

Re: [Xen-devel] [PATCH v2 3/6] i386/relocator: Add grub_relocator64_efi relocator

2015-11-12 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 10.11.2015 15:38, Daniel Kiper wrote: >>> - if (entry_specified) >>> > > + if (keep_bs && efi_entry_specified) >>> > > +grub_multiboot_payload_eip = efi_entry; >>> > > + else if (entry_specified) >>> > > grub_multiboot_payload_eip = entry; >>> > > >> > This seems redundant. > What

Re: [Xen-devel] [PATCH GRUB] Allow initrd concatenation on ARM64

2015-11-12 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 12.11.2015 14:27, Ian Campbell wrote: > On Thu, 2015-11-12 at 14:08 +0100, Vladimir 'φ-coder/phcoder' Serbinenko > wrote: >> While on it also change "xen_linux" to "xen_kernel" to be vendor-neutral >> Could someone test please? I only compile-tested it >

Re: [Xen-devel] [PATCH v2 2/6] relocator: Do not use memory region if its starta is smaller than size

2015-11-09 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 21.07.2015 08:42, Andrei Borzenkov wrote: > On Mon, Jul 20, 2015 at 5:35 PM, Daniel Kiper wrote: >> malloc_in_range() should not use memory region if its starta is smaller >> than size. Otherwise target wraps around and points to region which is >> usually not a RAM,

Re: [Xen-devel] [PATCH v2 1/6] gitignore: Ignore *.orig, *.rej and *.swp files

2015-11-09 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 09.11.2015 16:29, Daniel Kiper wrote: > On Wed, Nov 04, 2015 at 01:03:56PM +0100, Vladimir 'phcoder' Serbinenko wrote: >> Le 12 août 2015 11:04 AM, "Ian Campbell" a écrit : >>> >>> >>> (Having written the below I see too late that this is a grub patch not a >>> Xen

Re: [Xen-devel] [PATCH v2 1/6] gitignore: Ignore *.orig, *.rej and *.swp files

2015-11-09 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 09.11.2015 16:39, Daniel Kiper wrote: > On Mon, Nov 09, 2015 at 04:34:20PM +0100, Vladimir 'φ-coder/phcoder' > Serbinenko wrote: >> On 09.11.2015 16:29, Daniel Kiper wrote: >>> On Wed, Nov 04, 2015 at 01:03:56PM +0100, Vladimir 'phcoder' Serbinenko >>> wrote:

Re: [Xen-devel] [PATCH v2 3/6] i386/relocator: Add grub_relocator64_efi relocator

2015-11-09 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 20.07.2015 16:35, Daniel Kiper wrote: > Add grub_relocator64_efi relocator. It will be used on EFI 64-bit platforms > when multiboot2 compatible image requests MULTIBOOT_TAG_TYPE_EFI_BS. Relocator > will set lower parts of %rax and %rbx accordingly to multiboot2 specification. > On the other

Re: [Xen-devel] [PATCH v2 5/6] multiboot2: Add support for relocatable images

2015-11-09 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 20.07.2015 16:35, Daniel Kiper wrote: > Signed-off-by: Daniel Kiper What is handling the actual relocation? Why doesn't this patch include support for ELF relocations? > --- > grub-core/loader/i386/multiboot_mbi.c |6 ++-- > grub-core/loader/multiboot.c |

Re: [Xen-devel] [PATCH v3 3/4] * util/grub.d/20_linux_xen.in: Add support of the XEN boot on aarch64

2015-10-30 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 30.10.2015 09:44, Fu Wei wrote: > Hi Vladimir, > > Great thanks for your suggestion! :-) > > On 29 October 2015 at 23:25, Vladimir 'φ-coder/phcoder' Serbinenko > <phco...@gmail.com> wrote: >>> +if [ "x$machine" != xaarch64 ]; then >>> +

Re: [Xen-devel] [PATCH v3 1/4] arm64: Add and export some accessor functions for xen boot

2015-10-29 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 23.07.2015 07:16, fu@linaro.org wrote: > From: Fu Wei > > Add accessor functions of "loaded" flag in > grub-core/loader/arm64/linux.c. > > Export accessor functions of "loaded" flag and > grub_linux_get_fdt function in include/grub/arm64/linux.h. > > Purpose: Reuse

Re: [Xen-devel] [PATCH v3 3/4] * util/grub.d/20_linux_xen.in: Add support of the XEN boot on aarch64

2015-10-29 Thread Vladimir 'φ-coder/phcoder' Serbinenko
> +if [ "x$machine" != xaarch64 ]; then > + multiboot_cmd="multiboot" > + module_linux_cmd="module" > + module_initrd_cmd="module --nounzip" > +else > + multiboot_cmd="xen_hypervisor" > + module_linux_cmd="xen_linux" > + module_initrd_cmd="xen_initrd" > +fi > + Please do

Re: [Xen-devel] [PATCH v3 2/4] arm64: Add xen_boot module file

2015-10-29 Thread Vladimir 'φ-coder/phcoder' Serbinenko
Committed without the xen_module command. Its argument parsing was non-trivial and I don't quite get what its intent is. Can you resubmit? On 23.07.2015 07:16, fu@linaro.org wrote: > From: Fu Wei > > grub-core/loader/arm64/xen_boot.c > > - This adds support for the Xen

Re: [Xen-devel] [PATCH 3/5] i386/relocator: Remove unused avoid_efi_bootservices argument

2015-07-15 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 30.05.2015 07:25, Andrei Borzenkov wrote: В Fri, 29 May 2015 22:58:43 +0200 Daniel Kiper daniel.ki...@oracle.com пишет: Another questions is why grub_relocator_alloc_chunk_addr() does not consult EFI memory map if grub_relocator_alloc_chunk_align() does. Should not we fix it? My best

Re: [Xen-devel] [PATCH v2 0/3] arm64: Add multiboot support (via fdt) for Xen boot

2015-07-15 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 13.07.2015 10:53, fu@linaro.org wrote: From: Fu Wei fu@linaro.org - This adds support for the Xen boot on ARM specification for arm64. - The implementation for Xen is following Multiboot on ARM Specification:

Re: [Xen-devel] [PATCH v2 1/3] arm64: Add Xen boot support file

2015-07-15 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 13.07.2015 10:53, fu@linaro.org wrote: From: Fu Wei fu@linaro.org This patch adds Xen boot support file: grub-core/loader/arm64/xen_boot.c include/grub/arm64/xen_boot.h This patch also adds commands register code and hearder file into grub-core/loader/arm64/linux.c - This

Re: [Xen-devel] [PATCH v2 2/3] util/grub.d/20_linux_xen.in: Add arm64 support

2015-07-15 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 14.07.2015 15:09, Fu Wei wrote: Hi Andrei, Great thanks for your review. So Are you suggesting this: (1) in util/grub.d/20_linux_xen.in, we only use xen_hypervisor/xen_module. (2) in xen_boot.c, we only register command xen_hypervisor/xen_module. (3) in grub-core/loader/i386/xen.c, we

Re: [Xen-devel] [PATCH 2/5] i386/relocator: Remove unused extern grub_relocator64_rip_addr

2015-05-07 Thread Vladimir 'φ-coder/phcoder' Serbinenko
Committed, thanks. signature.asc Description: OpenPGP digital signature ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel