[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/3] Add pci_bus_reset() function.
From: |
Gleb Natapov |
Subject: |
Re: [Qemu-devel] [PATCH 2/3] Add pci_bus_reset() function. |
Date: |
Mon, 8 Jun 2009 17:37:10 +0300 |
On Mon, Jun 08, 2009 at 05:35:11PM +0300, Avi Kivity wrote:
> Gleb Natapov wrote:
>> To reset internal irq handling data structures.
>>
>> Signed-off-by: Gleb Natapov <address@hidden>
>> Signed-off-by: Yaniv Kamay <address@hidden>
>> ---
>> hw/pci.c | 16 ++++++++++++++++
>> 1 files changed, 16 insertions(+), 0 deletions(-)
>>
>> diff --git a/hw/pci.c b/hw/pci.c
>> index 02b335f..89fefdf 100644
>> --- a/hw/pci.c
>> +++ b/hw/pci.c
>> @@ -88,6 +88,21 @@ static int pcibus_load(QEMUFile *f, void *opaque, int
>> version_id)
>> return 0;
>> }
>> +static void pci_bus_reset(void *opaque)
>> +{
>> + PCIBus *bus = (PCIBus *)opaque;
>> + int i;
>> +
>> + for (i = 0; i < bus->nirq; i++) {
>> + bus->irq_count[i] = 0;
>> + }
>> + for (i = 0; i < PCI_DEVICES_MAX; i++) {
>> + if (bus->devices[i])
>> + memset(bus->devices[i]->irq_state, 0,
>> + sizeof(bus->devices[i]->irq_state));
>> + }
>> +}
>> +
>>
>
> Shouldn't each device's reset function bring its line low, thus zeroing
> the irq_state naturally?
>
> If not, we have a bug somewhere. Note we have exactly the same issue
> with save/restore.
>
They should, but I haven't found one that does.
--
Gleb.
- [Qemu-devel] [PATCH 1/3] Add rtc reset function., Gleb Natapov, 2009/06/08
- [Qemu-devel] [PATCH 2/3] Add pci_bus_reset() function., Gleb Natapov, 2009/06/08
- Re: [Qemu-devel] [PATCH 2/3] Add pci_bus_reset() function., Avi Kivity, 2009/06/08
- Re: [Qemu-devel] [PATCH 2/3] Add pci_bus_reset() function.,
Gleb Natapov <=
- Re: [Qemu-devel] [PATCH 2/3] Add pci_bus_reset() function., Yaniv Kamay, 2009/06/08
- Re: [Qemu-devel] [PATCH 2/3] Add pci_bus_reset() function., Dor Laor, 2009/06/08
- Re: [Qemu-devel] [PATCH 2/3] Add pci_bus_reset() function., Gleb Natapov, 2009/06/09
- Re: [Qemu-devel] [PATCH 2/3] Add pci_bus_reset() function., Blue Swirl, 2009/06/09
- Re: [Qemu-devel] [PATCH 2/3] Add pci_bus_reset() function., Gleb Natapov, 2009/06/09
- Re: [Qemu-devel] [PATCH 2/3] Add pci_bus_reset() function., Gleb Natapov, 2009/06/09
- Re: [Qemu-devel] [PATCH 2/3] Add pci_bus_reset() function., Blue Swirl, 2009/06/09
[Qemu-devel] [PATCH 3/3] Call piix3_reset() on system reset., Gleb Natapov, 2009/06/08
[Qemu-devel] Re: [PATCH 1/3] Add rtc reset function., Jan Kiszka, 2009/06/08