On Thu, Jul 10, 2014 at 07:32:14AM -0400, Jeff Layton wrote: > So, you're correct that we never take an fi_access reference for O_RDWR, > which is why the array doesn't have a slot for it: > > atomic_t fi_access[2]; > > ...it's tracked by the union of the O_RDONLY and O_WRONLY counters. Which will be racy as long we try to use an atomic_t and not a proper lock over access to all of fi_access. I'm still trying to understand why we even need fi_access and can't just use the file references directly, though. -- 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