From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from [140.186.70.92] (port=52034 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1PB4mu-0005yj-Lw for qemu-devel@nongnu.org; Wed, 27 Oct 2010 08:08:21 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1PB4lw-0007c2-0m for qemu-devel@nongnu.org; Wed, 27 Oct 2010 08:07:28 -0400 Received: from mx1.redhat.com ([209.132.183.28]:2089) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1PB4lv-0007bO-Q8 for qemu-devel@nongnu.org; Wed, 27 Oct 2010 08:06:27 -0400 Date: Wed, 27 Oct 2010 16:05:58 +0200 From: "Michael S. Tsirkin" Message-ID: <20101027140558.GA3380@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Subject: [Qemu-devel] [PATCH] pci: improve w1c mask handling List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org, yamahata@valinux.co.jp - save/restore must not check w1c bits since they are in fact guest controlled - clear w1c bits on reset Note: for express there are different kinds of reset, some leave part of config space alone. We will likely need a sticky bit mask to implement this. Signed-off-by: Michael S. Tsirkin --- Isaku, does the below make sense? hw/pci.c | 6 ++++-- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/hw/pci.c b/hw/pci.c index e50de14..1a14a0e 100644 --- a/hw/pci.c +++ b/hw/pci.c @@ -141,7 +141,8 @@ static void pci_device_reset(PCIDevice *dev) pci_update_irq_status(dev); /* Clear all writeable bits */ pci_word_test_and_clear_mask(dev->config + PCI_COMMAND, - pci_get_word(dev->wmask + PCI_COMMAND)); + pci_get_word(dev->wmask + PCI_COMMAND) | + pci_get_word(dev->w1cmask + PCI_COMMAND)); dev->config[PCI_CACHE_LINE_SIZE] = 0x0; dev->config[PCI_INTERRUPT_LINE] = 0x0; for (r = 0; r < PCI_NUM_REGIONS; ++r) { @@ -293,7 +294,8 @@ static int get_pci_config_device(QEMUFile *f, void *pv, size_t size) qemu_get_buffer(f, config, size); for (i = 0; i < size; ++i) { - if ((config[i] ^ s->config[i]) & s->cmask[i] & ~s->wmask[i]) { + if ((config[i] ^ s->config[i]) & + s->cmask[i] & ~s->wmask[i] & ~s->w1cmask) { qemu_free(config); return -EINVAL; } -- 1.7.3.2.91.g446ac