From mboxrd@z Thu Jan 1 00:00:00 1970 From: Avi Kivity Subject: Re: [PATCH][v2] kvm-userspace: Load PCI option ROMs Date: Sun, 04 Jan 2009 21:51:39 +0200 Message-ID: <4961134B.70204@redhat.com> References: <49588A4A.8020902@redhat.com> <49589D21.9070201@redhat.com> <61563CE63B4F854986A895DA7AD3C177044E6ECA@pdsmsx502.ccr.corp.intel.com> <495A4367.6050203@redhat.com> <61563CE63B4F854986A895DA7AD3C177044E7098@pdsmsx502.ccr.corp.intel.com> <495B3B43.2090200@redhat.com> <61563CE63B4F854986A895DA7AD3C177044E7306@pdsmsx502.ccr.corp.intel.com> <00e301c96e20$35c2fdb0$a148f910$@org> <49608EBC.2090409@redhat.com> <010101c96e8f$a42d4a20$ec87de60$@org> <4960F1D2.4080600@redhat.com> <010201c96e95$72cd4f10$5867ed30$@org> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: "'Kevin O'Connor'" , "'Shan, Haitao'" , "'Liu, Kechao'" , kvm@vger.kernel.org To: Leendert van Doorn Return-path: Received: from mx2.redhat.com ([66.187.237.31]:52945 "EHLO mx2.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1758316AbZADTvZ (ORCPT ); Sun, 4 Jan 2009 14:51:25 -0500 In-Reply-To: <010201c96e95$72cd4f10$5867ed30$@org> Sender: kvm-owner@vger.kernel.org List-ID: Leendert van Doorn wrote: >> With a switch, please. Default behaviour should be to virtualize. >> >> One way to implement it is to pass pci devfn -> BAR hints through the >> firmware interface. This way you can choose which BARs to place where, >> and where to allow the default placement. >> > > I'm already providing the hints, but you are proposing that the default > behavior for the BIOS should be to allocate BARs and when given a flag it > should try to preserve the host BAR mappings. I can do that. > It's a general policy -- isolate the guest from the host wherever possible. In a few cases it isn't possible (time) and it really hurts. >>> Of course, my goal is to run unmodified BIOS/drivers. You could always >>> change the drivers. >>> >> Not Windows drivers. >> > > Well, at least I have a fighting chance to change the ATI windows drivers > :-) > :-) -- I have a truly marvellous patch that fixes the bug which this signature is too narrow to contain.