Re: [Qemu-devel] [PATCHv2 0/2] dma-helpers: explicitly pass alignment into DMA helpers

2016-10-14 Thread John Snow



On 10/14/2016 07:41 AM, Mark Cave-Ayland wrote:

This is a follow-up to the thread at
https://lists.gnu.org/archive/html/qemu-devel/2016-09/msg01396.html which
introduces an explicit alignment to the DMA helpers to facilitate conversion
of the macio controller over to use the now byte-aligned DMA helpers.

Patch 1 introduces an alignment parameter as suggested by Paolo above, whilst
patch 2 performs the conversion for the macio controller.

Signed-off-by: Mark Cave-Ayland 

v2:
- Use QEMU_IS_ALIGNED and QEMU_ALIGN_DOWN macros suggested by Eric
- Add Reviewed-by/Acked-by tags from Eric and John
- Rebase onto master

Mark Cave-Ayland (2):
  dma-helpers: explicitly pass alignment into DMA helpers
  macio: switch over to new byte-aligned DMA helpers

 dma-helpers.c|   21 ++---
 hw/block/nvme.c  |6 +-
 hw/ide/ahci.c|2 +
 hw/ide/core.c|6 +-
 hw/ide/macio.c   |  213 +++---
 hw/scsi/scsi-disk.c  |2 +
 include/sysemu/dma.h |6 +-
 7 files changed, 54 insertions(+), 202 deletions(-)



Did you know: "PATCHv2" apparently confuses our patches scraper?

Anyway:

Thanks, applied to my IDE tree:

https://github.com/jnsnow/qemu/commits/ide
https://github.com/jnsnow/qemu.git

--js



[Qemu-devel] [PATCHv2 0/2] dma-helpers: explicitly pass alignment into DMA helpers

2016-10-14 Thread Mark Cave-Ayland
This is a follow-up to the thread at
https://lists.gnu.org/archive/html/qemu-devel/2016-09/msg01396.html which
introduces an explicit alignment to the DMA helpers to facilitate conversion
of the macio controller over to use the now byte-aligned DMA helpers.

Patch 1 introduces an alignment parameter as suggested by Paolo above, whilst
patch 2 performs the conversion for the macio controller.

Signed-off-by: Mark Cave-Ayland 

v2:
- Use QEMU_IS_ALIGNED and QEMU_ALIGN_DOWN macros suggested by Eric
- Add Reviewed-by/Acked-by tags from Eric and John
- Rebase onto master

Mark Cave-Ayland (2):
  dma-helpers: explicitly pass alignment into DMA helpers
  macio: switch over to new byte-aligned DMA helpers

 dma-helpers.c|   21 ++---
 hw/block/nvme.c  |6 +-
 hw/ide/ahci.c|2 +
 hw/ide/core.c|6 +-
 hw/ide/macio.c   |  213 +++---
 hw/scsi/scsi-disk.c  |2 +
 include/sysemu/dma.h |6 +-
 7 files changed, 54 insertions(+), 202 deletions(-)

-- 
1.7.10.4