From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dave Airlie Subject: Re: [RFC v2] Revive the work on render-nodes branch Date: Fri, 20 Apr 2012 11:20:45 +0100 Message-ID: References: <1334254784-3200-1-git-send-email-ihadzic@research.bell-labs.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail-yx0-f177.google.com (mail-yx0-f177.google.com [209.85.213.177]) by gabe.freedesktop.org (Postfix) with ESMTP id AD5E59E745 for ; Fri, 20 Apr 2012 03:20:46 -0700 (PDT) Received: by yenm10 with SMTP id m10so5618693yen.36 for ; Fri, 20 Apr 2012 03:20:46 -0700 (PDT) In-Reply-To: <1334254784-3200-1-git-send-email-ihadzic@research.bell-labs.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: Ilija Hadzic Cc: dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org On Thu, Apr 12, 2012 at 7:19 PM, Ilija Hadzic wrote: > The following set of patches is the reword of the series > sent two weeks ago [2] that will revive the drm-render-nodes [1] > branch. Details of the original series are described in [2]. Thanks for looking at this, So one thing about adding render nodes, is it gives us the chance to maybe change the userspace API we expose to be more secure and have less cruft in it. Now I'm weary of a major API redesign as this could bog things now and we'd never make forward progress, but I'd like to at least consider it before adding new device nodes and locking us into the old APIs. The areas suggested before: 1) GEM, drop flink/open ioctls - these make security hard, since once you share a buffer any GEM opener can get access to it. Solutions to this proposed before are flink_to and maybe using PRIME/dma-buf. 2) master/device ownership. We might be able to drop the first opener is magically master, and require openers to ask for it somehow. 3) drop the old map ioctls from this interface, irq by busid, drop the AGP. I'm not talking about changing ioctl operation, more about introducing new ioctls and not allowing the old ones on the new nodes. Dave.