From patchwork Fri Jul 22 15:59:03 2011 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Donald Dutile X-Patchwork-Id: 999872 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by demeter2.kernel.org (8.14.4/8.14.4) with ESMTP id p6MGBP3C007169 for ; Fri, 22 Jul 2011 16:11:26 GMT Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751588Ab1GVP7G (ORCPT ); Fri, 22 Jul 2011 11:59:06 -0400 Received: from mx1.redhat.com ([209.132.183.28]:9338 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751111Ab1GVP7F (ORCPT ); Fri, 22 Jul 2011 11:59:05 -0400 Received: from int-mx10.intmail.prod.int.phx2.redhat.com (int-mx10.intmail.prod.int.phx2.redhat.com [10.5.11.23]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id p6MFx4V5026047 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK) for ; Fri, 22 Jul 2011 11:59:04 -0400 Received: from dddsys0.bos.redhat.com (dddsys0.bos.redhat.com [10.16.16.40]) by int-mx10.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id p6MFx391015211; Fri, 22 Jul 2011 11:59:03 -0400 From: Donald Dutile Subject: [PATCH v3] pci: correct pci config size default for cap version 2 endpoints To: kvm@vger.kernel.org Cc: alex.williamson@redhat.com, mst@redhat.com Date: Fri, 22 Jul 2011 11:59:03 -0400 Message-ID: <20110722155338.43049.12587.stgit@dddsys0.bos.redhat.com> User-Agent: StGIT/0.14.3 MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.68 on 10.5.11.23 Sender: kvm-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: kvm@vger.kernel.org X-Greylist: IP, sender and recipient auto-whitelisted, not delayed by milter-greylist-4.2.6 (demeter2.kernel.org [140.211.167.43]); Fri, 22 Jul 2011 16:11:26 +0000 (UTC) v3: remove all boundary tests. just fix the obvious bug. : boundary test is not necessary; get this fix in & post boundary test in another separate patch. v2: do local boundary check with respect to legacy PCI header length, and don't depend on it in pci_add_capability(). : fix compilation, and change else>2 to simple else for all other cases. v1: first patch: boundary check in pci_add_capability(). Doing device assignement using a PCIe device with it's PCI Cap structure at offset 0xcc showed a problem in the default size mapped for this cap-id. The failure caused a corruption which might have gone unnoticed otherwise. Fix assigned_device_pci_cap_init() to set the default size of PCIe Cap structure (cap-id 0x10) to 0x34 instead of 0x3c. 0x34 is default, min, for endpoint device with a cap version of 2. Signed-off-by: Donald Dutile cc: Alex Williamson cc: Michael S. Tsirkin tested-by: ebenes@redhat.com Acked-by: Alex Williamson --- hw/device-assignment.c | 8 +++++--- 1 files changed, 5 insertions(+), 3 deletions(-) -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html diff --git a/hw/device-assignment.c b/hw/device-assignment.c index 36ad6b0..34db52e 100644 --- a/hw/device-assignment.c +++ b/hw/device-assignment.c @@ -1419,16 +1419,18 @@ static int assigned_device_pci_cap_init(PCIDevice *pci_dev) } if ((pos = pci_find_cap_offset(pci_dev, PCI_CAP_ID_EXP, 0))) { - uint8_t version; + uint8_t version, size; uint16_t type, devctl, lnkcap, lnksta; uint32_t devcap; - int size = 0x3c; /* version 2 size */ version = pci_get_byte(pci_dev->config + pos + PCI_EXP_FLAGS); version &= PCI_EXP_FLAGS_VERS; if (version == 1) { size = 0x14; - } else if (version > 2) { + } else if (version == 2) { + /* don't include slot cap/stat/ctrl 2 regs; only support endpoints */ + size = 0x34; + } else { fprintf(stderr, "Unsupported PCI express capability version %d\n", version); return -EINVAL;