From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rob Clark Subject: Re: [PATCH 1/2] drm: store the gem vma offset manager in a typed pointer Date: Tue, 17 Dec 2013 20:22:06 -0500 Message-ID: References: <1386768287-30353-1-git-send-email-daniel.vetter@ffwll.ch> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail-ie0-f169.google.com (mail-ie0-f169.google.com [209.85.223.169]) by gabe.freedesktop.org (Postfix) with ESMTP id 11DC7FD7EC for ; Tue, 17 Dec 2013 17:22:08 -0800 (PST) Received: by mail-ie0-f169.google.com with SMTP id e14so9571021iej.28 for ; Tue, 17 Dec 2013 17:22:07 -0800 (PST) In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dri-devel-bounces@lists.freedesktop.org Errors-To: dri-devel-bounces@lists.freedesktop.org To: Dave Airlie Cc: Daniel Vetter , DRI Development List-Id: dri-devel@lists.freedesktop.org On Tue, Dec 17, 2013 at 8:04 PM, Dave Airlie wrote: > On Wed, Dec 11, 2013 at 11:24 PM, Daniel Vetter wrote: >> This was hidden in a generic void * dev->mm_private. But only ever >> used for gem. But thanks to this fake generic pretension no one >> noticed that Rob's drm drivers are now all broken. > > Should this go to fixes? I think should go in fixes probably.. wasn't it broken since vma offset mgr? I guess I don't test debugfs enough. oh, and: Signed-off-by: Rob Clark (sorry, overlooked this thread earlier, even though I think Daniel mentioned the issue on irc) > Or is the problem only introduced in this series? > > Dave. > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/dri-devel