diff options
| author | David Woodhouse <David.Woodhouse@intel.com> | 2008-10-18 15:43:14 +0100 |
|---|---|---|
| committer | David Woodhouse <David.Woodhouse@intel.com> | 2008-10-18 15:45:48 +0100 |
| commit | f82851a8a480a26611175f064f54e17f5f7b01ae (patch) | |
| tree | 6649d01d7adb0f1e3e5357f10082cb9823d2a40d /drivers/pci | |
| parent | bb9e6d65078da2f38cfe1067cfd31a896ca867c0 (diff) | |
| download | linux-f82851a8a480a26611175f064f54e17f5f7b01ae.tar.gz linux-f82851a8a480a26611175f064f54e17f5f7b01ae.tar.bz2 linux-f82851a8a480a26611175f064f54e17f5f7b01ae.zip | |
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')
| -rw-r--r-- | drivers/pci/dmar.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/pci/dmar.c b/drivers/pci/dmar.c index b65173828bc2..7b3751136e63 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; |