From mboxrd@z Thu Jan 1 00:00:00 1970 From: Christoph Hellwig Subject: Re: twa generates WARNING upon boot Date: Wed, 30 Sep 2015 09:41:23 -0700 Message-ID: <20150930164123.GA24596@infradead.org> References: <1443548266.2207.19.camel@HansenPartnership.com> <1443549768.2207.22.camel@HansenPartnership.com> <95d34bcc4c0fcf84d02d76a4443ba07c.squirrel@atoth.sote.hu> <1443551581.2207.26.camel@HansenPartnership.com> <20150930160835.GA16534@infradead.org> <1443629730.2185.25.camel@HansenPartnership.com> <20150930163147.GA2928@infradead.org> <1443630984.2185.27.camel@HansenPartnership.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from bombadil.infradead.org ([198.137.202.9]:52855 "EHLO bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754687AbbI3Qlf (ORCPT ); Wed, 30 Sep 2015 12:41:35 -0400 Content-Disposition: inline In-Reply-To: <1443630984.2185.27.camel@HansenPartnership.com> Sender: linux-scsi-owner@vger.kernel.org List-Id: linux-scsi@vger.kernel.org To: James Bottomley Cc: Christoph Hellwig , "\"T?th Attila\"" , adam radford , linux-scsi On Wed, Sep 30, 2015 at 09:36:24AM -0700, James Bottomley wrote: > Are you sure? The init trace that kicked all this off still has > twa_init in it. I was figuring the most likely suspect is > twa_get_param(). That one doesnt set ->srb at all, but given that it's called before any SCSI commands can sent ->srb will be all NULL. It certainly doesn't use a scsi_cmnd anywhere and does not expect to hit any code path calling scsi_dma_unmap. If it ends up there it's a bad bug and would blow up a lot more spectacularly.