[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [Xen-devel] [RFC 1/8] move qemu_irq typedef out of cpu-
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [Xen-devel] [RFC 1/8] move qemu_irq typedef out of cpu-common.h |
Date: |
Wed, 29 Aug 2012 16:38:13 +0100 |
On 29 August 2012 16:36, Igor Mammedov <address@hidden> wrote:
> Peter Maydell <address@hidden> wrote:
>> I'm not objecting to this patch if it helps us move forwards,
>> but adding the #include to sysemu.h is effectively just adding
>> the definition to another grabbag header (183 files include
>> sysemu.h). It would be nicer long-term to separate out the
>> one thing in this header that cares about qemu_irq (the extern
>> declaration of qemu_system_powerdown).
>> [I'm not really convinced that a qemu_irq is even the right
>> way to signal "hey the system has actually powered down now"...]
>
> Instead of global qemu_system_powerdown we could use notifiers like it's done
> for suspend, I'll post patches today after testing them on target-i386.
>
> BTW getting rid of qemu_system_powerdown is orthogonal to topic of this
> series.
> I hope you won't object to this patch providing there will be follow on series
> to deal with qemu_system_powerdown.
Yes, as I say, I don't object if this patch is useful in the
meantime.
-- PMM
- [Qemu-devel] [RFC 0/8] include qdev core in *-user, make CPU child of DeviceState, Eduardo Habkost, 2012/08/21
- [Qemu-devel] [RFC 4/8] cleanup error.h, included qapi-types.h aready has stdbool.h, Eduardo Habkost, 2012/08/21
- [Qemu-devel] [RFC 7/8] include core qdev code into *-user, too, Eduardo Habkost, 2012/08/21
- [Qemu-devel] [RFC 3/8] qapi-types.h doesn't really need to include qemu-common.h, Eduardo Habkost, 2012/08/21
- [Qemu-devel] [RFC 2/8] qdev: split up header so it can be used in cpu.h, Eduardo Habkost, 2012/08/21
- [Qemu-devel] [RFC 5/8] split qdev into a core and code used only by qemu-system-*, Eduardo Habkost, 2012/08/21