OpenE2K/qemu-e2k
13
4
Fork 0
Code Issues 4 Pull Requests Projects 2 Releases Activity

qdev: Do not use slow [*] expansion for GPIO creation

Browse Source 
Expansion of [*] suffix is very slow because index expansion is done using
trial and error strategy, starting every time from zero and retrying with
the next index until insertion succeeds. With large number of already added
properties this process takes huge amount of time (O(n^2) complexity).

Some architectures (like ARM) use very large amount of IRQ pins in interrupt
controller models. This flaw makes machine startup extremely slow
(~20 seconds for ARM64 with 32 CPUs). This patch decreases this time down to
~10 seconds.

Also in qdev_init_gpio_out_named() memset() is now called only once for the
whole array instead of per-cell cleaning

Signed-off-by: Pavel Fedin <p.fedin@samsung.com>
Reviewed-by: Daniel P. Berrange <berrange@redhat.com>
Signed-off-by: Andreas Färber <afaerber@suse.de>
This commit is contained in:
Pavel Fedin 2015-07-31 15:23:22 +03:00 committed by Andreas Färber
parent e1c8237df5
commit 6c76b37742
1 changed files with 15 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
hw/core/qdev.c
View File

@ -417,17 +417,21 @@ void qdev_init_gpio_in_named(DeviceState *dev, qemu_irq_handler handler,
{ {
int i; int i;
NamedGPIOList *gpio_list = qdev_get_named_gpio_list(dev, name); NamedGPIOList *gpio_list = qdev_get_named_gpio_list(dev, name);
char *propname = g_strdup_printf("%s[*]", name ? name : "unnamed-gpio-in");
assert(gpio_list->num_out == 0 || !name); assert(gpio_list->num_out == 0 || !name);
gpio_list->in = qemu_extend_irqs(gpio_list->in, gpio_list->num_in, handler, gpio_list->in = qemu_extend_irqs(gpio_list->in, gpio_list->num_in, handler,
dev, n); dev, n);
if (!name) {
name = "unnamed-gpio-in";
}
for (i = gpio_list->num_in; i < gpio_list->num_in + n; i++) { for (i = gpio_list->num_in; i < gpio_list->num_in + n; i++) {
gchar *propname = g_strdup_printf("%s[%u]", name, i);
object_property_add_child(OBJECT(dev), propname, object_property_add_child(OBJECT(dev), propname,
OBJECT(gpio_list->in[i]), &error_abort); OBJECT(gpio_list->in[i]), &error_abort);
g_free(propname);
} }
g_free(propname);
gpio_list->num_in += n; gpio_list->num_in += n;
} }
@ -442,20 +446,25 @@ void qdev_init_gpio_out_named(DeviceState *dev, qemu_irq *pins,
{ {
int i; int i;
NamedGPIOList *gpio_list = qdev_get_named_gpio_list(dev, name); NamedGPIOList *gpio_list = qdev_get_named_gpio_list(dev, name);
char *propname = g_strdup_printf("%s[*]", name ? name : "unnamed-gpio-out");
assert(gpio_list->num_in == 0 || !name); assert(gpio_list->num_in == 0 || !name);
gpio_list->num_out += n;
if (!name) {
name = "unnamed-gpio-out";
}
memset(pins, 0, sizeof(*pins) * n);
for (i = 0; i < n; ++i) { for (i = 0; i < n; ++i) {
memset(&pins[i], 0, sizeof(*pins)); gchar *propname = g_strdup_printf("%s[%u]", name,
gpio_list->num_out + i);
object_property_add_link(OBJECT(dev), propname, TYPE_IRQ, object_property_add_link(OBJECT(dev), propname, TYPE_IRQ,
(Object **)&pins[i], (Object **)&pins[i],
object_property_allow_set_link, object_property_allow_set_link,
OBJ_PROP_LINK_UNREF_ON_RELEASE, OBJ_PROP_LINK_UNREF_ON_RELEASE,
&error_abort); &error_abort);
g_free(propname);
} }
g_free(propname); gpio_list->num_out += n;
} }
void qdev_init_gpio_out(DeviceState *dev, qemu_irq *pins, int n) void qdev_init_gpio_out(DeviceState *dev, qemu_irq *pins, int n)