From: BALATON Zoltan <balaton@eik.bme.hu>
To: qemu-devel@nongnu.org
Cc: philmd@linaro.org
Subject: [PATCH 0/2] Separate memory access logs from guest_errors
Date: Sun, 06 Oct 2024 18:49:12 +0200 (CEST) [thread overview]
Message-ID: <cover.1728232526.git.balaton@eik.bme.hu> (raw)
Originally memory access logs were a debug define that then were
converted to log messages but were classified as guest_errors which
already logs misc errors. As invalid memory access logs can come from
accessing not emulated peripherals or memory areas, these often
generate a lot of messages that are better be controlled separately
from other errors to avoid obscuring those. As an example try
'qemu-system-ppc -d guest_errors' to see the problem. After this
series the actual guest error logs are easier to spot. I've tried to
submit this before but there were some people who liked the current
behaviour so now this series has another patch that preserves the old
optino printing a warning to allow time to get used to the new
behaviour (which actually brings back the old behaviour when mem
access logs were a debug define).
Regards,
BALATON Zoltan
BALATON Zoltan (2):
log: Add separate debug option for logging invalid memory accesses
log: Suggest using -d guest_error,memaccess instead of guest_errors
docs/devel/secure-coding-practices.rst | 2 +-
include/qemu/log.h | 1 +
system/memory.c | 6 +++---
system/physmem.c | 2 +-
tests/avocado/smmu.py | 2 +-
tests/qtest/pnv-host-i2c-test.c | 2 +-
util/log.c | 8 +++++++-
7 files changed, 15 insertions(+), 8 deletions(-)
--
2.30.9
next reply other threads:[~2024-10-06 16:49 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-10-06 16:49 BALATON Zoltan [this message]
2024-10-06 16:49 ` [PATCH 1/2] log: Add separate debug option for logging invalid memory accesses BALATON Zoltan
2024-10-14 14:13 ` Peter Maydell
2024-10-14 16:48 ` BALATON Zoltan
2024-10-06 16:49 ` [PATCH 2/2] log: Suggest using -d guest_error,memaccess instead of guest_errors BALATON Zoltan
2024-10-14 14:18 ` Peter Maydell
2024-10-14 16:46 ` BALATON Zoltan
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=cover.1728232526.git.balaton@eik.bme.hu \
--to=balaton@eik.bme.hu \
--cc=philmd@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).