From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [PATCH 1/3] drm/radeon: GPU virtual memory support v22 Date: Mon, 9 Jan 2012 21:05:35 +0100 Message-ID: <20120109200535.GI3723@phenom.ffwll.local> References: <1325819467-2216-1-git-send-email-alexdeucher@gmail.com> <20120108140532.GA4456@phenom.ffwll.local> <20120109083116.GB3723@phenom.ffwll.local> <20120109154415.GC3588@homer.localdomain> <20120109164946.GG3723@phenom.ffwll.local> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Return-path: Received: from mail-ww0-f43.google.com (mail-ww0-f43.google.com [74.125.82.43]) by gabe.freedesktop.org (Postfix) with ESMTP id A48F59E903 for ; Mon, 9 Jan 2012 12:03:33 -0800 (PST) Received: by wgbdt11 with SMTP id dt11so1720402wgb.12 for ; Mon, 09 Jan 2012 12:03:32 -0800 (PST) Content-Disposition: inline In-Reply-To: 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: Alex Deucher Cc: Alex Deucher , Jerome Glisse , dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org On Mon, Jan 09, 2012 at 11:56:20AM -0500, Alex Deucher wrote: > On Mon, Jan 9, 2012 at 11:49 AM, Daniel Vetter wrote: > > On Mon, Jan 09, 2012 at 11:07:44AM -0500, Alex Deucher wrote: > >> We would have liked to as well, but with the holidays and internal > >> review process and people being on vacation, and the kernel merge > >> window timing it didn't work out this time. =A0Still as Jerome said, o= ur > >> VM setup is tightly coupled with the way our HW works. =A0I'm not sure > >> how much code could actually be shared. > > > > Again to clarify, I'm not so much interesting in sharing code here - I > > know it'll be nigh to impossible by having looked through nouveau alrea= dy > > and quickly glossed through your stuff. I see the value is just in havi= ng > > a minimal set of common idioms that we can use to build cool stuff like > > resource management. > > > > Think of gem, which is actually about next to nothing. There's a the fl= ink > > stuff and a common close ioctl and recently also a dummy create unusable > > for anythine else than boot spalsh screens. Plus the shmem backing store > > which seems to be not so much in favour with embedded chips, i.e doesn't > > really count. Ridiculous compared to all the other stuff required to dr= aw > > triangles on a gpu, but good enough to make things like dri2 and wayland > > possible. > > > = > I generally agree here, but I don't think this patch prevents > something like that in the future. Also, I think we need to actually > use GPU VM for a bit and see how everyone's implementations and plans > shake out before we can decide which idioms are common. Agreed. My sligh miff has only really ben that this got posted only recently and there was no chance to have this quick discussion before it got merged. Cheers, Daniel -- = Daniel Vetter Mail: daniel@ffwll.ch Mobile: +41 (0)79 365 57 48