lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Tue, 8 Jul 2014 00:08:05 +0000 From: "Zheng, Lv" <lv.zheng@...el.com> To: "Rafael J. Wysocki" <rjw@...ysocki.net> CC: "Wysocki, Rafael J" <rafael.j.wysocki@...el.com>, "Brown, Len" <len.brown@...el.com>, Lv Zheng <zetalog@...il.com>, "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>, "linux-acpi@...r.kernel.org" <linux-acpi@...r.kernel.org>, Len Brown <lenb@...nel.org>, "sfi-devel@...plefirmware.org" <sfi-devel@...plefirmware.org> Subject: RE: [PATCH v2 5/7] ACPI/SFI: Fix wrong <acpi/acpi.h> inclusion in SFI/ACPI wrapper - table definitions. Hi, > From: Rafael J. Wysocki [mailto:rjw@...ysocki.net] > Sent: Tuesday, July 08, 2014 5:25 AM > > On Monday, July 07, 2014 12:17:28 PM Lv Zheng wrote: > > This patch removes <acpi/acpi.h> inclusions from <linux/sfi_acpi.h> as > > <linux/acpi.h> has already included it for CONFIG_ACPI=n builds. > > So are patches [1-4/5] only needed to fix this particular issue? To fix this issue, we only need patch 1-5. After applying these patches, all ACPICA structures are visible to the entire linux kernel. Thanks and best regards -Lv > > > Cc: Len Brown <lenb@...nel.org> > > Cc: sfi-devel@...plefirmware.org > > Signed-off-by: Lv Zheng <lv.zheng@...el.com> > > --- > > include/linux/sfi_acpi.h | 3 --- > > 1 file changed, 3 deletions(-) > > > > diff --git a/include/linux/sfi_acpi.h b/include/linux/sfi_acpi.h > > index 4723bbf..a6e555c 100644 > > --- a/include/linux/sfi_acpi.h > > +++ b/include/linux/sfi_acpi.h > > @@ -63,8 +63,6 @@ > > #include <linux/sfi.h> > > > > #ifdef CONFIG_SFI > > -#include <acpi/acpi.h> /* FIXME: inclusion should be removed */ > > - > > extern int sfi_acpi_table_parse(char *signature, char *oem_id, > > char *oem_table_id, > > int (*handler)(struct acpi_table_header *)); > > @@ -78,7 +76,6 @@ static inline int __init acpi_sfi_table_parse(char *signature, > > return sfi_acpi_table_parse(signature, NULL, NULL, handler); > > } > > #else /* !CONFIG_SFI */ > > - > > static inline int sfi_acpi_table_parse(char *signature, char *oem_id, > > char *oem_table_id, > > int (*handler)(struct acpi_table_header *)) > > > > -- > I speak only for myself. > Rafael J. Wysocki, Intel Open Source Technology Center.
Powered by blists - more mailing lists