qemuhotplugtest: Fix timeouts

[PATCH 0/4] qemuhotplugtest: Fix timeouts

Peter Krempa posted 4 patches 4 years ago

Download series mbox

Test syntax-check failed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/cover.1587634711.git.pkrempa@redhat.com

src/qemu/qemu_hotplug.c | 2 +-
tests/qemuhotplugmock.c | 6 +++---
tests/qemuhotplugtest.c | 9 +++------
3 files changed, 7 insertions(+), 10 deletions(-)

Expand all Fold all

[PATCH 0/4] qemuhotplugtest: Fix timeouts

Posted by Peter Krempa 4 years ago

The test takes too long for no good reason. One was caused by the
compiler inlining too much and second one was a deliberate decision.

Peter Krempa (4):
  qemuDomainGetUnplugTimeout: Add G_GNUC_NO_INLINE
  mock:qemuDomainGetUnplugTimeout: Remove unused attribute for '@vm'
  mock: qemuDomainGetUnplugTimeout: Decrease timeout
  qemuhotplugtest: detach: Remove commands which are not issued

 src/qemu/qemu_hotplug.c | 2 +-
 tests/qemuhotplugmock.c | 6 +++---
 tests/qemuhotplugtest.c | 9 +++------
 3 files changed, 7 insertions(+), 10 deletions(-)

-- 
2.26.0

Re: [PATCH 0/4] qemuhotplugtest: Fix timeouts

Posted by Michal Privoznik 4 years ago

On 4/23/20 11:40 AM, Peter Krempa wrote:
> The test takes too long for no good reason. One was caused by the
> compiler inlining too much and second one was a deliberate decision.
> 
> Peter Krempa (4):
>    qemuDomainGetUnplugTimeout: Add G_GNUC_NO_INLINE
>    mock:qemuDomainGetUnplugTimeout: Remove unused attribute for '@vm'
>    mock: qemuDomainGetUnplugTimeout: Decrease timeout
>    qemuhotplugtest: detach: Remove commands which are not issued
> 
>   src/qemu/qemu_hotplug.c | 2 +-
>   tests/qemuhotplugmock.c | 6 +++---
>   tests/qemuhotplugtest.c | 9 +++------
>   3 files changed, 7 insertions(+), 10 deletions(-)
> 

Reviewed-by: Michal Privoznik <mprivozn@redhat.com>

Michal