From mboxrd@z Thu Jan 1 00:00:00 1970 From: Konrad Rzeszutek Wilk Subject: Re: [RFC] ttm merge ttm_backend & ttm_tt Date: Tue, 1 Nov 2011 21:29:42 -0400 Message-ID: <20111102012942.GA20327@phenom.dumpdata.com> References: Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from acsinet15.oracle.com (acsinet15.oracle.com [141.146.126.227]) by gabe.freedesktop.org (Postfix) with ESMTP id EAAAA9E7F4 for ; Tue, 1 Nov 2011 18:29:52 -0700 (PDT) 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: Jerome Glisse Cc: Thomas Hellstrom , dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org On Tue, Nov 01, 2011 at 09:11:37PM -0400, Jerome Glisse wrote: > Hi, > > So attached is patch serie (sorry for that i am away of my normal mail > config) which ultimately merge ttm_backend & ttm_tt it allows to get > rid of data duplication btw the two, especialy btw ttm_tt and driver > specific backend. So net result is less 300lines of code accross ttm > and driver. > > Konrad given some of the upstream nouveau change, your patchserie > conflict and would lead to revert some nouveau fixes. I believe the > intention is to get the ttm-dma code into 3.3 (3.2 seems late but > dunno). If 3.3 is the aim than i will rebase your patch on top of this Yup. 3.3. > serie, this should lower the number of change your serie needed. OK. > > Note that this is early code, only compile tested, i just wanted to > get feedback to know if anyone has objection on that. Let me take a look then at them tomorrow. > > For quick review, first 5 patches can be ignored, they are cleanup and > dead code removal or small fixes. The real action is in the last > patch. > > Next set is to move page allocation through ttm_helper into the driver > thus making the whole dma stuff a driver business, driver who don't > care about dma (like vmw) could ignore it, while driver that have to > would deal with it through ttm_helper. > > Cheers, > Jerome