On 08/26/2011 02:54 PM, J. Bruce Fields wrote: >> You might get by, by rechecking the grace period at the end of the >> processing and if passed issue a "reclaim request failed" anyway. >> So the first check is only for optimization but the final disposition >> is the post-check. (Just as if you dropped that refcount above) > > I guess that could work, but you'd have to back out the operation you > just did if the check showed you'd left the grace period. I'd rather > avoid that. > Yes, you'll need to "back out" of the operation. It was just a suggestion. You'll have to see what is easier to implement. The above is less invasive and does have merits. It is a bit like the RCU_FREE pattern when at the end of the operation you see that you lost the race and needs to "back out". But in the hot path it is very cheap, (No reference counts no locking). And only in the very very rare event, Those that today we fail on, do you do the extra "backing out". So overall runtime and coding wise it might be the cheapest solution. Just my $0.017 > --b. Boaz -- 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