On Tue, Sep 11, 2007 at 04:10:47AM +0900, FUJITA Tomonori wrote:

> > The only patch which touches qla1280 is git-block.patch.  From a quick
> > squizz the change looks OK, although it's tricky and something might have
> > broken.
> 
> Can you try this patch (against 2.6.23-rc4-mm1)?

Yep this patch seems to sort out booting on these boxes.  The other one
is also testing.  Results later.

> >From 592bd2049cb3e6e1f1dde7cf631879f26ddffeaa Mon Sep 17 00:00:00 2001
> From: FUJITA Tomonori <[EMAIL PROTECTED]>
> Date: Mon, 10 Sep 2007 04:17:13 +0100
> Subject: [PATCH] qla1280: sg chaining fixes
> 
> Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]>
> ---
>  drivers/scsi/qla1280.c |    5 ++++-
>  1 files changed, 4 insertions(+), 1 deletions(-)
> 
> diff --git a/drivers/scsi/qla1280.c b/drivers/scsi/qla1280.c
> index bd805ec..7c1eaec 100644
> --- a/drivers/scsi/qla1280.c
> +++ b/drivers/scsi/qla1280.c
> @@ -2977,8 +2977,8 @@ qla1280_64bit_start_scsi(struct scsi_qla_host *ha, 
> struct srb * sp)
>                                               
> cpu_to_le32(pci_dma_hi32(dma_handle)),
>                                               
> cpu_to_le32(pci_dma_lo32(dma_handle)),
>                                               cpu_to_le32(sg_dma_len(s)));
> -                                     remseg--;
>                               }
> +                             remseg -= cnt;
>                               dprintk(5, "qla1280_64bit_start_scsi: "
>                                       "continuation packet data - b %i, t "
>                                       "%i, l %i \n", SCSI_BUS_32(cmd),
> @@ -3250,6 +3250,8 @@ qla1280_32bit_start_scsi(struct scsi_qla_host *ha, 
> struct srb * sp)
>  
>                               /* Load continuation entry data segments. */
>                               for_each_sg(sg, s, remseg, cnt) {
> +                                     if (cnt == 7)
> +                                             break;
>                                       *dword_ptr++ =
>                                               
> cpu_to_le32(pci_dma_lo32(sg_dma_address(s)));
>                                       *dword_ptr++ =
> @@ -3260,6 +3262,7 @@ qla1280_32bit_start_scsi(struct scsi_qla_host *ha, 
> struct srb * sp)
>                                               
> cpu_to_le32(pci_dma_lo32(sg_dma_address(s))),
>                                               cpu_to_le32(sg_dma_len(s)));
>                               }
> +                             remseg -= cnt;
>                               dprintk(5, "qla1280_32bit_start_scsi: "
>                                       "continuation packet data - "
>                                       "scsi(%i:%i:%i)\n", SCSI_BUS_32(cmd),
> -- 
> 1.5.2.4
> 
> 

-apw
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to