Re: [PATCH] target/sh4: Honor QEMU_LOG_FILENAME with QEMU_LOG=cpu

[Yoshinori Sato]

On Mon, 25 Jul 2022 23:28:54 +0900,
Ilya Leoshkevich wrote:
> 
> When using QEMU_LOG=cpu on sh4, QEMU_LOG_FILENAME is partially ignored.
> Fix by using qemu_fprintf() instead of qemu_printf() in the respective
> places.
> 
> Fixes: 90c84c560067 ("qom/cpu: Simplify how CPUClass:cpu_dump_state() prints")
> Signed-off-by: Ilya Leoshkevich <iii@linux.ibm.com>
> ---
>  target/sh4/translate.c | 14 +++++++-------
>  1 file changed, 7 insertions(+), 7 deletions(-)
> 
> diff --git a/target/sh4/translate.c b/target/sh4/translate.c
> index f1b190e7cf..9aadaf52cd 100644
> --- a/target/sh4/translate.c
> +++ b/target/sh4/translate.c
> @@ -171,16 +171,16 @@ void superh_cpu_dump_state(CPUState *cs, FILE *f, int 
> flags)
>      qemu_fprintf(f, "sgr=0x%08x dbr=0x%08x delayed_pc=0x%08x fpul=0x%08x\n",
>                   env->sgr, env->dbr, env->delayed_pc, env->fpul);
>      for (i = 0; i < 24; i += 4) {
> -        qemu_printf("r%d=0x%08x r%d=0x%08x r%d=0x%08x r%d=0x%08x\n",
> -                 i, env->gregs[i], i + 1, env->gregs[i + 1],
> -                 i + 2, env->gregs[i + 2], i + 3, env->gregs[i + 3]);
> +        qemu_fprintf(f, "r%d=0x%08x r%d=0x%08x r%d=0x%08x r%d=0x%08x\n",
> +                     i, env->gregs[i], i + 1, env->gregs[i + 1],
> +                     i + 2, env->gregs[i + 2], i + 3, env->gregs[i + 3]);
>      }
>      if (env->flags & DELAY_SLOT) {
> -        qemu_printf("in delay slot (delayed_pc=0x%08x)\n",
> -                 env->delayed_pc);
> +        qemu_fprintf(f, "in delay slot (delayed_pc=0x%08x)\n",
> +                     env->delayed_pc);
>      } else if (env->flags & DELAY_SLOT_CONDITIONAL) {
> -        qemu_printf("in conditional delay slot (delayed_pc=0x%08x)\n",
> -                 env->delayed_pc);
> +        qemu_fprintf(f, "in conditional delay slot (delayed_pc=0x%08x)\n",
> +                     env->delayed_pc);
>      } else if (env->flags & DELAY_SLOT_RTE) {
>          qemu_fprintf(f, "in rte delay slot (delayed_pc=0x%08x)\n",
>                       env->delayed_pc);
> -- 
> 2.35.3
> 

Reviewd-by: Yoshinori Sato <ysato@users.sourceforge.jp>

-- 
Yosinori Sato