[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] What's the intended use of log.h logging? (was: [PATCH
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] What's the intended use of log.h logging? (was: [PATCH 2/3] log: report hmp/qmp command and qmp event) |
Date: |
Thu, 22 Oct 2015 13:26:10 +0100 |
User-agent: |
Mutt/1.5.24 (2015-08-30) |
On Wed, Oct 21, 2015 at 01:22:22PM +0100, Peter Maydell wrote:
> On 21 October 2015 at 11:41, Stefan Hajnoczi <address@hidden> wrote:
> > DPRINTF() falls under both #1 and #2. Care needs to be taken to
> > separate the logging from the tracing when converting DPRINTF().
> >
> > Side note: we don't have effect "once only" or rate-limiting built into
> > error_report(). Guest-triggerable errors or warnings should be limited
> > to prevent log flooding.
> >
> > It would be nice to unify TCG's "qemu log" and error_report() into a
> > single logging mechanism that is thread-safe and has rate-limiting.
>
> As long as you can disable the rate-limiting: for almost all
> of the use cases for the qemu-log code you really don't want
> to have it drop logging messages.
In the Linux kernel there are a few primitives:
printk() - plain old printf
printk_once() - uses a static bool for once-only semantics,
modulo races between CPUs
printk_ratelimited() - time-based rate-limiting
(And if you need to configure the ratelimit threshold, you can use the
underlying __ratelimit() function.)
The same could work for QEMU. Existing "qemu log" stuff is mostly
printk(). Any messages that are not errors or warnings don't need
rate-limiting because they shouldn't be enabled by default (and when you
do want them for troubleshooting, then rate-limiting gets in the way).
Stefan
- Re: [Qemu-devel] What's the intended use of log.h logging?, (continued)
- Re: [Qemu-devel] What's the intended use of log.h logging?, Dr. David Alan Gilbert, 2015/10/19
- Re: [Qemu-devel] What's the intended use of log.h logging?, Kevin Wolf, 2015/10/20
- Re: [Qemu-devel] What's the intended use of log.h logging? (was: [PATCH 2/3] log: report hmp/qmp command and qmp event), Alex Bennée, 2015/10/16
- Re: [Qemu-devel] What's the intended use of log.h logging?, Markus Armbruster, 2015/10/19
- Re: [Qemu-devel] What's the intended use of log.h logging?, Peter Maydell, 2015/10/19
- Re: [Qemu-devel] What's the intended use of log.h logging? (was: [PATCH 2/3] log: report hmp/qmp command and qmp event), Stefan Hajnoczi, 2015/10/21
- Re: [Qemu-devel] What's the intended use of log.h logging?, Denis V. Lunev, 2015/10/21
- Re: [Qemu-devel] What's the intended use of log.h logging? (was: [PATCH 2/3] log: report hmp/qmp command and qmp event), Peter Maydell, 2015/10/21
- Re: [Qemu-devel] What's the intended use of log.h logging? (was: [PATCH 2/3] log: report hmp/qmp command and qmp event),
Stefan Hajnoczi <=
- Re: [Qemu-devel] What's the intended use of log.h logging?, Paolo Bonzini, 2015/10/22
[Qemu-devel] [PATCH 3/3] log: adds a timestamp to each log entry, Denis V. Lunev, 2015/10/15
[Qemu-devel] [PATCH 1/3] log: improve performance of qemu_log and qemu_log_mask if disabled, Denis V. Lunev, 2015/10/15
Re: [Qemu-devel] [PATCH 1/3] log: improve performance of qemu_log and qemu_log_mask if disabled, Markus Armbruster, 2015/10/16