Received: by 2002:a05:6902:102b:0:0:0:0 with SMTP id x11csp11990ybt; Tue, 30 Jun 2020 13:48:00 -0700 (PDT) X-Google-Smtp-Source: ABdhPJwvo7ukX64syY+DEPx+jdwnWY+tLusaEVTGlV5MoyCpw1iGkGq+J9BqFGr0h1x2ScECxtDr X-Received: by 2002:a17:906:7d9:: with SMTP id m25mr19452875ejc.25.1593549607563; Tue, 30 Jun 2020 13:40:07 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1593549607; cv=none; d=google.com; s=arc-20160816; b=oNs/g+9sfTrdyBjazkeLFyfLN0GGqDmHwPXBFAFU46SvsukaqPhJRndXtAoLkaMKMg rOPqL4Wt1u9/KTnLtyAmQ0zKkzqDUZvcj7CJzmN54KkYRbS/2yq5woastLjziCu/gFIM H2iNh/IgFyJi7YtUgMibnldPvCZ4gjXOlEEDc0jN6C6kkRKT7aHNJw3VeOG6taGF3L4n haK2S9fOMBAKoLTum7QVHIudmLcbIiFGw++v5cKo5p9pCHUK1Nul9KmBs3ErnroFwMuB eHPzf4GG/+YpbCr80t8GOzmVQFonhLSkkrkUqPU78F1jYSlGYmr80AN5iZr6DGG7MGL4 T4YA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date; bh=dkUNXagucMQdCPHyYTIJ7J17aRXemA6Z1Ft/mgr6Hbk=; b=nc4wld7r6r9G/IuV8vNdHTiSKLKy5lq0stZpeIFD404V8ZdGCi/EQNxwuCBZzKcnv8 tZMLDPaOprPQFQZijZPQRBiVZXiXoj8qSEkZDV0YiAhiJl6KXL14dpzKvG7q8ElaC86E bZGvNDiXqVkCMVL0pg2QedqOPxKzhL9J8VjrSNbe5s/qL+mp7KScy+9jjnPv8aSPQGI9 q5Fklch5u54DDZkOCxXdQxIER9GZOdwuoExe23dI1KZ0TyuYDWy/VFnPidC9JPrJgZyK rOkg7DHZVwpd3BjpD1cBnlKdREV03qV5vT7m5UqbmsGBKWWMyvarkxU9ZEztZY+MU5/b F+YQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id br6si2335250ejb.203.2020.06.30.13.39.45; Tue, 30 Jun 2020 13:40:07 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2387658AbgF3STe (ORCPT + 99 others); Tue, 30 Jun 2020 14:19:34 -0400 Received: from verein.lst.de ([213.95.11.211]:36961 "EHLO verein.lst.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726736AbgF3STe (ORCPT ); Tue, 30 Jun 2020 14:19:34 -0400 Received: by verein.lst.de (Postfix, from userid 2407) id 2C5A868B05; Tue, 30 Jun 2020 20:19:29 +0200 (CEST) Date: Tue, 30 Jun 2020 20:19:28 +0200 From: Christoph Hellwig To: Jens Axboe Cc: Christoph Hellwig , dm-devel@redhat.com, linux-kernel@vger.kernel.org, linux-m68k@lists.linux-m68k.org, linux-xtensa@linux-xtensa.org, drbd-dev@lists.linbit.com, linuxppc-dev@lists.ozlabs.org, linux-bcache@vger.kernel.org, linux-raid@vger.kernel.org, linux-nvdimm@lists.01.org, linux-nvme@lists.infradead.org, linux-s390@vger.kernel.org Subject: Re: rename ->make_request_fn and move it to the block_device_operations Message-ID: <20200630181928.GA7853@lst.de> References: <20200629193947.2705954-1-hch@lst.de> <6ddbe343-0fc2-58c8-3726-c4ba9952994f@kernel.dk> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <6ddbe343-0fc2-58c8-3726-c4ba9952994f@kernel.dk> User-Agent: Mutt/1.5.17 (2007-11-01) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Jun 30, 2020 at 09:43:31AM -0600, Jens Axboe wrote: > On 6/30/20 7:57 AM, Jens Axboe wrote: > > On 6/29/20 1:39 PM, Christoph Hellwig wrote: > >> 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?