From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alex Deucher Subject: Re: [PATCH v2 0/6] DRM: VMA Access Management and Render Nodes Date: Fri, 23 Aug 2013 09:34:17 -0400 Message-ID: References: <1377256408-746-1-git-send-email-dh.herrmann@gmail.com> <52174745.306@amd.com> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Return-path: Received: from mail-vc0-f181.google.com (mail-vc0-f181.google.com [209.85.220.181]) by gabe.freedesktop.org (Postfix) with ESMTP id 34575E5DFC for ; Fri, 23 Aug 2013 06:34:18 -0700 (PDT) Received: by mail-vc0-f181.google.com with SMTP id hz10so416716vcb.26 for ; Fri, 23 Aug 2013 06:34:17 -0700 (PDT) In-Reply-To: <52174745.306@amd.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org Errors-To: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org To: =?ISO-8859-1?Q?Christian_K=F6nig?= Cc: Alex Deucher , Maling list - DRI developers List-Id: dri-devel@lists.freedesktop.org On Fri, Aug 23, 2013 at 7:28 AM, Christian K=F6nig wrote: > Hi David, > > Am 23.08.2013 13:13, schrieb David Herrmann: > >> Hi >> >> I reduced the vma access-management patches to a minimum. I now do filp* >> tracking in gem unconditionally and force drm_gem_mmap() to check this. >> Hence, >> all gem drivers are safe now. For TTM drivers, I now use the already >> available >> verify_access() callback to get access to the underlying gem-object. >> Pretty >> simple.. Why hadn't I thought of that before? >> >> Long story short: All drivers using GEM are safe now. This leaves vmwgfx= .. >> But >> they do their own access-management, anyway. >> >> The 3 patches on top implement render-nodes. I added a "drm_rnodes" modu= le >> parameter to core drm. You need to pass "drm.rnodes=3D1" on the kernel >> command-line or via sysfs _before_ loading a driver. Otherwise, render >> nodes >> will not be created. >> >> This allows us to test render-nodes and play with the API. I added FLINK >> for >> now so we can better test it. Not sure whether we should allow it in the >> end, >> though. >> >> Maybe we can get this into 3.11? > > > A bit unlikely, but 3.12 should work fine. > > I'm working on a project that can make good use of this, so if Alex doesn= 't > mind like to add the necessary radeon flags (should be only a few one lin= ers > anyway). > No objections from me. Alex