On Wed, Jun 10, 2009 at 12:39 AM, Ramirez Luna, Omar<omar.ramirez@xxxxxx> wrote:
> Hi,
>
>
>
> Could you please comment on this patch.
>
>
>
> From: Sripal Bagadia <bagadia@xxxxxx>
>
> Date: Tue, 9 Jun 2009 16:05:09 -0500
>
> Subject: [PATCH] DSPBRIDGE: Video Playback Cache Optimization
>
>
>
> Avoid get_user_pages cache flush overheads for uncached & reserved
>
> buffers (e.g. display & camera buffers).
>
>
>
> Signed-off-by: Sripal Bagadia <bagadia@xxxxxx>
>
> ---
>
> drivers/dsp/bridge/wmd/tiomap3430.c | 4 +++-
>
> 1 files changed, 3 insertions(+), 1 deletions(-)
>
>
>
> diff --git a/drivers/dsp/bridge/wmd/tiomap3430.c
> b/drivers/dsp/bridge/wmd/tiomap3430.c
>
> index 7a9603d..a2f32e8 100644
>
> --- a/drivers/dsp/bridge/wmd/tiomap3430.c
>
> +++ b/drivers/dsp/bridge/wmd/tiomap3430.c
>
> @@ -1461,7 +1461,9 @@ static DSP_STATUS WMD_BRD_MemMap(struct
> WMD_DEV_CONTEXT *hDevContext,
>
> goto func_cont;
>
> }
>
>
>
> - if (vma->vm_flags & VM_IO) {
>
> + if ((vma->vm_flags & VM_IO) | ((vma->vm_flags & VM_RESERVED) &&
>
> + (~pgprot_val(vma->vm_page_prot) & L_PTE_CACHEABLE) &&
>
> + (~pgprot_val(vma->vm_page_prot) & L_PTE_BUFFERABLE))) {
L_PTE_BUFFERABLE is obsolete, isn't it? L_PTE_MT_BUFFERABLE should be
used instead.
--
Felipe Contreras
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
