On 19.11.2014 16:40, Cole Robinson wrote:
On 11/19/2014 10:30 AM, Michal Privoznik wrote:
> Currently, we are whitelisting architectures, that we know how to run
> OVMF on. So far, only x86_64 was enabled. However, looking at qemu
> code, the same commandline can be used to enable OVMF for aarch64.
>
> Signed-off-by: Michal Privoznik <mprivozn(a)redhat.com>
> ---
> src/qemu/qemu_command.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c
> index d2e6991..ca57e35 100644
> --- a/src/qemu/qemu_command.c
> +++ b/src/qemu/qemu_command.c
> @@ -7749,7 +7749,8 @@ qemuBuildDomainLoaderCommandLine(virCommandPtr cmd,
>
> case VIR_DOMAIN_LOADER_TYPE_PFLASH:
> /* UEFI is supported only for x86_64 currently */
> - if (def->os.arch != VIR_ARCH_X86_64) {
> + if (def->os.arch != VIR_ARCH_X86_64 &&
> + def->os.arch != VIR_ARCH_AARCH64) {
> virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
> _("pflash is not supported for %s guest
> architecture"),
> virArchToString(def->os.arch));
>
Please add armv7hl as well, it should work completely identically
(if/when we have an OS that supports it). ACK with that
(sorry for not following up on the original patch, still haven't gotten
back to doing virt-* and aarch64 testing since KVM forum.
- Cole
Thanks, fixed and pushed.
Michal