From: Peter Maydell <peter.maydell@linaro.org>
To: qemu-devel@nongnu.org
Cc: Blue Swirl <blauwirbel@gmail.com>, patches@linaro.org
Subject: [Qemu-devel] [PATCH v2 0/8] Add LOG_GUEST_ERROR for reporting guest bugs
Date: Thu, 18 Oct 2012 14:11:34 +0100 [thread overview]
Message-ID: <1350565902-23940-1-git-send-email-peter.maydell@linaro.org> (raw)
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 3-8 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.
Changes v1->v2:
* add patch 2 including qemu-log.h in hw/hw.h
* update 3..8 to not include qemu-log.h directly
Peter Maydell (8):
qemu-log: Add new log category for guest bugs
hw/hw.h: Add include of qemu-log.h
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/hw.h | 1 +
hw/pl011.c | 11 +++++++----
hw/pl022.c | 8 +++++---
hw/pl031.c | 16 ++++++++++------
hw/pl041.c | 5 +++--
hw/pl181.c | 18 ++++++++++--------
hw/pl190.c | 6 ++++--
qemu-log.c | 3 +++
qemu-log.h | 1 +
9 files changed, 44 insertions(+), 25 deletions(-)
--
1.7.9.5
next reply other threads:[~2012-10-18 13:12 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-10-18 13:11 Peter Maydell [this message]
2012-10-18 13:11 ` [Qemu-devel] [PATCH v2 1/8] qemu-log: Add new log category for guest bugs Peter Maydell
2012-10-18 13:11 ` [Qemu-devel] [PATCH v2 2/8] hw/hw.h: Add include of qemu-log.h Peter Maydell
2012-10-18 14:26 ` Igor Mammedov
2012-10-18 14:30 ` Peter Maydell
2012-10-18 13:11 ` [Qemu-devel] [PATCH v2 3/8] hw/pl181: Use LOG_UNIMP and LOG_GUEST_ERROR Peter Maydell
2012-10-18 13:11 ` [Qemu-devel] [PATCH v2 4/8] hw/pl041: Use LOG_UNIMP Peter Maydell
2012-10-18 13:11 ` [Qemu-devel] [PATCH v2 5/8] hw/pl190: Use LOG_GUEST_ERROR Peter Maydell
2012-10-18 13:11 ` [Qemu-devel] [PATCH v2 6/8] hw/pl011: Use LOG_UNIMP and LOG_GUEST_ERROR Peter Maydell
2012-10-18 13:11 ` [Qemu-devel] [PATCH v2 7/8] hw/pl022: " Peter Maydell
2012-10-18 13:11 ` [Qemu-devel] [PATCH v2 8/8] hw/pl031: Use LOG_GUEST_ERROR Peter Maydell
2012-10-20 8:48 ` [Qemu-devel] [PATCH v2 0/8] Add LOG_GUEST_ERROR for reporting guest bugs Blue Swirl
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1350565902-23940-1-git-send-email-peter.maydell@linaro.org \
--to=peter.maydell@linaro.org \
--cc=blauwirbel@gmail.com \
--cc=patches@linaro.org \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).