[v2] vl: Fixes for cleanups to -accel

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

Richard Henderson posted 4 patches 4 years, 3 months ago

Diff against v1
Download series mbox

Test docker-quick@centos7 failed

Test docker-mingw@fedora failed

Test FreeBSD passed

Test checkpatch passed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20200116210528.31953-1-richard.henderson@linaro.org

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

Expand all Fold all

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

Posted by Richard Henderson 4 years, 3 months ago

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(-)

-- 
2.20.1

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

Posted by Paolo Bonzini 4 years, 3 months ago

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>