Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753643AbcCLV6b (ORCPT ); Sat, 12 Mar 2016 16:58:31 -0500 Received: from mail-pa0-f66.google.com ([209.85.220.66]:36191 "EHLO mail-pa0-f66.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753498AbcCLV6V (ORCPT ); Sat, 12 Mar 2016 16:58:21 -0500 From: Wei Yang To: joro@8bytes.org, jiang.liu@linux.intel.com, tglx@linutronix.de Cc: iommu@lists.linux-foundation.org, linux-kernel@vger.kernel.org, Wei Yang Subject: [PATCH 2/2] iommu/vt-d: use zero-sized array in DMAR related ACPI structures Date: Sat, 12 Mar 2016 21:57:39 +0000 Message-Id: <1457819859-28668-2-git-send-email-richard.weiyang@gmail.com> X-Mailer: git-send-email 2.5.0 In-Reply-To: <1457819859-28668-1-git-send-email-richard.weiyang@gmail.com> References: <1457819859-28668-1-git-send-email-richard.weiyang@gmail.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 5623 Lines: 167 1. DMAR table has variable number of remapping entries 2. DMAR hardware unit has variable number of device scope 3. DMAR device scope has variable number of pci path In current implementation, we use (head + 1) to access these variable number elements, which may not be obvious for audience. Zero-sized array is usually used for this purpose. This patch adds zero-sized array for variable elements in DMAR ACPI structures, which tries to make the code more audience friendly. Signed-off-by: Wei Yang --- drivers/iommu/dmar.c | 22 ++++++++++------------ include/acpi/actbl2.h | 31 +++++++++++++++++-------------- 2 files changed, 27 insertions(+), 26 deletions(-) diff --git a/drivers/iommu/dmar.c b/drivers/iommu/dmar.c index d6dd23f..04f199c 100644 --- a/drivers/iommu/dmar.c +++ b/drivers/iommu/dmar.c @@ -225,7 +225,6 @@ int dmar_insert_dev_scope(struct dmar_pci_notify_info *info, int i, level; struct device *tmp, *dev = &info->dev->dev; struct acpi_dmar_device_scope *scope; - struct acpi_dmar_pci_path *path; if (segment != info->seg) return 0; @@ -236,9 +235,8 @@ int dmar_insert_dev_scope(struct dmar_pci_notify_info *info, scope->entry_type != ACPI_DMAR_SCOPE_TYPE_BRIDGE) continue; - path = (struct acpi_dmar_pci_path *)(scope + 1); - level = (scope->length - sizeof(*scope)) / sizeof(*path); - if (!dmar_match_pci_path(info, scope->bus, path, level)) + level = (scope->length - sizeof(*scope)) / sizeof(*scope->path); + if (!dmar_match_pci_path(info, scope->bus, scope->path, level)) continue; if ((scope->entry_type == ACPI_DMAR_SCOPE_TYPE_ENDPOINT) ^ @@ -393,7 +391,7 @@ static int dmar_parse_one_drhd(struct acpi_dmar_header *header, void *arg) dmaru->reg_base_addr = drhd->address; dmaru->segment = drhd->segment; dmaru->include_all = drhd->flags & 0x1; /* BIT0: INCLUDE_ALL */ - dmaru->devices = dmar_alloc_dev_scope((void *)(drhd + 1), + dmaru->devices = dmar_alloc_dev_scope(drhd->dev_scope, ((void *)drhd) + drhd->header.length, &dmaru->devices_cnt); if (dmaru->devices_cnt && dmaru->devices == NULL) { @@ -579,7 +577,7 @@ static int dmar_walk_remapping_entries(struct acpi_dmar_header *start, static inline int dmar_walk_dmar_table(struct acpi_table_dmar *dmar, struct dmar_res_callback *cb) { - return dmar_walk_remapping_entries((void *)(dmar + 1), + return dmar_walk_remapping_entries(dmar->remapping_entries, dmar->header.length - sizeof(*dmar), cb); } @@ -685,12 +683,11 @@ static void __init dmar_acpi_insert_dev_scope(u8 device_number, struct acpi_dmar_device_scope *scope; struct device *tmp; int i; - struct acpi_dmar_pci_path *path; for_each_drhd_unit(dmaru) { drhd = (struct acpi_dmar_hardware_unit *)dmaru->hdr; - for (scope = (void *)(drhd + 1); + for (scope = drhd->dev_scope; (unsigned long)scope < ((unsigned long)drhd) + drhd->header.length; scope = ((void *)scope) + scope->length) { if (scope->entry_type != ACPI_DMAR_SCOPE_TYPE_NAMESPACE) @@ -698,15 +695,16 @@ static void __init dmar_acpi_insert_dev_scope(u8 device_number, if (scope->enumeration_id != device_number) continue; - path = (void *)(scope + 1); pr_info("ACPI device \"%s\" under DMAR at %llx as %02x:%02x.%d\n", dev_name(&adev->dev), dmaru->reg_base_addr, - scope->bus, path->device, path->function); + scope->bus, scope->path->device, + scope->path->function); for_each_dev_scope(dmaru->devices, dmaru->devices_cnt, i, tmp) if (tmp == NULL) { dmaru->devices[i].bus = scope->bus; - dmaru->devices[i].devfn = PCI_DEVFN(path->device, - path->function); + dmaru->devices[i].devfn = + PCI_DEVFN(scope->path->device, + scope->path->function); rcu_assign_pointer(dmaru->devices[i].dev, get_device(&adev->dev)); return; diff --git a/include/acpi/actbl2.h b/include/acpi/actbl2.h index 6e28f54..fd5c615 100644 --- a/include/acpi/actbl2.h +++ b/include/acpi/actbl2.h @@ -403,17 +403,6 @@ struct acpi_table_dbgp { * ******************************************************************************/ -struct acpi_table_dmar { - struct acpi_table_header header; /* Common ACPI table header */ - u8 width; /* Host Address Width */ - u8 flags; - u8 reserved[10]; -}; - -/* Masks for Flags field above */ - -#define ACPI_DMAR_INTR_REMAP (1) - /* DMAR subtable header */ struct acpi_dmar_header { @@ -434,12 +423,18 @@ enum acpi_dmar_type { /* DMAR Device Scope structure */ +struct acpi_dmar_pci_path { + u8 device; + u8 function; +}; + struct acpi_dmar_device_scope { u8 entry_type; u8 length; u16 reserved; u8 enumeration_id; u8 bus; + struct acpi_dmar_pci_path path[0]; }; /* Values for entry_type in struct acpi_dmar_device_scope - device types */ @@ -454,11 +449,18 @@ enum acpi_dmar_scope_type { ACPI_DMAR_SCOPE_TYPE_RESERVED = 6 /* 6 and greater are reserved */ }; -struct acpi_dmar_pci_path { - u8 device; - u8 function; +struct acpi_table_dmar { + struct acpi_table_header header; /* Common ACPI table header */ + u8 width; /* Host Address Width */ + u8 flags; + u8 reserved[10]; + struct acpi_dmar_header remapping_entries[0]; }; +/* Masks for Flags field above */ + +#define ACPI_DMAR_INTR_REMAP (1) + /* * DMAR Subtables, correspond to Type in struct acpi_dmar_header */ @@ -471,6 +473,7 @@ struct acpi_dmar_hardware_unit { u8 reserved; u16 segment; u64 address; /* Register Base Address */ + struct acpi_dmar_device_scope dev_scope[0]; }; /* Masks for Flags field above */ -- 2.5.0