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

[Brian King <brking@linux.vnet.ibm.com>]

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