From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([208.118.235.92]:43347)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <kraxel@redhat.com>) id 1SzM3v-00066c-SK
	for qemu-devel@nongnu.org; Thu, 09 Aug 2012 02:17:40 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <kraxel@redhat.com>) id 1SzM3s-0004Xg-6b
	for qemu-devel@nongnu.org; Thu, 09 Aug 2012 02:17:39 -0400
Received: from mx1.redhat.com ([209.132.183.28]:7489)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <kraxel@redhat.com>) id 1SzM3r-0004Xc-V3
	for qemu-devel@nongnu.org; Thu, 09 Aug 2012 02:17:36 -0400
Message-ID: <502355FA.1010708@redhat.com>
Date: Thu, 09 Aug 2012 08:17:30 +0200
From: Gerd Hoffmann <kraxel@redhat.com>
MIME-Version: 1.0
References: <1343629462.21647.32.camel@pasglop> <50165D0A.6060608@redhat.com>
	<1343647217.21647.40.camel@pasglop> <50166F2A.1040507@redhat.com>
	<1343649267.21647.44.camel@pasglop>
	<20120730161954.GM29361@garlic.redhat.com>
	<50194E79.5090409@suse.de> <8739465sea.fsf@codemonkey.ws>
	<20120803064532.GR29361@garlic.redhat.com>
	<87boism6sv.fsf@codemonkey.ws>
	<20120807070055.GA25376@garlic.redhat.com>
	<5020CB4A.10501@redhat.com>
	<CAF+CadvjqF+brCAgc2ApGi4brbOF=A-Yz1kGcwJs=auF7oYX1A@mail.gmail.com>
	<50212125.6010209@redhat.com>
	<CAF+Cadsrg4-coPE0H5oVZnD9cMUDDhLAF6b4JFpEkAR+8VoJdw@mail.gmail.com>
	<502204CE.3000303@redhat.com>
	<CAF+CadvpWYtSz2LC8Ko7D8gUYC3Yfm=CW8dS+Mx3jQVxoO5mYw@mail.gmail.com>
In-Reply-To: <CAF+CadvpWYtSz2LC8Ko7D8gUYC3Yfm=CW8dS+Mx3jQVxoO5mYw@mail.gmail.com>
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit
Subject: Re: [Qemu-devel] Cirrus bugs vs endian: how two bugs cancel each
 other out
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: Erlon Cruz <sombrafam@gmail.com>
Cc: Avi Kivity <avi@redhat.com>, Alon Levy <alevy@redhat.com>, =?ISO-8859-1?Q?Andreas_F=E4?= =?ISO-8859-1?Q?rber?= <afaerber@suse.de>, Anthony Liguori <anthony@codemonkey.ws>, qemu-devel@nongnu.org

  Hi,

>> Maybe the guest driver places the qxl commands in native endian instead
>> of big endian into memory?
> 
> The guest drivers places the commands in the memory in its native
> endianess. That why the device only works when guest/host has the same
> endianess. That's sanit-checks doesn't fails.

Ah, ok. this explains it.  So we must fix red_parse_qxl.c to byteswap
and also the guest driver to byteswap.  Then it will work even in case
guest and host have different endianess.

cheers,
  Gerd