linux-kernel - 3.10.33 stable lockdep warning

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Message-ID: <5321B378.9000603@borntraeger.net>
Date:	Thu, 13 Mar 2014 14:32:40 +0100
From:	Christian Borntraeger <christian@...ntraeger.net>
To:	stable@...r.kernel.org
CC:	Heiko Carstens <heiko.carstens@...ibm.com>,
	Jens Axboe <axboe@...nel.dk>,
	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
	linux-s390 <linux-s390@...r.kernel.org>
Subject: 3.10.33 stable lockdep warning

3.10.33 spits the following warning:

[   14.463408] ------------[ cut here ]------------
[   14.463411] WARNING: at block/elevator.c:193
[   14.463413] Modules linked in: vhost_net macvtap macvlan kvm dasd_eckd_mod dasd_mod zfcp scsi_transport_fc scsi_tgt qeth qdio ccwgroup dm_multipath sunrpc
[   14.463430] CPU: 10 PID: 18762 Comm: dasdconf.sh Tainted: G        W    3.10.33+ #344
[   14.463433] task: 00000004a4020000 ti: 00000004cfca8000 task.ti: 00000004cfca8000
[   14.463435] Krnl PSW : 0704c00180000000 00000000003c6d56 (elevator_init+0xfe/0x144)
[   14.463441]            R:0 T:1 IO:1 EX:1 Key:0 M:1 W:0 P:0 AS:3 CC:0 PM:0 EA:3
Krnl GPRS: 0000000000000000 00000004a4020000 0000000000000000 00000004d199f028
[   14.463446]            000000000019b4ae 0000000000000f13 0000000000000000 00000004ddcbb038
[   14.463449]            00000004ddcbb000 ffffffff00000000 00000004bfcfd850 00000004b820c800
[   14.463451]            000003ff801bcd2a 00000004d199e440 00000000003c6d4c 00000004cfcabb10
[   14.463460] Krnl Code: 00000000003c6d46: c0e5ffeea38f	brasl	%r14,19b464
           00000000003c6d4c: ec26ff9f007e	cij	%r2,0,6,3c6c8a
          #00000000003c6d52: a7f40001		brc	15,3c6d54
          >00000000003c6d56: a7f4ff9a		brc	15,3c6c8a
           00000000003c6d5a: c020008b0327	larl	%r2,15273a8
           00000000003c6d60: a7390000		lghi	%r3,0
           00000000003c6d64: c0e5fffffb58	brasl	%r14,3c6414
           00000000003c6d6a: b9040012		lgr	%r1,%r2
[   14.463481] Call Trace:
[   14.463482] ([<00000000003c6d4c>] elevator_init+0xf4/0x144)
[   14.463486]  [<000003ff801b4daa>] dasd_change_state+0x36a/0xbe0 [dasd_mod]
[   14.463492]  [<000003ff801b56b6>] dasd_set_target_state+0x96/0x110 [dasd_mod]
[   14.463498]  [<000003ff801b5bc8>] dasd_generic_set_online+0xc4/0x2e4 [dasd_mod]
[   14.463503]  [<00000000004d2ecc>] ccw_device_set_online+0xcc/0x4e0
[   14.463507]  [<00000000004d3336>] online_store_recog_and_online+0x56/0x12c
[   14.463510]  [<00000000004d58fa>] online_store+0x1b2/0x248
[   14.463514]  [<00000000002f1b38>] sysfs_write_file+0xe4/0x188
[   14.463517]  [<0000000000270a26>] vfs_write+0xa6/0x1cc
[   14.463520]  [<0000000000270f68>] SyS_write+0x60/0xac
[   14.463523]  [<0000000000636a9c>] sysc_nr_ok+0x22/0x28
[   14.463526]  [<000003fffd6a8984>] 0x3fffd6a8984
[   14.463529] 4 locks held by dasdconf.sh/18762:
[   14.463531]  #0:  (sb_writers#3){.+.+.+}, at: [<0000000000270b3e>] vfs_write+0x1be/0x1cc
[   14.463539]  #1:  (&buffer->mutex){+.+.+.}, at: [<00000000002f1a9a>] sysfs_write_file+0x46/0x188
[   14.463546]  #2:  (s_active#28){.+.+.+}, at: [<00000000002f1b16>] sysfs_write_file+0xc2/0x188
[   14.463554]  #3:  (&device->state_mutex){+.+.+.}, at: [<000003ff801b566c>] dasd_set_target_state+0x4c/0x110 [dasd_mod]
[   14.463563] Last Breaking-Event-Address:
[   14.463565]  [<00000000003c6d52>] elevator_init+0xfa/0x144
[   14.463568] ---[ end trace 3ec4802a2d4e0d3e ]---

So I guess we should pull in
commit ef0899410ff630b2e75306da49996dbbfa318165
Author:     Heiko Carstens <heiko.carstens@...ibm.com>
AuthorDate: Thu Oct 31 13:24:28 2013 +0100
Commit:     Jens Axboe <axboe@...nel.dk>
CommitDate: Fri Nov 8 09:10:29 2013 -0700

    s390/dasd: hold request queue sysfs lock when calling elevator_init()

for 3.10.34 or later.
I can confirm that this patch fixes the issue.
Thanks

Christian
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/