Thread (32 messages) 32 messages, 5 authors, 2020-07-02

Re: rename ->make_request_fn and move it to the block_device_operations

From: Jens Axboe <axboe@kernel.dk>
Date: 2020-06-30 18:22:00
Also in: dm-devel, linux-bcache, linux-m68k, linux-nvme, linux-raid, linux-s390, lkml, nvdimm

On 6/30/20 12:19 PM, Christoph Hellwig wrote:
On Tue, Jun 30, 2020 at 09:43:31AM -0600, Jens Axboe wrote:
quoted
On 6/30/20 7:57 AM, Jens Axboe wrote:
quoted
On 6/29/20 1:39 PM, Christoph Hellwig wrote:
quoted
Hi Jens,

this series moves the make_request_fn method into block_device_operations
with the much more descriptive ->submit_bio name.  It then also gives
generic_make_request a more descriptive name, and further optimize the
path to issue to blk-mq, removing the need for the direct_make_request
bypass.
Looks good to me, and it's a nice cleanup as well. Applied.
Dropped, insta-crashes with dm:
Hmm.  Can you send me what is at "submit_bio_noacct+0x1f6" from gdb?
Or your .config?
I'd have to apply and compile again. But it's a bad RIP, so I'm guessing
it's ->submit_bio == NULL. Let me know if you really need it, and I can
re-generate the OOPS and have the vmlinux too.

-- 
Jens Axboe
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help