On Wed, 2015-09-30 at 09:41 -0700, Christoph Hellwig wrote:
> 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.

OK, post a compilable version of the patch and lets get the reporter to
try it out.  Not resurrecting esoteric flags suits me.

James


--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to