[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 0/7] Add LOG_GUEST_ERROR for reporting guest bug
From: |
Blue Swirl |
Subject: |
Re: [Qemu-devel] [PATCH 0/7] Add LOG_GUEST_ERROR for reporting guest bugs |
Date: |
Sun, 14 Oct 2012 15:55:45 +0000 |
On Sun, Oct 14, 2012 at 3:41 PM, Peter Maydell <address@hidden> wrote:
> On 14 October 2012 15:02, Blue Swirl <address@hidden> wrote:
>> On Sun, Oct 14, 2012 at 1:11 PM, Peter Maydell <address@hidden> wrote:
>>> One question that comes to mind -- should we include qemu-log.h
>>> in qemu-common.h (or some other common header?) rather than
>>> having to include qemu-log.h directly in lots of device model
>>> source files?
>>
>> Yes.
>
> This turns out not to be quite as trivial as I'd thought,
> since qemu-log.h requires (if NEED_CPU_H is defined) definitions
> of types like target_ulong. So we can't include it from qemu-common.h
> unless we change all the target-*/cpu.h to include cpu-defs.h
> before qemu-common.h. Alternatively we could split the NEED_CPU_H
> bits of qemu-log.h out into a qemu-cpu-log.h which would be
> manually included by the bits of qemu that needed it.
>
> The other option would be to include qemu-log.h from hw/hw.h.
> This just works without requiring any complicated workarounds,
> and all the device code includes hw.h already...
I was also thinking about that but then I thought that it would be
nice to use qemu-log.h elsewhere too. But hw/hw.h sounds much easier
for now.
>
> -- PMM
- [Qemu-devel] [PATCH 0/7] Add LOG_GUEST_ERROR for reporting guest bugs, Peter Maydell, 2012/10/14
- [Qemu-devel] [PATCH 6/7] hw/pl022: Use LOG_UNIMP and LOG_GUEST_ERROR, Peter Maydell, 2012/10/14
- [Qemu-devel] [PATCH 5/7] hw/pl011: Use LOG_UNIMP and LOG_GUEST_ERROR, Peter Maydell, 2012/10/14
- [Qemu-devel] [PATCH 3/7] hw/pl041: Use LOG_UNIMP, Peter Maydell, 2012/10/14
- [Qemu-devel] [PATCH 2/7] hw/pl181: Use LOG_UNIMP and LOG_GUEST_ERROR, Peter Maydell, 2012/10/14
- [Qemu-devel] [PATCH 7/7] hw/pl031: Use LOG_GUEST_ERROR, Peter Maydell, 2012/10/14
- [Qemu-devel] [PATCH 4/7] hw/pl190: Use LOG_GUEST_ERROR, Peter Maydell, 2012/10/14
- [Qemu-devel] [PATCH 1/7] qemu-log: Add new log category for guest bugs, Peter Maydell, 2012/10/14
- Re: [Qemu-devel] [PATCH 0/7] Add LOG_GUEST_ERROR for reporting guest bugs, Blue Swirl, 2012/10/14