Re: [PATCH] hmp-commands-info.hx: Move definition of "info accel"

["Daniel P. Berrangé" <berrange@redhat.com>]

On Thu, Jan 08, 2026 at 09:32:49AM +0100, Markus Armbruster wrote:
> Commit c10eb740108 (accel/system: Add 'info accel' on human monitor)
> inserted "info accel" in the middle of "info sync-profile".  Move it
> behind "info sync-profile".

Opps, we have this kind of mistake over & over again. The .hx format
is really horrible and easy to make this mistake with, and makes me
rather wish we could eliminate it :-(



> 
> Signed-off-by: Markus Armbruster <armbru@redhat.com>
> ---
>  hmp-commands-info.hx | 24 ++++++++++++------------
>  1 file changed, 12 insertions(+), 12 deletions(-)

Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>

> 
> diff --git a/hmp-commands-info.hx b/hmp-commands-info.hx
> index 41674dcbe1..74c741f80e 100644
> --- a/hmp-commands-info.hx
> +++ b/hmp-commands-info.hx
> @@ -267,18 +267,6 @@ ERST
>          .cmd        = hmp_info_sync_profile,
>      },
>  
> -    {
> -        .name       = "accel",
> -        .args_type  = "",
> -        .params     = "",
> -        .help       = "show accelerator statistics",
> -    },
> -
> -SRST
> -  ``info accel``
> -    Show accelerator statistics.
> -ERST
> -
>  SRST
>    ``info sync-profile [-m|-n]`` [*max*]
>      Show synchronization profiling info, up to *max* entries (default: 10),
> @@ -294,6 +282,18 @@ SRST
>      being coalesced.
>  ERST
>  
> +    {
> +        .name       = "accel",
> +        .args_type  = "",
> +        .params     = "",
> +        .help       = "show accelerator statistics",
> +    },
> +
> +SRST
> +  ``info accel``
> +    Show accelerator statistics.
> +ERST
> +
>      {
>          .name       = "kvm",
>          .args_type  = "",
> -- 
> 2.52.0
> 
> 

With regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|