NSM: Avoid return code checking in NSM XDR encoder functions
authorChuck Lever <chuck.lever@oracle.com>
Tue, 14 Dec 2010 14:58:30 +0000 (14:58 +0000)
committerTrond Myklebust <Trond.Myklebust@netapp.com>
Thu, 16 Dec 2010 17:37:24 +0000 (12:37 -0500)
commit49b170047f4a9fe1483132e14a11bdf493bdb8af
tree626d5a764350c6a0f57c7566df86e26aede2cdd3
parentead00597882c4ee3c534d6880cc3bcb4d412cc4b
NSM: Avoid return code checking in NSM XDR encoder functions

Clean up.

The trend in the other XDR encoder functions is to BUG() when encoding
problems occur, since a problem here is always due to a local coding
error.  Then, instead of a status, zero is unconditionally returned.

Update the NSM XDR encoders to behave this way.

To finish the update, use the new-style be32_to_cpup() and
cpu_to_be32() macros, and compute the buffer sizes using raw integers
instead of sizeof().  This matches the conventions used in other XDR
functions

Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Tested-by: J. Bruce Fields <bfields@redhat.com>
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
fs/lockd/mon.c