All of lore.kernel.org
 help / color / mirror / Atom feed
From: Thierry Reding <thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
To: Mikko Perttunen <mperttunen-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
Cc: linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org,
	airlied-cv59FeDIM0c@public.gmane.org
Subject: Re: [PATCH 2/8] drm/tegra: Allocate BOs from lower 4G when without IOMMU
Date: Mon, 5 Dec 2016 19:39:55 +0100	[thread overview]
Message-ID: <20161205183955.GB22918@ulmo.ba.sec> (raw)
In-Reply-To: <20161110182345.31777-3-mperttunen-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>

[-- Attachment #1: Type: text/plain, Size: 567 bytes --]

On Thu, Nov 10, 2016 at 08:23:39PM +0200, Mikko Perttunen wrote:
> On 64-bit Tegras, buffer object memory allocation may
> return memory above 4G that units behind Host1x cannot
> access. Add the GFP_DMA flag to these allocation when
> IOMMU is not enabled to ensure units can always access
> BO memory.

I don't think that's entirely true. The display controller can address
more than 32 bits. It's perhaps slightly different from other units that
are behind host1x, but maybe we need a special case here to allow frame-
buffers to reside in higher memory?

Thierry

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 801 bytes --]

  parent reply	other threads:[~2016-12-05 18:39 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-11-10 18:23 [PATCH 0/8] Host1x IOMMU support + VIC support Mikko Perttunen
2016-11-10 18:23 ` [PATCH 1/8] drm/tegra: Add Tegra DRM allocation API Mikko Perttunen
     [not found]   ` <20161110182345.31777-2-mperttunen-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2016-12-05 18:37     ` Thierry Reding
     [not found]       ` <20161205183726.GA22918-EkSeR96xj6Pcmrwk2tT4+A@public.gmane.org>
2016-12-07  9:23         ` Mikko Perttunen
2016-11-10 18:23 ` [PATCH 2/8] drm/tegra: Allocate BOs from lower 4G when without IOMMU Mikko Perttunen
     [not found]   ` <20161110182345.31777-3-mperttunen-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2016-12-05 18:39     ` Thierry Reding [this message]
     [not found]       ` <20161205183955.GB22918-EkSeR96xj6Pcmrwk2tT4+A@public.gmane.org>
2016-12-07  9:32         ` Mikko Perttunen
     [not found] ` <20161110182345.31777-1-mperttunen-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2016-11-10 18:23   ` [PATCH 3/8] drm/tegra: Add falcon helper library Mikko Perttunen
2016-12-05 19:13     ` Thierry Reding
     [not found]       ` <20161205191346.GC22918-EkSeR96xj6Pcmrwk2tT4+A@public.gmane.org>
2016-12-07 11:54         ` Mikko Perttunen
2016-11-10 18:23   ` [PATCH 4/8] drm/tegra: Add VIC support Mikko Perttunen
2016-12-05 19:35     ` Thierry Reding
     [not found]       ` <20161205193559.GD22918-EkSeR96xj6Pcmrwk2tT4+A@public.gmane.org>
2016-12-14  8:28         ` Mikko Perttunen
2016-11-10 18:23   ` [PATCH 7/8] arm64: tegra: Enable VIC on T210 Mikko Perttunen
2016-11-10 18:23   ` [PATCH 8/8] arm64: tegra: Enable IOMMU for Host1x on Tegra210 Mikko Perttunen
2016-12-05 13:25   ` [PATCH 0/8] Host1x IOMMU support + VIC support Mikko Perttunen
2016-11-10 18:23 ` [PATCH 5/8] gpu: host1x: Add IOMMU support Mikko Perttunen
2016-12-05 19:49   ` Thierry Reding
     [not found]     ` <20161205194924.GE22918-EkSeR96xj6Pcmrwk2tT4+A@public.gmane.org>
2016-12-14  8:50       ` Mikko Perttunen
2016-11-10 18:23 ` [PATCH 6/8] dt-bindings: Add bindings for the Tegra VIC Mikko Perttunen
2016-12-05 19:51 ` [PATCH 0/8] Host1x IOMMU support + VIC support Thierry Reding
     [not found]   ` <20161205195131.GF22918-EkSeR96xj6Pcmrwk2tT4+A@public.gmane.org>
2016-12-05 19:58     ` Mikko Perttunen
2016-12-06  7:14     ` Daniel Vetter
     [not found]       ` <20161206071450.uedxvvtcxbkwsoza-dv86pmgwkMBes7Z6vYuT8azUEOm+Xw19@public.gmane.org>
2016-12-06  9:33         ` Mikko Perttunen
2016-12-07 10:36           ` Daniel Vetter

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=20161205183955.GB22918@ulmo.ba.sec \
    --to=thierry.reding-re5jqeeqqe8avxtiumwx3w@public.gmane.org \
    --cc=airlied-cv59FeDIM0c@public.gmane.org \
    --cc=dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org \
    --cc=linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=mperttunen-DDmLM1+adcrQT0dZR+AlfA@public.gmane.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 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.