From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43)
	id 1NxAPg-0000IL-Bl
	for qemu-devel@nongnu.org; Wed, 31 Mar 2010 22:45:44 -0400
Received: from [140.186.70.92] (port=38403 helo=eggs.gnu.org)
	by lists.gnu.org with esmtp (Exim 4.43) id 1NxAPe-0000IB-5h
	for qemu-devel@nongnu.org; Wed, 31 Mar 2010 22:45:43 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.69)
	(envelope-from <anthony@codemonkey.ws>) id 1NxAPc-0007oT-7p
	for qemu-devel@nongnu.org; Wed, 31 Mar 2010 22:45:41 -0400
Received: from mail-gx0-f222.google.com ([209.85.217.222]:40215)
	by eggs.gnu.org with esmtp (Exim 4.69)
	(envelope-from <anthony@codemonkey.ws>) id 1NxAPc-0007oE-4n
	for qemu-devel@nongnu.org; Wed, 31 Mar 2010 22:45:40 -0400
Received: by gxk22 with SMTP id 22so618810gxk.4
	for <qemu-devel@nongnu.org>; Wed, 31 Mar 2010 19:45:39 -0700 (PDT)
Message-ID: <4BB408CF.9040004@codemonkey.ws>
Date: Wed, 31 Mar 2010 21:45:35 -0500
From: Anthony Liguori <anthony@codemonkey.ws>
MIME-Version: 1.0
Subject: Re: [Qemu-devel] Re: [PATCH] vhost: fix features ack
References: <20100331182031.GA5200@redhat.com> <4BB393CF.1040700@codemonkey.ws>
	<20100331153805.03ee142e@redhat.com>
	<20100331190753.GA6914@redhat.com> <4BB3A183.8000905@codemonkey.ws>
	<20100331224541.GB19306@hall.aurel32.net>
In-Reply-To: <20100331224541.GB19306@hall.aurel32.net>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
List-Id: qemu-devel.nongnu.org
List-Unsubscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.gnu.org/pipermail/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Aurelien Jarno <aurelien@aurel32.net>
Cc: blauwirbel@gmail.com, Luiz Capitulino <lcapitulino@redhat.com>, qemu-devel@nongnu.org, "Michael S. Tsirkin" <mst@redhat.com>

On 03/31/2010 05:45 PM, Aurelien Jarno wrote:
> While it probably make sense to achieve this goal, it doesn't mean it
> should be done the dirty way.
>
> For example it is known for a lot of time that the solution for the
> bswap in the device code is to add a bus model doing the byteswapping.
> Removing the #ifdef by deciding "this device will only be big/little
> endian" doesn't seem to go in the right direction.
>    

Yeah, I'm having real trouble with the KVM regression.  I thought I had 
it fixed but linux-user really made a mess of things.  There's no simple 
solution that doesn't require quite a bit of refactoring which I'd 
rather do in a less ugly way.  We've already been discussing getting rid 
of all the kvm_enabled() stuff and I think doing that properly is going 
to be needed to handle this correctly.

I'm thinking we should back out the vl.c changes and try to clean up the 
KVM bits first.  Does that sound reasonable blueswirl or can you think 
of a cleaner way to deal with kvm?

Regards,

Anthony Liguori