On Fri, Dec 13, 2013 at 2:56 PM, Weston Andros Adamson <dros@xxxxxxxxxx> wrote: > So should we make this fix generic and check gssd_running for every upcall, or should we just handle this regression and return -EACCES in gss_refresh_null when !gssd_running? I can't see any reason to attempt an upcall if gssd is not running. -->Andy > > -dros > > > On Dec 13, 2013, at 2:02 PM, Andy Adamson <androsadamson@xxxxxxxxx> wrote: > >> On Fri, Dec 13, 2013 at 12:32 PM, Weston Andros Adamson <dros@xxxxxxxxxx> wrote: >>> Commit c297c8b99b07f496ff69a719cfb8e8fe852832ed (SUNRPC: do not fail gss proc NULL calls with EACCES) introduces a hang on reboot if there are any mounts that use AUTH_GSS. >>> >>> Due to recent changes, this can even happen when mounting sec=sys, because the non-fsid specific operations use KRB5 if possible. >>> >>> To reproduce: >>> >>> 1) mount a server with sec=krb5 (or sec=sys if you know krb5 will work for nfs_client ops) >>> 2) reboot >>> 3) notice hang (output below) >>> >>> >>> I can see why it’s hanging - the reboot forced unmount is happening after gssd is killed, so the upcall will never succeed…. Any ideas on how this should be fixed? Should we timeout after a certain number of tries? Should we detect that gssd isn’t running anymore (if this is even possible)? >> >> This patch : commit e2f0c83a9de331d9352185ca3642616c13127539 >> Author: Jeff Layton <jlayton@xxxxxxxxxx> >> Date: Thu Dec 5 07:34:44 2013 -0500 >> >> sunrpc: add an "info" file for the dummy gssd pipe >> >> solves the "is gssd running" problem. >> >> -->Andy >> >>> >>> -dros >>> >>> >>> BUG: soft lockup - CPU#0 stuck for 22s! [kworker/0:1:27] >>> Modules linked in: rpcsec_gss_krb5 nfsv4 nfs fscache crc32c_intel ppdev i2c_piix4 aesni_intel aes_x86_64 glue_helper lrw gf128mul serio_raw ablk_helper cryptd i2c_core e1000 parport_pc parport shpchp nfsd auth_rpcgss oid_registry exportfs nfs_acl lockd sunrpc autofs4 mptspi scsi_transport_spi mptscsih mptbase ata_generic floppy >>> irq event stamp: 279178 >>> hardirqs last enabled at (279177): [<ffffffff814a925c>] restore_args+0x0/0x30 >>> hardirqs last disabled at (279178): [<ffffffff814b0a6a>] apic_timer_interrupt+0x6a/0x80 >>> softirqs last enabled at (279176): [<ffffffff8103f583>] __do_softirq+0x1df/0x276 >>> softirqs last disabled at (279171): [<ffffffff8103f852>] irq_exit+0x53/0x9a >>> CPU: 0 PID: 27 Comm: kworker/0:1 Not tainted 3.13.0-rc3-branch-dros_testing+ #1 >>> Hardware name: VMware, Inc. VMware Virtual Platform/440BX Desktop Reference Platform, BIOS 6.00 07/31/2013 >>> Workqueue: rpciod rpc_async_schedule [sunrpc] >>> task: ffff88007b87a130 ti: ffff88007ad08000 task.ti: ffff88007ad08000 >>> RIP: 0010:[<ffffffffa00a562d>] [<ffffffffa00a562d>] rpcauth_refreshcred+0x17/0x15f [sunrpc] >>> RSP: 0018:ffff88007ad09c88 EFLAGS: 00000286 >>> RAX: ffffffffa02ba650 RBX: ffffffff81073f47 RCX: 0000000000000007 >>> RDX: 0000000000000007 RSI: ffff88007a885d70 RDI: ffff88007a158b40 >>> RBP: ffff88007ad09ce8 R08: ffff88007a5ce9f8 R09: ffffffffa00993d7 >>> R10: ffff88007a5ce7b0 R11: ffff88007a158b40 R12: ffffffffa009943d >>> R13: 0000000000000a81 R14: ffff88007a158bb0 R15: ffffffff814a925c >>> FS: 0000000000000000(0000) GS:ffff88007f200000(0000) knlGS:0000000000000000 >>> CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033 >>> CR2: 00007f2d03056000 CR3: 0000000001a0b000 CR4: 00000000001407f0 >>> Stack: >>> ffffffffa009943d ffff88007a5ce9f8 0000000000000000 0000000000000007 >>> 0000000000000007 ffff88007a885d70 ffff88007a158b40 ffffffffffffff10 >>> ffff88007a158b40 0000000000000000 ffff88007a158bb0 0000000000000a81 >>> Call Trace: >>> [<ffffffffa009943d>] ? call_refresh+0x66/0x66 [sunrpc] >>> [<ffffffffa0099438>] call_refresh+0x61/0x66 [sunrpc] >>> [<ffffffffa00a403b>] __rpc_execute+0xf1/0x362 [sunrpc] >>> [<ffffffff81073f47>] ? trace_hardirqs_on_caller+0x145/0x1a1 >>> [<ffffffffa00a42d3>] rpc_async_schedule+0x27/0x32 [sunrpc] >>> [<ffffffff81052974>] process_one_work+0x211/0x3a5 >>> [<ffffffff810528d5>] ? process_one_work+0x172/0x3a5 >>> [<ffffffff81052eeb>] worker_thread+0x134/0x202 >>> [<ffffffff81052db7>] ? rescuer_thread+0x280/0x280 >>> [<ffffffff81052db7>] ? rescuer_thread+0x280/0x280 >>> [<ffffffff810584a0>] kthread+0xc9/0xd1 >>> [<ffffffff810583d7>] ? __kthread_parkme+0x61/0x61 >>> [<ffffffff814afd6c>] ret_from_fork+0x7c/0xb0 >>> [<ffffffff810583d7>] ? __kthread_parkme+0x61/0x61 >>> Code: 89 c2 41 ff d6 48 83 c4 58 5b 41 5c 41 5d 41 5e 41 5f 5d c3 0f 1f 44 00 00 55 48 89 e5 41 56 41 55 41 54 53 48 89 fb 48 83 ec 40 <4c> 8b 6f 20 4d 8b a5 90 00 00 00 4d 85 e4 0f 85 e4 00 00 00 8b-- >>> To unsubscribe from this list: send the line "unsubscribe linux-nfs" in >>> the body of a message to majordomo@xxxxxxxxxxxxxxx >>> More majordomo info at http://vger.kernel.org/majordomo-info.html > -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
