[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 4/6] xen: add capability to load initrd outside of initial
From: |
Daniel Kiper |
Subject: |
Re: [PATCH v2 4/6] xen: add capability to load initrd outside of initial mapping |
Date: |
Fri, 12 Feb 2016 09:15:47 +0100 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
Hey Juergen,
On Fri, Feb 12, 2016 at 07:25:02AM +0100, Juergen Gross wrote:
[...]
> Okay, let me do some cleanup work on the xen loader:
>
> - add the possibility to call it multiple times (state reset, free the
> allocated memory)
> - merge all necessary global variables into one state structure, use
> local variables where possible
> - introduce lots of constants instead of using numerical values all over
> the code
Make sense for me.
> The first two items should go in before my series. The third is more a
> question of style, so it might block the complete series if I put it
> early in the series and such a change isn't regarded to be positive.
> Would you be okay if I put the constant introduction at the end of the
> series? This would result in readable code (at least I hope so) in case
OK, let's do that in that way.
> it is accepted and wouldn't block the functionality in case it is
> rejected. In case there is an early okay from the maintainer(s), I can
> do the constant introduction early, too, of course.
Granted!
Daniel
- [PATCH v2 0/6] grub-xen: support booting huge pv-domains, Juergen Gross, 2016/02/11
- [PATCH v2 2/6] xen: factor out allocation of special pages into separate function, Juergen Gross, 2016/02/11
- [PATCH v2 6/6] xen: add capability to load p2m list outside of kernel mapping, Juergen Gross, 2016/02/11
- [PATCH v2 4/6] xen: add capability to load initrd outside of initial mapping, Juergen Gross, 2016/02/11
- Re: [PATCH v2 4/6] xen: add capability to load initrd outside of initial mapping, Vladimir 'φ-coder/phcoder' Serbinenko, 2016/02/12
- Re: [PATCH v2 4/6] xen: add capability to load initrd outside of initial mapping, Juergen Gross, 2016/02/12
[PATCH v2 1/6] xen: factor out p2m list allocation into separate function, Juergen Gross, 2016/02/11
[PATCH v2 3/6] xen: factor out allocation of page tables into separate function, Juergen Gross, 2016/02/11