From mboxrd@z Thu Jan 1 00:00:00 1970 From: Benjamin Herrenschmidt Date: Thu, 08 Dec 2016 21:57:29 +0000 Subject: Re: [RFC PATCH 0/3] staging: remove fbdev drivers Message-Id: <1481234249.26959.55.camel@kernel.crashing.org> List-Id: References: <1479888193-23908-1-git-send-email-tomi.valkeinen@ti.com> <1481158879.26959.41.camel@kernel.crashing.org> <20161208101005.6ufl3d4qvwprosju@phenom.ffwll.local> <20161208140210.rfyjf2265flsfpfj@phenom.ffwll.local> <20161208153735.74d7d350@free-electrons.com> <20161208152134.wnv4j4i6m5xpoycp@phenom.ffwll.local> <1481232877.26959.52.camel@kernel.crashing.org> In-Reply-To: <1481232877.26959.52.camel@kernel.crashing.org> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Daniel Vetter , Geert Uytterhoeven Cc: Thomas Petazzoni , Linux Fbdev development list , Teddy Wang , Greg Kroah-Hartman , "linux-kernel@vger.kernel.org" , DRI Development , Tomi Valkeinen , Sudip Mukherjee , Arnaud Patard On Fri, 2016-12-09 at 08:34 +1100, Benjamin Herrenschmidt wrote: > As I mentioned earlier, probably 1 or 2 years ago, Dave made the > argument that shadowing through memory was necessary and precluded 2D > accel, though I don't fully remember the root of the argument. If that > is indeed not the case, then my main objection is lifted. Things seem to change quickly as Daniel pointed out. So ast and cirrus seem to still use a manual dirty tracking and shadowing (though I'm not sure why), but the infrastructure for that has moved from the drivers to the helpers. bochs (qemu) doesn't seem to anymore from what I can see as it doesn't have a ->dirty callback. Cheers, Ben. From mboxrd@z Thu Jan 1 00:00:00 1970 From: Benjamin Herrenschmidt Subject: Re: [RFC PATCH 0/3] staging: remove fbdev drivers Date: Fri, 09 Dec 2016 08:57:29 +1100 Message-ID: <1481234249.26959.55.camel@kernel.crashing.org> References: <1479888193-23908-1-git-send-email-tomi.valkeinen@ti.com> <1481158879.26959.41.camel@kernel.crashing.org> <20161208101005.6ufl3d4qvwprosju@phenom.ffwll.local> <20161208140210.rfyjf2265flsfpfj@phenom.ffwll.local> <20161208153735.74d7d350@free-electrons.com> <20161208152134.wnv4j4i6m5xpoycp@phenom.ffwll.local> <1481232877.26959.52.camel@kernel.crashing.org> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from gate.crashing.org (gate.crashing.org [63.228.1.57]) by gabe.freedesktop.org (Postfix) with ESMTPS id 5BF426E873 for ; Thu, 8 Dec 2016 21:57:56 +0000 (UTC) In-Reply-To: <1481232877.26959.52.camel@kernel.crashing.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Daniel Vetter , Geert Uytterhoeven Cc: Thomas Petazzoni , Linux Fbdev development list , Teddy Wang , Greg Kroah-Hartman , "linux-kernel@vger.kernel.org" , DRI Development , Tomi Valkeinen , Sudip Mukherjee , Arnaud Patard List-Id: dri-devel@lists.freedesktop.org T24gRnJpLCAyMDE2LTEyLTA5IGF0IDA4OjM0ICsxMTAwLCBCZW5qYW1pbiBIZXJyZW5zY2htaWR0 IHdyb3RlOgo+IEFzIEkgbWVudGlvbmVkIGVhcmxpZXIsIHByb2JhYmx5IDEgb3IgMiB5ZWFycyBh Z28sIERhdmUgbWFkZSB0aGUKPiBhcmd1bWVudCB0aGF0IHNoYWRvd2luZyB0aHJvdWdoIG1lbW9y eSB3YXMgbmVjZXNzYXJ5IGFuZCBwcmVjbHVkZWQgMkQKPiBhY2NlbCwgdGhvdWdoIEkgZG9uJ3Qg ZnVsbHkgcmVtZW1iZXIgdGhlIHJvb3Qgb2YgdGhlIGFyZ3VtZW50LiBJZiB0aGF0Cj4gaXMgaW5k ZWVkIG5vdCB0aGUgY2FzZSwgdGhlbiBteSBtYWluIG9iamVjdGlvbiBpcyBsaWZ0ZWQuCgpUaGlu Z3Mgc2VlbSB0byBjaGFuZ2UgcXVpY2tseSBhcyBEYW5pZWwgcG9pbnRlZCBvdXQuCgpTbyBhc3Qg YW5kIGNpcnJ1cyBzZWVtIHRvIHN0aWxsIHVzZSBhIG1hbnVhbCBkaXJ0eSB0cmFja2luZyBhbmQK c2hhZG93aW5nICh0aG91Z2ggSSdtIG5vdCBzdXJlIHdoeSksIGJ1dCB0aGUgaW5mcmFzdHJ1Y3R1 cmUgZm9yCnRoYXQgaGFzIG1vdmVkIGZyb20gdGhlIGRyaXZlcnMgdG8gdGhlIGhlbHBlcnMuCgpi b2NocyAocWVtdSkgZG9lc24ndCBzZWVtIHRvIGFueW1vcmUgZnJvbSB3aGF0IEkgY2FuIHNlZSBh cyBpdApkb2Vzbid0IGhhdmUgYSAtPmRpcnR5IGNhbGxiYWNrLgoKQ2hlZXJzLApCZW4uCgpfX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpkcmktZGV2ZWwgbWFp bGluZyBsaXN0CmRyaS1kZXZlbEBsaXN0cy5mcmVlZGVza3RvcC5vcmcKaHR0cHM6Ly9saXN0cy5m cmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9kcmktZGV2ZWwK From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932931AbcLHV6P (ORCPT ); Thu, 8 Dec 2016 16:58:15 -0500 Received: from gate.crashing.org ([63.228.1.57]:36843 "EHLO gate.crashing.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932898AbcLHV6N (ORCPT ); Thu, 8 Dec 2016 16:58:13 -0500 Message-ID: <1481234249.26959.55.camel@kernel.crashing.org> Subject: Re: [RFC PATCH 0/3] staging: remove fbdev drivers From: Benjamin Herrenschmidt To: Daniel Vetter , Geert Uytterhoeven Cc: Thomas Petazzoni , Tomi Valkeinen , Greg Kroah-Hartman , Noralf =?ISO-8859-1?Q?Tr=F8nnes?= , Sudip Mukherjee , Teddy Wang , Arnaud Patard , DRI Development , Linux Fbdev development list , "linux-kernel@vger.kernel.org" Date: Fri, 09 Dec 2016 08:57:29 +1100 In-Reply-To: <1481232877.26959.52.camel@kernel.crashing.org> References: <1479888193-23908-1-git-send-email-tomi.valkeinen@ti.com> <1481158879.26959.41.camel@kernel.crashing.org> <20161208101005.6ufl3d4qvwprosju@phenom.ffwll.local> <20161208140210.rfyjf2265flsfpfj@phenom.ffwll.local> <20161208153735.74d7d350@free-electrons.com> <20161208152134.wnv4j4i6m5xpoycp@phenom.ffwll.local> <1481232877.26959.52.camel@kernel.crashing.org> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.22.2 (3.22.2-1.fc25) Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, 2016-12-09 at 08:34 +1100, Benjamin Herrenschmidt wrote: > As I mentioned earlier, probably 1 or 2 years ago, Dave made the > argument that shadowing through memory was necessary and precluded 2D > accel, though I don't fully remember the root of the argument. If that > is indeed not the case, then my main objection is lifted. Things seem to change quickly as Daniel pointed out. So ast and cirrus seem to still use a manual dirty tracking and shadowing (though I'm not sure why), but the infrastructure for that has moved from the drivers to the helpers. bochs (qemu) doesn't seem to anymore from what I can see as it doesn't have a ->dirty callback. Cheers, Ben.