linux-kernel - RE: ISP1760 driver crashes

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <62DDBB9E5E23CC4A929EE46F9427CEAF015BFB56@BUDMLVEM04.e2k.ad.ge.com>
Date:	Fri, 21 Nov 2008 11:58:28 +0100
From:	"Hommel, Thomas (GE EntSol, Intelligent Platforms)" 
	<Thomas.Hommel@...anuc.com>
To:	"Alan Stern" <stern@...land.harvard.edu>,
	"FUJITA Tomonori" <fujita.tomonori@....ntt.co.jp>
Cc:	<jens.axboe@...cle.com>, <bigeasy@...utronix.de>,
	<linux-usb@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
	<James.Bottomley@...senPartnership.com>
Subject: RE: ISP1760 driver crashes

> Alan Stern wrote:
> 
> On Thu, 20 Nov 2008, FUJITA Tomonori wrote:
> 
> > We have been used 4GB for long time if dma_mask is zero (I guess we 
> > use 4GB as kinda the default dma address limit at several 
> places). The 
> > majority of drivers (such as pci) sets properly 
> dev->dma_mask so the 
> > patch might not change anything but suddenly changing the 
> > long-standing rule in an odd way (use BLK_BOUNCE_HIGH if dma_mask is
> > zero) doesn't sound a good idea to me.
> > 
> > Why not calling blk_queue_bounce_limit() in the 
> slave_configure hook?
> > I think that it's the common way for SCSI LLDs with odd 
> bounce limit.
> 
> Thomas, here's a patch to do what Tomonori suggests.  Try 
> replacing the old patch with this one.
> 
Thanks, I've tried the new patch and everything works ok.
Thomas

> Alan Stern
> 
> 
> Index: usb-2.6/drivers/usb/storage/scsiglue.c
> ===================================================================
> --- usb-2.6.orig/drivers/usb/storage/scsiglue.c
> +++ usb-2.6/drivers/usb/storage/scsiglue.c
> @@ -129,6 +129,14 @@ static int slave_configure(struct scsi_d
>  					      max_sectors);
>  	}
>  
> +	/* Some USB host controllers can't do DMA; they have to use PIO.
> +	 * They indicate this by setting their dma_mask to NULL.  For
> +	 * such controllers we need to make sure the block layer sets
> +	 * up bounce buffers in addressable memory.
> +	 */
> +	if (!us->pusb_dev->bus->controller->dma_mask)
> +		blk_queue_bounce_limit(sdev->request_queue, 
> BLK_BOUNCE_HIGH);
> +
>  	/* We can't put these settings in slave_alloc() because 
> that gets
>  	 * called before the device type is known.  Consequently these
>  	 * settings can't be overridden via the scsi devinfo 
> mechanism. */
> 
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/