summaryrefslogtreecommitdiffstats
path: root/drivers/pci/dmar.c
diff options
context:
space:
mode:
authorDavid Woodhouse <David.Woodhouse@intel.com>2008-10-18 15:43:14 +0100
committerDavid Woodhouse <David.Woodhouse@intel.com>2008-10-18 15:45:48 +0100
commitf82851a8a480a26611175f064f54e17f5f7b01ae (patch)
tree6649d01d7adb0f1e3e5357f10082cb9823d2a40d /drivers/pci/dmar.c
parentbb9e6d65078da2f38cfe1067cfd31a896ca867c0 (diff)
downloadop-kernel-dev-f82851a8a480a26611175f064f54e17f5f7b01ae.zip
op-kernel-dev-f82851a8a480a26611175f064f54e17f5f7b01ae.tar.gz
dmar: fix uninitialised 'ret' variable in dmar_parse_dev()
This was introduced by commit 1886e8a90a580f3ad343f2065c84c1b9e1dac9ef ("x64, x2apic/intr-remap: code re-structuring, to be used by both DMA and Interrupt remapping"). It was causing bogus results to be returned from dmar_parse_dev() when the first unit with the INCLUDE_ALL flag was processed. Signed-off-by: David Woodhouse <David.Woodhouse@intel.com>
Diffstat (limited to 'drivers/pci/dmar.c')
-rw-r--r--drivers/pci/dmar.c5
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/pci/dmar.c b/drivers/pci/dmar.c
index b651738..7b37511 100644
--- a/drivers/pci/dmar.c
+++ b/drivers/pci/dmar.c
@@ -188,12 +188,11 @@ dmar_parse_one_drhd(struct acpi_dmar_header *header)
return 0;
}
-static int __init
-dmar_parse_dev(struct dmar_drhd_unit *dmaru)
+static int __init dmar_parse_dev(struct dmar_drhd_unit *dmaru)
{
struct acpi_dmar_hardware_unit *drhd;
static int include_all;
- int ret;
+ int ret = 0;
drhd = (struct acpi_dmar_hardware_unit *) dmaru->hdr;
OpenPOWER on IntegriCloud