CID#23120 Logically dead code
Maximum concurrent RPC requests is now hard coded.
To improve readability and avoid this issue, it's
better to directly assign the value to 16.
CID#18514 Dead code

The last instruction cannot be reached under any
circumstances. The same return status is reached
from the first else if instruction.
CID#18479 Argument cannot be negative

close doesn't accept a negative value for
fd. Moving this close instruction at the
end of the if statement is the correct
way to proceed.
CID#18192 Unsigned compared against 0

Here namesize was of type size_t and could store
a negative value. ssize_t is prefered in this case
since we want to catch a negative return value
treated as an error.
CID#18162 Unsigned compared against 0

Here namesize was of type size_t and could store
a negative value. ssize_t is prefered in this case
since we want to catch a negative return value
treated as an error.
CID#18482 Unsigned compared against 0

Here namesize was of type size_t and could store
a negative value. ssize_t is prefered in this case
since we want to catch a negative return value
treated as an error.
CID#18458 Missing unlock

When returning EINVAL, the handle mutex wasn't released.
This is to be more consistent with the normal case where
the lock is released. I don't think it's required to
destroy it though.
Added ignoring vim swap files
Forgot the wildcard to get rid of vim swap files
CID#18457 Missing unlock

The mutex hold by fsal_lock wasn't released at the end of
the function. Though it might not be truly necessary, we
unlock it anyway.
CID#18502 Dereference before NULL check

recall is dereferenced before checking whether it's NULL
or not.
CID#18449 Logically dead code

Setting rc cannot be reached under any circumstances.
recall memory allocation is checked at the beginning,
therefore it can't be NULL at this stage.
Code cleanup: glist function name change

To prepare for 2.0 RC, this patch renames init_glist to
glist_init. The function was named this way to avoid a
conflict when building with MySQL support. Since the
later has been dropped along with the POSIX FSAL, a
general rename is done.
Hpss build: fix install path, don't link MainNFSD/ganesha.nfsd against hpss, only set rpath for the fsal
9P uid2grp: use realloc properly
CMake: do not create "rpm" and "srpm" targets if rpmbuild is not installed.
RPM: better management of files related to XFS support.
CMake : USE_FSAL_SHOOK is to be turned OFF is USE_FSAL_LUSTRE is turned OFF
RPM: for the moment do not generate RPM for HPSS
Merge branch 'phdeniel_next_46' into merge_next

Stripped out the "mount 9p..." patch because all the good stuff
was taken earlier.  This is a remnant of an un-updated submodule

Signed-off-by: Jim Lieb <jlieb@panasas.com>