On Wed, Sep 14, 2016 at 08:56:57AM +0800, Ian Kent wrote: [snip] > > > > Hi, Ian, > > > > > > > > Just wanted to check on the status of this fix. Is this still the > > > > approach you wanted to take/is there anything else you wanted to do with > > > > this? > > > > > > The problem is that someone tested this back ported to an older kernel and > > > claimed it caused file system corruption. > > > > > > That leaves me in a bad place indeed. > > > > > > Ian > > > > Hi, Ian, > > > > Dredging this up again because I forgot to reply in a timely manner last > > time. Do you have more details on that report? I'm having a hard time > > seeing how this change would cause filesystem corruption, and I still > > think a fix for this really needs to go in. > > You and me both. > > I recently re-factored the patch a bit and I'm thinking the best thing to do is > to send it to Andrew Morton so it can get plenty of testing before being > considered for mainline. > > I ran the kernel with the patches for several days without problem but didn't do > a lot with autofs during that time. > > I'll also have another look at it based on a comment from All Viro but I > couldn't see anything wrong with it myself, perhaps he will comment further when > I send it over to Andrew. > > Ian Awesome, please Cc me on the patch and I'll take it for a spin on some of our servers. -- Omar -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
