qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] migration: Remove unused VMSTATE_ARRAY_TEST() macro
@ 2024-06-21  7:03 Philippe Mathieu-Daudé
  2024-06-21  8:13 ` Zhijian Li (Fujitsu) via
  2024-06-21 17:38 ` Fabiano Rosas
  0 siblings, 2 replies; 4+ messages in thread
From: Philippe Mathieu-Daudé @ 2024-06-21  7:03 UTC (permalink / raw)
  To: qemu-devel
  Cc: Thomas Huth, Fabiano Rosas, Peter Xu, Philippe Mathieu-Daudé

Last use of VMSTATE_ARRAY_TEST() was removed in commit 46baa9007f
("migration/i386: Remove old non-softfloat 64bit FP support"), we
can safely get rid of it.

Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
---
 include/migration/vmstate.h | 10 ----------
 1 file changed, 10 deletions(-)

diff --git a/include/migration/vmstate.h b/include/migration/vmstate.h
index 294d2d8486..f313f2f408 100644
--- a/include/migration/vmstate.h
+++ b/include/migration/vmstate.h
@@ -388,16 +388,6 @@ extern const VMStateInfo vmstate_info_qlist;
     .offset     = vmstate_offset_varray(_state, _field, _type),      \
 }
 
-#define VMSTATE_ARRAY_TEST(_field, _state, _num, _test, _info, _type) {\
-    .name         = (stringify(_field)),                              \
-    .field_exists = (_test),                                          \
-    .num          = (_num),                                           \
-    .info         = &(_info),                                         \
-    .size         = sizeof(_type),                                    \
-    .flags        = VMS_ARRAY,                                        \
-    .offset       = vmstate_offset_array(_state, _field, _type, _num),\
-}
-
 #define VMSTATE_SUB_ARRAY(_field, _state, _start, _num, _version, _info, _type) { \
     .name       = (stringify(_field)),                               \
     .version_id = (_version),                                        \
-- 
2.41.0



^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] migration: Remove unused VMSTATE_ARRAY_TEST() macro
  2024-06-21  7:03 [PATCH] migration: Remove unused VMSTATE_ARRAY_TEST() macro Philippe Mathieu-Daudé
@ 2024-06-21  8:13 ` Zhijian Li (Fujitsu) via
  2024-06-21 14:16   ` Peter Xu
  2024-06-21 17:38 ` Fabiano Rosas
  1 sibling, 1 reply; 4+ messages in thread
From: Zhijian Li (Fujitsu) via @ 2024-06-21  8:13 UTC (permalink / raw)
  To: Philippe Mathieu-Daudé, qemu-devel@nongnu.org
  Cc: Thomas Huth, Fabiano Rosas, Peter Xu



On 21/06/2024 15:03, Philippe Mathieu-Daudé wrote:
> Last use of VMSTATE_ARRAY_TEST() was removed in commit 46baa9007f
> ("migration/i386: Remove old non-softfloat 64bit FP support"), we
> can safely get rid of it.
> 
> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>

Reviewed-by: Li Zhijian <lizhijian@fujitsu.com>


> ---
>   include/migration/vmstate.h | 10 ----------
>   1 file changed, 10 deletions(-)
> 
> diff --git a/include/migration/vmstate.h b/include/migration/vmstate.h
> index 294d2d8486..f313f2f408 100644
> --- a/include/migration/vmstate.h
> +++ b/include/migration/vmstate.h
> @@ -388,16 +388,6 @@ extern const VMStateInfo vmstate_info_qlist;
>       .offset     = vmstate_offset_varray(_state, _field, _type),      \
>   }
>   
> -#define VMSTATE_ARRAY_TEST(_field, _state, _num, _test, _info, _type) {\
> -    .name         = (stringify(_field)),                              \
> -    .field_exists = (_test),                                          \
> -    .num          = (_num),                                           \
> -    .info         = &(_info),                                         \
> -    .size         = sizeof(_type),                                    \
> -    .flags        = VMS_ARRAY,                                        \
> -    .offset       = vmstate_offset_array(_state, _field, _type, _num),\
> -}
> -
>   #define VMSTATE_SUB_ARRAY(_field, _state, _start, _num, _version, _info, _type) { \
>       .name       = (stringify(_field)),                               \
>       .version_id = (_version),                                        \

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] migration: Remove unused VMSTATE_ARRAY_TEST() macro
  2024-06-21  8:13 ` Zhijian Li (Fujitsu) via
@ 2024-06-21 14:16   ` Peter Xu
  0 siblings, 0 replies; 4+ messages in thread
From: Peter Xu @ 2024-06-21 14:16 UTC (permalink / raw)
  To: Zhijian Li (Fujitsu)
  Cc: Philippe Mathieu-Daudé, qemu-devel@nongnu.org, Thomas Huth,
	Fabiano Rosas

On Fri, Jun 21, 2024 at 08:13:35AM +0000, Zhijian Li (Fujitsu) via wrote:
> 
> 
> On 21/06/2024 15:03, Philippe Mathieu-Daudé wrote:
> > Last use of VMSTATE_ARRAY_TEST() was removed in commit 46baa9007f
> > ("migration/i386: Remove old non-softfloat 64bit FP support"), we
> > can safely get rid of it.
> > 
> > Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> 
> Reviewed-by: Li Zhijian <lizhijian@fujitsu.com>

Reviewed-by: Peter Xu <peterx@redhat.com>

-- 
Peter Xu



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] migration: Remove unused VMSTATE_ARRAY_TEST() macro
  2024-06-21  7:03 [PATCH] migration: Remove unused VMSTATE_ARRAY_TEST() macro Philippe Mathieu-Daudé
  2024-06-21  8:13 ` Zhijian Li (Fujitsu) via
@ 2024-06-21 17:38 ` Fabiano Rosas
  1 sibling, 0 replies; 4+ messages in thread
From: Fabiano Rosas @ 2024-06-21 17:38 UTC (permalink / raw)
  To: qemu-devel, Philippe Mathieu-Daudé; +Cc: Thomas Huth, Peter Xu

On Fri, 21 Jun 2024 09:03:17 +0200, Philippe Mathieu-Daudé wrote:
> Last use of VMSTATE_ARRAY_TEST() was removed in commit 46baa9007f
> ("migration/i386: Remove old non-softfloat 64bit FP support"), we
> can safely get rid of it.
> 
> 

Queued, thanks!


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2024-06-21 17:39 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-06-21  7:03 [PATCH] migration: Remove unused VMSTATE_ARRAY_TEST() macro Philippe Mathieu-Daudé
2024-06-21  8:13 ` Zhijian Li (Fujitsu) via
2024-06-21 14:16   ` Peter Xu
2024-06-21 17:38 ` Fabiano Rosas

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).