[Qemu-devel] [PATCH 0/4] Clean up accelerator options

Thomas Huth posted 4 patches 5 years, 10 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/1528866321-23886-1-git-send-email-thuth@redhat.com
Test checkpatch passed
Test docker-mingw@fedora passed
Test docker-quick@centos7 passed
Test s390x passed
docs/COLO-FT.txt                    |  8 +++----
docs/can.txt                        |  4 ++--
docs/multi-thread-compression.txt   |  2 +-
docs/multiseat.txt                  |  2 +-
docs/specs/tpm.txt                  |  8 +++----
hw/block/dataplane/virtio-blk.c     |  4 ++--
hw/scsi/virtio-scsi-dataplane.c     |  4 ++--
qemu-doc.texi                       |  3 +--
qemu-options.hx                     | 44 ++++++++++++-------------------------
scripts/qtest.py                    |  2 +-
tests/bios-tables-test.c            |  2 +-
tests/boot-serial-test.c            |  2 +-
tests/libqtest.c                    |  2 +-
tests/migration-test.c              |  8 +++----
tests/migration/guestperf/engine.py |  2 +-
tests/pnv-xscom-test.c              |  2 +-
tests/prom-env-test.c               |  2 +-
tests/pxe-test.c                    |  2 +-
tests/qemu-iotests/172              |  2 +-
tests/qemu-iotests/check            |  2 +-
tests/vmgenid-test.c                |  2 +-
21 files changed, 46 insertions(+), 63 deletions(-)
[Qemu-devel] [PATCH 0/4] Clean up accelerator options
Posted by Thomas Huth 5 years, 10 months ago
The current state of the accelerator options is quite messy: There are
three ways of enabling an accelerator (-machine accel=xyz, -accel xyz
and -enable-xyz) and this can be very confusing for the users. This
patch series now tries to standardize our documentation and examples
on the preferred -accel option.

Thomas Huth (4):
  Replace '-machine accel=xyz' with '-accel xyz'
  Replace '-enable-kvm' with '-accel kvm' in docs and help texts
  qemu-options: Improve the documentation of '-accel' and '-machine
    accel=...'
  qemu-options: Do not show -enable-kvm and -enable-hax in the docs
    anymore

 docs/COLO-FT.txt                    |  8 +++----
 docs/can.txt                        |  4 ++--
 docs/multi-thread-compression.txt   |  2 +-
 docs/multiseat.txt                  |  2 +-
 docs/specs/tpm.txt                  |  8 +++----
 hw/block/dataplane/virtio-blk.c     |  4 ++--
 hw/scsi/virtio-scsi-dataplane.c     |  4 ++--
 qemu-doc.texi                       |  3 +--
 qemu-options.hx                     | 44 ++++++++++++-------------------------
 scripts/qtest.py                    |  2 +-
 tests/bios-tables-test.c            |  2 +-
 tests/boot-serial-test.c            |  2 +-
 tests/libqtest.c                    |  2 +-
 tests/migration-test.c              |  8 +++----
 tests/migration/guestperf/engine.py |  2 +-
 tests/pnv-xscom-test.c              |  2 +-
 tests/prom-env-test.c               |  2 +-
 tests/pxe-test.c                    |  2 +-
 tests/qemu-iotests/172              |  2 +-
 tests/qemu-iotests/check            |  2 +-
 tests/vmgenid-test.c                |  2 +-
 21 files changed, 46 insertions(+), 63 deletions(-)

-- 
1.8.3.1


Re: [Qemu-devel] [PATCH 0/4] Clean up accelerator options
Posted by Stefan Hajnoczi 5 years, 10 months ago
On Wed, Jun 13, 2018 at 07:05:17AM +0200, Thomas Huth wrote:
> The current state of the accelerator options is quite messy: There are
> three ways of enabling an accelerator (-machine accel=xyz, -accel xyz
> and -enable-xyz) and this can be very confusing for the users. This
> patch series now tries to standardize our documentation and examples
> on the preferred -accel option.

Nice, thank you for keeping things tidy.

Stefan

> Thomas Huth (4):
>   Replace '-machine accel=xyz' with '-accel xyz'
>   Replace '-enable-kvm' with '-accel kvm' in docs and help texts
>   qemu-options: Improve the documentation of '-accel' and '-machine
>     accel=...'
>   qemu-options: Do not show -enable-kvm and -enable-hax in the docs
>     anymore
> 
>  docs/COLO-FT.txt                    |  8 +++----
>  docs/can.txt                        |  4 ++--
>  docs/multi-thread-compression.txt   |  2 +-
>  docs/multiseat.txt                  |  2 +-
>  docs/specs/tpm.txt                  |  8 +++----
>  hw/block/dataplane/virtio-blk.c     |  4 ++--
>  hw/scsi/virtio-scsi-dataplane.c     |  4 ++--
>  qemu-doc.texi                       |  3 +--
>  qemu-options.hx                     | 44 ++++++++++++-------------------------
>  scripts/qtest.py                    |  2 +-
>  tests/bios-tables-test.c            |  2 +-
>  tests/boot-serial-test.c            |  2 +-
>  tests/libqtest.c                    |  2 +-
>  tests/migration-test.c              |  8 +++----
>  tests/migration/guestperf/engine.py |  2 +-
>  tests/pnv-xscom-test.c              |  2 +-
>  tests/prom-env-test.c               |  2 +-
>  tests/pxe-test.c                    |  2 +-
>  tests/qemu-iotests/172              |  2 +-
>  tests/qemu-iotests/check            |  2 +-
>  tests/vmgenid-test.c                |  2 +-
>  21 files changed, 46 insertions(+), 63 deletions(-)
> 
> -- 
> 1.8.3.1
>