diff options
| author | Jan Höppner <hoeppner@linux.ibm.com> | 2023-10-25 15:24:37 +0200 |
|---|---|---|
| committer | Jens Axboe <axboe@kernel.dk> | 2023-11-20 11:50:54 -0700 |
| commit | db46cd1e0426f52999d50fa72cfa97fa39952885 (patch) | |
| tree | 2a1112c964ae25fb82efb63edf67dcee1ada7c69 /scripts/rustdoc_test_builder.rs | |
| parent | 5029c5e4f20d8d6b41cefbde4b3eeadaec4662c6 (diff) | |
s390/dasd: protect device queue against concurrent access
In dasd_profile_start() the amount of requests on the device queue are
counted. The access to the device queue is unprotected against
concurrent access. With a lot of parallel I/O, especially with alias
devices enabled, the device queue can change while dasd_profile_start()
is accessing the queue. In the worst case this leads to a kernel panic
due to incorrect pointer accesses.
Fix this by taking the device lock before accessing the queue and
counting the requests. Additionally the check for a valid profile data
pointer can be done earlier to avoid unnecessary locking in a hot path.
Cc: <stable@vger.kernel.org>
Fixes: 4fa52aa7a82f ("[S390] dasd: add enhanced DASD statistics interface")
Reviewed-by: Stefan Haberland <sth@linux.ibm.com>
Signed-off-by: Jan Höppner <hoeppner@linux.ibm.com>
Signed-off-by: Stefan Haberland <sth@linux.ibm.com>
Link: https://lore.kernel.org/r/20231025132437.1223363-3-sth@linux.ibm.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'scripts/rustdoc_test_builder.rs')
0 files changed, 0 insertions, 0 deletions