[Qemu-commits] [qemu/qemu] 2b8985: qtest: Rename qtest.c:qtest

qemu-commits
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-commits] [qemu/qemu] 2b8985: qtest: Rename qtest.c:qtest_init()

From:	Peter Maydell
Subject:	[Qemu-commits] [qemu/qemu] 2b8985: qtest: Rename qtest.c:qtest_init()
Date:	Fri, 16 Aug 2019 06:52:55 -0700
  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: 2b8985f1b82ea994dbe37986536c5c623c38ea86
      
https://github.com/qemu/qemu/commit/2b8985f1b82ea994dbe37986536c5c623c38ea86
  Author: Oleinik, Alexander <address@hidden>
  Date:   2019-08-15 (Thu, 15 Aug 2019)

  Changed paths:
    M include/sysemu/qtest.h
    M qtest.c
    M vl.c

  Log Message:
  -----------
  qtest: Rename qtest.c:qtest_init()

Both the qtest client, libqtest.c, and server, qtest.c, used the same
name for initialization functions which can cause confusion.

Signed-off-by: Alexander Oleinik <address@hidden>
Message-Id: <address@hidden>
Reviewed-by: John Snow <address@hidden>
Reviewed-by: Stefan Hajnoczi <address@hidden>
Signed-off-by: Thomas Huth <address@hidden>


  Commit: 375eae1c716bd55936d65ad545ae2ea0c9909b91
      
https://github.com/qemu/qemu/commit/375eae1c716bd55936d65ad545ae2ea0c9909b91
  Author: Oleinik, Alexander <address@hidden>
  Date:   2019-08-15 (Thu, 15 Aug 2019)

  Changed paths:
    M tests/libqos/virtio-net.c
    M tests/libqos/virtio-net.h

  Log Message:
  -----------
  libqos: Account for the ctrl queue in virtio-net

The number of queues is 2n+1, where n == 1 when multiqueue is disabled

Signed-off-by: Alexander Oleinik <address@hidden>
Message-Id: <address@hidden>
[thuth: fixed "intefaces" typo]
Signed-off-by: Thomas Huth <address@hidden>


  Commit: ca1ef1e62e581f61c40f8be49d222007739691c4
      
https://github.com/qemu/qemu/commit/ca1ef1e62e581f61c40f8be49d222007739691c4
  Author: Andrey Shinkevich <address@hidden>
  Date:   2019-08-15 (Thu, 15 Aug 2019)

  Changed paths:
    M tests/drive_del-test.c
    M tests/libqos/virtio-scsi.c
    M tests/megasas-test.c
    M tests/nvme-test.c
    M tests/qmp-test.c
    M tests/test-blockjob-txn.c
    M tests/test-blockjob.c
    M tests/usb-hcd-uhci-test.c
    M tests/usb-hcd-xhci-test.c
    M tests/virtio-blk-test.c
    M tests/virtio-ccw-test.c
    M tests/virtio-scsi-test.c

  Log Message:
  -----------
  tests: Set read-zeroes on for null-co driver

This patch is to reduce the number of Valgrind report messages about
using uninitialized memory with the null-co driver. It helps to filter
real memory issues and is the same work done for the iotests with the
commit ID a6862418fec4072.

Suggested-by: Kevin Wolf <address@hidden>
Signed-off-by: Andrey Shinkevich <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Thomas Huth <address@hidden>


  Commit: 1999a70a05ad603d1c76e0a7d230bcfed0d626ac
      
https://github.com/qemu/qemu/commit/1999a70a05ad603d1c76e0a7d230bcfed0d626ac
  Author: Thomas Huth <address@hidden>
  Date:   2019-08-15 (Thu, 15 Aug 2019)

  Changed paths:
    M tests/libqos/virtio.c
    M tests/libqos/virtio.h
    M tests/virtio-9p-test.c
    M tests/virtio-blk-test.c
    M tests/virtio-net-test.c
    M tests/virtio-scsi-test.c

  Log Message:
  -----------
  tests/libqos: Make generic virtio code independent from global_qtest

The libqos library functions should never depend on global_qtest,
since these functions might be used in tests that track multiple
test states. Pass around a pointer to the QTestState instead.

Message-Id: <address@hidden>
Signed-off-by: Thomas Huth <address@hidden>


  Commit: 2380d31e19ee32e6466c7724ab72ae9d2fa2db96
      
https://github.com/qemu/qemu/commit/2380d31e19ee32e6466c7724ab72ae9d2fa2db96
  Author: Thomas Huth <address@hidden>
  Date:   2019-08-15 (Thu, 15 Aug 2019)

  Changed paths:
    M tests/libqos/virtio-pci.c

  Log Message:
  -----------
  tests/libqos: Make virtio-pci code independent from global_qtest

The libqos library functions should never depend on global_qtest,
since these functions might be used in tests that track multiple
test states. So let's use the test state of the QPCIDevice instead.

Reviewed-by: Stefan Hajnoczi <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Thomas Huth <address@hidden>


  Commit: f945b84a2b1307086efcf97a95702eb74a8aaba5
      
https://github.com/qemu/qemu/commit/f945b84a2b1307086efcf97a95702eb74a8aaba5
  Author: Thomas Huth <address@hidden>
  Date:   2019-08-15 (Thu, 15 Aug 2019)

  Changed paths:
    M tests/libqtest.c
    M tests/libqtest.h

  Log Message:
  -----------
  tests/libqtest: Remove unused function hmp()

No test is using hmp() anymore, and since this function uses the disliked
global_qtest variable, we should also make sure that nobody adds new code
with this function again. qtest_hmp() should be used instead.

Reviewed-by: Eric Blake <address@hidden>
Reviewed-by: Stefan Hajnoczi <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Thomas Huth <address@hidden>


  Commit: 17de4741296af8ed3361b14feda130a7d7102987
      
https://github.com/qemu/qemu/commit/17de4741296af8ed3361b14feda130a7d7102987
  Author: Thomas Huth <address@hidden>
  Date:   2019-08-15 (Thu, 15 Aug 2019)

  Changed paths:
    M tests/libqtest.c

  Log Message:
  -----------
  tests/libqtest: Clean up qtest_cb_for_every_machine() wrt global_qtest

The generic libqtest library functions should not use functions that
require the global_qtest variable.

Reviewed-by: Stefan Hajnoczi <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Thomas Huth <address@hidden>


  Commit: e5758de4e836c3b2edc2befd904651fc6967d74f
      
https://github.com/qemu/qemu/commit/e5758de4e836c3b2edc2befd904651fc6967d74f
  Author: Thomas Huth <address@hidden>
  Date:   2019-08-15 (Thu, 15 Aug 2019)

  Changed paths:
    M tests/cpu-plug-test.c
    M tests/e1000e-test.c
    M tests/ivshmem-test.c
    M tests/libqos/usb.c
    M tests/libqos/usb.h
    M tests/libqtest.c
    M tests/libqtest.h
    M tests/usb-hcd-ohci-test.c
    M tests/usb-hcd-uhci-test.c
    M tests/usb-hcd-xhci-test.c
    M tests/virtio-blk-test.c
    M tests/virtio-ccw-test.c
    M tests/virtio-net-test.c
    M tests/virtio-rng-test.c
    M tests/virtio-scsi-test.c
    M tests/virtio-serial-test.c

  Log Message:
  -----------
  tests/libqtest: Make qtest_qmp_device_add/del independent from global_qtest

Generic library functions like qtest_qmp_device_add() and _del()
should not depend on the global_qtest variable. Pass the test
state via parameter instead.

Reviewed-by: Eric Blake <address@hidden>
Reviewed-by: Stefan Hajnoczi <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Thomas Huth <address@hidden>


  Commit: 6fc9f3d347aee337421f8afc4d0984294f8ea6c3
      
https://github.com/qemu/qemu/commit/6fc9f3d347aee337421f8afc4d0984294f8ea6c3
  Author: Thomas Huth <address@hidden>
  Date:   2019-08-15 (Thu, 15 Aug 2019)

  Changed paths:
    M tests/libqtest.c
    M tests/libqtest.h
    M tests/virtio-scsi-test.c

  Log Message:
  -----------
  tests/libqtest: Make qmp_assert_success() independent from global_qtest

The normal libqtest library functions should never depend on global_qtest.
Pass in the test state via parameter instead. And while we're at it,
also rename this function to qtest_qmp_assert_success() to make it clear
that it is part of libqtest.

Reviewed-by: Stefan Hajnoczi <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Thomas Huth <address@hidden>


  Commit: 1f3a51f3feabb5235c073a9ec9211156a519d058
      
https://github.com/qemu/qemu/commit/1f3a51f3feabb5235c073a9ec9211156a519d058
  Author: Peter Maydell <address@hidden>
  Date:   2019-08-16 (Fri, 16 Aug 2019)

  Changed paths:
    M include/sysemu/qtest.h
    M qtest.c
    M tests/cpu-plug-test.c
    M tests/drive_del-test.c
    M tests/e1000e-test.c
    M tests/ivshmem-test.c
    M tests/libqos/usb.c
    M tests/libqos/usb.h
    M tests/libqos/virtio-net.c
    M tests/libqos/virtio-net.h
    M tests/libqos/virtio-pci.c
    M tests/libqos/virtio-scsi.c
    M tests/libqos/virtio.c
    M tests/libqos/virtio.h
    M tests/libqtest.c
    M tests/libqtest.h
    M tests/megasas-test.c
    M tests/nvme-test.c
    M tests/qmp-test.c
    M tests/test-blockjob-txn.c
    M tests/test-blockjob.c
    M tests/usb-hcd-ohci-test.c
    M tests/usb-hcd-uhci-test.c
    M tests/usb-hcd-xhci-test.c
    M tests/virtio-9p-test.c
    M tests/virtio-blk-test.c
    M tests/virtio-ccw-test.c
    M tests/virtio-net-test.c
    M tests/virtio-rng-test.c
    M tests/virtio-scsi-test.c
    M tests/virtio-serial-test.c
    M vl.c

  Log Message:
  -----------
  Merge remote-tracking branch 
'remotes/huth-gitlab/tags/pull-request-2019-08-15' into staging

- Fix for ctrl queue in the virtio-net QOS driver
- Improve Valgrind reports in the tests that use the null-co driver
- Get rid of global_qtest related code in libqtest and libqos

# gpg: Signature made Thu 15 Aug 2019 18:28:16 BST
# gpg:                using RSA key 27B88847EEE0250118F3EAB92ED9D774FE702DB5
# gpg:                issuer "address@hidden"
# gpg: Good signature from "Thomas Huth <address@hidden>" [full]
# gpg:                 aka "Thomas Huth <address@hidden>" [full]
# gpg:                 aka "Thomas Huth <address@hidden>" [full]
# gpg:                 aka "Thomas Huth <address@hidden>" [unknown]
# Primary key fingerprint: 27B8 8847 EEE0 2501 18F3  EAB9 2ED9 D774 FE70 2DB5

* remotes/huth-gitlab/tags/pull-request-2019-08-15:
  tests/libqtest: Make qmp_assert_success() independent from global_qtest
  tests/libqtest: Make qtest_qmp_device_add/del independent from global_qtest
  tests/libqtest: Clean up qtest_cb_for_every_machine() wrt global_qtest
  tests/libqtest: Remove unused function hmp()
  tests/libqos: Make virtio-pci code independent from global_qtest
  tests/libqos: Make generic virtio code independent from global_qtest
  tests: Set read-zeroes on for null-co driver
  libqos: Account for the ctrl queue in virtio-net
  qtest: Rename qtest.c:qtest_init()

Signed-off-by: Peter Maydell <address@hidden>


Compare: https://github.com/qemu/qemu/compare/27608c7c66bd...1f3a51f3feab
[Prev in Thread]
Current Thread
[Next in Thread]
[Qemu-commits] [qemu/qemu] 2b8985: qtest: Rename qtest.c:qtest_init(), Peter Maydell <=
Prev by Date: [Qemu-commits] [qemu/qemu] 3d661c: migration: Add error_desc for file channel errors
Next by Date: [Qemu-commits] [qemu/qemu] ec150c: include: Make headers more self-contained
Previous by thread: [Qemu-commits] [qemu/qemu] 3d661c: migration: Add error_desc for file channel errors
Next by thread: [Qemu-commits] [qemu/qemu] ec150c: include: Make headers more self-contained
Index(es):
- Date
- Thread