[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [PATCH] savevm: unbreak register_savevm_live()/vmstate_
From: |
Alex Williamson |
Subject: |
[Qemu-devel] Re: [PATCH] savevm: unbreak register_savevm_live()/vmstate_register_with_alias_id() |
Date: |
Wed, 26 Jan 2011 07:04:51 -0700 |
On Wed, 2011-01-26 at 18:45 +0900, Isaku Yamahata wrote:
> This patch unbreaks 7685ee6abcb939104801f84b3fe9645412528088.
> With the changeset, more than one instances of same device on bus
> that provides get_dev_path method can't be created because it hits
> the assertion.
> This patch removes the assertion whose assumption isn't correct.
Can you give an example of where this is needed? The point of
get_dev_path is that when it's available, it's supposed to provide a
unique string per device. Since get_dev_path is really only implemented
for PCI, does this mean you have two independent devices at the same PCI
address? Thanks,
Alex
> Cc: Alex Williamson <address@hidden>
> Signed-off-by: Isaku Yamahata <address@hidden>
> ---
> savevm.c | 2 --
> 1 files changed, 0 insertions(+), 2 deletions(-)
>
> diff --git a/savevm.c b/savevm.c
> index fcd8db4..cd29eb8 100644
> --- a/savevm.c
> +++ b/savevm.c
> @@ -1126,7 +1126,6 @@ int register_savevm_live(DeviceState *dev,
> } else {
> se->instance_id = instance_id;
> }
> - assert(!se->compat || se->instance_id == 0);
> /* add at the end of list */
> QTAILQ_INSERT_TAIL(&savevm_handlers, se, entry);
> return 0;
> @@ -1236,7 +1235,6 @@ int vmstate_register_with_alias_id(DeviceState *dev,
> int instance_id,
> } else {
> se->instance_id = instance_id;
> }
> - assert(!se->compat || se->instance_id == 0);
> /* add at the end of list */
> QTAILQ_INSERT_TAIL(&savevm_handlers, se, entry);
> return 0;