]> git.ipfire.org Git - thirdparty/systemd.git/commitdiff
src/boot/efi/shim: elide __attribute__((sysv_abi)) on non-intel archs
authorDan Streetman <ddstreet@canonical.com>
Tue, 13 Aug 2019 10:45:04 +0000 (06:45 -0400)
committerDan Streetman <ddstreet@canonical.com>
Thu, 15 Aug 2019 20:36:10 +0000 (16:36 -0400)
This attribute is x86-only, so when building on non-intel archs it
generates a compiler warning.  When building with -Werror this turns
into an error, so only include the attribute on intel archs.

src/boot/efi/shim.c

index 9e072d294f9764dfa2e5bc7b03fb8fae0099b417..8db27547cca3f24ee88291d95a0fd689b4aae8b3 100644 (file)
 #include "util.h"
 #include "shim.h"
 
+#if defined(__x86_64__) || defined(__i386__)
+#define __sysv_abi__ __attribute__((sysv_abi))
+#else
+#define __sysv_abi__
+#endif
+
 struct ShimLock {
-        EFI_STATUS __attribute__((sysv_abi)) (*shim_verify) (VOID *buffer, UINT32 size);
+        EFI_STATUS __sysv_abi__ (*shim_verify) (VOID *buffer, UINT32 size);
 
         /* context is actually a struct for the PE header, but it isn't needed so void is sufficient just do define the interface
          * see shim.c/shim.h and PeHeader.h in the github shim repo */
-        EFI_STATUS __attribute__((sysv_abi)) (*generate_hash) (VOID *data, UINT32 datasize, VOID *context, UINT8 *sha256hash, UINT8 *sha1hash);
+        EFI_STATUS __sysv_abi__ (*generate_hash) (VOID *data, UINT32 datasize, VOID *context, UINT8 *sha256hash, UINT8 *sha1hash);
 
-        EFI_STATUS __attribute__((sysv_abi)) (*read_header) (VOID *data, UINT32 datasize, VOID *context);
+        EFI_STATUS __sysv_abi__ (*read_header) (VOID *data, UINT32 datasize, VOID *context);
 };
 
 static const EFI_GUID simple_fs_guid = SIMPLE_FILE_SYSTEM_PROTOCOL;