2007-12-20 20:04:00[permalink] [raw]
Here's a set of patches that remove lockd's dependence on the rpc_xprt.prot
field in nlmclnt_proc().
There are a few reasons to remove this dependence, including:
1. The NFS client is now fully in control of selecting which transport
protocol is chosen for the sideband NLM protocol (use UDP only if NFS
over UDP is specified; otherwise TCP is used).
2. Looking at fields in the rpc_xprt structure in an upper-level protocol
implementation is a layering violation.
3. The rpc_xprt.prot field is overloaded -- it's used to control RPC binding
as well as which transport protocol is used for sideband protocols.
4. Removing the nlmclnt_lookup_host() call from nlmclnt_proc() eliminates
work that is unnecessarily repeated for each NLM request.
This patch series should replace the four I sent you last week for review.
Please consider them for 2.6.25.
corporate: <chuck dot lever at oracle dot com>