[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/2] qdev: add qbus_reset_all
From: |
Andreas Färber |
Subject: |
Re: [Qemu-devel] [PATCH 1/2] qdev: add qbus_reset_all |
Date: |
Fri, 11 Jan 2013 18:34:22 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/17.0 Thunderbird/17.0 |
Am 10.01.2013 19:34, schrieb Andreas Färber:
> Am 10.01.2013 15:49, schrieb Paolo Bonzini:
>> Signed-off-by: Paolo Bonzini <address@hidden>
>> ---
>> hw/qdev-core.h | 12 ++++++++++++
>> hw/qdev.c | 7 ++++++-
>> 2 files changed, 18 insertions(+), 1 deletion(-)
>>
>> diff --git a/hw/qdev-core.h b/hw/qdev-core.h
>> index fdf14ec..853bd08 100644
>> --- a/hw/qdev-core.h
>> +++ b/hw/qdev-core.h
>> @@ -182,6 +182,18 @@ int qbus_walk_children(BusState *bus, qdev_walkerfn
>> *devfn,
>> int qdev_walk_children(DeviceState *dev, qdev_walkerfn *devfn,
>> qbus_walkerfn *busfn, void *opaque);
>> void qdev_reset_all(DeviceState *dev);
>> +
>> +/**
>> + * @qbus_reset_all:
>
> Just " * qbus_reset_all:" please.
Patch was applied nontheless, please fix in a follow-up.
Andreas
>> + * @bus: Bus to be reset.
>> + *
>> + * Reset @bus and perform a bus-level ("hard") reset of all devices
>> connected
>> + * to it, including recursive processing of all buses below @bus itself. A
>> + * hard reset means that qbus_reset_all will reset all state of the device.
>
> qbus_reset_all()?
>
>> + * For PCI devices, for example, this will include the base address
>> registers
>
> #PCIDevice?
>
>> + * or configuration space.
>> + */
>> +void qbus_reset_all(BusState *bus);
>
> Otherwise looks trivial and okay.
>
> Andreas
>
>> void qbus_reset_all_fn(void *opaque);
>>
>> void qbus_free(BusState *bus);
[snip]
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg