[Qemu-devel] [PATCH v2 00/13] Use unsigned types for the 'len' argument of all memory read/write functions

[Martin Galvan]

Changes in v2:
* Split the patch in several parts so the various submaintainers can look at it.

When looking at address_space_read_continue I noticed the 'len' variable was
a signed int, while it clearly should be a hwaddr or some other unsigned type.
I started looking around and saw that this error had spread all over the code.

I replaced the signed int by their unsigned counterparts as much as possible.
At the suggestion of Paolo I'm splitting the patch in several parts according
to the MAINTAINERS file.

 cpus.c                     |  2 +-
 disas/cris.c               |  2 +-
 disas/s390.c               |  3 ++-
 disas/sh4.c                |  2 +-
 dma-helpers.c              |  2 +-
 exec.c                     | 29 +++++++++++++++--------------
 gdbstub.c                  |  3 ++-
 hw/arm/boot.c              |  2 +-
 hw/arm/musicpal.c          |  2 +-
 hw/audio/marvell_88w8618.c |  2 +-
 hw/audio/milkymist-ac97.c  |  4 ++--
 hw/dma/sun4m_iommu.c       |  4 ++--
 hw/net/dp8393x.c           |  8 ++++----
 hw/net/mcf_fec.c           |  4 ++--
 hw/net/vmware_utils.h      | 17 +++++++++--------
 hw/net/xgmac.c             |  4 ++--
 hw/s390x/virtio-ccw.c      |  2 +-
 include/exec/cpu-all.h     |  2 +-
 include/exec/cpu-common.h  | 10 +++++-----
 include/exec/memory.h      | 10 +++++-----
 include/hw/sparc/sun4m.h   |  6 +++---
 include/qom/cpu.h          |  2 +-
 kvm-all.c                  |  2 +-
 monitor.c                  |  3 ++-
 scripts/coverity-model.c   |  2 +-
 target-s390x/mmu_helper.c  |  4 ++--
 target-sparc/cpu.h         |  2 +-
 target-sparc/mmu_helper.c  |  6 +++---
 28 files changed, 73 insertions(+), 68 deletions(-)

--
2.7.1