[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 14/17] hw/arm/spitz: Use LOG_GUEST_ERROR for bad guest regist
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [PATCH 14/17] hw/arm/spitz: Use LOG_GUEST_ERROR for bad guest register accesses |
Date: |
Mon, 29 Jun 2020 11:14:07 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.5.0 |
On 6/28/20 4:24 PM, Peter Maydell wrote:
> Instead of logging guest accesses to invalid register offsets in the
> Spitz flash device with zaurus_printf() (which just prints to stderr),
> use the usual qemu_log_mask(LOG_GUEST_ERROR,...).
>
> Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
> ---
> hw/arm/spitz.c | 12 +++++++-----
> 1 file changed, 7 insertions(+), 5 deletions(-)
>
> diff --git a/hw/arm/spitz.c b/hw/arm/spitz.c
> index 6eb46869157..49eae3fce4e 100644
> --- a/hw/arm/spitz.c
> +++ b/hw/arm/spitz.c
> @@ -23,6 +23,7 @@
> #include "hw/ssi/ssi.h"
> #include "hw/block/flash.h"
> #include "qemu/timer.h"
> +#include "qemu/log.h"
> #include "hw/arm/sharpsl.h"
> #include "ui/console.h"
> #include "hw/audio/wm8750.h"
> @@ -65,9 +66,6 @@ typedef struct {
> #define zaurus_printf(format, ...) \
> fprintf(stderr, "%s: " format, __func__, ##__VA_ARGS__)
>
> -#undef REG_FMT
> -#define REG_FMT "0x%02lx"
> -
> /* Spitz Flash */
> #define FLASH_BASE 0x0c000000
> #define FLASH_ECCLPLB 0x00 /* Line parity 7 - 0 bit */
> @@ -137,7 +135,9 @@ static uint64_t sl_read(void *opaque, hwaddr addr,
> unsigned size)
> return ecc_digest(&s->ecc, nand_getio(s->nand));
>
> default:
> - zaurus_printf("Bad register offset " REG_FMT "\n", (unsigned
> long)addr);
> + qemu_log_mask(LOG_GUEST_ERROR,
> + "sl_read: bad register offset 0x%02" HWADDR_PRIx "\n",
> + addr);
> }
> return 0;
> }
> @@ -168,7 +168,9 @@ static void sl_write(void *opaque, hwaddr addr,
> break;
>
> default:
> - zaurus_printf("Bad register offset " REG_FMT "\n", (unsigned
> long)addr);
> + qemu_log_mask(LOG_GUEST_ERROR,
> + "sl_write: bad register offset 0x%02" HWADDR_PRIx "\n",
> + addr);
> }
> }
>
>
- [PATCH 11/17] hw/misc/max111x: Create header file for documentation, TYPE_ macros, (continued)
- [PATCH 12/17] hw/arm/spitz: Encapsulate misc GPIO handling in a device, Peter Maydell, 2020/06/28
- [PATCH 13/17] hw/gpio/zaurus.c: Use LOG_GUEST_ERROR for bad guest register accesses, Peter Maydell, 2020/06/28
- [PATCH 14/17] hw/arm/spitz: Use LOG_GUEST_ERROR for bad guest register accesses, Peter Maydell, 2020/06/28
- [PATCH 16/17] hw/arm/spitz: Provide usual QOM macros for corgi-ssp and spitz-lcdtg, Peter Maydell, 2020/06/28
- [PATCH 15/17] hw/arm/pxa2xx_pic: Use LOG_GUEST_ERROR for bad guest register accesses, Peter Maydell, 2020/06/28
- [PATCH 17/17] Replace uses of FROM_SSI_SLAVE() macro with QOM casts, Peter Maydell, 2020/06/28
- Re: [PATCH 00/17] spitz: fix hacks, fix CID 1421913, various cleanups, no-reply, 2020/06/28