On Mon, 2004-01-05 at 16:58, Alan Stern wrote:
> ===== st.c 1.45 vs edited =====
> --- 1.45/drivers/scsi/st.c    Fri Sep  5 12:16:40 2003
> +++ edited/drivers/scsi/st.c  Fri Nov 21 12:30:34 2003
> @@ -1267,7 +1267,8 @@
>               i = STp->try_dio && try_rdio;
>       else
>               i = STp->try_dio && try_wdio;
> -     if (i) {
> +     if (i && ((unsigned int)buf & queue_dma_alignment(

This needs to be unsigned long, otherwise it will print a warning on a
64 bit compile.

> ===== scsi_lib.c 1.49 vs edited =====
> --- 1.49/drivers/scsi/scsi_lib.c      Sat Nov 22 11:20:45 2003
> +++ edited/drivers/scsi/scsi_lib.c    Mon Jan  5 16:55:32 2004
> @@ -1287,6 +1287,15 @@
>       blk_queue_max_sectors(q, shost->max_sectors);
>       blk_queue_bounce_limit(q, scsi_calculate_bounce_limit(shost));
>       blk_queue_segment_boundary(q, shost->dma_boundary);
> + 
> +     /*
> +      * Set the queue's mask to require a mere 8-byte alignment for
> +      * DMA buffers, rather than the default 512.  This shouldn't
> +      * inconvenience any user programs and should be okay for most
> +      * host adapters.  A host driver can alter this mask in its
> +      * slave_alloc() or slave_configure() callback if necessary.
> +      */
> +     blk_queue_dma_alignment(sdev->request_queue, (8 - 1));

sdev->request_queue is NULL here (it's filled in by the return from this
function.  The argument should be q.

I'll make the changes and apply the patch.

James




-------------------------------------------------------
The SF.Net email is sponsored by EclipseCon 2004
Premiere Conference on Open Tools Development and Integration
See the breadth of Eclipse activity. February 3-5 in Anaheim, CA.
http://www.eclipsecon.org/osdn
_______________________________________________
[EMAIL PROTECTED]
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel

Reply via email to