[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [Qemu-devel] [PATCH 1/9] IDE: replace DEBUG_IDE with tr
From: |
John Snow |
Subject: |
Re: [Qemu-block] [Qemu-devel] [PATCH 1/9] IDE: replace DEBUG_IDE with tracing system |
Date: |
Tue, 22 Aug 2017 15:03:15 -0400 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1 |
On 08/08/2017 04:00 PM, Eric Blake wrote:
> On 08/08/2017 01:32 PM, John Snow wrote:
>> Out with the old, in with the new.
>>
>> Signed-off-by: John Snow <address@hidden>
>> ---
>
>> hw/ide/piix.c | 11 ++++----
>> hw/ide/trace-events | 33 ++++++++++++++++++++++++
>> hw/ide/via.c | 10 +++-----
>
> Hmm - should we tweak scripts/git.orderfile to prioritize trace-events
> over .c files? Then again, right now it prioritizes all .c files before
> anything that didn't match, so that things like trace-events will at
> least avoid falling in the middle of a patch if you use the project's
> orderfile.
>
>> +++ b/hw/ide/cmd646.c
>> @@ -32,6 +32,7 @@
>> #include "sysemu/dma.h"
>>
>> #include "hw/ide/pci.h"
>> +#include "trace.h"
>>
>> /* CMD646 specific */
>> #define CFR 0x50
>> @@ -195,9 +196,8 @@ static uint64_t bmdma_read(void *opaque, hwaddr addr,
>> val = 0xff;
>> break;
>> }
>> -#ifdef DEBUG_IDE
>> - printf("bmdma: readb " TARGET_FMT_plx " : 0x%02x\n", addr, val);
>> -#endif
>
> Yay for killing code prone to bitrot.
>
>> +++ b/hw/ide/core.c
>
>> @@ -2054,18 +2044,18 @@ void ide_exec_cmd(IDEBus *bus, uint32_t val)
>> }
>
>> hob = 0;
>> - switch(addr) {
>> + switch(reg_num) {
>
> Worth fixing the style to put space after switch while touching this?
>
>> +++ b/hw/ide/trace-events
>> @@ -0,0 +1,33 @@
>> +# See docs/devel/tracing.txt for syntax documentation.
>> +
>> +# hw/ide/core.c
>
>> +
>> +# hw/ide/pci.c
>> +bmdma_reset(void) ""
>
> An empty trace? Do all the backends support it?
>
Not the first instance of this, so I'm assuming yes.
- [Qemu-block] [PATCH 4/9] ATAPI: Replace DEBUG_IDE_ATAPI with tracing events, (continued)
- [Qemu-block] [PATCH 4/9] ATAPI: Replace DEBUG_IDE_ATAPI with tracing events, John Snow, 2017/08/08
- [Qemu-block] [PATCH 5/9] IDE: replace DEBUG_AIO with trace events, John Snow, 2017/08/08
- [Qemu-block] [PATCH 1/9] IDE: replace DEBUG_IDE with tracing system, John Snow, 2017/08/08
- [Qemu-block] [PATCH 9/9] AHCI: remove DPRINTF macro, John Snow, 2017/08/08
- [Qemu-block] [PATCH 7/9] AHCI: Rework IRQ constants, John Snow, 2017/08/08
- [Qemu-block] [PATCH 3/9] IDE: add tracing for data ports, John Snow, 2017/08/08