[BUG] Argument-alignment build error with clang

From: Paul E. McKenney
Date: Wed Oct 23 2024 - 18:27:07 EST


Hello!

Running rcutorture on next-20241023 got me lots of these:

drivers/acpi/prmt.c:156:29: error: passing 1-byte aligned argument to 4-byte aligned parameter 1 of 'efi_pa_va_lookup' may result in an unaligned pointer access [-Werror,-Walign-mismatch]
156 | (void *)efi_pa_va_lookup(&th->guid, handler_info->handler_address);

This is built with CC=clang. I don't see this diagnostic with GCC.
But we are supposed to be able to build with clang, so...

The first argument is the address of one of these:

typedef struct {
__u8 b[UUID_SIZE];
} guid_t;

Where UUID_SIZE is as follows:

#define UUID_SIZE 16

But this guid_t is a member of one of these:

struct prm_handler_info {
guid_t guid;
efi_status_t (__efiapi *handler_addr)(u64, void *);
u64 static_data_buffer_addr;
u64 acpi_param_buffer_addr;

struct list_head handler_list;
};

One can argue that this structure must be 16-bit aligned on a
64-bit build. So maybe this is a bug in clang's diagnostics, hence
linux-toolchains on CC.

Thoughts?

Thanx, Paul