diff options
| author | Brian Behlendorf <[email protected]> | 2022-03-23 08:51:00 -0700 |
|---|---|---|
| committer | GitHub <[email protected]> | 2022-03-23 08:51:00 -0700 |
| commit | 460748d4aeb1ed3c78efdd562fcf3d2eaa9ff536 (patch) | |
| tree | 4306b254cee276ba810e507173a0a75d42b9a311 /include/sys/zfs_context.h | |
| parent | b73505c7e056417cba73dcf51b49062c84fd2b59 (diff) | |
Switch from _Noreturn to __attribute__((noreturn))
Parts of the Linux kernel build system struggle with _Noreturn. This
results in the following warnings when building on RHEL 8.5, and likely
other environments. Switch to using the __attribute__((noreturn)).
warning: objtool: dbuf_free_range()+0x2b8:
return with modified stack frame
warning: objtool: dbuf_free_range()+0x0:
stack state mismatch: cfa1=7+40 cfa2=7+8
...
WARNING: EXPORT symbol "arc_buf_size" [zfs.ko] version generation
failed, symbol will not be versioned.
WARNING: EXPORT symbol "spa_open" [zfs.ko] version generation
failed, symbol will not be versioned.
...
Additionally, __thread_exit() has been renamed spl_thread_exit() and
made a static inline function. This was needed because the kernel
will generate a warning for symbols which are __attribute__((noreturn))
and then exported with EXPORT_SYMBOL.
While we could continue to use _Noreturn in user space I've also
switched it to __attribute__((noreturn)) purely for consistency
throughout the code base.
Reviewed-by: Ryan Moeller <[email protected]>
Reviewed-by: Brian Atkinson <[email protected]>
Signed-off-by: Brian Behlendorf <[email protected]>
Closes #13238
Diffstat (limited to 'include/sys/zfs_context.h')
| -rw-r--r-- | include/sys/zfs_context.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/include/sys/zfs_context.h b/include/sys/zfs_context.h index 61a6365f5..944acdcfb 100644 --- a/include/sys/zfs_context.h +++ b/include/sys/zfs_context.h @@ -152,8 +152,8 @@ extern void dprintf_setup(int *argc, char **argv); extern void cmn_err(int, const char *, ...); extern void vcmn_err(int, const char *, va_list); -extern _Noreturn void panic(const char *, ...); -extern _Noreturn void vpanic(const char *, va_list); +extern __attribute__((noreturn)) void panic(const char *, ...); +extern __attribute__((noreturn)) void vpanic(const char *, va_list); #define fm_panic panic |