[PATCH 1/1] ipr: set the data list length in the request control block

[PATCH 1/1] ipr: set the data list length in the request control block

[Wayne Boyer]

In bring up testing for the new 64 bit adapters, the first read command failed
after loading the driver.  The cause was that the command requires more than 
one scatter gather element and the corresponding code to set the data list
length in the request control block was missing.  This patch adds the correct 
assignment.

Signed-off-by: Wayne Boyer <wayneb@linux.vnet.ibm.com>
---
 drivers/scsi/ipr.c |    2 ++
 1 file changed, 2 insertions(+)

Index: b/drivers/scsi/ipr.c
===================================================================
--- a/drivers/scsi/ipr.c	2010-05-13 12:38:47.000000000 -0700
+++ b/drivers/scsi/ipr.c	2010-05-13 12:45:11.000000000 -0700
@@ -5018,6 +5018,8 @@ static int ipr_build_ioadl64(struct ipr_
 	ipr_cmd->dma_use_sg = nseg;

 	ioarcb->data_transfer_length = cpu_to_be32(length);
+	ioarcb->ioadl_len =
+		cpu_to_be32(sizeof(struct ipr_ioadl64_desc) * ipr_cmd->dma_use_sg);

 	if (scsi_cmd->sc_data_direction == DMA_TO_DEVICE) {
 		ioadl_flags = IPR_IOADL_FLAGS_WRITE;

Re: [PATCH 1/1] ipr: set the data list length in the request control block

[Brian King]

Acked-by: Brian King <brking@linux.vnet.ibm.com>

On 05/14/2010 10:55 AM, Wayne Boyer wrote:
> In bring up testing for the new 64 bit adapters, the first read command failed
> after loading the driver.  The cause was that the command requires more than 
> one scatter gather element and the corresponding code to set the data list
> length in the request control block was missing.  This patch adds the correct 
> assignment.
> 
> Signed-off-by: Wayne Boyer <wayneb@linux.vnet.ibm.com>
> ---
>  drivers/scsi/ipr.c |    2 ++
>  1 file changed, 2 insertions(+)
> 
> Index: b/drivers/scsi/ipr.c
> ===================================================================
> --- a/drivers/scsi/ipr.c	2010-05-13 12:38:47.000000000 -0700
> +++ b/drivers/scsi/ipr.c	2010-05-13 12:45:11.000000000 -0700
> @@ -5018,6 +5018,8 @@ static int ipr_build_ioadl64(struct ipr_
>  	ipr_cmd->dma_use_sg = nseg;
> 
>  	ioarcb->data_transfer_length = cpu_to_be32(length);
> +	ioarcb->ioadl_len =
> +		cpu_to_be32(sizeof(struct ipr_ioadl64_desc) * ipr_cmd->dma_use_sg);
> 
>  	if (scsi_cmd->sc_data_direction == DMA_TO_DEVICE) {
>  		ioadl_flags = IPR_IOADL_FLAGS_WRITE;
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html


-- 
Brian King
Linux on Power Virtualization
IBM Linux Technology Center