All of lore.kernel.org
 help / color / mirror / Atom feed
From: Chris Simmonds <chris.simmonds@2net.co.uk>
To: Eric Nelson <eric.nelson@boundarydevices.com>
Cc: video4linux-list@redhat.com
Subject: Re: Contiguous memory allocations
Date: Mon, 05 Jul 2010 16:31:37 +0100	[thread overview]
Message-ID: <4C31FAD9.6050808@2net.co.uk> (raw)
In-Reply-To: <4C31EBD3.2010607@boundarydevices.com>

On 05/07/10 15:27, Eric Nelson wrote:
> On 07/05/2010 03:10 AM, Chris Simmonds wrote:
>> On 02/07/10 21:47, Eric Nelson wrote:
>>> Does anyone know if there's a common infrastructure for allocation
>>> of DMA'able memory by drivers and applications above the straight
>>> kernel API (dma_alloc_coherent)?
>>>
>>> I'm working with Freescale i.MX51 drivers to do 720P video
>>> input and output and the embedded calls to dma_alloc_coherent
>>> fail except when used right after boot because of fragmentation.
>>>
>>> I'm fighting the urge to write yet another special-purpose allocator
>>> for video buffers thinking this must be a common problem with a
>>> solution already, but I can't seem to locate one.
>>>
>>> The closest thing I've found is the bigphysarea patch, which doesn't
>>> appear to be supported or headed toward main-line.
>>>
>>> Thanks in advance,
>>
>> dma_alloc_coherent is pretty much just a wrapper round get_free_pages,
>> which is the lowest level allocator in the kernel. So, no there is no
>> other option (but see below). The simplest thing is to make sure your
>> driver is loaded at boot time and to grab all the memory you need then
>> and never let it go. That's what I do.
>>
> Thanks Chris.
>
> The trouble is always "how much"? If we don't know at startup what kind of
> video's needed or what size(s) of camera input may be needed, it's
> impossible
> to tune. In the current Freescale kernels, there are at least 4 separate
> drivers that allocate RAM, sometimes for internal use, but mostly in
> response
> to userspace calls (ioctl).
>
> - frame-buffer driver
> - Video Processing Unit (VPU) - video encode/decode
> - V4L2 output device - allows access to YUV output layer, color blending
> - Image Processing Unit (IPU) - allows userspace bitblts through DMA
>
> With this number of calls, tuning with separate kernel command-line args
> seems
> unworkable.

I think the kernel developers don't like this kind of on-the-side 
allocator because they tend to be dedicated to solving one kind of problem.

Here are a few thoughts about the imx51 specifically, based on my 
experience. First, the size of the memory pool used for 
dma_alloc_coherent is set in plat-mxc/include/mach/memory.h where it is 
hard coded to 64 MiB. You could try bumping that up a bit.

Second, you could re-do the buffer allocation and replace 
dma_alloc_coherent with kmalloc and then use dma_map_single to lock it 
down while dma is taking place. This way you avoid the 64M dma pool 
limit and you speed up buffer access via mmap because the memory is 
cached. In my case I got a two fold speed improvement reading frames 
into application memory. I have to admit that my case was a bit 
specialised though and it may not be worth the effort for you.

Bye for now,
Chris.


-- 
Chris Simmonds                   2net Limited
chris@2net.co.uk                 http://www.2net.co.uk/

--
video4linux-list mailing list
Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list

  reply	other threads:[~2010-07-05 15:31 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-07-02 20:47 Contiguous memory allocations Eric Nelson
2010-07-05 10:10 ` Chris Simmonds
2010-07-05 14:27   ` Eric Nelson
2010-07-05 15:31     ` Chris Simmonds [this message]
2010-07-05 16:14       ` Eric Nelson

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=4C31FAD9.6050808@2net.co.uk \
    --to=chris.simmonds@2net.co.uk \
    --cc=chris@2net.co.uk \
    --cc=eric.nelson@boundarydevices.com \
    --cc=video4linux-list@redhat.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.