From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Herrmann Subject: Re: [PATCH v2 0/6] DRM: VMA Access Management and Render Nodes Date: Fri, 23 Aug 2013 14:31:32 +0200 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-ie0-f173.google.com (mail-ie0-f173.google.com [209.85.223.173]) by gabe.freedesktop.org (Postfix) with ESMTP id 9528AE81B2 for ; Fri, 23 Aug 2013 05:31:32 -0700 (PDT) Received: by mail-ie0-f173.google.com with SMTP id qd12so685086ieb.18 for ; Fri, 23 Aug 2013 05:31:32 -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 , "dri-devel@lists.freedesktop.org" List-Id: dri-devel@lists.freedesktop.org Hi On Fri, Aug 23, 2013 at 1:28 PM, 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. whoops, 3.12 of course. > 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). Feel free to send a patch to dri-devel or just let me know the ioctls and I will include it in this series. Regards David