From mboxrd@z Thu Jan 1 00:00:00 1970 From: Anthony Liguori Subject: Re: [PATCH kvm-unit-tests 2/4] Introduce a C++ wrapper for the kvm APIs Date: Wed, 24 Nov 2010 10:57:35 -0600 Message-ID: <4CED43FF.4030201@codemonkey.ws> References: <1290595933-13122-1-git-send-email-avi@redhat.com> <1290595933-13122-3-git-send-email-avi@redhat.com> <50DD1E97-0ECD-41E6-B6F8-1D78AA4A4876@suse.de> <4CED2416.1040102@codemonkey.ws> <20101124154006.GE15111@redhat.com> <4CED344B.3030000@codemonkey.ws> <4CED3FE6.50900@redhat.com> <4CED41A0.8060003@redhat.com> <4CED4276.9090103@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: Avi Kivity , Gleb Natapov , Alexander Graf , Marcelo Tosatti , kvm@vger.kernel.org To: Jes Sorensen Return-path: Received: from mail-qw0-f46.google.com ([209.85.216.46]:37291 "EHLO mail-qw0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754937Ab0KXQ6G (ORCPT ); Wed, 24 Nov 2010 11:58:06 -0500 Received: by qwb7 with SMTP id 7so1057643qwb.19 for ; Wed, 24 Nov 2010 08:58:05 -0800 (PST) In-Reply-To: <4CED4276.9090103@redhat.com> Sender: kvm-owner@vger.kernel.org List-ID: On 11/24/2010 10:51 AM, Jes Sorensen wrote: > On 11/24/10 17:47, Avi Kivity wrote: > >> On 11/24/2010 06:40 PM, Jes Sorensen wrote: >> >>> Well the problem here is that the i8042 is in the i440fx.c file, it >>> shouldn't be there in the first place. The gluing together things in >>> silicon is really just a way to shorten the wires and make it easier, >>> they are still separate devices and as long as the i8042 requires ISA >>> access, and to be treated like an ISA device, we should glue it onto the >>> virtual ISA bus within QEMU. >>> >>> What you did above is making the exact same mistake as is done with the >>> current i440fx.c code. >>> >> If a real life 440fx has an i8042, then an emulated 440fx should have an >> emulated i8042. It's not complicated. >> > It's a question of how it is accessed, if it is treated like an ISA > device by the silicon, we should treat it like an ISA device in QEMU, > rather than pretend it is something that it isn't. > Does anyone have any evidence that the i8042 has anything to do with the ISA bus at all other than the fact that people have some weird notion in their head that if a pio/mmio operation isn't for a PCI device, it must be ISA? Regards, Anthony Liguori