From mboxrd@z Thu Jan 1 00:00:00 1970 From: Avi Kivity Subject: Re: [PATCH 2/3] qemu: make cirrus init value pci spec compliant Date: Thu, 08 Oct 2009 17:29:29 +0200 Message-ID: <4ACE0559.5040206@redhat.com> References: <20091008145241.GC12275@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: Anthony Liguori , "qemu-devel@nongnu.org" , kvm-devel To: "Michael S. Tsirkin" Return-path: Received: from mx1.redhat.com ([209.132.183.28]:48308 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1758500AbZJHP37 (ORCPT ); Thu, 8 Oct 2009 11:29:59 -0400 In-Reply-To: <20091008145241.GC12275@redhat.com> Sender: kvm-owner@vger.kernel.org List-ID: On 10/08/2009 04:52 PM, Michael S. Tsirkin wrote: > PCI memory should be disabled at reset, otherwise > we might claim transactions at address 0. > I/O should also be disabled, although for cirrus > it is harmless to enable it as we do not > have I/O bar. > > Note: need bios fix for this patch to work: > currently pc-bios incorrently assumes that it does not > need to enable i/o unless device has i/o bar. > > Signed-off-by: Michael S. Tsirkin > --- > hw/cirrus_vga.c | 1 - > 1 files changed, 0 insertions(+), 1 deletions(-) > > diff --git a/hw/cirrus_vga.c b/hw/cirrus_vga.c > index 1feb827..c54ac65 100644 > --- a/hw/cirrus_vga.c > +++ b/hw/cirrus_vga.c > @@ -3248,7 +3248,6 @@ static int pci_cirrus_vga_initfn(PCIDevice *dev) > /* setup PCI */ > pci_config_set_vendor_id(pci_conf, PCI_VENDOR_ID_CIRRUS); > pci_config_set_device_id(pci_conf, device_id); > - pci_conf[0x04] = PCI_COMMAND_IOACCESS | PCI_COMMAND_MEMACCESS; > pci_config_set_class(pci_conf, PCI_CLASS_DISPLAY_VGA); > pci_conf[PCI_HEADER_TYPE] = PCI_HEADER_TYPE_NORMAL; > This needs to be conditional on the machine type. Old machines must retain this code for live migration to work (we need to live migrate the bios, so we can't assume the bios fix is in during live migration from older qemus). -- error compiling committee.c: too many arguments to function