From: Omar Ramirez Luna <omar.ramirez@ti.com>
To: Felipe Contreras <felipe.contreras@gmail.com>
Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
"linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>,
Tony Lindgren <tony@atomide.com>,
Ohad Ben-Cohen <ohad@wizery.com>,
Greg Kroah-Hartman <gregkh@suse.de>,
Russell King <linux@arm.linux.org.uk>,
"Gomez Castellanos, Ivan" <ivan.gomez@ti.com>,
"Ramos Falcon, Ernesto" <ernesto@ti.com>,
"linux-omap@vger.kernel.org" <linux-omap@vger.kernel.org>,
Ameya Palande <ameya.palande@nokia.com>
Subject: Re: [RFC] tidspbridge: use a parameter to allocate shared memory
Date: Thu, 7 Oct 2010 14:16:49 -0500 [thread overview]
Message-ID: <4CAE1CA1.4050006@ti.com> (raw)
In-Reply-To: <AANLkTim8H0ibSucP3Nq=q19t0QY0pvijim1kOZSbygY1@mail.gmail.com>
On 10/7/2010 1:22 PM, Felipe Contreras wrote:
...
>
> Note that the "shared memory" described in the document you share has
> nothing to do with the SHM pool. AFAIK that memory is used for other
> things, like MMU PTEs, and storing the base image and socket-nodes,
> thus it needs to be contiguous.
hmmm, no. it is the same memory. i.e.: we propagate the current opp
through the shared memory so the dsp can read it if it went to sleep,
with the proper offset you can read that variable starting from the
mempool base address.
"storing the base image and socket-nodes" symbols is also donde in the
shared memory but this is done in different sections of the SHM.
>
> Right now allocating contiguous memory can only be done with memblock
> (bootmem), but in the future it could be done with CMA/VCMM. But the
> cacheability is a separate issue.
yes
>
> I don't see any problem flushing the SHM area when needed, which
> probably has performance implications when mmaping/unmapping buffers,
> at which point you need to flush bigger memory areas anyway, so that's
> not an issue.
well, you would have to flush when loading the base image, or allocating
a socket node, but also minor flushes for opp propagation, SHM messages
to DSP, chnl params, those are the ones I can quickly think of.
>
> Anyway, we will not know for sure until we try... Right?
yes we can try, at least we now for sure arm side can be done.
Regards,
omar
next prev parent reply other threads:[~2010-10-07 19:17 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-10-07 5:45 [RFC] tidspbridge: use a parameter to allocate shared memory Omar Ramirez Luna
2010-10-07 7:40 ` Laurent Pinchart
2010-10-07 8:32 ` Russell King - ARM Linux
2010-10-07 14:01 ` Laurent Pinchart
2010-10-07 17:13 ` Omar Ramirez Luna
2010-10-07 19:07 ` Russell King - ARM Linux
2010-10-07 17:01 ` Omar Ramirez Luna
2010-10-07 18:22 ` Felipe Contreras
2010-10-07 19:16 ` Omar Ramirez Luna [this message]
2010-10-08 8:18 ` Felipe Contreras
2010-10-08 17:20 ` Omar Ramirez Luna
2010-10-08 8:20 ` Felipe Contreras
2010-10-08 17:31 ` Omar Ramirez Luna
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=4CAE1CA1.4050006@ti.com \
--to=omar.ramirez@ti.com \
--cc=ameya.palande@nokia.com \
--cc=ernesto@ti.com \
--cc=felipe.contreras@gmail.com \
--cc=gregkh@suse.de \
--cc=ivan.gomez@ti.com \
--cc=laurent.pinchart@ideasonboard.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-omap@vger.kernel.org \
--cc=linux@arm.linux.org.uk \
--cc=ohad@wizery.com \
--cc=tony@atomide.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