mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Mason <slash.tmp@free.fr>
To: Mans Rullgard <mans@mansr.com>
Cc: Geert Uytterhoeven <geert@linux-m68k.org>,
	Vinod Koul <vinod.koul@intel.com>,
	Russell King <linux@arm.linux.org.uk>,
	dmaengine@vger.kernel.org,
	Linus Walleij <linus.walleij@linaro.org>,
	Dan Williams <dan.j.williams@intel.com>,
	LKML <linux-kernel@vger.kernel.org>,
	Linux ARM <linux-arm-kernel@lists.infradead.org>,
	Jon Mason <jdmason@kudzu.us>, Mark Brown <broonie@kernel.org>,
	Lars-Peter Clausen <lars@metafoo.de>,
	Lee Jones <lee.jones@linaro.org>,
	Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
	Arnd Bergmann <arnd@arndb.de>,
	Maxime Ripard <maxime.ripard@free-electrons.com>,
	Dave Jiang <dave.jiang@intel.com>,
	Peter Ujfalusi <peter.ujfalusi@ti.com>,
	Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>,
	Sebastian Frias <sf84@laposte.net>,
	Thibaud Cornic <thibaud_cornic@sigmadesigns.com>,
	Thomas Gambier <thomas_gambier@sigmadesigns.com>
Subject: Re: Tearing down DMA transfer setup after DMA client has finished
Date: Thu, 8 Dec 2016 14:29:17 +0100	[thread overview]
Message-ID: <68bbe77a-c991-4e64-c189-fafbcda8e7ae@free.fr> (raw)
In-Reply-To: <yw1xfulyzj2b.fsf@unicorn.mansr.com>

On 08/12/2016 13:44, Måns Rullgård wrote:

> Mason <slash.tmp@free.fr> writes:
> 
>> On 08/12/2016 13:20, Måns Rullgård wrote:
>>
>>> The only problem we have is that nobody envisioned hardware where the
>>> dma engine indicates completion slightly too soon.  I suspect there's a
>>> fifo or such somewhere, and the interrupt is triggered when the last
>>> byte has been placed in the fifo rather than when it has been removed
>>> which would have been more correct.
>>
>> As I (tried to) explain here:
>> https://marc.info/?l=dmaengine&m=148007808418242&w=2
>>
>> A *read* MBUS agent raises its IRQ when it is safe for the memory
>> to be overwritten (i.e. every byte has been pushed into the pipe).
>>
>> A *write* MBUS agent raises its IRQ when it is safe for another
>> agent to read any one of the transferred bytes.
>>
>> The issue comes from the fact that, for a memory-to-device transfer,
>> the system will receive the read agent's IRQ, but most devices
>> (NFC, SATA) don't have an IRQ line to signal that their part of the
>> operation is complete.
> 
> SATA does, actually.  Nevertheless, it's an unusual design.

Thanks, I was mistaken about the SATA controller.

On tango3 (and also tango4, I assume)

IRQ 41 = Serial ATA #0
IRQ 42 = Serial ATA DMA #0
IRQ 54 = Serial ATA #1
IRQ 55 = Serial ATA DMA #1

But in the end, whether there is a device interrupt (SATA)
or not (NFC), for a memory-to-device transfer, the DMA
driver will get the read agent notification (which should
be ignored) and the client driver should either spin until
idle (NFC) or wait for its completion IRQ (SATA).

Correct?

Regards.

  reply	other threads:[~2016-12-08 13:30 UTC|newest]

Thread overview: 82+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-11-23 10:25 Mason
2016-11-23 12:13 ` Måns Rullgård
2016-11-23 12:41   ` Mason
2016-11-23 17:21     ` Måns Rullgård
2016-11-24 10:53       ` Mason
2016-11-24 14:17         ` Måns Rullgård
2016-11-24 15:20           ` Mason
2016-11-24 16:37             ` Måns Rullgård
2016-11-25  4:55 ` Vinod Koul
2016-11-25 11:57   ` Måns Rullgård
2016-11-25 14:05     ` Mason
2016-11-25 14:12       ` Måns Rullgård
2016-11-25 14:28         ` Mason
2016-11-25 14:42           ` Måns Rullgård
2016-11-25 12:45   ` Russell King - ARM Linux
2016-11-25 13:07     ` Måns Rullgård
2016-11-25 13:34       ` Russell King - ARM Linux
2016-11-25 13:50         ` Måns Rullgård
2016-11-25 13:58           ` Russell King - ARM Linux
2016-11-25 14:03             ` Måns Rullgård
2016-11-25 14:17               ` Russell King - ARM Linux
2016-11-25 14:40                 ` Måns Rullgård
2016-11-25 14:56                   ` Russell King - ARM Linux
2016-11-25 15:08                     ` Måns Rullgård
2016-11-25 15:02                 ` Mason
2016-11-25 15:12                   ` Måns Rullgård
2016-11-25 15:21                     ` Mason
2016-11-25 15:28                       ` Måns Rullgård
2016-11-25 12:46   ` Mason
2016-11-25 13:11     ` Måns Rullgård
2016-11-25 14:21       ` Mason
2016-11-25 14:37         ` Måns Rullgård
2016-11-25 15:35           ` Mason
2016-11-29 18:25     ` Mason
2016-12-06  5:12       ` Vinod Koul
2016-12-06 12:42         ` Mason
2016-12-06 13:14           ` Måns Rullgård
2016-12-06 15:24             ` Mason
2016-12-06 15:34               ` Måns Rullgård
2016-12-06 22:55                 ` Mason
2016-12-07 16:43             ` Vinod Koul
2016-12-07 16:45               ` Måns Rullgård
2016-12-08 10:39                 ` Vinod Koul
2016-12-08 10:54                   ` Mason
2016-12-08 11:18                     ` Geert Uytterhoeven
2016-12-08 11:47                       ` Måns Rullgård
2016-12-08 12:03                         ` Geert Uytterhoeven
2016-12-08 12:17                           ` Mason
2016-12-08 12:21                           ` Måns Rullgård
2016-12-08 16:37                     ` Vinod Koul
2016-12-08 16:48                       ` Måns Rullgård
2016-12-09  6:59                         ` Vinod Koul
2016-12-09 10:25                           ` Sebastian Frias
2016-12-09 11:34                             ` Måns Rullgård
2016-12-09 11:35                             ` 1Måns Rullgård
2016-12-09 17:17                             ` Vinod Koul
2016-12-09 17:28                               ` Måns Rullgård
2016-12-09 17:53                                 ` Vinod Koul
2016-12-09 17:34                               ` Mason
2016-12-09 17:56                                 ` Vinod Koul
2016-12-09 18:17                                   ` Vinod Koul
2016-12-09 18:23                                   ` Mason
2016-12-12  5:01                                     ` Vinod Koul
2016-12-15 11:17                                     ` Mark Brown
2016-12-08 11:44                   ` Måns Rullgård
2016-12-08 11:59                     ` Geert Uytterhoeven
2016-12-08 12:20                       ` Måns Rullgård
2016-12-08 12:31                         ` Geert Uytterhoeven
2016-12-08 12:41                         ` Mason
2016-12-08 12:44                           ` Måns Rullgård
2016-12-08 13:29                             ` Mason [this message]
2016-12-08 13:39                               ` Måns Rullgård
2016-12-08 15:50                         ` Vinod Koul
2016-12-08 16:36                           ` Måns Rullgård
2016-12-08 15:40                     ` Vinod Koul
2016-12-08 15:43                       ` Mason
2016-12-08 16:21                         ` Vinod Koul
2016-12-08 16:46                       ` Måns Rullgård
2016-12-07 16:41           ` Vinod Koul
2016-12-07 16:44             ` Måns Rullgård
2016-12-08 10:37               ` Vinod Koul
2016-12-08 11:44                 ` Måns Rullgård

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=68bbe77a-c991-4e64-c189-fafbcda8e7ae@free.fr \
    --to=slash.tmp@free.fr \
    --cc=arnd@arndb.de \
    --cc=b.zolnierkie@samsung.com \
    --cc=broonie@kernel.org \
    --cc=dan.j.williams@intel.com \
    --cc=dave.jiang@intel.com \
    --cc=dmaengine@vger.kernel.org \
    --cc=geert@linux-m68k.org \
    --cc=jdmason@kudzu.us \
    --cc=lars@metafoo.de \
    --cc=laurent.pinchart@ideasonboard.com \
    --cc=lee.jones@linaro.org \
    --cc=linus.walleij@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux@arm.linux.org.uk \
    --cc=mans@mansr.com \
    --cc=maxime.ripard@free-electrons.com \
    --cc=peter.ujfalusi@ti.com \
    --cc=sf84@laposte.net \
    --cc=thibaud_cornic@sigmadesigns.com \
    --cc=thomas_gambier@sigmadesigns.com \
    --cc=vinod.koul@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®