On 06/17/2011 06:10 PM, kxie@xxxxxxxxxxx wrote:
> [PATCH] cxgb3i: fixed programing of the dma page sizes
>
> From: Karen Xie <kxie@xxxxxxxxxxx>
>
> Fixed missing programming of the pages sizes for DMA.
>
> Signed-off-by: Karen Xie <kxie@xxxxxxxxxxx>
> ---
> 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@xxxxxxxxxxx>
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
