From: joelf@ti.com (Joel Fernandes)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v4 0/6] dma: edma: Support scatter-lists of any length
Date: Thu, 29 Aug 2013 18:05:39 -0500 [thread overview]
Message-ID: <1377817545-18015-1-git-send-email-joelf@ti.com> (raw)
The following series adds support to EDMA driver to enable DMA of
scatter-gather lists of arbitrary length, but still make use of only
a certain MAX number of slots at a time for a given channel. Thus
free-ing up the rest of the slots to other slaves/channels. With this
there is no need for slave drivers to query the EDMA driver about how
much is the MAX it can send at a time as done in [1]. Drivers can send
SG lists of any number of entries to DMA. Reference discussion at [2].
With this, all the patches for MMC and EDMA related to "sg limits" can be
dropped.
Tested omap-aes and omap_hsmmc drivers with different MAX number of slots,
even just 1. In the case where it is 1, only 1-slot is used to DMA an
entire scatter list of arbitrary length.
Since this series touches EDMA private API code also shared with davinci-pcm,
playback of a 16-bit 44.1KHz audio file with davinci-pcm has been tested.
Sample test run with 1 vs 16 (MAX number of slots/SG) in omap-aes driver:
MAX slots = 1:
(128 bit key, 8192 byte blocks): 1266 operations in 1 seconds (10371072 bytes)
MAX slots = 16:
(128 bit key, 8192 byte blocks): 1601 operations in 1 seconds (13115392 bytes)
Note: For the above test, 8K buffer is mapped into SG list of size 2 so
only 2 slots are required. So beyond size 2, there will not be any noticeable
performance improvement. But above experiment just shows as proof of concept
that even using 1 slot is managed by just DMA'ing 1 SG entry at a time.
Patch series history:
v1: The initial patch series
v2: Several style related cleanups.
v3: Rewrote major portions of the series to avoid usage of error interrupts.
v4: Go back to v1: After performing tests, it is determined that the v1
series is a good enough approach and that v3 triggers problems in certain cases.
Tests have shown that there is not any noticeable performance difference between
using vs not using error interrupts. v4 also squashes some patches in v1.
At this point, we are freezing the overall architecture of the patch series to v4.
[1] https://lkml.org/lkml/2013/7/18/432
[2] http://marc.info/?l=linux-omap&m=137416733628831&w=2
Joel Fernandes (6):
dma: edma: Setup parameters to DMA MAX_NR_SG at a time
dma: edma: Write out and handle MAX_NR_SG at a given time
ARM: edma: Add function to manually trigger an EDMA channel
dma: edma: Find missed events and issue them
dma: edma: Leave linked to Null slot instead of DUMMY slot
dma: edma: Remove limits on number of slots
arch/arm/common/edma.c | 17 ++++
drivers/dma/edma.c | 164 ++++++++++++++++++++++++++++---------
include/linux/platform_data/edma.h | 2 +
3 files changed, 144 insertions(+), 39 deletions(-)
--
1.8.1.2
next reply other threads:[~2013-08-29 23:05 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-08-29 23:05 Joel Fernandes [this message]
2013-08-29 23:05 ` [PATCH v4 1/6] dma: edma: Setup parameters to DMA MAX_NR_SG at a time Joel Fernandes
2013-08-29 23:05 ` [PATCH v4 2/6] dma: edma: Write out and handle MAX_NR_SG at a given time Joel Fernandes
2013-09-03 4:08 ` Vinod Koul
2013-09-03 15:02 ` Joel Fernandes
2013-08-29 23:05 ` [PATCH v4 3/6] ARM: edma: Add function to manually trigger an EDMA channel Joel Fernandes
2013-09-02 14:08 ` Sekhar Nori
2013-08-29 23:05 ` [PATCH v4 4/6] dma: edma: Find missed events and issue them Joel Fernandes
2013-08-29 23:05 ` [PATCH v4 5/6] dma: edma: Leave linked to Null slot instead of DUMMY slot Joel Fernandes
2013-08-29 23:05 ` [PATCH v4 6/6] dma: edma: Remove limits on number of slots Joel Fernandes
2013-09-02 11:59 ` [PATCH v4 0/6] dma: edma: Support scatter-lists of any length Vinod Koul
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=1377817545-18015-1-git-send-email-joelf@ti.com \
--to=joelf@ti.com \
--cc=linux-arm-kernel@lists.infradead.org \
/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;
as well as URLs for NNTP newsgroup(s).