efivarfs: Remove unused internal struct members
authorTim Schumacher <timschumi@gmx.de>
Thu, 28 Mar 2024 20:50:32 +0000 (21:50 +0100)
committerArd Biesheuvel <ardb@kernel.org>
Sat, 13 Apr 2024 08:33:02 +0000 (10:33 +0200)
The structure was moved to the efivarfs internals in commit 2d82e6227ea1
("efi: vars: Move efivar caching layer into efivarfs") after previously
being used as the data ABI for efivars until its removal in commit
0f5b2c69a4cb ("efi: vars: Remove deprecated 'efivars' sysfs interface").

As efivarfs only uses the structure for the variable name caching layer,
the data-related members were never in use. Remove them to avoid
implying that efivarfs is bound by the same restrictions that efivars
once had. While at it, remove the packed attribute, since we no longer
have to guarantee a stable layout.

Signed-off-by: Tim Schumacher <timschumi@gmx.de>
Signed-off-by: Ard Biesheuvel <ardb@kernel.org>
fs/efivarfs/internal.h

index f7206158ee81385eeaab387fd16b05aea5a7634b..d71d2e08422f09250c282faeb4ab8e58b6abc401 100644 (file)
@@ -24,11 +24,8 @@ struct efivarfs_fs_info {
 struct efi_variable {
        efi_char16_t  VariableName[EFI_VAR_NAME_LEN/sizeof(efi_char16_t)];
        efi_guid_t    VendorGuid;
-       unsigned long DataSize;
-       __u8          Data[1024];
-       efi_status_t  Status;
        __u32         Attributes;
-} __attribute__((packed));
+};
 
 struct efivar_entry {
        struct efi_variable var;