[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] don't call reset functions on cpu initializati
From: |
Laurent Desnogues |
Subject: |
Re: [Qemu-devel] [PATCH] don't call reset functions on cpu initialization |
Date: |
Tue, 3 Nov 2009 21:24:59 +0100 |
On Tue, Nov 3, 2009 at 9:16 PM, Glauber Costa <address@hidden> wrote:
> On Tue, Nov 03, 2009 at 09:12:41PM +0100, Laurent Desnogues wrote:
>> On Tue, Nov 3, 2009 at 8:50 PM, Glauber Costa <address@hidden> wrote:
>> > There is absolutely no need to call reset functions when initializing
>> > devices. Since we are already registering them, calling qemu_system_reset()
>> > should suffice. Actually, it is what happens when we reboot the machine,
>> > and using the same process instead of a special case semantics will even
>> > allow us to find bugs easier.
>> >
>> > Furthermore, the fact that we initialize things like the cpu quite early,
>> > leads to the need to introduce synchronization stuff like qemu_system_cond.
>> > This patch removes it entirely. All we need to do is call
>> > qemu_system_reset()
>> > only when we're already sure the system is up and running
>> >
>> > I tested it with qemu (with and without io-thread) and qemu-kvm, and it
>> > seems to be doing okay - although qemu-kvm uses a slightly different patch.
>> >
>> > Signed-off-by: Glauber Costa <address@hidden>
>> > ---
>> > hw/apic.c | 1 -
>> > hw/e1000.c | 1 -
>> > hw/hpet.c | 1 -
>> > hw/i8254.c | 2 --
>> > hw/ide/piix.c | 1 -
>> > hw/piix4.c | 1 -
>> > hw/piix_pci.c | 1 -
>> > hw/rtl8139.c | 2 +-
>> > hw/serial.c | 1 -
>> > hw/usb-ohci.c | 1 -
>> > hw/usb-uhci.c | 1 -
>> > hw/vga.c | 1 -
>> > target-i386/helper.c | 1 -
>> > vl.c | 15 +--------------
>> > 14 files changed, 2 insertions(+), 28 deletions(-)
>> [...]
>> > diff --git a/target-i386/helper.c b/target-i386/helper.c
>> > index c961544..957b3fc 100644
>> > --- a/target-i386/helper.c
>> > +++ b/target-i386/helper.c
>> > @@ -1885,7 +1885,6 @@ CPUX86State *cpu_x86_init(const char *cpu_model)
>> > return NULL;
>> > }
>> > mce_init(env);
>> > - cpu_reset(env);
>>
>> Who will call cpu_reset for user mode QEMU then?
>
> I believe we can then call it explicitly right after we call cpu_init
> for user mode, then
>
> or can't we?
Can't say :-)
Perhaps the safest thing to do would be to #ifdef/ifndef that
call without moving it?
Laurent