OpenAFS Master Repository branch, master, updated. openafs-devel-1_5_76-4322-g2e2bef2
Gerrit Code Review
Wed, 13 Aug 2014 10:02:03 -0400
The following commit has been merged in the master branch:
Author: Mark Vitale <firstname.lastname@example.org>
Date: Fri Jun 6 19:27:04 2014 -0400
opr: opr_AssertionFailed undefined in kernel module
The opr_Assert in opr_rbtree_remove is incompletely defined;
the opr_Assert macro is defined in opr.h, but the definition
for the opr_AssertionFailed routine it invokes is not included.
This allows the kernel module to build successfully even though
it retains a hidden undefined reference for opr_AssertionFailed.
However, the logic in obr_rbtree_remove ensures that this
particular opr_Assert can never fail - it is superfluous.
Some compilers (e.g. gcc for Linux AFS kernel module
builds) are able to recognize this and optimize it out. Others
(e.g. Solaris 5.12) do not, and when this happens the OpenAFS
build appears to succeed but the kernel module will fail to load
due to the undefined symbol.
Remove the superfluous opr_Assert.
This partially reverts commit 9f8b765bbdbb8913fcadbde8d3362039e9dc8e61.
Tested-by: BuildBot <email@example.com>
Reviewed-by: Benjamin Kaduk <firstname.lastname@example.org>
Reviewed-by: Perry Ruiter <email@example.com>
Reviewed-by: Jeffrey Altman <firstname.lastname@example.org>
src/opr/rbtree.c | 5 +----
1 files changed, 1 insertions(+), 4 deletions(-)
OpenAFS Master Repository