Re: malicious filesystems (was Re: [linux-pm] Re: [PATCH] Remove process freezer from suspend to RAM pathway)

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

> > > > > We can just wait for all fuse requests to be serviced before

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.


Sorry, didn't mean to offend you.


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.


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.


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?


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
-
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH] Remove process freezer from suspend to RAM pathway, Matthew Garrett, (Mon Jul 2, 9:29 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Mon Jul 2, 10:48 pm)
Re: [linux-pm] [PATCH] Remove process freezer from suspend ..., Benjamin Herrenschmidt, (Mon Jul 2, 10:49 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Mon Jul 2, 10:51 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Tue Jul 3, 12:19 am)
Re: [linux-pm] Re: [PATCH] Remove process freezer from sus ..., Benjamin Herrenschmidt, (Tue Jul 3, 4:40 am)
Re: [linux-pm] Re: [PATCH] Remove process freezer from sus ..., Benjamin Herrenschmidt, (Tue Jul 3, 4:44 am)
Re: [linux-pm] Re: [PATCH] Remove process freezer from sus ..., Benjamin Herrenschmidt, (Tue Jul 3, 4:45 am)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Tue Jul 3, 2:14 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Tue Jul 3, 2:16 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Tue Jul 3, 2:20 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Tue Jul 3, 2:35 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Tue Jul 3, 2:35 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Tue Jul 3, 2:56 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Tue Jul 3, 4:08 pm)
The big suspend mess, Adrian Bunk, (Wed Jul 4, 3:19 pm)
Re: The big suspend mess, Pavel Machek, (Wed Jul 4, 5:27 pm)
Re: The big suspend mess, Paul Mackerras, (Wed Jul 4, 5:53 pm)
Re: The big suspend mess, Adrian Bunk, (Wed Jul 4, 6:22 pm)
Re: The big suspend mess, Pavel Machek, (Thu Jul 5, 2:32 am)
Re: The big suspend mess, Gabriel C, (Thu Jul 5, 3:29 am)
Re: The big suspend mess, Rafael J. Wysocki, (Thu Jul 5, 5:18 am)
Re: [linux-pm] The big suspend mess, Alan Stern, (Thu Jul 5, 7:14 am)
Re: [linux-pm] Re: [PATCH] Remove process freezer from sus ..., Jeremy Maitin-Shepard, (Thu Jul 5, 9:06 am)
Re: [linux-pm] Re: [PATCH] Remove process freezer from sus ..., Benjamin Herrenschmidt, (Thu Jul 5, 3:38 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Thu Jul 5, 3:46 pm)
Re: [linux-pm] Re: [PATCH] Remove process freezer from sus ..., Benjamin Herrenschmidt, (Thu Jul 5, 3:59 pm)
Re: [linux-pm] Re: [PATCH] Remove process freezer from sus ..., Benjamin Herrenschmidt, (Thu Jul 5, 4:05 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Thu Jul 5, 4:20 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Thu Jul 5, 8:54 pm)
Re: [linux-pm] Re: [PATCH] Remove process freezer from sus ..., Jeremy Maitin-Shepard, (Thu Jul 5, 8:59 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Thu Jul 5, 8:59 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Thu Jul 5, 9:41 pm)
Re: [linux-pm] Re: [PATCH] Remove process freezer from sus ..., Rafael J. Wysocki, (Fri Jul 6, 12:04 am)
Re: [linux-pm] Re: [PATCH] Remove process freezer from sus ..., Rafael J. Wysocki, (Fri Jul 6, 12:13 am)
Re: [linux-pm] Re: [PATCH] Remove process freezer from sus ..., Rafael J. Wysocki, (Fri Jul 6, 12:16 am)
Re: [linux-pm] Re: [PATCH] Remove process freezer from sus ..., Rafael J. Wysocki, (Fri Jul 6, 12:20 am)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Rafael J. Wysocki, (Fri Jul 6, 12:35 am)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Rafael J. Wysocki, (Fri Jul 6, 12:40 am)
Re: [linux-pm] Re: [PATCH] Remove process freezer from sus ..., Benjamin Herrenschmidt, (Fri Jul 6, 1:59 am)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Fri Jul 6, 2:03 am)
Re: [linux-pm] Re: [PATCH] Remove process freezer from sus ..., Benjamin Herrenschmidt, (Fri Jul 6, 7:44 pm)
Re: [linux-pm] Re: [PATCH] Remove process freezer from sus ..., Benjamin Herrenschmidt, (Fri Jul 6, 7:46 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Fri Jul 6, 8:44 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Fri Jul 6, 9:06 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Sat Jul 7, 5:40 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Sat Jul 7, 5:42 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Sat Jul 7, 5:48 pm)
Re: [linux-pm] Re: [PATCH] Remove process freezer from sus ..., Benjamin Herrenschmidt, (Sat Jul 7, 5:50 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Sat Jul 7, 9:39 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Sat Jul 7, 10:14 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Sat Jul 7, 10:19 pm)
Re: malicious filesystems (was Re: [linux-pm] Re: [PATCH] ..., Rafael J. Wysocki, (Sun Jul 8, 7:06 am)
Re: malicious filesystems (was Re: [linux-pm] Re: [PATCH] ..., Miklos Szeredi, (Sun Jul 8, 7:23 am)
Re: malicious filesystems (was Re: [linux-pm] Re: [PATCH] ..., Rafael J. Wysocki, (Sun Jul 8, 11:08 am)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Rafael J. Wysocki, (Sun Jul 8, 11:20 am)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Rafael J. Wysocki, (Sun Jul 8, 11:26 am)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Rafael J. Wysocki, (Sun Jul 8, 11:46 am)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Rafael J. Wysocki, (Sun Jul 8, 12:15 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Sun Jul 8, 2:03 pm)
Re: malicious filesystems (was Re: [linux-pm] Re: [PATCH] ..., Rafael J. Wysocki, (Sun Jul 8, 2:07 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Sun Jul 8, 2:20 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Sun Jul 8, 2:21 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Sun Jul 8, 2:54 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Sun Jul 8, 5:33 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Sun Jul 8, 6:32 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Mon Jul 9, 2:13 am)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Mon Jul 9, 2:14 am)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Mon Jul 9, 2:33 am)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Mon Jul 9, 3:05 am)
Re: hibernation/snapshot design, Jeremy Maitin-Shepard, (Mon Jul 9, 8:23 am)
Re: sysrq-t dumps of s2ram/fuse deadlock, Jeremy Maitin-Shepard, (Wed Jul 11, 6:45 am)