mbox series

[v2,0/4] vl: Fixes for cleanups to -accel

Message ID 20200116210528.31953-1-richard.henderson@linaro.org (mailing list archive)
Headers show
Series vl: Fixes for cleanups to -accel | expand

Message

Richard Henderson Jan. 16, 2020, 9:05 p.m. UTC
Running qemu-system-foo with no options should not generate
a warning for "invalid accelerator bar".

Changes in v2:
  * Rebase on master, getting the free accel_list fix from upstream.
    Re-word the resulting patch 2 to merely reduce the scope of the
    local variables.
  * Use g_str_has_suffix (ajb)


r~


Richard Henderson (4):
  vl: Remove unused variable in configure_accelerators
  vl: Reduce scope of variables in configure_accelerators
  vl: Remove useless test in configure_accelerators
  vl: Only choose enabled accelerators in configure_accelerators

 vl.c | 27 ++++++++++++++++-----------
 1 file changed, 16 insertions(+), 11 deletions(-)

Comments

Paolo Bonzini Jan. 17, 2020, 11:58 a.m. UTC | #1
On 16/01/20 22:05, Richard Henderson wrote:
> Running qemu-system-foo with no options should not generate
> a warning for "invalid accelerator bar".
> 
> Changes in v2:
>   * Rebase on master, getting the free accel_list fix from upstream.
>     Re-word the resulting patch 2 to merely reduce the scope of the
>     local variables.
>   * Use g_str_has_suffix (ajb)
> 
> 
> r~
> 
> 
> Richard Henderson (4):
>   vl: Remove unused variable in configure_accelerators
>   vl: Reduce scope of variables in configure_accelerators
>   vl: Remove useless test in configure_accelerators
>   vl: Only choose enabled accelerators in configure_accelerators
> 
>  vl.c | 27 ++++++++++++++++-----------
>  1 file changed, 16 insertions(+), 11 deletions(-)
> 

Acked-by: Paolo Bonzini <pbonzini@redhat.com>