]> git.michaelhowe.org Git - packages/o/openafs.git/commit
Linux: Fix USE_UCONTEXT detection
authorSimon Wilkinson <sxw@your-file-system.com>
Fri, 1 Apr 2011 15:37:32 +0000 (16:37 +0100)
committerDerrick Brashear <shadow@dementix.org>
Fri, 16 Dec 2011 10:57:33 +0000 (02:57 -0800)
commit3262f111028093ed31da3fe9e3ed7efcc0e8b769
treec24418f080d80a6140d74d925ff5328470a37a73
parent8c74ddc6c7658acdc827069a64e9b7a05cfe1bf5
Linux: Fix USE_UCONTEXT detection

On Linux, afs/param.h determines whether to use the ucontext()
function based on the glibc version. However, the glibc version
macros aren't available until a C library header has been included.

There have been a couple of attempts to fix this. The first included
afs_sysnames.h before the C library check, but this wasn't sufficient
to pull in a header for all builds. The second included stdio.h before
including afs/param.h in process.c.

Whilst the second approach works, it has the drawback that it breaks
our convention that afsconfig.h and afs/param.h must always be the first
includes in a file. More seriously, it means that the behaviour of
afs/param.h is different depending on where it appears in the include
order - which is especially dangerous for 3rd party user.

So, this patch tries to resolve all of this by explicitly including
features.h before doing the glibc version checks. The only danger I
can see here is that Linux platforms without glibc may lack a
features.h, but I suspect we have trouble building on such platforms
in any case.

Reviewed-on: http://gerrit.openafs.org/4389
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Derrick Brashear <shadow@dementia.org>
(cherry picked from commit f5f0ce525a63264098cd0625769d642fe6f106ba)

Change-Id: Ia94487f35ad0fc3143faafd2fa12a6c71d40f116
Reviewed-on: http://gerrit.openafs.org/6278
Tested-by: Derrick Brashear <shadow@dementix.org>
Reviewed-by: Derrick Brashear <shadow@dementix.org>
src/config/param.linux24.h
src/config/param.linux26.h
src/lwp/process.c