On Apr 20, 2012, at 5:58 PM, Myklebust, Trond wrote: > On Fri, 2012-04-20 at 21:48 +0000, Adamson, Dros wrote: >> On Apr 20, 2012, at 5:43 PM, Myklebust, Trond wrote: >> >>> On Fri, 2012-04-20 at 21:38 +0000, Adamson, Dros wrote: >>> >>>> Yes, something like that could work, but we still have the same issue - dentry->d_fsdata never has [] escaping. >>> >>> That's a bug too... >> >> Right! This is what I've been talking about the whole time - I guess I didn't do a good job explaining it :) The original bug (splitting return of nfs_path()) would be trivial to fix if d_fsdata had [] escaping. >> >> So I guess that answers my earlier question - we *do* want to change dev_name to include [] escaping. >> >> -dros > > Here is a patch from Jan Kara that does just this. I'll start by > applying that... Cool! dprintk debugging just led me there! Why wasn't this patch applied earlier? Just curious. I'll post a new patch to fix splitting nfs_path(). -dros
Attachment:
smime.p7s
Description: S/MIME cryptographic signature