On Thu, 13 Jun 2013 13:47:37 -0500 Malahal Naineni <malahal@xxxxxxxxxx> wrote: > Hi Trond, > > I saw Bryan's patches here https://patchwork.kernel.org/patch/987402/ > that fix issues after loss of a lock. What is the status on this patch > set? Do they need more work? We have an application that uses range > locks on a file. Two threads from two different clients end up writing > to the same a file due to this bug after a lease expiry from a client. > > Regards, Malahal. (cc'ing Bryan since he did the original set) Yeah, this set would be a nice thing to have. A couple of comments: - I still think it would be best to make SIGLOST its own signal, but as Bryan points out, it would need to be larger than SIGRTMAX. I'm not sure that's possible on all arches with the way the RT signals were done. It's probably worth investigating that though before settling on SIGIO since it would be hard to change that retroactively. - This is not really a v4.1 specific thing. It should also be done for v4.0 and v2/3, though the latter two really need to be done within lockd. -- Jeff Layton <jlayton@xxxxxxxxxx> -- 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
