[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20150911105430.7b0bb2fa@voom.fritz.box>
Date: Fri, 11 Sep 2015 10:54:30 +1000
From: David Gibson <dgibson@...hat.com>
To: Laurent Vivier <lvivier@...hat.com>
Cc: linux-scsi@...r.kernel.org, linux-kernel@...r.kernel.org,
"James E.J. Bottomley" <JBottomley@...n.com>,
Tyrel Datwyler <tyreld@...ux.vnet.ibm.com>, thuth@...hat.com,
pbonzini@...hat.com
Subject: Re: [PATCH 3/3] ibmvsci: Allow to configure maximum LUN
On Thu, 10 Sep 2015 11:23:14 +0200
Laurent Vivier <lvivier@...hat.com> wrote:
> QEMU allows until 32 LUNs.
>
> Signed-off-by: Laurent Vivier <lvivier@...hat.com>
Reviewed-by: David Gibson <david@...son.dropbear.id.au>
> ---
> drivers/scsi/ibmvscsi/ibmvscsi.c | 5 ++++-
> 1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/scsi/ibmvscsi/ibmvscsi.c b/drivers/scsi/ibmvscsi/ibmvscsi.c
> index f9d7ec4..e5478b0 100644
> --- a/drivers/scsi/ibmvscsi/ibmvscsi.c
> +++ b/drivers/scsi/ibmvscsi/ibmvscsi.c
> @@ -84,6 +84,7 @@
> */
> static int max_id = 64;
> static int max_channel = 3;
> +static int max_lun = 8;
> static int init_timeout = 300;
> static int login_timeout = 60;
> static int info_timeout = 30;
> @@ -117,6 +118,8 @@ module_param_named(fast_fail, fast_fail, int, S_IRUGO | S_IWUSR);
> MODULE_PARM_DESC(fast_fail, "Enable fast fail. [Default=1]");
> module_param_named(client_reserve, client_reserve, int, S_IRUGO );
> MODULE_PARM_DESC(client_reserve, "Attempt client managed reserve/release");
> +module_param(max_lun, int, S_IRUGO);
> +MODULE_PARM_DESC(max_lun, "Maximum LUN value [Default=8]");
>
> static void ibmvscsi_handle_crq(struct viosrp_crq *crq,
> struct ibmvscsi_host_data *hostdata);
> @@ -2289,7 +2292,7 @@ static int ibmvscsi_probe(struct vio_dev *vdev, const struct vio_device_id *id)
> goto init_pool_failed;
> }
>
> - host->max_lun = 8;
> + host->max_lun = max_lun;
> host->max_id = max_id;
> host->max_channel = max_channel;
> host->max_cmd_len = 16;
> --
> 2.1.0
>
--
David Gibson <dgibson@...hat.com>
Senior Software Engineer, Virtualization, Red Hat
--
David Gibson <dgibson@...hat.com>
Senior Software Engineer, Virtualization, Red Hat
Content of type "application/pgp-signature" skipped
Powered by blists - more mailing lists