[lkml]   [2008]   [Oct]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [linux-pm] Freezer: Don't count threads waiting for frozen filesystems.
    On Mon, 27 Oct 2008, Nigel Cunningham wrote:
    > Hi.
    > On Mon, 2008-10-27 at 12:37 +0100, Rafael J. Wysocki wrote:
    > > Well, I guess it's better if you post the entire thing so that we can see
    > > what the role of the $subject patch is in it, even if this patch finally gets
    > > merged separately.
    > Ah.. that makes me see how vfs_check_frozen was getting triggered...
    > (fs/namei.c, below).

    Nigel, thanks for the patch, it makes thinks a lot clearer.

    From a quick look through the patch it seems to solve a bunch of cases
    where new fuse requests during the freezing could cause problems. But
    it doesn't do anything with requests that are already under way when
    the freezing starts, which would still result in all the same

    Take this scenario:

    1) process A does rename("/mnt/fuse/a", "/mnt/fuse/b")
    2) request goes to process B serving the fuse filesystem
    3) filesystems are frozen, no new fuse requests
    4) processes are frozen, let's say B first, then A
    5) freezing A will fail, since it's still waiting for the request to finish

    Several solutions have been posted, none of which really solve the problem:

    a) Let's tag fuse server processes and freeze them later. This is
    basically impossible, because many processes could be interoperating
    and there's no way to know which is depending on which (example:
    sshfs uses ssh for communication, which possibly relies on openvpn
    process for packet transmission).

    b) While waiting for replies to fuse request, allow process to
    freeze. Does not fully solve the problem, as VFS might be holding
    locks, and other processes waiting for those locks will not be


     \ /
      Last update: 2008-10-27 13:41    [W:0.020 / U:61.580 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site