[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v8 13/14] sdhci: fix the PCI device, using the P
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [Qemu-devel] [PATCH v8 13/14] sdhci: fix the PCI device, using the PCI address space for DMA |
Date: |
Mon, 15 Jan 2018 15:39:39 -0300 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.5.2 |
On 01/15/2018 03:24 PM, Philippe Mathieu-Daudé wrote:
> While SysBus devices can use the get_system_memory() address space,
> PCI devices should use the bus master address space for DMA.
>
> Suggested-by: Peter Maydell <address@hidden>
> Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
I forgot to add Peter's R-b:
Reviewed-by: Peter Maydell <address@hidden>
> ---
> include/hw/sd/sdhci.h | 1 +
> hw/sd/sdhci.c | 29 +++++++++++++++--------------
> 2 files changed, 16 insertions(+), 14 deletions(-)
>
> diff --git a/include/hw/sd/sdhci.h b/include/hw/sd/sdhci.h
> index 442e30aff2..4a102b86ce 100644
> --- a/include/hw/sd/sdhci.h
> +++ b/include/hw/sd/sdhci.h
> @@ -41,6 +41,7 @@ typedef struct SDHCIState {
> /*< public >*/
> SDBus sdbus;
> MemoryRegion iomem;
> + AddressSpace *dma_as;
>
> QEMUTimer *insert_timer; /* timer for 'changing' sd card. */
> QEMUTimer *transfer_timer;
> diff --git a/hw/sd/sdhci.c b/hw/sd/sdhci.c
> index 9bdbcd0a04..dd400695e4 100644
> --- a/hw/sd/sdhci.c
> +++ b/hw/sd/sdhci.c
> @@ -496,7 +496,7 @@ static void sdhci_sdma_transfer_multi_blocks(SDHCIState
> *s)
> s->blkcnt--;
> }
> }
> - dma_memory_write(&address_space_memory, s->sdmasysad,
> + dma_memory_write(s->dma_as, s->sdmasysad,
> &s->fifo_buffer[begin], s->data_count - begin);
> s->sdmasysad += s->data_count - begin;
> if (s->data_count == block_size) {
> @@ -518,7 +518,7 @@ static void sdhci_sdma_transfer_multi_blocks(SDHCIState
> *s)
> s->data_count = block_size;
> boundary_count -= block_size - begin;
> }
> - dma_memory_read(&address_space_memory, s->sdmasysad,
> + dma_memory_read(s->dma_as, s->sdmasysad,
> &s->fifo_buffer[begin], s->data_count - begin);
> s->sdmasysad += s->data_count - begin;
> if (s->data_count == block_size) {
> @@ -556,11 +556,9 @@ static void sdhci_sdma_transfer_single_block(SDHCIState
> *s)
> for (n = 0; n < datacnt; n++) {
> s->fifo_buffer[n] = sdbus_read_data(&s->sdbus);
> }
> - dma_memory_write(&address_space_memory, s->sdmasysad, s->fifo_buffer,
> - datacnt);
> + dma_memory_write(s->dma_as, s->sdmasysad, s->fifo_buffer, datacnt);
> } else {
> - dma_memory_read(&address_space_memory, s->sdmasysad, s->fifo_buffer,
> - datacnt);
> + dma_memory_read(s->dma_as, s->sdmasysad, s->fifo_buffer, datacnt);
> for (n = 0; n < datacnt; n++) {
> sdbus_write_data(&s->sdbus, s->fifo_buffer[n]);
> }
> @@ -584,7 +582,7 @@ static void get_adma_description(SDHCIState *s, ADMADescr
> *dscr)
> hwaddr entry_addr = (hwaddr)s->admasysaddr;
> switch (SDHC_DMA_TYPE(s->hostctl)) {
> case SDHC_CTRL_ADMA2_32:
> - dma_memory_read(&address_space_memory, entry_addr, (uint8_t *)&adma2,
> + dma_memory_read(s->dma_as, entry_addr, (uint8_t *)&adma2,
> sizeof(adma2));
> adma2 = le64_to_cpu(adma2);
> /* The spec does not specify endianness of descriptor table.
> @@ -596,7 +594,7 @@ static void get_adma_description(SDHCIState *s, ADMADescr
> *dscr)
> dscr->incr = 8;
> break;
> case SDHC_CTRL_ADMA1_32:
> - dma_memory_read(&address_space_memory, entry_addr, (uint8_t *)&adma1,
> + dma_memory_read(s->dma_as, entry_addr, (uint8_t *)&adma1,
> sizeof(adma1));
> adma1 = le32_to_cpu(adma1);
> dscr->addr = (hwaddr)(adma1 & 0xFFFFF000);
> @@ -609,12 +607,12 @@ static void get_adma_description(SDHCIState *s,
> ADMADescr *dscr)
> }
> break;
> case SDHC_CTRL_ADMA2_64:
> - dma_memory_read(&address_space_memory, entry_addr,
> + dma_memory_read(s->dma_as, entry_addr,
> (uint8_t *)(&dscr->attr), 1);
> - dma_memory_read(&address_space_memory, entry_addr + 2,
> + dma_memory_read(s->dma_as, entry_addr + 2,
> (uint8_t *)(&dscr->length), 2);
> dscr->length = le16_to_cpu(dscr->length);
> - dma_memory_read(&address_space_memory, entry_addr + 4,
> + dma_memory_read(s->dma_as, entry_addr + 4,
> (uint8_t *)(&dscr->addr), 8);
> dscr->attr = le64_to_cpu(dscr->attr);
> dscr->attr &= 0xfffffff8;
> @@ -673,7 +671,7 @@ static void sdhci_do_adma(SDHCIState *s)
> s->data_count = block_size;
> length -= block_size - begin;
> }
> - dma_memory_write(&address_space_memory, dscr.addr,
> + dma_memory_write(s->dma_as, dscr.addr,
> &s->fifo_buffer[begin],
> s->data_count - begin);
> dscr.addr += s->data_count - begin;
> @@ -697,7 +695,7 @@ static void sdhci_do_adma(SDHCIState *s)
> s->data_count = block_size;
> length -= block_size - begin;
> }
> - dma_memory_read(&address_space_memory, dscr.addr,
> + dma_memory_read(s->dma_as, dscr.addr,
> &s->fifo_buffer[begin],
> s->data_count - begin);
> dscr.addr += s->data_count - begin;
> @@ -1312,7 +1310,8 @@ static void sdhci_pci_realize(PCIDevice *dev, Error
> **errp)
> dev->config[PCI_CLASS_PROG] = 0x01; /* Standard Host supported DMA */
> dev->config[PCI_INTERRUPT_PIN] = 0x01; /* interrupt pin A */
> s->irq = pci_allocate_irq(dev);
> - pci_register_bar(dev, 0, 0, &s->iomem);
> + s->dma_as = pci_get_address_space(dev);
> + pci_register_bar(dev, 0, PCI_BASE_ADDRESS_SPACE_MEMORY, &s->iomem);
> }
>
> static void sdhci_pci_exit(PCIDevice *dev)
> @@ -1381,6 +1380,8 @@ static void sdhci_sysbus_realize(DeviceState *dev,
> Error ** errp)
> return;
> }
>
> + s->dma_as = &address_space_memory;
> +
> sysbus_init_irq(sbd, &s->irq);
> sysbus_init_mmio(sbd, &s->iomem);
> }
>
- [Qemu-devel] [PATCH v8 04/14] sdhci: refactor common sysbus/pci class_init() into sdhci_common_class_init(), (continued)
- [Qemu-devel] [PATCH v8 04/14] sdhci: refactor common sysbus/pci class_init() into sdhci_common_class_init(), Philippe Mathieu-Daudé, 2018/01/15
- [Qemu-devel] [PATCH v8 05/14] sdhci: refactor common sysbus/pci realize() into sdhci_common_realize(), Philippe Mathieu-Daudé, 2018/01/15
- [Qemu-devel] [PATCH v8 06/14] sdhci: refactor common sysbus/pci unrealize() into sdhci_common_unrealize(), Philippe Mathieu-Daudé, 2018/01/15
- [Qemu-devel] [PATCH v8 07/14] sdhci: use qemu_log_mask(UNIMP) instead of fprintf(), Philippe Mathieu-Daudé, 2018/01/15
- [Qemu-devel] [PATCH v8 08/14] sdhci: convert the DPRINT() calls into trace events, Philippe Mathieu-Daudé, 2018/01/15
- [Qemu-devel] [PATCH v8 09/14] sdhci: move MASK_TRNMOD with other SDHC_TRN* defines in "sd-internal.h", Philippe Mathieu-Daudé, 2018/01/15
- [Qemu-devel] [PATCH v8 10/14] sdhci: rename the SDHC_CAPAB register, Philippe Mathieu-Daudé, 2018/01/15
- [Qemu-devel] [PATCH v8 11/14] sdhci: fix CAPAB/MAXCURR registers, both are 64bit and read-only, Philippe Mathieu-Daudé, 2018/01/15
- [Qemu-devel] [PATCH v8 12/14] sdhci: Implement write method of ACMD12ERRSTS register, Philippe Mathieu-Daudé, 2018/01/15
- [Qemu-devel] [PATCH v8 13/14] sdhci: fix the PCI device, using the PCI address space for DMA, Philippe Mathieu-Daudé, 2018/01/15
- Re: [Qemu-devel] [PATCH v8 13/14] sdhci: fix the PCI device, using the PCI address space for DMA,
Philippe Mathieu-Daudé <=
- [Qemu-devel] [PATCH v8 14/14] sdhci: add a 'dma' property to the sysbus devices, Philippe Mathieu-Daudé, 2018/01/15
- Re: [Qemu-devel] [PATCH v8 00/14] SDHCI: housekeeping (part 1), Peter Maydell, 2018/01/16