[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH] qdev: Reset hotplugged devices
From: |
Alex Williamson |
Subject: |
[Qemu-devel] [PATCH] qdev: Reset hotplugged devices |
Date: |
Tue, 03 Aug 2010 10:19:47 -0600 |
User-agent: |
StGIT/0.14.3 |
Several devices rely on their reset() function being called to
initialize device state, e1000 and rtl8139 in particular. When
the device is hot added, the reset doesn't occur, often leaving
the device in an unusable state. Adding a call to reset() after
init() for hotplugged devices puts the device in the expected
state for the guest.
Signed-off-by: Alex Williamson <address@hidden>
---
0.13 candidate?
hw/qdev.c | 3 +++
1 files changed, 3 insertions(+), 0 deletions(-)
diff --git a/hw/qdev.c b/hw/qdev.c
index e99c73f..b156272 100644
--- a/hw/qdev.c
+++ b/hw/qdev.c
@@ -278,6 +278,9 @@ int qdev_init(DeviceState *dev)
qdev_free(dev);
return rc;
}
+ if (dev->hotplugged) {
+ qdev_reset(dev);
+ }
qemu_register_reset(qdev_reset, dev);
if (dev->info->vmsd) {
vmstate_register_with_alias_id(dev, -1, dev->info->vmsd, dev,
- [Qemu-devel] [PATCH] qdev: Reset hotplugged devices,
Alex Williamson <=
- [Qemu-devel] Re: [PATCH] qdev: Reset hotplugged devices, Glauber Costa, 2010/08/03
- Re: [Qemu-devel] [PATCH] qdev: Reset hotplugged devices, Markus Armbruster, 2010/08/20
- Re: [Qemu-devel] [PATCH] qdev: Reset hotplugged devices, Alex Williamson, 2010/08/20
- Re: [Qemu-devel] [PATCH] qdev: Reset hotplugged devices, Markus Armbruster, 2010/08/20
- Re: [Qemu-devel] [PATCH] qdev: Reset hotplugged devices, Anthony Liguori, 2010/08/20
- Re: [Qemu-devel] [PATCH] qdev: Reset hotplugged devices, Markus Armbruster, 2010/08/20
- Re: [Qemu-devel] [PATCH] qdev: Reset hotplugged devices, Anthony Liguori, 2010/08/20
- Re: [Qemu-devel] [PATCH] qdev: Reset hotplugged devices, Alex Williamson, 2010/08/20
- Re: [Qemu-devel] [PATCH] qdev: Reset hotplugged devices, Markus Armbruster, 2010/08/21
- Re: [Qemu-devel] [PATCH] qdev: Reset hotplugged devices, Anthony Liguori, 2010/08/21