From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from bombadil.infradead.org ([65.50.211.133]:46683 "EHLO bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751195AbdJCJRr (ORCPT ); Tue, 3 Oct 2017 05:17:47 -0400 Date: Tue, 3 Oct 2017 02:17:46 -0700 From: Christoph Hellwig To: Ming Lei Cc: Jens Axboe , linux-block@vger.kernel.org, Christoph Hellwig , Mike Snitzer , dm-devel@redhat.com, Bart Van Assche , Laurence Oberman , Paolo Valente , Oleksandr Natalenko , Tom Nguyen , linux-kernel@vger.kernel.org, Omar Sandoval Subject: Re: [PATCH V5 5/8] block: add check on elevator for supporting bio merge via hashtable from blk-mq sw queue Message-ID: <20171003091746.GC2771@infradead.org> References: <20170930112655.31451-1-ming.lei@redhat.com> <20170930112655.31451-6-ming.lei@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <20170930112655.31451-6-ming.lei@redhat.com> Sender: linux-block-owner@vger.kernel.org List-Id: linux-block@vger.kernel.org On Sat, Sep 30, 2017 at 07:26:52PM +0800, Ming Lei wrote: > blk_mq_sched_try_merge() will be reused in following patches > to support bio merge to blk-mq sw queue, so add checkes to > related functions which are called from blk_mq_sched_try_merge(). The changes themselves look ok, but don't make sense on their own, I think this needs to be merged with the code tor actually introduce the bio merge.