Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752392AbbKYR5M (ORCPT ); Wed, 25 Nov 2015 12:57:12 -0500 Received: from mx0b-00082601.pphosted.com ([67.231.153.30]:2091 "EHLO mx0b-00082601.pphosted.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751180AbbKYR5B (ORCPT ); Wed, 25 Nov 2015 12:57:01 -0500 Subject: Re: kernel BUG at drivers/scsi/scsi_lib.c:1096! To: Hannes Reinecke , References: <1447838334.1564.2.camel@ellerman.id.au> <1447855399.3974.24.camel@redhat.com> <1447894964.15206.0.camel@ellerman.id.au> <20151119082325.GA11419@infradead.org> <564DEC41.5010600@suse.de> <1448030316.4067.18.camel@localhost.localdomain> <564F3453.9040603@suse.de> <1448033323.4067.24.camel@localhost.localdomain> <565579A2.4000005@suse.de> CC: Christoph Hellwig , Michael Ellerman , Mark Salter , "James E. J. Bottomley" , brking , , , , , Mike Snitzer , "Jun'ichi Nomura" From: Jens Axboe Message-ID: <5655F635.5040805@fb.com> Date: Wed, 25 Nov 2015 10:56:05 -0700 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 MIME-Version: 1.0 In-Reply-To: <565579A2.4000005@suse.de> Content-Type: text/plain; charset="utf-8"; format=flowed Content-Transfer-Encoding: 7bit X-Originating-IP: [192.168.54.13] X-Proofpoint-Spam-Reason: safe X-FB-Internal: Safe X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:,, definitions=2015-11-25_12:,, signatures=0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2771 Lines: 81 On 11/25/2015 02:04 AM, Hannes Reinecke wrote: > On 11/20/2015 04:28 PM, Ewan Milne wrote: >> On Fri, 2015-11-20 at 15:55 +0100, Hannes Reinecke wrote: >>> Can't we have a joint effort here? >>> I've been spending a _LOT_ of time trying to debug things here, but >>> none of the ideas I've come up with have been able to fix anything. >> >> Yes. I'm not the one primarily looking at it, and we don't have a >> reproducer in-house. We just have the one dump right now. >> >>> >>> I'm almost tempted to increase the count from scsi_alloc_sgtable() >>> by one and be done with ... >>> >> >> That might not fix it if it is a problem with the merge code, though. >> > And indeed, it doesn't. > Seems I finally found the culprit. > > What happens is this: > We have two paths, with these seg_boundary_masks: > > path-1: seg_boundary_mask = 65535, > path-2: seg_boundary_mask = 4294967295, > > consequently the DM request queue has this: > > md-1: seg_boundary_mask = 65535, > > What happens now is that a request is being formatted, and sent > to path 2. During submission req->nr_phys_segments is formatted > with the limits of path 2, arriving at a count of 3. > Now the request gets retried on path 1, but as the NOMERGE request > flag is set req->nr_phys_segments is never updated. > But blk_rq_map_sg() ignores all counters, and just uses the > bi_vec directly, resulting in a count of 4 -> boom. > > So the culprit here is the NOMERGE flag, which is evaluated > via > ->dm_dispatch_request() > ->blk_insert_cloned_request() > ->blk_rq_check_limits() > > If the above assessment is correct, the following patch should > fix it: > > diff --git a/block/blk-core.c b/block/blk-core.c > index 801ced7..12cccd6 100644 > --- a/block/blk-core.c > +++ b/block/blk-core.c > @@ -1928,7 +1928,7 @@ EXPORT_SYMBOL(submit_bio); > */ > int blk_rq_check_limits(struct request_queue *q, struct request *rq) > { > - if (!rq_mergeable(rq)) > + if (rq->cmd_type != REQ_TYPE_FS) > return 0; > > if (blk_rq_sectors(rq) > blk_queue_get_max_sectors(q, > rq->cmd_flags)) { > > > Mike? Jens? > Can you comment on it? We only support merging on REQ_TYPE_FS already, so how is the above making it any different? In general, NOMERGE being set or not should not make a difference. It's only a hint that we need not check further if we should be merging on this request, since we already tried it once, found we'd exceed various limits, then set NOMERGE to reflect that. -- Jens Axboe -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/