> > > > > We can just wait for all fuse requests to be serviced before > > > > > proceeding further with freeze, right? > > > > > > > > Right. Nice way to slow down or stop the suspend with an unprivileged > > > > process. Avoiding that sort of DoS is one of the design goals of > > > > fuse. > > > > > > So you want me to handle _malicious_ filesystems now? > > > > What I'd like, is a suspend, that works reliably, regardless of the > > state of any userspace filesystem, network servers and such. > > Well, fix userspace filesystems and maybe NFS. If they react to > sigstop in timely manner, they will work with suspend properly, too. Which is pretty much impossible, given the unix filesystem API. To be able to react to sigstop, the operations in question need to be restartable. Which they are not, so they can't react to sigstop. End of story. > > > That should be easy... :-). You already have nasty deadlocks in FUSE, > > > and you solve them by "root can echo 1 > abort"... so allow me the > > > same possibility. > > > > > > We can tell fused we are freezing, and if all the requests are not > > > serviced within, say, 30 seconds, we call the filesystem malicious and > > > do echo 1 > abort. > > > > Arbitrary time limits, nice. Not. > > > > This freezer is like an old house that's close to collapsing, and you > > Nice way to have useful discussion. Not. Sorry, didn't mean to offend you. > Look, Linux was not designed with malicious filesystems in mind. In > particular, suspend was not designed with malicious filesystems in > mind, and VFS was not designed with malicious filesystems in mind. > That's why you have the nastiness with deadlocks... which you are > unable/unwilling to solve because you'd have to redesign VFS and meet > Al Viro. No, I don't think we need to redesign the VFS, and also I think the malicious filesystem thing has been adequately taken care of in fuse. You may not like the fact that one process can cause another to go into uninterruptible sleep, but in fact there's nothing wrong with that. The effects are localized to the mount owner, it won't cause any system-wide ill, it's in fact no worse than a malicious process doing ptrace(). And as explained above it's unavoidable due to the well established userspace API. The _point_ of fuse is to let this API be used by unmodified programs and the filesystem be provided by a userspace process. After showing me the right way to do this with Podfuk, this should not come as a surprise to you. So the fact that the freezer can't handle this is unfortunate, but it's just a symptom of the brokenness of it, not something that fuse introduced. Not being able to suspend with NFS (or other network filesystems) when the network is lost shows that this is a deeper problem. And for some reason you seem not to accept that. You think that the problem is with fuse, NFS, CIFS, whatever, and not the freezer, when in fact it's quite clear, that neither of the above should have anything to do with power management. Even if it was possible to fix them, it would still be just fixing the symptoms. > Now.. freezer already includes timeout; if some part of kernel knows > nothing about suspend or had crashed, it will abort after some time, > telling you which part to blame. Another timeout for detecting > malicious userspace filesystems will not hurt much in this old house. > (Maybe you don't want to auto abort them, just tell user what happened. > > We are stuck with refrigerator for now, and at least for hibernation, > I don't see any feasible alternative. Even for hibernation, I don't see, why we would need all processes being effectively in a stopped state. As stated otherwise in the thread, suspend2 in fact allowed processes to be in uninterruptible sleep instead, without negative side effects. The current freezer does too much for suspend and for hibernate too. > > Malicious programs are not something specific to fuse. A lot of the > > multiuser/multitasking OS design is about isolating things, so such a > > program is limited in the damage it can do. > > I'm talking malicious _filesystems_ here, and yes, fuse is first of > this kind. We want to handle unresponding NFS, but I believe handling > malicious NFS server nicely is slightly out of scope. It doesn't have to be malicious, it's enough if the server crashes, or the network connection is lost. Do we want to maintain the status quo, just because we can live with it? > > > Not nice, but we don't know any better for now. "Just fix all the > > > drivers" basically means "just fix 90% of kernel". > > > > And how much of that 90% currently has any power management? > > Anything that's used in today's notebooks, I'd say... Which is what, 10% of all the drivers? Then it really not as bad as you try to make it sound. And with the late suspend call (whatever it does) that can take care of most of those, it really becomes just a few drivers and subsystems to fix. Miklos _______________________________________________ linux-pm mailing list linux-pm@xxxxxxxxxxxxxxxxxxxxxxxxxx https://lists.linux-foundation.org/mailman/listinfo/linux-pm
