|
From: | Alexey Kardashevskiy |
Subject: | Re: [Qemu-devel] [PATCH qemu v2] qemu-options: Add documentation about PPC64's "-cpu compat" option |
Date: | Tue, 28 Apr 2015 17:25:37 +1000 |
User-agent: | Mozilla/5.0 (X11; Linux i686 on x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0 |
On 04/10/2015 12:16 PM, David Gibson wrote:
On Wed, Apr 08, 2015 at 07:33:21PM +1000, Alexey Kardashevskiy wrote:Since 8dfa3a5e "target-ppc: Add "compat" CPU option" (which was a part of client-architecture-support patchset) there is a "compat" option of "-cpu". However it was not documented at all and this is what this patch is trying to do. Signed-off-by: Alexey Kardashevskiy <address@hidden>I've updated the previous version in spapr-next.
That was bad idea. First of all this patch is missing this: diff --git a/vl.c b/vl.c index 694deb4..12ffa1c 100644 --- a/vl.c +++ b/vl.c @@ -2876,6 +2876,7 @@ int main(int argc, char **argv, char **envp) break; } case QEMU_OPTION_cpu: + case QEMU_OPTION_cpu_ppc: /* hw initialization will check this */ cpu_model = optarg; break;But even with it, "-cpu help" does not work as the "if (!(popt->arch_mask & arch_type))" fails at vl.c:2868 because QEMU_OPTION_cpu does not have PPC64bit set (in order to hide unwanted options on x86) and QEMU_OPTION_cpu_ppc is not "-cpu".
We should use the original patch or none at all. -- Alexey
[Prev in Thread] | Current Thread | [Next in Thread] |