Re: [Qemu-devel] [PULL 00/14] Ide patches

2015-05-12 Thread John Snow


On 05/12/2015 11:25 AM, Peter Maydell wrote:
 On 12 May 2015 at 16:22, John Snow js...@redhat.com wrote:
 On 05/12/2015 06:44 AM, Peter Maydell wrote:
 Doesn't build on 32-bit:

 /root/qemu/qtest.c: In function ‘qtest_process_command’:
 /root/qemu/qtest.c:519:28: error: format ‘%zu’ expects argument of
 type ‘size_t’, but argument 2 has type ‘uint64_t’ [-Werror=format]
 
 More motivation for me to try another stab at fixing up the -m32
 support, I guess.
 
 You could also build for Windows 32 bit, or on OSX (where it's
 only a warning but does detect this even on 64-bit hosts).
 
 -- PMM
 

Limping along using -m32 and some hacks for now. v2 sent, sorry for the
noise.

--js



Re: [Qemu-devel] [PULL 00/14] Ide patches

2015-05-12 Thread Peter Maydell
On 11 May 2015 at 19:12, John Snow js...@redhat.com wrote:
 The following changes since commit 9ad2c8cd41a086020e21aa6d616b73bd5e2a800b:

   Merge remote-tracking branch 
 'remotes/mjt/tags/pull-trivial-patches-2015-05-09' into staging (2015-05-11 
 13:54:00 +0100)

 are available in the git repository at:

   https://github.com/jnsnow/qemu.git tags/ide-pull-request

 for you to fetch changes up to 6e8d74ed17e0526c34386283df6b7935076d983a:

   qtest: pre-buffer hex nibs (2015-05-11 11:00:05 -0400)


Doesn't build on 32-bit:

/root/qemu/qtest.c: In function ‘qtest_process_command’:
/root/qemu/qtest.c:519:28: error: format ‘%zu’ expects argument of
type ‘size_t’, but argument 2 has type ‘uint64_t’ [-Werror=format]

-- PMM



Re: [Qemu-devel] [PULL 00/14] Ide patches

2015-05-12 Thread John Snow


On 05/12/2015 06:44 AM, Peter Maydell wrote:
 On 11 May 2015 at 19:12, John Snow js...@redhat.com wrote:
 The following changes since commit 9ad2c8cd41a086020e21aa6d616b73bd5e2a800b:

   Merge remote-tracking branch 
 'remotes/mjt/tags/pull-trivial-patches-2015-05-09' into staging (2015-05-11 
 13:54:00 +0100)

 are available in the git repository at:

   https://github.com/jnsnow/qemu.git tags/ide-pull-request

 for you to fetch changes up to 6e8d74ed17e0526c34386283df6b7935076d983a:

   qtest: pre-buffer hex nibs (2015-05-11 11:00:05 -0400)

 
 Doesn't build on 32-bit:
 
 /root/qemu/qtest.c: In function ‘qtest_process_command’:
 /root/qemu/qtest.c:519:28: error: format ‘%zu’ expects argument of
 type ‘size_t’, but argument 2 has type ‘uint64_t’ [-Werror=format]
 
 -- PMM
 

More motivation for me to try another stab at fixing up the -m32
support, I guess.

--js



Re: [Qemu-devel] [PULL 00/14] Ide patches

2015-05-12 Thread Peter Maydell
On 12 May 2015 at 16:22, John Snow js...@redhat.com wrote:
 On 05/12/2015 06:44 AM, Peter Maydell wrote:
 Doesn't build on 32-bit:

 /root/qemu/qtest.c: In function ‘qtest_process_command’:
 /root/qemu/qtest.c:519:28: error: format ‘%zu’ expects argument of
 type ‘size_t’, but argument 2 has type ‘uint64_t’ [-Werror=format]

 More motivation for me to try another stab at fixing up the -m32
 support, I guess.

You could also build for Windows 32 bit, or on OSX (where it's
only a warning but does detect this even on 64-bit hosts).

-- PMM



[Qemu-devel] [PULL 00/14] Ide patches

2015-05-11 Thread John Snow
The following changes since commit 9ad2c8cd41a086020e21aa6d616b73bd5e2a800b:

  Merge remote-tracking branch 
'remotes/mjt/tags/pull-trivial-patches-2015-05-09' into staging (2015-05-11 
13:54:00 +0100)

are available in the git repository at:

  https://github.com/jnsnow/qemu.git tags/ide-pull-request

for you to fetch changes up to 6e8d74ed17e0526c34386283df6b7935076d983a:

  qtest: pre-buffer hex nibs (2015-05-11 11:00:05 -0400)





John Snow (14):
  libqos/ahci: Add halted command helpers
  libqos/ahci: Fix sector set method
  libqos: Add migration helpers
  ich9/ahci: Enable Migration
  qtest/ahci: Add migration test
  qtest/ahci: add migrate dma test
  qtest/ahci: add flush migrate test
  qtest/ahci: add halted dma test
  qtest/ahci: add migrate halted dma test
  qtest: allow arbitrarily long sends
  qtest: Add base64 encoded read/write
  qtest: add memset to qtest protocol
  libqos/ahci: Swap memread/write with bufread/write
  qtest: pre-buffer hex nibs

 hw/ide/ahci.c |   1 -
 hw/ide/ich.c  |   1 -
 qtest.c   | 138 ++---
 tests/ahci-test.c | 326 +-
 tests/libqos/ahci.c   |  38 +-
 tests/libqos/ahci.h   |   3 +
 tests/libqos/libqos.c |  85 +
 tests/libqos/libqos.h |   2 +
 tests/libqos/malloc.c |  74 +---
 tests/libqos/malloc.h |   1 +
 tests/libqtest.c  |  47 ++--
 tests/libqtest.h  |  49 
 12 files changed, 709 insertions(+), 56 deletions(-)

-- 
2.1.0