Re: [PATCH] xen-guest-image-minimal: example image for a PV/HVM/PVHVM xen guest built with the 'xen-guest' kernel fragment.

[Philip Tricca <flihp@twobit.us>]

I'm a bit confused about this one. Maybe it's my misunderstanding. Questions in line:

On 10/06/2013 05:22 PM, Chris Patterson wrote:
> Signed-off-by: Chris Patterson <cjp256@gmail.com>
> ---
>  recipes-extended/images/xen-guest-image-minimal.bb | 31 ++++++++++++++++++++++
>  1 file changed, 31 insertions(+)
>  create mode 100644 recipes-extended/images/xen-guest-image-minimal.bb
> 
> diff --git a/recipes-extended/images/xen-guest-image-minimal.bb b/recipes-extended/images/xen-guest-image-minimal.bb
> new file mode 100644
> index 0000000..05b0b61
> --- /dev/null
> +++ b/recipes-extended/images/xen-guest-image-minimal.bb
> @@ -0,0 +1,31 @@
> +DESCRIPTION = "A Xen PV guest image."
> +
> +inherit core-image
> +
> +IMAGE_INSTALL += " \
> +    packagegroup-core-boot \
> +    kernel-module-xen-acpi-processor \
> +    kernel-module-xen-blkback \
> +    kernel-module-xen-netback \
> +    kernel-module-xen-pciback \

For a minimal guest I'd expect you wouldn't want any of the *back drivers. Did you mean for these to be the *front drivers?

Cheers,
- Philip

> +    kernel-module-xen-pcifront \
> +    "
> +
> +IMAGE_INSTALL += "${@base_contains('DISTRO_FEATURES', 'x11', ' xf86-video-fbdev', '', d)}"
> +IMAGE_INSTALL += "${@base_contains('DISTRO_FEATURES', 'x11', ' xf86-video-vesa', '', d)}"
> +
> +LICENSE = "MIT"
> +
> +SYSLINUX_KERNEL_ARGS ?= "LABEL=boot ramdisk_size=32768 root=/dev/ram0 rootimg=rootfs.img rw console=hvc0 console=tty0 panic=10 debugshell=5"
> +
> +# syslinux is not required for PV guest configuration, but for HVM
> +build_syslinux_cfg () {
> +	echo "ALLOWOPTIONS 1" > ${SYSLINUXCFG}
> +	echo "DEFAULT boot" >> ${SYSLINUXCFG}
> +	echo "TIMEOUT 10" >> ${SYSLINUXCFG}
> +	echo "PROMPT 1" >> ${SYSLINUXCFG}
> +	echo "LABEL boot" >> ${SYSLINUXCFG}
> +	echo "  KERNEL /vmlinuz" >> ${SYSLINUXCFG}
> +	echo "  INITRD /initrd" >> ${SYSLINUXCFG}
> +	echo "  APPEND ${SYSLINUX_KERNEL_ARGS}" >> ${SYSLINUXCFG}
> +}
>