On 07/07/2014 03:31 PM, J. Bruce Fields wrote: > On Mon, Jul 07, 2014 at 03:14:50PM -0400, Jeff Layton wrote: >> Sigh...after staring at this patch all day, I think I now see a >> potential problem with the fault injection piece. >> >> In the "recall" case, we may find a delegation on the cl_delegations >> list that has already been recalled. If that's the case, then we >> probably should just skip it. >> >> I'll fix this patch, retest and resend. Sorry for the noise... > > OK, so sounds like the fix isn't a big deal, but it also sounds like the > fault injection code has come up as needing special attention a few > times during this work now. > > Has anyone actually ended up using that code? Anna? I used that code more often when I first wrote it, but I haven't played around with anything in at least a year. Anna > > In retrospect I wonder if it's worth the trouble. It's very > special-purpose stuff and if nobody's using it then maybe we could just > remove it.... > > --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
