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

Re: ARM LPAE flavour (Was: Re: [PATCH] udebs for armmp flavour)



On Fri, 2013-08-30 at 21:53 +0100, Ian Campbell wrote:
> On Fri, 2013-08-30 at 21:47 +0100, Ian Campbell wrote:
> > > I've built a netboot installer image and run through it on a Calxeda
> > > system.
> 
> This system has 8GB of RAM, so an LPAE kernel flavour would be quite
> useful. The patch (below) seems remarkably simple and, build, installs
> and and boots for me.

Yes, we should certainly be adding LPAE.

> Perhaps we want to consider getting rid of the vexpress and/or mx5
> flavours before adding another one? The armmp flavour appears to have
> the relevant options turned on, but has anyone tried it? I guess we'd
> want to add dummy patches to aid in the transition?

linux-latest could build transitional packages (as for the i386/686
flavour in squeeze->wheezy).

[...]
> --- /dev/null
> +++ b/linux/debian/config/armhf/config.lpae
> @@ -0,0 +1,4 @@
> +##
> +## file: arch/arm/mm/Kconfig
> +##
> +CONFIG_ARM_LPAE=y

Shouldn't this also enable KVM (which depends on ARM_LPAE)?

(I'm surprised you haven't asked to enable XEN & XEN_DOM0 on armhf
yet...)

> diff --git a/linux/debian/config/armhf/defines b/linux/debian/config/armhf/defines
> index 643a4d3..0b4c6c2 100644
> --- a/linux/debian/config/armhf/defines
> +++ b/linux/debian/config/armhf/defines
> @@ -1,6 +1,7 @@
>  [base]
>  flavours:
>   armmp
> + armmp-lpae
>   mx5
>   omap
>   vexpress
> @@ -19,6 +20,18 @@ image-file: arch/arm/boot/zImage
>  hardware: ARMv7 multiplatform compatible SoCs
>  hardware-long: ARMv7 multiplatform kernel for Marvell Armada 370/xp, Freescale iMX5x/iMX6
>  
> +[armmp-lpae_build]
> +image-file: arch/arm/boot/zImage
> +
> +[armmp-lpae_description]
> +hardware: ARMv7 multiplatform compatible SoCs
[...]

The short description should be different from that for armmp as well.

Ben.

-- 
Ben Hutchings
If God had intended Man to program,
we'd have been born with serial I/O ports.

Attachment: signature.asc
Description: This is a digitally signed message part


Reply to: