[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 1/2] qtest: Fix rtas dependencies
From: |
Thomas Huth |
Subject: |
Re: [PATCH 1/2] qtest: Fix rtas dependencies |
Date: |
Wed, 5 Feb 2020 21:46:17 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.9.0 |
On 05/02/2020 21.41, Laurent Vivier wrote:
> qtest "rtas" command is only available with pseries not all ppc64 targets,
> so if I try to compile only powernv machine, the build fails with:
>
> /usr/bin/ld: qtest.o: in function `qtest_process_command':
> .../qtest.c:645: undefined reference to `qtest_rtas_call'
>
> We fix this by enabling rtas command only with pseries machine.
>
> Fixes: eeddd59f5962 ("tests: add RTAS command in the protocol")
Well, at that point in time CONFIG_PSERIES was not available yet ... so
not sure if the "Fixes:" really makes sense here...
But apart from that:
Reviewed-by: Thomas Huth <address@hidden>
> Signed-off-by: Laurent Vivier <address@hidden>
> ---
> qtest.c | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/qtest.c b/qtest.c
> index 12432f99cf44..587dcbb4b515 100644
> --- a/qtest.c
> +++ b/qtest.c
> @@ -27,7 +27,8 @@
> #include "qemu/error-report.h"
> #include "qemu/module.h"
> #include "qemu/cutils.h"
> -#ifdef TARGET_PPC64
> +#include "config-devices.h"
> +#ifdef CONFIG_PSERIES
> #include "hw/ppc/spapr_rtas.h"
> #endif
>
> @@ -628,7 +629,7 @@ static void qtest_process_command(CharBackend *chr, gchar
> **words)
> #else
> qtest_sendf(chr, "OK little\n");
> #endif
> -#ifdef TARGET_PPC64
> +#ifdef CONFIG_PSERIES
> } else if (strcmp(words[0], "rtas") == 0) {
> uint64_t res, args, ret;
> unsigned long nargs, nret;
>