Re: [Qemu-block] [Qemu-devel] [PATCH 1/9] IDE: replace DEBUG_IDE with tracing system

[Philippe Mathieu-Daudé]

On 08/08/2017 05: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.

It sounds like a good idea, although I'd rather prioritize .c, having 
trace-events at bottom. At least we can agree about top-to-bottom 
scripting here :)