[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v5 1/3] Check if GlobalProperty exists before re
From: |
Anthony Liguori |
Subject: |
Re: [Qemu-devel] [PATCH v5 1/3] Check if GlobalProperty exists before registering |
Date: |
Wed, 11 Jul 2012 12:52:41 -0500 |
User-agent: |
Notmuch/0.13.2+60~g7ecf77d (http://notmuchmail.org) Emacs/23.3.1 (x86_64-pc-linux-gnu) |
CrÃstian Viana <address@hidden> writes:
> If a GlobalProperty has already been registered, it won't have its
> value overwritten.
>
> Signed-off-by: CrÃstian Viana <address@hidden>
> ---
>
> Changes since v4:
> - Only register a GlobalProperty if it hasn't already been registered. That
> avoids the property values to be overwritten (fix previous known bug).
>
> hw/qdev-properties.c | 8 ++++++++
> 1 file changed, 8 insertions(+)
>
> diff --git a/hw/qdev-properties.c b/hw/qdev-properties.c
> index 099a7aa..fda94dd 100644
> --- a/hw/qdev-properties.c
> +++ b/hw/qdev-properties.c
> @@ -1105,6 +1105,14 @@ static QTAILQ_HEAD(, GlobalProperty) global_props =
> QTAILQ_HEAD_INITIALIZER(glob
>
> static void qdev_prop_register_global(GlobalProperty *prop)
> {
> + GlobalProperty *p;
> +
> + QTAILQ_FOREACH(p, &global_props, next) {
> + if (strcmp(prop->driver, p->driver) == 0) {
> + return;
> + }
> + }
> +
> QTAILQ_INSERT_TAIL(&global_props, prop, next);
> }
This is relying on a very subtle characteristic: that -global properties
are registered before machine options. We probably need to be more
explicit about who we want to win. In the very least, you should add a
comment here that says this check is specifically to allow -global to
"win" over machine options.
Regards,
Anthony Liguori
>
> --
> 1.7.9.5
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-devel] [PATCH v5 1/3] Check if GlobalProperty exists before registering,
Anthony Liguori <=