Messages in this thread | | | Date | Sat, 18 Oct 2014 11:38:28 -0700 | Subject | Re: [PATCH 0/5] fuse: handle release synchronously (v4) | From | Linus Torvalds <> |
| |
On Sat, Oct 18, 2014 at 11:01 AM, Miklos Szeredi <miklos@szeredi.hu> wrote: > > And what you don't get is that there's a deep difference between those > and the /proc file access case.
No there isn't.
Your "action by the holder" argument is pure and utter garbage, for a very simple and core reason: the *filesystem* doesn't know or care. So from a fuse standpoint, the difference is totally and entirely irrelevant.
Your "synchronous fput" fails. It fails totally regardless of that "who incremented the file user count" issue. Face it, your patch is broken. And it's *fundamentally* broken, which is why I'm so tired of your stupid ad-hoc hacks that cannot possibly work.
Your umount EBUSY case is somewhat relevant to the "who holds the file open", but quite frankly, that's not a filesystem issue, it's more of a system management issue. So umount might get EBUSY. That's not something the filesystem should/could care about, that's a MIS issue that is entirely irrelevant, and the answer is "if somebody does lsof, that might keep the filesystem busy". Tough.
Linus
| |