On 2011-07-27 15:33, Trond Myklebust wrote: > On Wed, 2011-07-27 at 14:40 -0400, Jim Rees wrote: >> From: Andy Adamson <andros@xxxxxxxxxx> ... >> @@ -6902,6 +7029,7 @@ struct rpc_procinfo nfs4_procedures[] = { >> PROC(GET_LEASE_TIME, enc_get_lease_time, dec_get_lease_time), >> PROC(RECLAIM_COMPLETE, enc_reclaim_complete, dec_reclaim_complete), >> PROC(GETDEVICEINFO, enc_getdeviceinfo, dec_getdeviceinfo), >> + PROC(GETDEVICELIST, enc_getdevicelist, dec_getdevicelist), > > This needs to go at the end of the list of procedures in order to work > correctly with nfsstat. > FWIW, what matters is the value of NFSPROC4_CLNT_GETDEVICELIST rather than the order of instantiation of the PROC() macro. It is defined in end of the respective enum in include/linux/nfs4.h. Benny >> PROC(LAYOUTGET, enc_layoutget, dec_layoutget), >> PROC(LAYOUTCOMMIT, enc_layoutcommit, dec_layoutcommit), >> PROC(LAYOUTRETURN, enc_layoutreturn, dec_layoutreturn), > > -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html