Hi Jens, On 10/12/2018 05:58 PM, Jens Axboe wrote: > On 10/12/18 4:47 PM, Geoff Levand wrote: >> On 10/11/2018 09:58 AM, Jens Axboe wrote: >>> Convert from the old request_fn style driver to blk-mq. >> >> I tested this on PS3 applied to v4.19-rc7, and on boot it either >> gets a deadlock message or becomes unstable with various other >> block related error messages. >> >> For what its worth, I put an image of the deadlock message here: >> >> https://storage.googleapis.com/public-tmp-files/blk-deadlock.jpg > > Thanks for testing! Can you try this incremental? It's mostly just > a cleanup on top, but the last hunk should be the thing that > hopefully makes this tick again. If you prefer just using a new > patch, this one is updated: > > http://git.kernel.dk/cgit/linux-block/commit/?h=mq-conversions&id=5c86f76f45e7c8d5a47a1f20b3e7b93fc945168a I tested your V2 applied to v4.19-rc8 and it works a little better, in that the system will now boot up, but the ps3disk still gets hung up. I added a '#define DEBUG' to ps3disk.c and saved the dmesg output with and without the patch here: https://storage.googleapis.com/public-tmp-files/dmesg-ok.txt https://storage.googleapis.com/public-tmp-files/dmesg-ng.txt Looking at dmesg-ng.txt, it seems things are going OK, but then go wrong with a 'tag mismatch': [ 6.068298] ps3disk sb_03: ps3disk_do_request:193 [ 6.068318] ps3disk sb_03: ps3disk_submit_request_sg:140: read req has 2 bvecs for 16 sectors [ 6.068339] ps3disk sb_03: ps3disk_submit_request_sg:146: read 16 sectors starting at 256 [ 6.074231] ps3disk sb_03: ps3disk_interrupt:239: tag mismatch, got 0, expected 1 [ 6.084125] ps3disk sb_03: ps3disk_interrupt:271: read completed [ 6.084147] ps3disk sb_03: ps3disk_scatter_gather:107: bio 0: 16 sectors from 256 [ 6.084174] ps3disk sb_03: ps3disk_scatter_gather:107: bio 1: 16 sectors from 256 [ 6.084405] ps3disk sb_03: ps3disk_interrupt:252 non-block layer request completed ... [ 242.657694] INFO: task blkid:563 blocked for more than 120 seconds. Geert is the original author of the PS3 disk support. Geert, can you see anything wrong in the patch? -Geoff
