mbox

[PULL,00/17] Block and testing patches

Message ID 20180815031248.14908-1-famz@redhat.com
State New
Headers show

Pull-request

git://github.com/famz/qemu.git tags/block-and-testing-pull-request

Message

Fam Zheng Aug. 15, 2018, 3:12 a.m. UTC 
The following changes since commit 38441756b70eec5807b5f60dad11a93a91199866:

  Update version for v3.0.0 release (2018-08-14 16:38:43 +0100)

are available in the Git repository at:

  git://github.com/famz/qemu.git tags/block-and-testing-pull-request

for you to fetch changes up to 37a81812f7b8367422a039eb09d915df543983ee:

  aio-posix: Improve comment around marking node deleted (2018-08-15 10:12:35 +0800)

----------------------------------------------------------------
Block and testing patches for 3.1

- aio fixes by me
- nvme fixes by Paolo and me
- test improvements by Peter, Phil and me

----------------------------------------------------------------

Fam Zheng (10):
  nvme: Fix nvme_init error handling
  aio-posix: Don't count ctx->notifier as progress when polling
  aio: Do aio_notify_accept only during blocking aio_poll
  docker: Install more packages in centos7
  tests: Add an option for snapshot (default: off)
  tests: Allow overriding archive path with SRC_ARCHIVE
  tests: Add centos VM testing
  tests: vm: Add vm-clean-all
  tests/vm: Add vm-build-all/vm-clean-all in help text
  aio-posix: Improve comment around marking node deleted

Paolo Bonzini (1):
  nvme: simplify plug/unplug

Peter Maydell (4):
  tests/vm: Pass the jobs parallelism setting to 'make check'
  tests/vm: Propagate V=1 down into the make inside the VM
  tests/vm: Bump guest RAM up from 2G to 4G
  tests/vm: Use make's --output-sync option

Philippe Mathieu-Daudé (2):
  tests/vm: Only use -cpu 'host' if KVM is available
  tests/vm: Add flex and bison to the vm image

 block/nvme.c                            | 57 +++++++----------
 docs/devel/testing.rst                  |  1 +
 tests/docker/Makefile.include           |  7 ++-
 tests/docker/dockerfiles/centos7.docker |  3 +
 tests/vm/Makefile.include               | 12 +++-
 tests/vm/basevm.py                      | 17 +++--
 tests/vm/centos                         | 84 +++++++++++++++++++++++++
 tests/vm/freebsd                        |  4 +-
 tests/vm/netbsd                         |  4 +-
 tests/vm/openbsd                        |  4 +-
 tests/vm/ubuntu.i386                    |  6 +-
 util/aio-posix.c                        |  9 +--
 util/aio-win32.c                        |  3 +-
 13 files changed, 154 insertions(+), 57 deletions(-)
 create mode 100755 tests/vm/centos

Comments

Peter Maydell Aug. 15, 2018, 3:31 p.m. UTC | #1 
On 15 August 2018 at 04:12, Fam Zheng <famz@redhat.com> wrote:
> The following changes since commit 38441756b70eec5807b5f60dad11a93a91199866:
>
>   Update version for v3.0.0 release (2018-08-14 16:38:43 +0100)
>
> are available in the Git repository at:
>
>   git://github.com/famz/qemu.git tags/block-and-testing-pull-request
>
> for you to fetch changes up to 37a81812f7b8367422a039eb09d915df543983ee:
>
>   aio-posix: Improve comment around marking node deleted (2018-08-15 10:12:35 +0800)
>
> ----------------------------------------------------------------
> Block and testing patches for 3.1
>
> - aio fixes by me
> - nvme fixes by Paolo and me
> - test improvements by Peter, Phil and me
>
> ----------------------------------------------------------------

Applied, thanks.

-- PMM
Peter Maydell Aug. 15, 2018, 3:35 p.m. UTC | #2 
On 15 August 2018 at 04:12, Fam Zheng <famz@redhat.com> wrote:
> The following changes since commit 38441756b70eec5807b5f60dad11a93a91199866:
>
>   Update version for v3.0.0 release (2018-08-14 16:38:43 +0100)


>   tests/vm: Only use -cpu 'host' if KVM is available

Incidentally, why did you go with this one rather than just
using -cpu max all the time? The point of -cpu max is to work
everywhere, so you don't need to have special casing to decide
whether to use it or something else...

thanks
-- PMM
Fam Zheng Aug. 16, 2018, 4:34 a.m. UTC | #3 
On Wed, 08/15 16:35, Peter Maydell wrote:
> On 15 August 2018 at 04:12, Fam Zheng <famz@redhat.com> wrote:
> > The following changes since commit 38441756b70eec5807b5f60dad11a93a91199866:
> >
> >   Update version for v3.0.0 release (2018-08-14 16:38:43 +0100)
> 
> 
> >   tests/vm: Only use -cpu 'host' if KVM is available
> 
> Incidentally, why did you go with this one rather than just
> using -cpu max all the time? The point of -cpu max is to work
> everywhere, so you don't need to have special casing to decide
> whether to use it or something else...

My apology for dropping the patch but forgot to send a note.

The only reason is this was queued first, and yours couldn't be applied without
rebasing (which effectively drops the former). But I agree with your point, and
am open for a further clean up.

Fam