Commit 0eb3bf33 authored by Michael Chang's avatar Michael Chang Committed by Colin Watson

Fix packed-not-aligned error on GCC 8

When building with GCC 8, there are several errors regarding packed-not-aligned.

./include/grub/gpt_partition.h:79:1: error: alignment 1 of ‘struct grub_gpt_partentry’ is less than 8 [-Werror=packed-not-aligned]

This patch fixes the build error by cleaning up the ambiguity of placing
aligned structure in a packed one. In "struct grub_btrfs_time" and "struct
grub_gpt_part_type", the aligned attribute seems to be superfluous, and also
has to be packed, to ensure the structure is bit-to-bit mapped to the format
laid on disk. I think we could blame to copy and paste error here for the
mistake. In "struct efi_variable", we have to use grub_efi_packed_guid_t, as
the name suggests. :)
Signed-off-by: 's avatarMichael Chang <>
Tested-by: 's avatarMichael Chang <>
Tested-by: 's avatarPaul Menzel <>
Reviewed-by: 's avatarDaniel Kiper <>

Origin: upstream,
Last-Update: 2018-10-07

Patch-Name: gcc-8-packed-not-aligned.patch
parent b514a78a
......@@ -175,7 +175,7 @@ struct grub_btrfs_time
grub_int64_t sec;
grub_uint32_t nanosec;
} __attribute__ ((aligned (4)));
struct grub_btrfs_inode
......@@ -29,7 +29,7 @@ struct grub_efiemu_ptv_rel
struct efi_variable
grub_efi_guid_t guid;
grub_efi_packed_guid_t guid;
grub_uint32_t namelen;
grub_uint32_t size;
grub_efi_uint32_t attributes;
......@@ -28,7 +28,7 @@ struct grub_gpt_part_type
grub_uint16_t data2;
grub_uint16_t data3;
grub_uint8_t data4[8];
} __attribute__ ((aligned(8)));
typedef struct grub_gpt_part_type grub_gpt_part_type_t;
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment