On Sun, May 11, 2014 at 10:33:17PM -0700, Christoph Hellwig wrote: > > + if (op->status == nfserr_resource && nfsd4_has_session(&resp->cstate)) { > > + struct nfsd4_slot *slot = resp->cstate.slot; > > + > > + if (slot->sl_flags & NFSD4_SLOT_CACHETHIS) > > + op->status = nfserr_rep_too_big_to_cache; > > + else > > + op->status = nfserr_rep_too_big; > > There is a closing brace missing here, which breaks the compile for me. Aie, sorry, I guess I didn't test anything after moving that patch back in the series. I normally do a quick test-compile of whatever a patch touched while rebasing, then run a set of regression tests before posting. But in this case I think I finished the rebase in a hurry Friday, then posted Sunday without remembering what exactly I'd last done.... Anyway, fixed, new tests are running--thanks for the review. --b. -- 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