[PATCH] efi: remove UEFIv2.5 properties table from /sys/firmware/efi/systab

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



As pointed out by Dave Young, /sys/firmware/efi/systab violates the
sysfs one-value-per-file policy already, so adding more data items
to it should be avoided. So let's remove the PROP= line that lists
the address of the recently introduced Properties Table before it
appears in a release.

Signed-off-by: Ard Biesheuvel <ard.biesheuvel@xxxxxxxxxx>
---

Matt pointed out that there still needs to be a way for kexec/x86 to
access this data, since it needs to pass it on to the kexec kernel.
That is way out of my area of expertise, so I will leave it to you
to come up with something suitable there.

 drivers/firmware/efi/efi.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/drivers/firmware/efi/efi.c b/drivers/firmware/efi/efi.c
index 2eb31e28a35c..3804ecd4a77a 100644
--- a/drivers/firmware/efi/efi.c
+++ b/drivers/firmware/efi/efi.c
@@ -109,8 +109,6 @@ static ssize_t systab_show(struct kobject *kobj,
 		str += sprintf(str, "BOOTINFO=0x%lx\n", efi.boot_info);
 	if (efi.uga != EFI_INVALID_TABLE_ADDR)
 		str += sprintf(str, "UGA=0x%lx\n", efi.uga);
-	if (efi.properties_table != EFI_INVALID_TABLE_ADDR)
-		str += sprintf(str, "PROP=0x%lx\n", efi.properties_table);
 
 	return str - buf;
 }
-- 
1.9.1

--
To unsubscribe from this list: send the line "unsubscribe linux-efi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux