]> git.michaelhowe.org Git - packages/o/openafs.git/commit
vol: Move VOL_CV_TIMEDWAIT to volume_inline.h
authorAndrew Deason <adeason@sinenomine.net>
Tue, 23 Nov 2010 03:23:02 +0000 (22:23 -0500)
committerDerrick Brashear <shadow@dementix.org>
Fri, 16 Dec 2011 04:32:33 +0000 (20:32 -0800)
commit1a86d857adbf9b517c58a2770f93f7f1767f5e01
tree6795ce40419b3c8f507174bf566379b960214409
parentcd69fb1401bc7d9803fb26c7b5e9256efa0167d4
vol: Move VOL_CV_TIMEDWAIT to volume_inline.h

Some platforms will give errors for the vol package glock not being
defined in VOL_CV_TIMEDWAIT, since some utilities include volume.h
(with pthreaded ubik enabled) but do not have the vol glock. This
isn't a problem for the other CV_WAIT (et al) macros, since, being
macros, they are not expanded if never referenced. Since
VOL_CV_TIMEDWAIT is a static inline function, however, and at least
some platforms try to compile/link it even if it's not referenced.

So, move it to volume_inline.h, which is only used by the volume
package itself, where we will have the vol glock.

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

Change-Id: Ic1d6286bb148f202b8da010edf48fff93a5a48fb
Reviewed-on: http://gerrit.openafs.org/6268
Reviewed-by: Derrick Brashear <shadow@dementix.org>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
src/vol/volume.h
src/vol/volume_inline.h