[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [Qemu-block] [PATCH v2 0/9] IDE: replace printfs with t
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [Qemu-block] [PATCH v2 0/9] IDE: replace printfs with tracing |
Date: |
Wed, 30 Aug 2017 10:21:56 +0100 |
User-agent: |
Mutt/1.8.3 (2017-05-23) |
On Tue, Aug 29, 2017 at 04:49:25PM -0400, John Snow wrote:
> Wherever possible, replace all printfs with proper tracing.
> In most places I've tried to do a straight replacement, but
> forthcoming patches may calibrate the tracing to be a little nicer.
>
> For now, it's nice to just remove the all-or-nothing tracing.
>
> For V2, I opted to leave in the nearly redundant tracers for now
> with their individual names so that they can be targeted individually.
Looks good. I left a comment on one patch.
- [Qemu-devel] [PATCH v2 4/9] ATAPI: Replace DEBUG_IDE_ATAPI with tracing events, (continued)
- [Qemu-devel] [PATCH v2 4/9] ATAPI: Replace DEBUG_IDE_ATAPI with tracing events, John Snow, 2017/08/29
- [Qemu-devel] [PATCH v2 8/9] AHCI: pretty-print FIS to buffer instead of stderr, John Snow, 2017/08/29
- [Qemu-devel] [PATCH v2 6/9] AHCI: Replace DPRINTF with trace-events, John Snow, 2017/08/29
- [Qemu-devel] [PATCH v2 9/9] AHCI: remove DPRINTF macro, John Snow, 2017/08/29
- [Qemu-devel] [PATCH v2 7/9] AHCI: Rework IRQ constants, John Snow, 2017/08/29
- Re: [Qemu-devel] [PATCH v2 0/9] IDE: replace printfs with tracing, no-reply, 2017/08/29
- Re: [Qemu-devel] [Qemu-block] [PATCH v2 0/9] IDE: replace printfs with tracing,
Stefan Hajnoczi <=