From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:57906)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <benh@kernel.crashing.org>) id 1X3gW3-0005vn-Ng
	for qemu-devel@nongnu.org; Sun, 06 Jul 2014 03:05:45 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <benh@kernel.crashing.org>) id 1X3gVx-0008AQ-Ik
	for qemu-devel@nongnu.org; Sun, 06 Jul 2014 03:05:39 -0400
Received: from gate.crashing.org ([63.228.1.57]:41334)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <benh@kernel.crashing.org>) id 1X3gVx-0008AM-65
	for qemu-devel@nongnu.org; Sun, 06 Jul 2014 03:05:33 -0400
Message-ID: <1404630327.29546.225.camel@pasglop>
From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Date: Sun, 06 Jul 2014 17:05:27 +1000
In-Reply-To: <1404629182.29546.224.camel@pasglop>
References: <1402974463.7661.102.camel@pasglop>
	<1403001900.1614.10.camel@nilsson.home.kraxel.org>
	<1403003721.7661.148.camel@pasglop>
	<1403006267.1614.14.camel@nilsson.home.kraxel.org>
	<1403040734.7661.173.camel@pasglop>
	<1403090315.13406.3.camel@nilsson.home.kraxel.org>
	<1403096614.7661.208.camel@pasglop>
	<1403170570.22530.5.camel@nilsson.home.kraxel.org>
	<1403329021.4587.78.camel@pasglop> <1403403026.4587.108.camel@pasglop>
	<1404126876.24066.23.camel@nilsson.home.kraxel.org>
	<1404131521.29546.3.camel@pasglop>
	<1404202838.24066.52.camel@nilsson.home.kraxel.org>
	<53B270CD.7080907@suse.de> <53B271EE.4030308@redhat.com>
	<1404205639.24066.58.camel@nilsson.home.kraxel.org>
	<53B27D25.4030603@redhat.com>
	<1404213329.24066.64.camel@nilsson.home.kraxel.org>
	<1404213827.29546.58.camel@pasglop> <1404292780.29546.90.camel@pasglop>
	<1404303120.7258.10.camel@nilsson.home.kraxel.org>
	<1404613171.29546.211.camel@pasglop>
	<1404625798.29546.214.camel@pasglop>
	<1404629182.29546.224.camel@pasglop>
Content-Type: text/plain; charset="UTF-8"
Mime-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: Re: [Qemu-devel] [RFC] qemu VGA endian swap low level drawing
	changes
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Gerd Hoffmann <kraxel@redhat.com>
Cc: Alexey Kardashevskiy <aik@ozlabs.ru>, Paolo Bonzini <pbonzini@redhat.com>, Alexander Graf <agraf@suse.de>, "qemu-devel@nongnu.org" <qemu-devel@nongnu.org>

On Sun, 2014-07-06 at 16:46 +1000, Benjamin Herrenschmidt wrote:
> At this point, I"m tempted to just revert that commit. What do you
> think Gerd ?

I mean that hunk of the commit... I missed that the commit itself
added a whole lot more bound checking.

Cheers,
Ben.