Re: [PATCH] cxgb3i: fixed programing of the dma page sizes

[Mike Christie <michaelc@cs.wisc.edu>]

On 06/17/2011 06:10 PM, kxie@chelsio.com wrote:
> [PATCH] cxgb3i: fixed programing of the dma page sizes
> 
> From: Karen Xie <kxie@chelsio.com>
> 
> Fixed missing programming of the pages sizes for DMA. 
> 
> Signed-off-by: Karen Xie <kxie@chelsio.com>
> ---
>  drivers/scsi/cxgbi/cxgb3i/cxgb3i.c |    4 +++-
>  1 files changed, 3 insertions(+), 1 deletions(-)
> 
> diff --git a/drivers/scsi/cxgbi/cxgb3i/cxgb3i.c b/drivers/scsi/cxgbi/cxgb3i/cxgb3i.c
> index fc2cdb6..ffb9eea 100644
> --- a/drivers/scsi/cxgbi/cxgb3i/cxgb3i.c
> +++ b/drivers/scsi/cxgbi/cxgb3i/cxgb3i.c
> @@ -1245,7 +1245,7 @@ static int cxgb3i_ddp_init(struct cxgbi_device *cdev)
>  	struct cxgbi_ddp_info *ddp = tdev->ulp_iscsi;
>  	struct ulp_iscsi_info uinfo;
>  	unsigned int pgsz_factor[4];
> -	int err;
> +	int i, err;
>  
>  	if (ddp) {
>  		kref_get(&ddp->refcnt);
> @@ -1271,6 +1271,8 @@ static int cxgb3i_ddp_init(struct cxgbi_device *cdev)
>  
>  	uinfo.tagmask = ddp->idx_mask << PPOD_IDX_SHIFT;
>  	cxgbi_ddp_page_size_factor(pgsz_factor);
> +	for (i = 0; i < 4; i++)
> +		uinfo.pgsz_factor[i] = pgsz_factor[i];
>  	uinfo.ulimit = uinfo.llimit + (ddp->nppods << PPOD_SIZE_SHIFT);
>  
>  	err = tdev->ctl(tdev, ULP_ISCSI_SET_PARAMS, &uinfo);

Looks ok.

Reviewed-by: Mike Christie <michaelc@cs.wisc.edu>