diff for duplicates of <1495626265.8735.2.camel@primarydata.com> diff --git a/a/1.txt b/N1/1.txt index 97c36f1..97c1241 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,34 +1,53 @@ -T24gV2VkLCAyMDE3LTA1LTI0IGF0IDEwOjM2ICswODAwLCBrZXJuZWwgdGVzdCByb2JvdCB3cm90 -ZToNCj4gRllJLCB3ZSBub3RpY2VkIHRoZSBmb2xsb3dpbmcgY29tbWl0Og0KPiANCj4gY29tbWl0 -OiAxNGQxYmJiMGNhNDI1YzRmZGQ5YTM4ZWFhYzM0NmMwMjg2NjFiNzUyICgiTkZTOiBDcmVhdGUg -YQ0KPiBjb21tb24gbmZzNF9tYXRjaF9jbGllbnQoKSBmdW5jdGlvbiIpDQo+IGh0dHBzOi8vZ2l0 -Lmtlcm5lbC5vcmcvY2dpdC9saW51eC9rZXJuZWwvZ2l0L3RvcnZhbGRzL2xpbnV4LmdpdA0KPiBt -YXN0ZXINCj4gDQo+IGluIHRlc3RjYXNlOiBib290DQo+IA0KPiBvbiB0ZXN0IG1hY2hpbmU6IHFl -bXUtc3lzdGVtLXg4Nl82NCAtZW5hYmxlLWt2bSAtY3B1IGhvc3QgLXNtcCAyIC1tDQo+IDRHDQo+ -IA0KPiBjYXVzZWQgYmVsb3cgY2hhbmdlcyAocGxlYXNlIHJlZmVyIHRvIGF0dGFjaGVkIGRtZXNn -L2ttc2cgZm9yIGVudGlyZQ0KPiBsb2cvYmFja3RyYWNlKToNCj4gDQo+IA0KPiArLS0tLS0tLS0t -LS0tLS0tLS0tLS0tLS0tLS0tLS0tLSstLS0tLS0tLS0tLS0rLS0tLS0tLS0tLS0tKw0KPiA+IMKg -wqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDC -oHwgNWI2ZDNmZjYwNSB8IDE0ZDFiYmIwY2EgfA0KPiANCj4gKy0tLS0tLS0tLS0tLS0tLS0tLS0t -LS0tLS0tLS0tLS0rLS0tLS0tLS0tLS0tKy0tLS0tLS0tLS0tLSsNCj4gPiBib290X3N1Y2Nlc3Nl -c8KgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgfCA0NjnCoMKgwqDCoMKgwqDCoMKgfCA0 -NzDCoMKgwqDCoMKgwqDCoMKgfA0KPiA+IGJvb3RfZmFpbHVyZXPCoMKgwqDCoMKgwqDCoMKgwqDC -oMKgwqDCoMKgwqDCoMKgfCAwwqDCoMKgwqDCoMKgwqDCoMKgwqB8IDMxwqDCoMKgwqDCoMKgwqDC -oMKgfA0KPiA+IEJVRzpzY2hlZHVsaW5nX3doaWxlX2F0b21pY8KgwqDCoHwgMMKgwqDCoMKgwqDC -oMKgwqDCoMKgfCAzMcKgwqDCoMKgwqDCoMKgwqDCoHwNCj4gPiBCVUc6c3BpbmxvY2tfcmVjdXJz -aW9uX29uX0NQVSB8IDDCoMKgwqDCoMKgwqDCoMKgwqDCoHwgMzDCoMKgwqDCoMKgwqDCoMKgwqB8 -DQo+IA0KPiArLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLSstLS0tLS0tLS0tLS0rLS0t -LS0tLS0tLS0tKw0KPiANCj4gDQo+IA0KPiBbwqDCoDI5OS4xNTg5OTFdIEJVRzogc2NoZWR1bGlu -ZyB3aGlsZSBhdG9taWM6DQo+IG1vdW50Lm5mcy85Mzg3LzB4MDAwMDAwMDINCj4gW8KgwqAyOTku -MTY5NTg3XSAyIGxvY2tzIGhlbGQgYnkgbW91bnQubmZzLzkzODc6DQo+IFvCoMKgMjk5LjE3NjE2 -NV3CoMKgIzA6wqDCoChuZnNfY2xpZF9pbml0X211dGV4KXsuLi4uLi59LCBhdDoNCj4gWzxmZmZm -ZmZmZjgxMzBjYzkyPl0gbmZzNF9kaXNjb3Zlcl9zZXJ2ZXJfdHJ1bmtpbmcrMHg0Ny8weDFmYw0K -PiBbwqDCoDI5OS4yMDE4MDJdwqDCoCMxOsKgwqAoJigmbm4tPm5mc19jbGllbnRfbG9jayktPnJs -b2NrKXsuLi4uLi59LCBhdDoNCj4gWzxmZmZmZmZmZjgxMzEyNWZhPl0gbmZzNDBfd2Fsa19jbGll -bnRfbGlzdCsweDJlOS8weDMzOA0KPiANCg0KT29wcy4gQW5uYSwgaXQgbG9va3MgYXMgaWYgdGhh -dCBwYXRjaCBhZGRzIGFuIGluY29ycmVjdCBjYWxsIHRvDQoic3Bpbl9sb2NrKCZubi0+bmZzX2Ns -aWVudF9sb2NrKSIgZm9yIHRoZSAiZGVmYXVsdDoiIGNhc2Ugd2hlbg0KcHJvY2Vzc2luZyB0aGUg -cmVzdWx0IGZyb20gdGhlIFNFVENMSUVOVElEX0NPTkZJUk0gY2FsbC4gSSBjYW4gc2VuZCBhDQpm -aXguDQoNClRoYW5rcyBYaWFvbG9uZyENCg0KQ2hlZXJzDQogIFRyb25kDQoNCi0tIA0KVHJvbmQg -TXlrbGVidXN0DQpMaW51eCBORlMgY2xpZW50IG1haW50YWluZXIsIFByaW1hcnlEYXRhDQp0cm9u -ZC5teWtsZWJ1c3RAcHJpbWFyeWRhdGEuY29tDQo= +On Wed, 2017-05-24 at 10:36 +0800, kernel test robot wrote: +> FYI, we noticed the following commit: +> +> commit: 14d1bbb0ca425c4fdd9a38eaac346c028661b752 ("NFS: Create a +> common nfs4_match_client() function") +> https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git +> master +> +> in testcase: boot +> +> on test machine: qemu-system-x86_64 -enable-kvm -cpu host -smp 2 -m +> 4G +> +> caused below changes (please refer to attached dmesg/kmsg for entire +> log/backtrace): +> +> +> +-------------------------------+------------+------------+ +> > | 5b6d3ff605 | 14d1bbb0ca | +> +> +-------------------------------+------------+------------+ +> > boot_successes | 469 | 470 | +> > boot_failures | 0 | 31 | +> > BUG:scheduling_while_atomic | 0 | 31 | +> > BUG:spinlock_recursion_on_CPU | 0 | 30 | +> +> +-------------------------------+------------+------------+ +> +> +> +> [ 299.158991] BUG: scheduling while atomic: +> mount.nfs/9387/0x00000002 +> [ 299.169587] 2 locks held by mount.nfs/9387: +> [ 299.176165] #0: (nfs_clid_init_mutex){......}, at: +> [<ffffffff8130cc92>] nfs4_discover_server_trunking+0x47/0x1fc +> [ 299.201802] #1: (&(&nn->nfs_client_lock)->rlock){......}, at: +> [<ffffffff813125fa>] nfs40_walk_client_list+0x2e9/0x338 +> + +Oops. Anna, it looks as if that patch adds an incorrect call to +"spin_lock(&nn->nfs_client_lock)" for the "default:" case when +processing the result from the SETCLIENTID_CONFIRM call. I can send a +fix. + +Thanks Xiaolong! + +Cheers + Trond + +-- +Trond Myklebust +Linux NFS client maintainer, PrimaryData +trond.myklebust(a)primarydata.com diff --git a/a/content_digest b/N1/content_digest index ff2bb90..4520008 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -1,48 +1,62 @@ "ref\020170524023629.GK1795@yexl-desktop\0" "From\0Trond Myklebust <trondmy@primarydata.com>\0" - "Subject\0Re: [lkp-robot] [NFS] 14d1bbb0ca: BUG:scheduling_while_atomic\0" + "Subject\0Re: [lkp-robot] [NFS] 14d1bbb0ca: BUG:scheduling_while_atomic\0" "Date\0Wed, 24 May 2017 11:44:27 +0000\0" - "To\0Anna.Schumaker@netapp.com <Anna.Schumaker@netapp.com>" - " xiaolong.ye@intel.com <xiaolong.ye@intel.com>\0" - "Cc\0torvalds@linux-foundation.org <torvalds@linux-foundation.org>" - linux-kernel@vger.kernel.org <linux-kernel@vger.kernel.org> - lkp@01.org <lkp@01.org> - " linux-nfs@vger.kernel.org <linux-nfs@vger.kernel.org>\0" - "\00:1\0" + "To\0lkp@lists.01.org\0" + "\01:1\0" "b\0" - "T24gV2VkLCAyMDE3LTA1LTI0IGF0IDEwOjM2ICswODAwLCBrZXJuZWwgdGVzdCByb2JvdCB3cm90\n" - "ZToNCj4gRllJLCB3ZSBub3RpY2VkIHRoZSBmb2xsb3dpbmcgY29tbWl0Og0KPiANCj4gY29tbWl0\n" - "OiAxNGQxYmJiMGNhNDI1YzRmZGQ5YTM4ZWFhYzM0NmMwMjg2NjFiNzUyICgiTkZTOiBDcmVhdGUg\n" - "YQ0KPiBjb21tb24gbmZzNF9tYXRjaF9jbGllbnQoKSBmdW5jdGlvbiIpDQo+IGh0dHBzOi8vZ2l0\n" - "Lmtlcm5lbC5vcmcvY2dpdC9saW51eC9rZXJuZWwvZ2l0L3RvcnZhbGRzL2xpbnV4LmdpdA0KPiBt\n" - "YXN0ZXINCj4gDQo+IGluIHRlc3RjYXNlOiBib290DQo+IA0KPiBvbiB0ZXN0IG1hY2hpbmU6IHFl\n" - "bXUtc3lzdGVtLXg4Nl82NCAtZW5hYmxlLWt2bSAtY3B1IGhvc3QgLXNtcCAyIC1tDQo+IDRHDQo+\n" - "IA0KPiBjYXVzZWQgYmVsb3cgY2hhbmdlcyAocGxlYXNlIHJlZmVyIHRvIGF0dGFjaGVkIGRtZXNn\n" - "L2ttc2cgZm9yIGVudGlyZQ0KPiBsb2cvYmFja3RyYWNlKToNCj4gDQo+IA0KPiArLS0tLS0tLS0t\n" - "LS0tLS0tLS0tLS0tLS0tLS0tLS0tLSstLS0tLS0tLS0tLS0rLS0tLS0tLS0tLS0tKw0KPiA+IMKg\n" - "wqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDC\n" - "oHwgNWI2ZDNmZjYwNSB8IDE0ZDFiYmIwY2EgfA0KPiANCj4gKy0tLS0tLS0tLS0tLS0tLS0tLS0t\n" - "LS0tLS0tLS0tLS0rLS0tLS0tLS0tLS0tKy0tLS0tLS0tLS0tLSsNCj4gPiBib290X3N1Y2Nlc3Nl\n" - "c8KgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgfCA0NjnCoMKgwqDCoMKgwqDCoMKgfCA0\n" - "NzDCoMKgwqDCoMKgwqDCoMKgfA0KPiA+IGJvb3RfZmFpbHVyZXPCoMKgwqDCoMKgwqDCoMKgwqDC\n" - "oMKgwqDCoMKgwqDCoMKgfCAwwqDCoMKgwqDCoMKgwqDCoMKgwqB8IDMxwqDCoMKgwqDCoMKgwqDC\n" - "oMKgfA0KPiA+IEJVRzpzY2hlZHVsaW5nX3doaWxlX2F0b21pY8KgwqDCoHwgMMKgwqDCoMKgwqDC\n" - "oMKgwqDCoMKgfCAzMcKgwqDCoMKgwqDCoMKgwqDCoHwNCj4gPiBCVUc6c3BpbmxvY2tfcmVjdXJz\n" - "aW9uX29uX0NQVSB8IDDCoMKgwqDCoMKgwqDCoMKgwqDCoHwgMzDCoMKgwqDCoMKgwqDCoMKgwqB8\n" - "DQo+IA0KPiArLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLSstLS0tLS0tLS0tLS0rLS0t\n" - "LS0tLS0tLS0tKw0KPiANCj4gDQo+IA0KPiBbwqDCoDI5OS4xNTg5OTFdIEJVRzogc2NoZWR1bGlu\n" - "ZyB3aGlsZSBhdG9taWM6DQo+IG1vdW50Lm5mcy85Mzg3LzB4MDAwMDAwMDINCj4gW8KgwqAyOTku\n" - "MTY5NTg3XSAyIGxvY2tzIGhlbGQgYnkgbW91bnQubmZzLzkzODc6DQo+IFvCoMKgMjk5LjE3NjE2\n" - "NV3CoMKgIzA6wqDCoChuZnNfY2xpZF9pbml0X211dGV4KXsuLi4uLi59LCBhdDoNCj4gWzxmZmZm\n" - "ZmZmZjgxMzBjYzkyPl0gbmZzNF9kaXNjb3Zlcl9zZXJ2ZXJfdHJ1bmtpbmcrMHg0Ny8weDFmYw0K\n" - "PiBbwqDCoDI5OS4yMDE4MDJdwqDCoCMxOsKgwqAoJigmbm4tPm5mc19jbGllbnRfbG9jayktPnJs\n" - "b2NrKXsuLi4uLi59LCBhdDoNCj4gWzxmZmZmZmZmZjgxMzEyNWZhPl0gbmZzNDBfd2Fsa19jbGll\n" - "bnRfbGlzdCsweDJlOS8weDMzOA0KPiANCg0KT29wcy4gQW5uYSwgaXQgbG9va3MgYXMgaWYgdGhh\n" - "dCBwYXRjaCBhZGRzIGFuIGluY29ycmVjdCBjYWxsIHRvDQoic3Bpbl9sb2NrKCZubi0+bmZzX2Ns\n" - "aWVudF9sb2NrKSIgZm9yIHRoZSAiZGVmYXVsdDoiIGNhc2Ugd2hlbg0KcHJvY2Vzc2luZyB0aGUg\n" - "cmVzdWx0IGZyb20gdGhlIFNFVENMSUVOVElEX0NPTkZJUk0gY2FsbC4gSSBjYW4gc2VuZCBhDQpm\n" - "aXguDQoNClRoYW5rcyBYaWFvbG9uZyENCg0KQ2hlZXJzDQogIFRyb25kDQoNCi0tIA0KVHJvbmQg\n" - "TXlrbGVidXN0DQpMaW51eCBORlMgY2xpZW50IG1haW50YWluZXIsIFByaW1hcnlEYXRhDQp0cm9u\n" - ZC5teWtsZWJ1c3RAcHJpbWFyeWRhdGEuY29tDQo= + "On Wed, 2017-05-24 at 10:36 +0800, kernel test robot wrote:\n" + "> FYI, we noticed the following commit:\n" + "> \n" + "> commit: 14d1bbb0ca425c4fdd9a38eaac346c028661b752 (\"NFS: Create a\n" + "> common nfs4_match_client() function\")\n" + "> https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git\n" + "> master\n" + "> \n" + "> in testcase: boot\n" + "> \n" + "> on test machine: qemu-system-x86_64 -enable-kvm -cpu host -smp 2 -m\n" + "> 4G\n" + "> \n" + "> caused below changes (please refer to attached dmesg/kmsg for entire\n" + "> log/backtrace):\n" + "> \n" + "> \n" + "> +-------------------------------+------------+------------+\n" + "> > \302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240| 5b6d3ff605 | 14d1bbb0ca |\n" + "> \n" + "> +-------------------------------+------------+------------+\n" + "> > boot_successes\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240| 469\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240| 470\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240|\n" + "> > boot_failures\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240| 0\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240| 31\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240|\n" + "> > BUG:scheduling_while_atomic\302\240\302\240\302\240| 0\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240| 31\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240|\n" + "> > BUG:spinlock_recursion_on_CPU | 0\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240| 30\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240|\n" + "> \n" + "> +-------------------------------+------------+------------+\n" + "> \n" + "> \n" + "> \n" + "> [\302\240\302\240299.158991] BUG: scheduling while atomic:\n" + "> mount.nfs/9387/0x00000002\n" + "> [\302\240\302\240299.169587] 2 locks held by mount.nfs/9387:\n" + "> [\302\240\302\240299.176165]\302\240\302\240#0:\302\240\302\240(nfs_clid_init_mutex){......}, at:\n" + "> [<ffffffff8130cc92>] nfs4_discover_server_trunking+0x47/0x1fc\n" + "> [\302\240\302\240299.201802]\302\240\302\240#1:\302\240\302\240(&(&nn->nfs_client_lock)->rlock){......}, at:\n" + "> [<ffffffff813125fa>] nfs40_walk_client_list+0x2e9/0x338\n" + "> \n" + "\n" + "Oops. Anna, it looks as if that patch adds an incorrect call to\n" + "\"spin_lock(&nn->nfs_client_lock)\" for the \"default:\" case when\n" + "processing the result from the SETCLIENTID_CONFIRM call. I can send a\n" + "fix.\n" + "\n" + "Thanks Xiaolong!\n" + "\n" + "Cheers\n" + " Trond\n" + "\n" + "-- \n" + "Trond Myklebust\n" + "Linux NFS client maintainer, PrimaryData\n" + trond.myklebust(a)primarydata.com -13c67e7df5b373b4f980815a69654111dbe699c0b777c55c6cd9e4bf6cee69bd +69c79febc279bab0a80712ffff512a587dc3f29bf0ad8d9d01b006d595656bb0
diff --git a/a/1.txt b/N2/1.txt index 97c36f1..aed4338 100644 --- a/a/1.txt +++ b/N2/1.txt @@ -1,34 +1,53 @@ -T24gV2VkLCAyMDE3LTA1LTI0IGF0IDEwOjM2ICswODAwLCBrZXJuZWwgdGVzdCByb2JvdCB3cm90 -ZToNCj4gRllJLCB3ZSBub3RpY2VkIHRoZSBmb2xsb3dpbmcgY29tbWl0Og0KPiANCj4gY29tbWl0 -OiAxNGQxYmJiMGNhNDI1YzRmZGQ5YTM4ZWFhYzM0NmMwMjg2NjFiNzUyICgiTkZTOiBDcmVhdGUg -YQ0KPiBjb21tb24gbmZzNF9tYXRjaF9jbGllbnQoKSBmdW5jdGlvbiIpDQo+IGh0dHBzOi8vZ2l0 -Lmtlcm5lbC5vcmcvY2dpdC9saW51eC9rZXJuZWwvZ2l0L3RvcnZhbGRzL2xpbnV4LmdpdA0KPiBt -YXN0ZXINCj4gDQo+IGluIHRlc3RjYXNlOiBib290DQo+IA0KPiBvbiB0ZXN0IG1hY2hpbmU6IHFl -bXUtc3lzdGVtLXg4Nl82NCAtZW5hYmxlLWt2bSAtY3B1IGhvc3QgLXNtcCAyIC1tDQo+IDRHDQo+ -IA0KPiBjYXVzZWQgYmVsb3cgY2hhbmdlcyAocGxlYXNlIHJlZmVyIHRvIGF0dGFjaGVkIGRtZXNn -L2ttc2cgZm9yIGVudGlyZQ0KPiBsb2cvYmFja3RyYWNlKToNCj4gDQo+IA0KPiArLS0tLS0tLS0t -LS0tLS0tLS0tLS0tLS0tLS0tLS0tLSstLS0tLS0tLS0tLS0rLS0tLS0tLS0tLS0tKw0KPiA+IMKg -wqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDC -oHwgNWI2ZDNmZjYwNSB8IDE0ZDFiYmIwY2EgfA0KPiANCj4gKy0tLS0tLS0tLS0tLS0tLS0tLS0t -LS0tLS0tLS0tLS0rLS0tLS0tLS0tLS0tKy0tLS0tLS0tLS0tLSsNCj4gPiBib290X3N1Y2Nlc3Nl -c8KgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgfCA0NjnCoMKgwqDCoMKgwqDCoMKgfCA0 -NzDCoMKgwqDCoMKgwqDCoMKgfA0KPiA+IGJvb3RfZmFpbHVyZXPCoMKgwqDCoMKgwqDCoMKgwqDC -oMKgwqDCoMKgwqDCoMKgfCAwwqDCoMKgwqDCoMKgwqDCoMKgwqB8IDMxwqDCoMKgwqDCoMKgwqDC -oMKgfA0KPiA+IEJVRzpzY2hlZHVsaW5nX3doaWxlX2F0b21pY8KgwqDCoHwgMMKgwqDCoMKgwqDC -oMKgwqDCoMKgfCAzMcKgwqDCoMKgwqDCoMKgwqDCoHwNCj4gPiBCVUc6c3BpbmxvY2tfcmVjdXJz -aW9uX29uX0NQVSB8IDDCoMKgwqDCoMKgwqDCoMKgwqDCoHwgMzDCoMKgwqDCoMKgwqDCoMKgwqB8 -DQo+IA0KPiArLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLSstLS0tLS0tLS0tLS0rLS0t -LS0tLS0tLS0tKw0KPiANCj4gDQo+IA0KPiBbwqDCoDI5OS4xNTg5OTFdIEJVRzogc2NoZWR1bGlu -ZyB3aGlsZSBhdG9taWM6DQo+IG1vdW50Lm5mcy85Mzg3LzB4MDAwMDAwMDINCj4gW8KgwqAyOTku -MTY5NTg3XSAyIGxvY2tzIGhlbGQgYnkgbW91bnQubmZzLzkzODc6DQo+IFvCoMKgMjk5LjE3NjE2 -NV3CoMKgIzA6wqDCoChuZnNfY2xpZF9pbml0X211dGV4KXsuLi4uLi59LCBhdDoNCj4gWzxmZmZm -ZmZmZjgxMzBjYzkyPl0gbmZzNF9kaXNjb3Zlcl9zZXJ2ZXJfdHJ1bmtpbmcrMHg0Ny8weDFmYw0K -PiBbwqDCoDI5OS4yMDE4MDJdwqDCoCMxOsKgwqAoJigmbm4tPm5mc19jbGllbnRfbG9jayktPnJs -b2NrKXsuLi4uLi59LCBhdDoNCj4gWzxmZmZmZmZmZjgxMzEyNWZhPl0gbmZzNDBfd2Fsa19jbGll -bnRfbGlzdCsweDJlOS8weDMzOA0KPiANCg0KT29wcy4gQW5uYSwgaXQgbG9va3MgYXMgaWYgdGhh -dCBwYXRjaCBhZGRzIGFuIGluY29ycmVjdCBjYWxsIHRvDQoic3Bpbl9sb2NrKCZubi0+bmZzX2Ns -aWVudF9sb2NrKSIgZm9yIHRoZSAiZGVmYXVsdDoiIGNhc2Ugd2hlbg0KcHJvY2Vzc2luZyB0aGUg -cmVzdWx0IGZyb20gdGhlIFNFVENMSUVOVElEX0NPTkZJUk0gY2FsbC4gSSBjYW4gc2VuZCBhDQpm -aXguDQoNClRoYW5rcyBYaWFvbG9uZyENCg0KQ2hlZXJzDQogIFRyb25kDQoNCi0tIA0KVHJvbmQg -TXlrbGVidXN0DQpMaW51eCBORlMgY2xpZW50IG1haW50YWluZXIsIFByaW1hcnlEYXRhDQp0cm9u -ZC5teWtsZWJ1c3RAcHJpbWFyeWRhdGEuY29tDQo= +On Wed, 2017-05-24 at 10:36 +0800, kernel test robot wrote: +> FYI, we noticed the following commit: +> +> commit: 14d1bbb0ca425c4fdd9a38eaac346c028661b752 ("NFS: Create a +> common nfs4_match_client() function") +> https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git +> master +> +> in testcase: boot +> +> on test machine: qemu-system-x86_64 -enable-kvm -cpu host -smp 2 -m +> 4G +> +> caused below changes (please refer to attached dmesg/kmsg for entire +> log/backtrace): +> +> +> +-------------------------------+------------+------------+ +> > | 5b6d3ff605 | 14d1bbb0ca | +> +> +-------------------------------+------------+------------+ +> > boot_successes | 469 | 470 | +> > boot_failures | 0 | 31 | +> > BUG:scheduling_while_atomic | 0 | 31 | +> > BUG:spinlock_recursion_on_CPU | 0 | 30 | +> +> +-------------------------------+------------+------------+ +> +> +> +> [ 299.158991] BUG: scheduling while atomic: +> mount.nfs/9387/0x00000002 +> [ 299.169587] 2 locks held by mount.nfs/9387: +> [ 299.176165] #0: (nfs_clid_init_mutex){......}, at: +> [<ffffffff8130cc92>] nfs4_discover_server_trunking+0x47/0x1fc +> [ 299.201802] #1: (&(&nn->nfs_client_lock)->rlock){......}, at: +> [<ffffffff813125fa>] nfs40_walk_client_list+0x2e9/0x338 +> + +Oops. Anna, it looks as if that patch adds an incorrect call to +"spin_lock(&nn->nfs_client_lock)" for the "default:" case when +processing the result from the SETCLIENTID_CONFIRM call. I can send a +fix. + +Thanks Xiaolong! + +Cheers + Trond + +-- +Trond Myklebust +Linux NFS client maintainer, PrimaryData +trond.myklebust@primarydata.com diff --git a/a/content_digest b/N2/content_digest index ff2bb90..15c5aaa 100644 --- a/a/content_digest +++ b/N2/content_digest @@ -10,39 +10,58 @@ " linux-nfs@vger.kernel.org <linux-nfs@vger.kernel.org>\0" "\00:1\0" "b\0" - "T24gV2VkLCAyMDE3LTA1LTI0IGF0IDEwOjM2ICswODAwLCBrZXJuZWwgdGVzdCByb2JvdCB3cm90\n" - "ZToNCj4gRllJLCB3ZSBub3RpY2VkIHRoZSBmb2xsb3dpbmcgY29tbWl0Og0KPiANCj4gY29tbWl0\n" - "OiAxNGQxYmJiMGNhNDI1YzRmZGQ5YTM4ZWFhYzM0NmMwMjg2NjFiNzUyICgiTkZTOiBDcmVhdGUg\n" - "YQ0KPiBjb21tb24gbmZzNF9tYXRjaF9jbGllbnQoKSBmdW5jdGlvbiIpDQo+IGh0dHBzOi8vZ2l0\n" - "Lmtlcm5lbC5vcmcvY2dpdC9saW51eC9rZXJuZWwvZ2l0L3RvcnZhbGRzL2xpbnV4LmdpdA0KPiBt\n" - "YXN0ZXINCj4gDQo+IGluIHRlc3RjYXNlOiBib290DQo+IA0KPiBvbiB0ZXN0IG1hY2hpbmU6IHFl\n" - "bXUtc3lzdGVtLXg4Nl82NCAtZW5hYmxlLWt2bSAtY3B1IGhvc3QgLXNtcCAyIC1tDQo+IDRHDQo+\n" - "IA0KPiBjYXVzZWQgYmVsb3cgY2hhbmdlcyAocGxlYXNlIHJlZmVyIHRvIGF0dGFjaGVkIGRtZXNn\n" - "L2ttc2cgZm9yIGVudGlyZQ0KPiBsb2cvYmFja3RyYWNlKToNCj4gDQo+IA0KPiArLS0tLS0tLS0t\n" - "LS0tLS0tLS0tLS0tLS0tLS0tLS0tLSstLS0tLS0tLS0tLS0rLS0tLS0tLS0tLS0tKw0KPiA+IMKg\n" - "wqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDC\n" - "oHwgNWI2ZDNmZjYwNSB8IDE0ZDFiYmIwY2EgfA0KPiANCj4gKy0tLS0tLS0tLS0tLS0tLS0tLS0t\n" - "LS0tLS0tLS0tLS0rLS0tLS0tLS0tLS0tKy0tLS0tLS0tLS0tLSsNCj4gPiBib290X3N1Y2Nlc3Nl\n" - "c8KgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgfCA0NjnCoMKgwqDCoMKgwqDCoMKgfCA0\n" - "NzDCoMKgwqDCoMKgwqDCoMKgfA0KPiA+IGJvb3RfZmFpbHVyZXPCoMKgwqDCoMKgwqDCoMKgwqDC\n" - "oMKgwqDCoMKgwqDCoMKgfCAwwqDCoMKgwqDCoMKgwqDCoMKgwqB8IDMxwqDCoMKgwqDCoMKgwqDC\n" - "oMKgfA0KPiA+IEJVRzpzY2hlZHVsaW5nX3doaWxlX2F0b21pY8KgwqDCoHwgMMKgwqDCoMKgwqDC\n" - "oMKgwqDCoMKgfCAzMcKgwqDCoMKgwqDCoMKgwqDCoHwNCj4gPiBCVUc6c3BpbmxvY2tfcmVjdXJz\n" - "aW9uX29uX0NQVSB8IDDCoMKgwqDCoMKgwqDCoMKgwqDCoHwgMzDCoMKgwqDCoMKgwqDCoMKgwqB8\n" - "DQo+IA0KPiArLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLSstLS0tLS0tLS0tLS0rLS0t\n" - "LS0tLS0tLS0tKw0KPiANCj4gDQo+IA0KPiBbwqDCoDI5OS4xNTg5OTFdIEJVRzogc2NoZWR1bGlu\n" - "ZyB3aGlsZSBhdG9taWM6DQo+IG1vdW50Lm5mcy85Mzg3LzB4MDAwMDAwMDINCj4gW8KgwqAyOTku\n" - "MTY5NTg3XSAyIGxvY2tzIGhlbGQgYnkgbW91bnQubmZzLzkzODc6DQo+IFvCoMKgMjk5LjE3NjE2\n" - "NV3CoMKgIzA6wqDCoChuZnNfY2xpZF9pbml0X211dGV4KXsuLi4uLi59LCBhdDoNCj4gWzxmZmZm\n" - "ZmZmZjgxMzBjYzkyPl0gbmZzNF9kaXNjb3Zlcl9zZXJ2ZXJfdHJ1bmtpbmcrMHg0Ny8weDFmYw0K\n" - "PiBbwqDCoDI5OS4yMDE4MDJdwqDCoCMxOsKgwqAoJigmbm4tPm5mc19jbGllbnRfbG9jayktPnJs\n" - "b2NrKXsuLi4uLi59LCBhdDoNCj4gWzxmZmZmZmZmZjgxMzEyNWZhPl0gbmZzNDBfd2Fsa19jbGll\n" - "bnRfbGlzdCsweDJlOS8weDMzOA0KPiANCg0KT29wcy4gQW5uYSwgaXQgbG9va3MgYXMgaWYgdGhh\n" - "dCBwYXRjaCBhZGRzIGFuIGluY29ycmVjdCBjYWxsIHRvDQoic3Bpbl9sb2NrKCZubi0+bmZzX2Ns\n" - "aWVudF9sb2NrKSIgZm9yIHRoZSAiZGVmYXVsdDoiIGNhc2Ugd2hlbg0KcHJvY2Vzc2luZyB0aGUg\n" - "cmVzdWx0IGZyb20gdGhlIFNFVENMSUVOVElEX0NPTkZJUk0gY2FsbC4gSSBjYW4gc2VuZCBhDQpm\n" - "aXguDQoNClRoYW5rcyBYaWFvbG9uZyENCg0KQ2hlZXJzDQogIFRyb25kDQoNCi0tIA0KVHJvbmQg\n" - "TXlrbGVidXN0DQpMaW51eCBORlMgY2xpZW50IG1haW50YWluZXIsIFByaW1hcnlEYXRhDQp0cm9u\n" - ZC5teWtsZWJ1c3RAcHJpbWFyeWRhdGEuY29tDQo= + "On Wed, 2017-05-24 at 10:36 +0800, kernel test robot wrote:\n" + "> FYI, we noticed the following commit:\n" + "> \n" + "> commit: 14d1bbb0ca425c4fdd9a38eaac346c028661b752 (\"NFS: Create a\n" + "> common nfs4_match_client() function\")\n" + "> https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git\n" + "> master\n" + "> \n" + "> in testcase: boot\n" + "> \n" + "> on test machine: qemu-system-x86_64 -enable-kvm -cpu host -smp 2 -m\n" + "> 4G\n" + "> \n" + "> caused below changes (please refer to attached dmesg/kmsg for entire\n" + "> log/backtrace):\n" + "> \n" + "> \n" + "> +-------------------------------+------------+------------+\n" + "> > \302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240| 5b6d3ff605 | 14d1bbb0ca |\n" + "> \n" + "> +-------------------------------+------------+------------+\n" + "> > boot_successes\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240| 469\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240| 470\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240|\n" + "> > boot_failures\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240| 0\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240| 31\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240|\n" + "> > BUG:scheduling_while_atomic\302\240\302\240\302\240| 0\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240| 31\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240|\n" + "> > BUG:spinlock_recursion_on_CPU | 0\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240| 30\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240|\n" + "> \n" + "> +-------------------------------+------------+------------+\n" + "> \n" + "> \n" + "> \n" + "> [\302\240\302\240299.158991] BUG: scheduling while atomic:\n" + "> mount.nfs/9387/0x00000002\n" + "> [\302\240\302\240299.169587] 2 locks held by mount.nfs/9387:\n" + "> [\302\240\302\240299.176165]\302\240\302\240#0:\302\240\302\240(nfs_clid_init_mutex){......}, at:\n" + "> [<ffffffff8130cc92>] nfs4_discover_server_trunking+0x47/0x1fc\n" + "> [\302\240\302\240299.201802]\302\240\302\240#1:\302\240\302\240(&(&nn->nfs_client_lock)->rlock){......}, at:\n" + "> [<ffffffff813125fa>] nfs40_walk_client_list+0x2e9/0x338\n" + "> \n" + "\n" + "Oops. Anna, it looks as if that patch adds an incorrect call to\n" + "\"spin_lock(&nn->nfs_client_lock)\" for the \"default:\" case when\n" + "processing the result from the SETCLIENTID_CONFIRM call. I can send a\n" + "fix.\n" + "\n" + "Thanks Xiaolong!\n" + "\n" + "Cheers\n" + " Trond\n" + "\n" + "-- \n" + "Trond Myklebust\n" + "Linux NFS client maintainer, PrimaryData\n" + trond.myklebust@primarydata.com -13c67e7df5b373b4f980815a69654111dbe699c0b777c55c6cd9e4bf6cee69bd +4fdcabeb9f6397db22d43a4a6e4085e45fa1df33e8c78ce8110aef92ccc73ca4
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.