[dm-devel] Re: [PATCH 1/3][RESUBMIT] scsi_dh_rdac: changes to collect the debug information during rdac_activate
Chandra Seetharaman
sekharan at us.ibm.com
Fri Aug 28 17:57:08 UTC 2009
Hi Babu,
Sorry about this comment after few iterations. I just realized it.
We already do a C8 inquiry (get_lun()), why can't we just piggy back on
that to fill the array name also. You can change the name of the
function to be get_lun_and_array_name() or some such.
chandra
On Fri, 2009-08-28 at 08:22 -0600, Moger, Babu wrote:
> Adding the code to read the debug information during activate. This patch collects the information about storage and controllers during rdac_activate.
>
> Signed-off-by: Babu Moger <babu.moger at lsi.com>
> Reviewed-by: Vijay Chauhan <vijay.chauhan at lsi.com>
> Reviewed-by: Bob Stankey <Robert.stankey at lsi.com>
>
> ---
> --- linux-2.6.31-rc5/drivers/scsi/device_handler/scsi_dh_rdac.c.orig 2009-08-27 18:17:23.000000000 -0500
> +++ linux-2.6.31-rc5/drivers/scsi/device_handler/scsi_dh_rdac.c 2009-08-27 18:27:23.000000000 -0500
> @@ -112,6 +112,7 @@ struct c9_inquiry {
>
> #define SUBSYS_ID_LEN 16
> #define SLOT_ID_LEN 2
> +#define ARRAY_LABEL_LEN 31
>
> struct c4_inquiry {
> u8 peripheral_info;
> @@ -135,6 +136,8 @@ struct rdac_controller {
> struct rdac_pg_legacy legacy;
> struct rdac_pg_expanded expanded;
> } mode_select;
> + u8 index;
> + u8 array_name[ARRAY_LABEL_LEN];
> };
> struct c8_inquiry {
> u8 peripheral_info;
> @@ -324,6 +327,13 @@ static struct rdac_controller *get_contr
> /* initialize fields of controller */
> memcpy(ctlr->subsys_id, subsys_id, SUBSYS_ID_LEN);
> memcpy(ctlr->slot_id, slot_id, SLOT_ID_LEN);
> +
> + /* update the controller index */
> + if (slot_id[1] == 0x31)
> + ctlr->index = 0;
> + else
> + ctlr->index = 1;
> +
> kref_init(&ctlr->kref);
> ctlr->use_ms10 = -1;
> list_add(&ctlr->node, &ctlr_list);
> @@ -381,6 +391,26 @@ static int get_lun(struct scsi_device *s
> return err;
> }
>
> +static int get_array_name(struct scsi_device *sdev, struct rdac_dh_data *h)
> +{
> + int err, i;
> + struct c8_inquiry *inqp;
> +
> + err = submit_inquiry(sdev, 0xC8, sizeof(struct c8_inquiry), h);
> + if (err == SCSI_DH_OK) {
> + inqp = &h->inq.c8;
> + if (inqp->page_code != 0xc8)
> + return SCSI_DH_NOSYS;
> +
> + for(i=0; i<ARRAY_LABEL_LEN-1; ++i)
> + h->ctlr->array_name[i] = inqp->array_user_label[(2*i)+1];
> +
> + h->ctlr->array_name[ARRAY_LABEL_LEN-1] = '\0';
> + }
> +
> + return err;
> +}
> +
> static int check_ownership(struct scsi_device *sdev, struct rdac_dh_data *h)
> {
> int err;
> @@ -529,6 +559,10 @@ static int rdac_activate(struct scsi_dev
> err = initialize_controller(sdev, h);
> if (err != SCSI_DH_OK)
> goto done;
> +
> + err = get_array_name(sdev, h);
> + if (err != SCSI_DH_OK)
> + goto done;
> }
>
> if (h->ctlr->use_ms10 == -1) {
>
>
More information about the dm-devel
mailing list