summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authormlarkin <mlarkin@openbsd.org>2014-05-18 20:16:29 +0000
committermlarkin <mlarkin@openbsd.org>2014-05-18 20:16:29 +0000
commitd75e4e4052211c4a255b97968e2fc322767194d1 (patch)
treeb63280c114955d00515c01605f35e9a7db27eb6f
parentIf you need to allocate `a + b' bytes of memory, then don't allocate `a + b*2', (diff)
downloadwireguard-openbsd-d75e4e4052211c4a255b97968e2fc322767194d1.tar.xz
wireguard-openbsd-d75e4e4052211c4a255b97968e2fc322767194d1.zip
_PIC is an optional method, so we shouldn't be aborting the rest of the
MADT processing if it is missing. This, combined with an earlier diff from kettenis@ to fix MADT interrupt polarity processing, should resolve the issues reported with QEMU/KVM ACPI power button presses when using OpenBSD guests. ok kettenis@, pirofti@ tested by myself and kettenis for the past few weeks
-rw-r--r--sys/dev/acpi/acpimadt.c5
1 files changed, 2 insertions, 3 deletions
diff --git a/sys/dev/acpi/acpimadt.c b/sys/dev/acpi/acpimadt.c
index f012cf37365..1fc2e64aebf 100644
--- a/sys/dev/acpi/acpimadt.c
+++ b/sys/dev/acpi/acpimadt.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: acpimadt.c,v 1.26 2012/01/07 20:13:16 kettenis Exp $ */
+/* $OpenBSD: acpimadt.c,v 1.27 2014/05/18 20:16:29 mlarkin Exp $ */
/*
* Copyright (c) 2006 Mark Kettenis <kettenis@openbsd.org>
*
@@ -216,8 +216,7 @@ acpimadt_attach(struct device *parent, struct device *self, void *aux)
arg.type = AML_OBJTYPE_INTEGER;
arg.v_integer = 1;
- if (aml_evalname(acpi_sc, NULL, "\\_PIC", 1, &arg, NULL) != 0)
- return;
+ aml_evalname(acpi_sc, NULL, "\\_PIC", 1, &arg, NULL);
mp_busses = acpimadt_busses;
mp_nbusses = nitems(acpimadt_busses);