Re: [PATCH 2/5] include/migration: mark vmstate_register() as a legacy function

2023-06-21 Thread Juan Quintela 
Alex Bennée  wrote:
> Mention that QOM-ified devices already have support for registering
> the description.
>
> Signed-off-by: Alex Bennée 

Reviewed-by: Juan Quintela 

I really remove that function in a future series (well, I substitute it
with vmstate_register_id() and vmstate_register_any(), but the comment
applies to the new versions also).

Later, Juan.

Re: [PATCH 2/5] include/migration: mark vmstate_register() as a legacy function

2023-06-20 Thread Peter Xu 
On Mon, Jun 19, 2023 at 06:14:34PM +0100, Alex Bennée wrote:
> Mention that QOM-ified devices already have support for registering
> the description.
> 
> Signed-off-by: Alex Bennée 

Acked-by: Peter Xu 

-- 
Peter Xu

Re: [PATCH 2/5] include/migration: mark vmstate_register() as a legacy function

2023-06-20 Thread Richard Henderson 

On 6/19/23 19:14, Alex Bennée wrote:

Mention that QOM-ified devices already have support for registering
the description.

Signed-off-by: Alex Bennée
---
  include/migration/vmstate.h | 9 -
  1 file changed, 8 insertions(+), 1 deletion(-)


Reviewed-by: Richard Henderson 

r~

Re: [PATCH 2/5] include/migration: mark vmstate_register() as a legacy function

2023-06-19 Thread Philippe Mathieu-Daudé 

On 19/6/23 19:14, Alex Bennée wrote:

Mention that QOM-ified devices already have support for registering
the description.

Signed-off-by: Alex Bennée 
---
  include/migration/vmstate.h | 9 -
  1 file changed, 8 insertions(+), 1 deletion(-)


Reviewed-by: Philippe Mathieu-Daudé