[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 8/8] dma: Let dma_buf_read() / dma_buf_write() propagate MemT
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [PATCH 8/8] dma: Let dma_buf_read() / dma_buf_write() propagate MemTxResult |
Date: |
Thu, 16 Dec 2021 13:39:07 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.3.0 |
On 12/16/21 13:35, Philippe Mathieu-Daudé wrote:
> Since the previous commit, dma_buf_rw() returns a MemTxResult
> type. Do not discard it, return it to the caller.
>
> Since both dma_buf_read/dma_buf_write functions were previously
> returning the QEMUSGList size not consumed, add an extra argument
> where the unconsummed size can be stored.
>
> Update the few callers.
>
> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
> ---
> include/sysemu/dma.h | 6 ++++--
> hw/ide/ahci.c | 8 ++++----
> hw/nvme/ctrl.c | 4 ++--
> hw/scsi/megasas.c | 48 ++++++++++++++++++++++++++++++-------------
> hw/scsi/scsi-bus.c | 4 ++--
> softmmu/dma-helpers.c | 18 ++++++----------
> 6 files changed, 52 insertions(+), 36 deletions(-)
>
> diff --git a/include/sysemu/dma.h b/include/sysemu/dma.h
> index cdf379fecad..9f998edbea4 100644
> --- a/include/sysemu/dma.h
> +++ b/include/sysemu/dma.h
> @@ -303,8 +303,10 @@ BlockAIOCB *dma_blk_read(BlockBackend *blk,
> BlockAIOCB *dma_blk_write(BlockBackend *blk,
> QEMUSGList *sg, uint64_t offset, uint32_t align,
> BlockCompletionFunc *cb, void *opaque);
> -uint64_t dma_buf_read(void *ptr, int32_t len, QEMUSGList *sg, MemTxAttrs
> attrs);
> -uint64_t dma_buf_write(void *ptr, int32_t len, QEMUSGList *sg, MemTxAttrs
> attrs);
> +MemTxResult dma_buf_read(void *ptr, int32_t len, uint64_t *residp,
> + QEMUSGList *sg, MemTxAttrs attrs);
> +MemTxResult dma_buf_write(void *ptr, int32_t len, uint64_t *residp,
> + QEMUSGList *sg, MemTxAttrs attrs);
Side note, ideally all functions returning the MemTxResult
type should use the warn_unused_result attribute.
- [PATCH 0/8] hw: Have DMA API take MemTxAttrs arg & propagate MemTxResult (part 2), Philippe Mathieu-Daudé, 2021/12/16
- [PATCH 1/8] dma: Have dma_buf_rw() take a void pointer, Philippe Mathieu-Daudé, 2021/12/16
- [PATCH 2/8] dma: Have dma_buf_read() / dma_buf_write() take a void pointer, Philippe Mathieu-Daudé, 2021/12/16
- [PATCH 3/8] dma: Let pci_dma_rw() take MemTxAttrs argument, Philippe Mathieu-Daudé, 2021/12/16
- [PATCH 4/8] dma: Let dma_buf_rw() take MemTxAttrs argument, Philippe Mathieu-Daudé, 2021/12/16
- [PATCH 5/8] dma: Let dma_buf_write() take MemTxAttrs argument, Philippe Mathieu-Daudé, 2021/12/16
- [PATCH 6/8] dma: Let dma_buf_read() take MemTxAttrs argument, Philippe Mathieu-Daudé, 2021/12/16
- [PATCH 7/8] dma: Let dma_buf_rw() propagate MemTxResult, Philippe Mathieu-Daudé, 2021/12/16
- [PATCH 8/8] dma: Let dma_buf_read() / dma_buf_write() propagate MemTxResult, Philippe Mathieu-Daudé, 2021/12/16
- Re: [PATCH 8/8] dma: Let dma_buf_read() / dma_buf_write() propagate MemTxResult,
Philippe Mathieu-Daudé <=
- Re: [PATCH 0/8] hw: Have DMA API take MemTxAttrs arg & propagate MemTxResult (part 2), Klaus Jensen, 2021/12/16
- Re: [PATCH 0/8] hw: Have DMA API take MemTxAttrs arg & propagate MemTxResult (part 2), John Snow, 2021/12/16