[Qemu-devel] [PATCH 0/7] Add LOG_GUEST_ERROR for reporting guest bugs

[Peter Maydell <peter.maydell@linaro.org>]

This patch series introduces a new logging category to
qemu_log_mask(), for reporting situations which the device
model can cope with but which indicate bugs in the guest.
The most common cause of these is "guest tried to access
a register which doesn't exist". At the moment device models
use an inconsistent mix of "silently ignore", "hw_error(),
killing qemu" and "direct print to stderr" for reporting
this kind of event.

Patches 2-7 are a random selection of ARM devices which I've
switched to using the new LOG_GUEST_ERROR (or the existing
LOG_UNIMP where that is more appropriate). There are obviously
more conversions that could be made; this is just a sample
to demonstrate the utility of the log category.

One question that comes to mind -- should we include qemu-log.h
in qemu-common.h (or some other common header?) rather than
having to include qemu-log.h directly in lots of device model
source files?

Peter Maydell (7):
  qemu-log: Add new log category for guest bugs
  hw/pl181: Use LOG_UNIMP and LOG_GUEST_ERROR
  hw/pl041: Use LOG_UNIMP
  hw/pl190: Use LOG_GUEST_ERROR
  hw/pl011: Use LOG_UNIMP and LOG_GUEST_ERROR
  hw/pl022: Use LOG_UNIMP and LOG_GUEST_ERROR
  hw/pl031: Use LOG_GUEST_ERROR

 hw/pl011.c | 12 ++++++++----
 hw/pl022.c |  9 ++++++---
 hw/pl031.c | 17 +++++++++++------
 hw/pl041.c |  6 ++++--
 hw/pl181.c | 19 +++++++++++--------
 hw/pl190.c |  7 +++++--
 qemu-log.c |  3 +++
 qemu-log.h |  1 +
 8 files changed, 49 insertions(+), 25 deletions(-)

-- 
1.7.11.4