• David Howells's avatar
    FS-Cache: Fix signal handling during waits · 9c04caa8
    David Howells authored
    wait_on_bit() with TASK_INTERRUPTIBLE returns 1 rather than a negative error
    code, so change what we check for.  This means that the signal handling in
    fscache_wait_for_retrieval_activation()  should now work properly.
    
    Without this, the following bug can be seen if CTRL-C is pressed during
    fscache read operation:
    
    FS-Cache: Assertion failed
    2 == 3 is false
    ------------[ cut here ]------------
    kernel BUG at fs/fscache/page.c:347!
    invalid opcode: 0000 [#1] SMP
    Modules linked in: cachefiles(F) nfsv4(F) nfsv3(F) nfsv2(F) nfs(F) fscache(F) auth_rpcgss(F) nfs_acl(F) lockd(F) sunrpc(F)
    CPU 1
    Pid: 15006, comm: slurp-q Tainted: GF            3.7.0-rc8-fsdevel+ #411                  /DG965RY
    RIP: 0010:[<ffffffffa007fcb4>]  [<ffffffffa007fcb4>] fscache_wait_for_retrieval_activation+0x167/0x177 [fscache]
    RSP: 0018:ffff88002a4c39a8  EFLAGS: 00010292
    RAX: 000000000000001a RBX: ffff88002d3dc158 RCX: 0000000000008685
    RDX: ffffffff8102ccd6 RSI: 0000000000000001 RDI: ffffffff8102d1d6
    RBP: ffff88002a4c39c8 R08: 0000000000000002 R09: 0000000000000000
    R10: ffffffff8163afa0 R11: ffff88003bd11900 R12: ffffffffa00868c8
    R13: ffff880028306458 R14: ffff88002d3dc1b0 R15: ffff88001372e538
    FS:  00007f17426a0700(0000) GS:ffff88003bd00000(0000) knlGS:0000000000000000
    CS:  0010 DS: 0000 ES: 0000 CR0: 000000008005003b
    CR2: 00007f1742494a44 CR3: 0000000031bd7000 CR4: 00000000000007e0
    DR0: 0000000000000000 DR1: 0000000000000000 DR2: 0000000000000000
    DR3: 0000000000000000 DR6: 00000000ffff0ff0 DR7: 0000000000000400
    Process slurp-q (pid: 15006, threadinfo ffff88002a4c2000, task ffff880023de3040)
    Stack:
     ffff88002d3dc158 ffff88001372e538 ffff88002a4c3ab4 ffff8800283064e0
     ffff88002a4c3a38 ffffffffa0080f6d 0000000000000000 ffff880023de3040
     ffff88002a4c3ac8 ffffffff810ac8ae ffff880028306458 ffff88002a4c3bc8
    Call Trace:
     [<ffffffffa0080f6d>] __fscache_read_or_alloc_pages+0x24f/0x4bc [fscache]
     [<ffffffff810ac8ae>] ? __alloc_pages_nodemask+0x195/0x75c
     [<ffffffffa00aab0f>] __nfs_readpages_from_fscache+0x86/0x13d [nfs]
     [<ffffffffa00a5fe0>] nfs_readpages+0x186/0x1bd [nfs]
     [<ffffffff810d23c8>] ? alloc_pages_current+0xc7/0xe4
     [<ffffffff810a68b5>] ? __page_cache_alloc+0x84/0x91
     [<ffffffff810af912>] ? __do_page_cache_readahead+0xa6/0x2e0
     [<ffffffff810afaa3>] __do_page_cache_readahead+0x237/0x2e0
     [<ffffffff810af912>] ? __do_page_cache_readahead+0xa6/0x2e0
     [<ffffffff810afe3e>] ra_submit+0x1c/0x20
     [<ffffffff810b019b>] ondemand_readahead+0x359/0x382
     [<ffffffff810b0279>] page_cache_sync_readahead+0x38/0x3a
     [<ffffffff810a77b5>] generic_file_aio_read+0x26b/0x637
     [<ffffffffa00f1852>] ? nfs_mark_delegation_referenced+0xb/0xb [nfsv4]
     [<ffffffffa009cc85>] nfs_file_read+0xaa/0xcf [nfs]
     [<ffffffff810db5b3>] do_sync_read+0x91/0xd1
     [<ffffffff810dbb8b>] vfs_read+0x9b/0x144
     [<ffffffff810dbc78>] sys_read+0x44/0x75
     [<ffffffff81422892>] system_call_fastpath+0x16/0x1b
    Signed-off-by: default avatarDavid Howells <dhowells@redhat.com>
    9c04caa8
page.c 27.9 KB