* [PATCH 3/3][RESUBMIT] scsi_dh_rdac: changes for rdac debug logging
@ 2009-08-28 14:23 Moger, Babu
0 siblings, 0 replies; 6+ messages in thread
From: Moger, Babu @ 2009-08-28 14:23 UTC (permalink / raw)
To: 'linux-scsi@vger.kernel.org'
Cc: 'device mapper development',
'sekharan@linux.vnet.ibm.com', Chauhan, Vijay,
Stankey, Robert, Dachepalli, Sudhir
Patch to add debugging stuff for rdac device handler.
- Added a bit mask "module parameter" rdac_logging with 2 bits for each type of logging.
- currently defined only two types of logging(failover and sense logging). Can be enhanced later if required.
- By default only failover logging is enabled which is equivalent of current logging.
Signed-off-by: Babu Moger <babu.moger@lsi.com>
Reviewed-by: Vijay Chauhan <vijay.chauhan@lsi.com>
Reviewed-by: Bob Stankey <Robert.stankey@lsi.com>
---
--- linux-2.6.31-rc5/drivers/scsi/device_handler/scsi_dh_rdac.c.orig 2009-08-27 18:33:39.000000000 -0500
+++ linux-2.6.31-rc5/drivers/scsi/device_handler/scsi_dh_rdac.c 2009-08-27 18:35:24.000000000 -0500
@@ -201,6 +201,31 @@ static const char *lun_state[] =
static LIST_HEAD(ctlr_list);
static DEFINE_SPINLOCK(list_lock);
+/*
+ * module parameter to enable rdac debug logging.
+ * 2 bits for each type of logging, only two types defined for now
+ * Can be enhanced if required at later point
+ */
+static int rdac_logging = 1;
+module_param(rdac_logging, int, S_IRUGO|S_IWUSR);
+MODULE_PARM_DESC(rdac_logging, "A bit mask of rdac logging levels, "
+ "Default is 1 - failover logging enabled, "
+ "set it to 0xF to enable all the logs");
+
+#define RDAC_LOG_FAILOVER 0
+#define RDAC_LOG_SENSE 2
+
+#define RDAC_LOG_BITS 2
+
+#define RDAC_LOG_LEVEL(SHIFT) \
+ ((rdac_logging >> (SHIFT)) & ((1 << (RDAC_LOG_BITS)) - 1))
+
+#define RDAC_LOG(SHIFT, sdev, f, arg...) \
+do { \
+ if (unlikely(RDAC_LOG_LEVEL(SHIFT))) \
+ sdev_printk(KERN_INFO, sdev, RDAC_NAME ": " f "\n", ## arg); \
+} while (0);
+
static inline struct rdac_dh_data *get_rdac_data(struct scsi_device *sdev)
{
struct scsi_dh_data *scsi_dh_data = sdev->scsi_dh_data;
@@ -480,6 +505,7 @@ static int mode_select_handle_sense(stru
{
struct scsi_sense_hdr sense_hdr;
int err = SCSI_DH_IO, ret;
+ struct rdac_dh_data *h = get_rdac_data(sdev);
ret = scsi_normalize_sense(sensebuf, SCSI_SENSE_BUFFERSIZE, &sense_hdr);
if (!ret)
@@ -508,11 +534,14 @@ static int mode_select_handle_sense(stru
err = SCSI_DH_RETRY;
break;
default:
- sdev_printk(KERN_INFO, sdev,
- "MODE_SELECT failed with sense %02x/%02x/%02x.\n",
- sense_hdr.sense_key, sense_hdr.asc, sense_hdr.ascq);
+ break;
}
+ RDAC_LOG(RDAC_LOG_FAILOVER, sdev, "array %s, ctlr %d, "
+ "MODE_SELECT returned with sense %02x/%02x/%02x",
+ (char *) h->ctlr->array_name, h->ctlr->index,
+ sense_hdr.sense_key, sense_hdr.asc, sense_hdr.ascq);
+
done:
return err;
}
@@ -529,7 +558,9 @@ retry:
if (!rq)
goto done;
- sdev_printk(KERN_INFO, sdev, "%s MODE_SELECT command.\n",
+ RDAC_LOG(RDAC_LOG_FAILOVER, sdev, "array %s, ctlr %d, "
+ "%s MODE_SELECT command",
+ (char *) h->ctlr->array_name, h->ctlr->index,
(retry_cnt == RDAC_RETRY_COUNT) ? "queueing" : "retrying");
err = blk_execute_rq(q, NULL, rq, 1);
@@ -539,8 +570,12 @@ retry:
if (err == SCSI_DH_RETRY && retry_cnt--)
goto retry;
}
- if (err == SCSI_DH_OK)
+ if (err == SCSI_DH_OK) {
h->state = RDAC_STATE_ACTIVE;
+ RDAC_LOG(RDAC_LOG_FAILOVER, sdev, "array %s, ctlr %d, "
+ "MODE_SELECT completed",
+ (char *) h->ctlr->array_name, h->ctlr->index);
+ }
done:
return err;
@@ -578,6 +613,12 @@ static int rdac_check_sense(struct scsi_
struct scsi_sense_hdr *sense_hdr)
{
struct rdac_dh_data *h = get_rdac_data(sdev);
+
+ RDAC_LOG(RDAC_LOG_SENSE, sdev, "array %s, ctlr %d, "
+ "I/O returned with sense %02x/%02x/%02x",
+ (char *) h->ctlr->array_name, h->ctlr->index,
+ sense_hdr->sense_key, sense_hdr->asc, sense_hdr->ascq);
+
switch (sense_hdr->sense_key) {
case NOT_READY:
if (sense_hdr->asc == 0x04 && sense_hdr->ascq == 0x01)
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 3/3][RESUBMIT] scsi_dh_rdac: changes for rdac debug logging
@ 2009-09-02 23:13 Moger, Babu
0 siblings, 0 replies; 6+ messages in thread
From: Moger, Babu @ 2009-09-02 23:13 UTC (permalink / raw)
To: 'linux-scsi@vger.kernel.org'
Cc: 'device mapper development',
'sekharan@linux.vnet.ibm.com', Chauhan, Vijay,
Stankey, Robert, Dachepalli, Sudhir
Patch to add debugging stuff for rdac device handler.
- Added a bit mask "module parameter" rdac_logging with 2 bits for each type of logging.
- currently defined only two types of logging(failover and sense logging). Can be enhanced later if required.
- By default only failover logging is enabled which is equivalent of current logging.
Signed-off-by: Babu Moger <babu.moger@lsi.com>
Reviewed-by: Vijay Chauhan <vijay.chauhan@lsi.com>
Reviewed-by: Bob Stankey <Robert.stankey@lsi.com>
---
--- linux-2.6.31-rc5/drivers/scsi/device_handler/scsi_dh_rdac.c.orig 2009-08-31 15:16:54.000000000 -0500
+++ linux-2.6.31-rc5/drivers/scsi/device_handler/scsi_dh_rdac.c 2009-08-31 15:18:31.000000000 -0500
@@ -201,6 +201,31 @@ static const char *lun_state[] =
static LIST_HEAD(ctlr_list);
static DEFINE_SPINLOCK(list_lock);
+/*
+ * module parameter to enable rdac debug logging.
+ * 2 bits for each type of logging, only two types defined for now
+ * Can be enhanced if required at later point
+ */
+static int rdac_logging = 1;
+module_param(rdac_logging, int, S_IRUGO|S_IWUSR);
+MODULE_PARM_DESC(rdac_logging, "A bit mask of rdac logging levels, "
+ "Default is 1 - failover logging enabled, "
+ "set it to 0xF to enable all the logs");
+
+#define RDAC_LOG_FAILOVER 0
+#define RDAC_LOG_SENSE 2
+
+#define RDAC_LOG_BITS 2
+
+#define RDAC_LOG_LEVEL(SHIFT) \
+ ((rdac_logging >> (SHIFT)) & ((1 << (RDAC_LOG_BITS)) - 1))
+
+#define RDAC_LOG(SHIFT, sdev, f, arg...) \
+do { \
+ if (unlikely(RDAC_LOG_LEVEL(SHIFT))) \
+ sdev_printk(KERN_INFO, sdev, RDAC_NAME ": " f "\n", ## arg); \
+} while (0);
+
static inline struct rdac_dh_data *get_rdac_data(struct scsi_device *sdev)
{
struct scsi_dh_data *scsi_dh_data = sdev->scsi_dh_data;
@@ -469,6 +494,7 @@ static int mode_select_handle_sense(stru
{
struct scsi_sense_hdr sense_hdr;
int err = SCSI_DH_IO, ret;
+ struct rdac_dh_data *h = get_rdac_data(sdev);
ret = scsi_normalize_sense(sensebuf, SCSI_SENSE_BUFFERSIZE, &sense_hdr);
if (!ret)
@@ -497,11 +523,14 @@ static int mode_select_handle_sense(stru
err = SCSI_DH_RETRY;
break;
default:
- sdev_printk(KERN_INFO, sdev,
- "MODE_SELECT failed with sense %02x/%02x/%02x.\n",
- sense_hdr.sense_key, sense_hdr.asc, sense_hdr.ascq);
+ break;
}
+ RDAC_LOG(RDAC_LOG_FAILOVER, sdev, "array %s, ctlr %d, "
+ "MODE_SELECT returned with sense %02x/%02x/%02x",
+ (char *) h->ctlr->array_name, h->ctlr->index,
+ sense_hdr.sense_key, sense_hdr.asc, sense_hdr.ascq);
+
done:
return err;
}
@@ -518,7 +547,9 @@ retry:
if (!rq)
goto done;
- sdev_printk(KERN_INFO, sdev, "%s MODE_SELECT command.\n",
+ RDAC_LOG(RDAC_LOG_FAILOVER, sdev, "array %s, ctlr %d, "
+ "%s MODE_SELECT command",
+ (char *) h->ctlr->array_name, h->ctlr->index,
(retry_cnt == RDAC_RETRY_COUNT) ? "queueing" : "retrying");
err = blk_execute_rq(q, NULL, rq, 1);
@@ -528,8 +559,12 @@ retry:
if (err == SCSI_DH_RETRY && retry_cnt--)
goto retry;
}
- if (err == SCSI_DH_OK)
+ if (err == SCSI_DH_OK) {
h->state = RDAC_STATE_ACTIVE;
+ RDAC_LOG(RDAC_LOG_FAILOVER, sdev, "array %s, ctlr %d, "
+ "MODE_SELECT completed",
+ (char *) h->ctlr->array_name, h->ctlr->index);
+ }
done:
return err;
@@ -567,6 +602,12 @@ static int rdac_check_sense(struct scsi_
struct scsi_sense_hdr *sense_hdr)
{
struct rdac_dh_data *h = get_rdac_data(sdev);
+
+ RDAC_LOG(RDAC_LOG_SENSE, sdev, "array %s, ctlr %d, "
+ "I/O returned with sense %02x/%02x/%02x",
+ (char *) h->ctlr->array_name, h->ctlr->index,
+ sense_hdr->sense_key, sense_hdr->asc, sense_hdr->ascq);
+
switch (sense_hdr->sense_key) {
case NOT_READY:
if (sense_hdr->asc == 0x04 && sense_hdr->ascq == 0x01)
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 3/3][RESUBMIT] scsi_dh_rdac: changes for rdac debug logging
@ 2009-09-04 3:42 Moger, Babu
2009-09-05 15:19 ` James Bottomley
2009-09-08 15:36 ` Chandra Seetharaman
0 siblings, 2 replies; 6+ messages in thread
From: Moger, Babu @ 2009-09-04 3:42 UTC (permalink / raw)
To: 'linux-scsi@vger.kernel.org'
Cc: 'device mapper development',
'sekharan@linux.vnet.ibm.com', Chauhan, Vijay,
Stankey, Robert, Dachepalli, Sudhir
Patch to add debugging stuff for rdac device handler.
- Added a bit mask "module parameter" rdac_logging with 2 bits for each type of logging.
- currently defined only two types of logging(failover and sense logging). Can be enhanced later if required.
- By default only failover logging is enabled which is equivalent of current logging.
Signed-off-by: Babu Moger <babu.moger@lsi.com>
Reviewed-by: Vijay Chauhan <vijay.chauhan@lsi.com>
Reviewed-by: Bob Stankey <Robert.stankey@lsi.com>
---
--- linux-2.6.31-rc5/drivers/scsi/device_handler/scsi_dh_rdac.c.orig 2009-09-03 14:52:46.000000000 -0500
+++ linux-2.6.31-rc5/drivers/scsi/device_handler/scsi_dh_rdac.c 2009-09-03 14:53:06.000000000 -0500
@@ -201,6 +201,31 @@ static const char *lun_state[] =
static LIST_HEAD(ctlr_list);
static DEFINE_SPINLOCK(list_lock);
+/*
+ * module parameter to enable rdac debug logging.
+ * 2 bits for each type of logging, only two types defined for now
+ * Can be enhanced if required at later point
+ */
+static int rdac_logging = 1;
+module_param(rdac_logging, int, S_IRUGO|S_IWUSR);
+MODULE_PARM_DESC(rdac_logging, "A bit mask of rdac logging levels, "
+ "Default is 1 - failover logging enabled, "
+ "set it to 0xF to enable all the logs");
+
+#define RDAC_LOG_FAILOVER 0
+#define RDAC_LOG_SENSE 2
+
+#define RDAC_LOG_BITS 2
+
+#define RDAC_LOG_LEVEL(SHIFT) \
+ ((rdac_logging >> (SHIFT)) & ((1 << (RDAC_LOG_BITS)) - 1))
+
+#define RDAC_LOG(SHIFT, sdev, f, arg...) \
+do { \
+ if (unlikely(RDAC_LOG_LEVEL(SHIFT))) \
+ sdev_printk(KERN_INFO, sdev, RDAC_NAME ": " f "\n", ## arg); \
+} while (0);
+
static inline struct rdac_dh_data *get_rdac_data(struct scsi_device *sdev)
{
struct scsi_dh_data *scsi_dh_data = sdev->scsi_dh_data;
@@ -469,6 +494,7 @@ static int mode_select_handle_sense(stru
{
struct scsi_sense_hdr sense_hdr;
int err = SCSI_DH_IO, ret;
+ struct rdac_dh_data *h = get_rdac_data(sdev);
ret = scsi_normalize_sense(sensebuf, SCSI_SENSE_BUFFERSIZE, &sense_hdr);
if (!ret)
@@ -497,11 +523,14 @@ static int mode_select_handle_sense(stru
err = SCSI_DH_RETRY;
break;
default:
- sdev_printk(KERN_INFO, sdev,
- "MODE_SELECT failed with sense %02x/%02x/%02x.\n",
- sense_hdr.sense_key, sense_hdr.asc, sense_hdr.ascq);
+ break;
}
+ RDAC_LOG(RDAC_LOG_FAILOVER, sdev, "array %s, ctlr %d, "
+ "MODE_SELECT returned with sense %02x/%02x/%02x",
+ (char *) h->ctlr->array_name, h->ctlr->index,
+ sense_hdr.sense_key, sense_hdr.asc, sense_hdr.ascq);
+
done:
return err;
}
@@ -518,7 +547,9 @@ retry:
if (!rq)
goto done;
- sdev_printk(KERN_INFO, sdev, "%s MODE_SELECT command.\n",
+ RDAC_LOG(RDAC_LOG_FAILOVER, sdev, "array %s, ctlr %d, "
+ "%s MODE_SELECT command",
+ (char *) h->ctlr->array_name, h->ctlr->index,
(retry_cnt == RDAC_RETRY_COUNT) ? "queueing" : "retrying");
err = blk_execute_rq(q, NULL, rq, 1);
@@ -528,8 +559,12 @@ retry:
if (err == SCSI_DH_RETRY && retry_cnt--)
goto retry;
}
- if (err == SCSI_DH_OK)
+ if (err == SCSI_DH_OK) {
h->state = RDAC_STATE_ACTIVE;
+ RDAC_LOG(RDAC_LOG_FAILOVER, sdev, "array %s, ctlr %d, "
+ "MODE_SELECT completed",
+ (char *) h->ctlr->array_name, h->ctlr->index);
+ }
done:
return err;
@@ -567,6 +602,12 @@ static int rdac_check_sense(struct scsi_
struct scsi_sense_hdr *sense_hdr)
{
struct rdac_dh_data *h = get_rdac_data(sdev);
+
+ RDAC_LOG(RDAC_LOG_SENSE, sdev, "array %s, ctlr %d, "
+ "I/O returned with sense %02x/%02x/%02x",
+ (char *) h->ctlr->array_name, h->ctlr->index,
+ sense_hdr->sense_key, sense_hdr->asc, sense_hdr->ascq);
+
switch (sense_hdr->sense_key) {
case NOT_READY:
if (sense_hdr->asc == 0x04 && sense_hdr->ascq == 0x01)
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 3/3][RESUBMIT] scsi_dh_rdac: changes for rdac debug logging
2009-09-04 3:42 [PATCH 3/3][RESUBMIT] scsi_dh_rdac: changes for rdac debug logging Moger, Babu
@ 2009-09-05 15:19 ` James Bottomley
2009-09-05 19:43 ` Moger, Babu
2009-09-08 15:36 ` Chandra Seetharaman
1 sibling, 1 reply; 6+ messages in thread
From: James Bottomley @ 2009-09-05 15:19 UTC (permalink / raw)
To: Moger, Babu
Cc: 'linux-scsi@vger.kernel.org',
'device mapper development',
'sekharan@linux.vnet.ibm.com', Chauhan, Vijay,
Stankey, Robert, Dachepalli, Sudhir
On Thu, 2009-09-03 at 21:42 -0600, Moger, Babu wrote:
> Patch to add debugging stuff for rdac device handler.
> - Added a bit mask "module parameter" rdac_logging with 2 bits for each type of logging.
> - currently defined only two types of logging(failover and sense logging). Can be enhanced later if required.
> - By default only failover logging is enabled which is equivalent of current logging.
>
> Signed-off-by: Babu Moger <babu.moger@lsi.com>
> Reviewed-by: Vijay Chauhan <vijay.chauhan@lsi.com>
> Reviewed-by: Bob Stankey <Robert.stankey@lsi.com>
>
> ---
> --- linux-2.6.31-rc5/drivers/scsi/device_handler/scsi_dh_rdac.c.orig 2009-09-03 14:52:46.000000000 -0500
> +++ linux-2.6.31-rc5/drivers/scsi/device_handler/scsi_dh_rdac.c 2009-09-03 14:53:06.000000000 -0500
> @@ -201,6 +201,31 @@ static const char *lun_state[] =
> static LIST_HEAD(ctlr_list);
> static DEFINE_SPINLOCK(list_lock);
>
> +/*
> + * module parameter to enable rdac debug logging.
> + * 2 bits for each type of logging, only two types defined for now
> + * Can be enhanced if required at later point
> + */
> +static int rdac_logging = 1;
> +module_param(rdac_logging, int, S_IRUGO|S_IWUSR);
> +MODULE_PARM_DESC(rdac_logging, "A bit mask of rdac logging levels, "
> + "Default is 1 - failover logging enabled, "
> + "set it to 0xF to enable all the logs");
> +
> +#define RDAC_LOG_FAILOVER 0
> +#define RDAC_LOG_SENSE 2
> +
> +#define RDAC_LOG_BITS 2
> +
> +#define RDAC_LOG_LEVEL(SHIFT) \
> + ((rdac_logging >> (SHIFT)) & ((1 << (RDAC_LOG_BITS)) - 1))
> +
> +#define RDAC_LOG(SHIFT, sdev, f, arg...) \
> +do { \
> + if (unlikely(RDAC_LOG_LEVEL(SHIFT))) \
> + sdev_printk(KERN_INFO, sdev, RDAC_NAME ": " f "\n", ## arg); \
> +} while (0);
> +
> static inline struct rdac_dh_data *get_rdac_data(struct scsi_device *sdev)
> {
> struct scsi_dh_data *scsi_dh_data = sdev->scsi_dh_data;
> @@ -469,6 +494,7 @@ static int mode_select_handle_sense(stru
> {
> struct scsi_sense_hdr sense_hdr;
> int err = SCSI_DH_IO, ret;
> + struct rdac_dh_data *h = get_rdac_data(sdev);
>
> ret = scsi_normalize_sense(sensebuf, SCSI_SENSE_BUFFERSIZE, &sense_hdr);
> if (!ret)
> @@ -497,11 +523,14 @@ static int mode_select_handle_sense(stru
> err = SCSI_DH_RETRY;
> break;
> default:
> - sdev_printk(KERN_INFO, sdev,
> - "MODE_SELECT failed with sense %02x/%02x/%02x.\n",
> - sense_hdr.sense_key, sense_hdr.asc, sense_hdr.ascq);
> + break;
> }
>
> + RDAC_LOG(RDAC_LOG_FAILOVER, sdev, "array %s, ctlr %d, "
> + "MODE_SELECT returned with sense %02x/%02x/%02x",
> + (char *) h->ctlr->array_name, h->ctlr->index,
> + sense_hdr.sense_key, sense_hdr.asc, sense_hdr.ascq);
> +
> done:
> return err;
> }
> @@ -518,7 +547,9 @@ retry:
> if (!rq)
> goto done;
>
> - sdev_printk(KERN_INFO, sdev, "%s MODE_SELECT command.\n",
> + RDAC_LOG(RDAC_LOG_FAILOVER, sdev, "array %s, ctlr %d, "
> + "%s MODE_SELECT command",
> + (char *) h->ctlr->array_name, h->ctlr->index,
> (retry_cnt == RDAC_RETRY_COUNT) ? "queueing" : "retrying");
>
> err = blk_execute_rq(q, NULL, rq, 1);
> @@ -528,8 +559,12 @@ retry:
> if (err == SCSI_DH_RETRY && retry_cnt--)
> goto retry;
> }
> - if (err == SCSI_DH_OK)
> + if (err == SCSI_DH_OK) {
> h->state = RDAC_STATE_ACTIVE;
> + RDAC_LOG(RDAC_LOG_FAILOVER, sdev, "array %s, ctlr %d, "
> + "MODE_SELECT completed",
> + (char *) h->ctlr->array_name, h->ctlr->index);
> + }
>
> done:
> return err;
> @@ -567,6 +602,12 @@ static int rdac_check_sense(struct scsi_
> struct scsi_sense_hdr *sense_hdr)
> {
> struct rdac_dh_data *h = get_rdac_data(sdev);
> +
> + RDAC_LOG(RDAC_LOG_SENSE, sdev, "array %s, ctlr %d, "
> + "I/O returned with sense %02x/%02x/%02x",
> + (char *) h->ctlr->array_name, h->ctlr->index,
> + sense_hdr->sense_key, sense_hdr->asc, sense_hdr->ascq);
> +
> switch (sense_hdr->sense_key) {
> case NOT_READY:
> if (sense_hdr->asc == 0x04 && sense_hdr->ascq == 0x01)
Could you run these through checkpatch.pl before sending them next time?
ERROR: trailing whitespace
#30: FILE: drivers/scsi/device_handler/scsi_dh_rdac.c:204:
+/* $
ERROR: trailing whitespace
#32: FILE: drivers/scsi/device_handler/scsi_dh_rdac.c:206:
+ * 2 bits for each type of logging, only two types defined for now $
ERROR: code indent should use tabs where possible
#47: FILE: drivers/scsi/device_handler/scsi_dh_rdac.c:221:
+ ((rdac_logging >> (SHIFT)) & ((1 << (RDAC_LOG_BITS)) - 1))$
ERROR: trailing whitespace
#113: FILE: drivers/scsi/device_handler/scsi_dh_rdac.c:605:
+^I^I$
total: 4 errors, 0 warnings, 90 lines checked
I fixed it all up this time around.
James
^ permalink raw reply [flat|nested] 6+ messages in thread
* RE: [PATCH 3/3][RESUBMIT] scsi_dh_rdac: changes for rdac debug logging
2009-09-05 15:19 ` James Bottomley
@ 2009-09-05 19:43 ` Moger, Babu
0 siblings, 0 replies; 6+ messages in thread
From: Moger, Babu @ 2009-09-05 19:43 UTC (permalink / raw)
To: James Bottomley
Cc: 'linux-scsi@vger.kernel.org',
'device mapper development',
'sekharan@linux.vnet.ibm.com', Chauhan, Vijay,
Stankey, Robert, Dachepalli, Sudhir
Thanks James.. Will take care next time..
________________________________________
From: James Bottomley [James.Bottomley@suse.de]
Sent: Saturday, September 05, 2009 10:19 AM
To: Moger, Babu
Cc: 'linux-scsi@vger.kernel.org'; 'device mapper development'; 'sekharan@linux.vnet.ibm.com'; Chauhan, Vijay; Stankey, Robert; Dachepalli, Sudhir
Subject: Re: [PATCH 3/3][RESUBMIT] scsi_dh_rdac: changes for rdac debug logging
On Thu, 2009-09-03 at 21:42 -0600, Moger, Babu wrote:
> Patch to add debugging stuff for rdac device handler.
> - Added a bit mask "module parameter" rdac_logging with 2 bits for each type of logging.
> - currently defined only two types of logging(failover and sense logging). Can be enhanced later if required.
> - By default only failover logging is enabled which is equivalent of current logging.
>
> Signed-off-by: Babu Moger <babu.moger@lsi.com>
> Reviewed-by: Vijay Chauhan <vijay.chauhan@lsi.com>
> Reviewed-by: Bob Stankey <Robert.stankey@lsi.com>
>
> ---
> --- linux-2.6.31-rc5/drivers/scsi/device_handler/scsi_dh_rdac.c.orig 2009-09-03 14:52:46.000000000 -0500
> +++ linux-2.6.31-rc5/drivers/scsi/device_handler/scsi_dh_rdac.c 2009-09-03 14:53:06.000000000 -0500
> @@ -201,6 +201,31 @@ static const char *lun_state[] =
> static LIST_HEAD(ctlr_list);
> static DEFINE_SPINLOCK(list_lock);
>
> +/*
> + * module parameter to enable rdac debug logging.
> + * 2 bits for each type of logging, only two types defined for now
> + * Can be enhanced if required at later point
> + */
> +static int rdac_logging = 1;
> +module_param(rdac_logging, int, S_IRUGO|S_IWUSR);
> +MODULE_PARM_DESC(rdac_logging, "A bit mask of rdac logging levels, "
> + "Default is 1 - failover logging enabled, "
> + "set it to 0xF to enable all the logs");
> +
> +#define RDAC_LOG_FAILOVER 0
> +#define RDAC_LOG_SENSE 2
> +
> +#define RDAC_LOG_BITS 2
> +
> +#define RDAC_LOG_LEVEL(SHIFT) \
> + ((rdac_logging >> (SHIFT)) & ((1 << (RDAC_LOG_BITS)) - 1))
> +
> +#define RDAC_LOG(SHIFT, sdev, f, arg...) \
> +do { \
> + if (unlikely(RDAC_LOG_LEVEL(SHIFT))) \
> + sdev_printk(KERN_INFO, sdev, RDAC_NAME ": " f "\n", ## arg); \
> +} while (0);
> +
> static inline struct rdac_dh_data *get_rdac_data(struct scsi_device *sdev)
> {
> struct scsi_dh_data *scsi_dh_data = sdev->scsi_dh_data;
> @@ -469,6 +494,7 @@ static int mode_select_handle_sense(stru
> {
> struct scsi_sense_hdr sense_hdr;
> int err = SCSI_DH_IO, ret;
> + struct rdac_dh_data *h = get_rdac_data(sdev);
>
> ret = scsi_normalize_sense(sensebuf, SCSI_SENSE_BUFFERSIZE, &sense_hdr);
> if (!ret)
> @@ -497,11 +523,14 @@ static int mode_select_handle_sense(stru
> err = SCSI_DH_RETRY;
> break;
> default:
> - sdev_printk(KERN_INFO, sdev,
> - "MODE_SELECT failed with sense %02x/%02x/%02x.\n",
> - sense_hdr.sense_key, sense_hdr.asc, sense_hdr.ascq);
> + break;
> }
>
> + RDAC_LOG(RDAC_LOG_FAILOVER, sdev, "array %s, ctlr %d, "
> + "MODE_SELECT returned with sense %02x/%02x/%02x",
> + (char *) h->ctlr->array_name, h->ctlr->index,
> + sense_hdr.sense_key, sense_hdr.asc, sense_hdr.ascq);
> +
> done:
> return err;
> }
> @@ -518,7 +547,9 @@ retry:
> if (!rq)
> goto done;
>
> - sdev_printk(KERN_INFO, sdev, "%s MODE_SELECT command.\n",
> + RDAC_LOG(RDAC_LOG_FAILOVER, sdev, "array %s, ctlr %d, "
> + "%s MODE_SELECT command",
> + (char *) h->ctlr->array_name, h->ctlr->index,
> (retry_cnt == RDAC_RETRY_COUNT) ? "queueing" : "retrying");
>
> err = blk_execute_rq(q, NULL, rq, 1);
> @@ -528,8 +559,12 @@ retry:
> if (err == SCSI_DH_RETRY && retry_cnt--)
> goto retry;
> }
> - if (err == SCSI_DH_OK)
> + if (err == SCSI_DH_OK) {
> h->state = RDAC_STATE_ACTIVE;
> + RDAC_LOG(RDAC_LOG_FAILOVER, sdev, "array %s, ctlr %d, "
> + "MODE_SELECT completed",
> + (char *) h->ctlr->array_name, h->ctlr->index);
> + }
>
> done:
> return err;
> @@ -567,6 +602,12 @@ static int rdac_check_sense(struct scsi_
> struct scsi_sense_hdr *sense_hdr)
> {
> struct rdac_dh_data *h = get_rdac_data(sdev);
> +
> + RDAC_LOG(RDAC_LOG_SENSE, sdev, "array %s, ctlr %d, "
> + "I/O returned with sense %02x/%02x/%02x",
> + (char *) h->ctlr->array_name, h->ctlr->index,
> + sense_hdr->sense_key, sense_hdr->asc, sense_hdr->ascq);
> +
> switch (sense_hdr->sense_key) {
> case NOT_READY:
> if (sense_hdr->asc == 0x04 && sense_hdr->ascq == 0x01)
Could you run these through checkpatch.pl before sending them next time?
ERROR: trailing whitespace
#30: FILE: drivers/scsi/device_handler/scsi_dh_rdac.c:204:
+/* $
ERROR: trailing whitespace
#32: FILE: drivers/scsi/device_handler/scsi_dh_rdac.c:206:
+ * 2 bits for each type of logging, only two types defined for now $
ERROR: code indent should use tabs where possible
#47: FILE: drivers/scsi/device_handler/scsi_dh_rdac.c:221:
+ ((rdac_logging >> (SHIFT)) & ((1 << (RDAC_LOG_BITS)) - 1))$
ERROR: trailing whitespace
#113: FILE: drivers/scsi/device_handler/scsi_dh_rdac.c:605:
+^I^I$
total: 4 errors, 0 warnings, 90 lines checked
I fixed it all up this time around.
James
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 3/3][RESUBMIT] scsi_dh_rdac: changes for rdac debug logging
2009-09-04 3:42 [PATCH 3/3][RESUBMIT] scsi_dh_rdac: changes for rdac debug logging Moger, Babu
2009-09-05 15:19 ` James Bottomley
@ 2009-09-08 15:36 ` Chandra Seetharaman
1 sibling, 0 replies; 6+ messages in thread
From: Chandra Seetharaman @ 2009-09-08 15:36 UTC (permalink / raw)
To: Moger, Babu
Cc: 'linux-scsi@vger.kernel.org',
'device mapper development', Chauhan, Vijay,
Stankey, Robert, Dachepalli, Sudhir
Acked-by: Chandra Seetharaman <sekharan@us.ibm.com>
On Thu, 2009-09-03 at 21:42 -0600, Moger, Babu wrote:
> Patch to add debugging stuff for rdac device handler.
> - Added a bit mask "module parameter" rdac_logging with 2 bits for each type of logging.
> - currently defined only two types of logging(failover and sense logging). Can be enhanced later if required.
> - By default only failover logging is enabled which is equivalent of current logging.
>
> Signed-off-by: Babu Moger <babu.moger@lsi.com>
> Reviewed-by: Vijay Chauhan <vijay.chauhan@lsi.com>
> Reviewed-by: Bob Stankey <Robert.stankey@lsi.com>
>
> ---
> --- linux-2.6.31-rc5/drivers/scsi/device_handler/scsi_dh_rdac.c.orig 2009-09-03 14:52:46.000000000 -0500
> +++ linux-2.6.31-rc5/drivers/scsi/device_handler/scsi_dh_rdac.c 2009-09-03 14:53:06.000000000 -0500
> @@ -201,6 +201,31 @@ static const char *lun_state[] =
> static LIST_HEAD(ctlr_list);
> static DEFINE_SPINLOCK(list_lock);
>
> +/*
> + * module parameter to enable rdac debug logging.
> + * 2 bits for each type of logging, only two types defined for now
> + * Can be enhanced if required at later point
> + */
> +static int rdac_logging = 1;
> +module_param(rdac_logging, int, S_IRUGO|S_IWUSR);
> +MODULE_PARM_DESC(rdac_logging, "A bit mask of rdac logging levels, "
> + "Default is 1 - failover logging enabled, "
> + "set it to 0xF to enable all the logs");
> +
> +#define RDAC_LOG_FAILOVER 0
> +#define RDAC_LOG_SENSE 2
> +
> +#define RDAC_LOG_BITS 2
> +
> +#define RDAC_LOG_LEVEL(SHIFT) \
> + ((rdac_logging >> (SHIFT)) & ((1 << (RDAC_LOG_BITS)) - 1))
> +
> +#define RDAC_LOG(SHIFT, sdev, f, arg...) \
> +do { \
> + if (unlikely(RDAC_LOG_LEVEL(SHIFT))) \
> + sdev_printk(KERN_INFO, sdev, RDAC_NAME ": " f "\n", ## arg); \
> +} while (0);
> +
> static inline struct rdac_dh_data *get_rdac_data(struct scsi_device *sdev)
> {
> struct scsi_dh_data *scsi_dh_data = sdev->scsi_dh_data;
> @@ -469,6 +494,7 @@ static int mode_select_handle_sense(stru
> {
> struct scsi_sense_hdr sense_hdr;
> int err = SCSI_DH_IO, ret;
> + struct rdac_dh_data *h = get_rdac_data(sdev);
>
> ret = scsi_normalize_sense(sensebuf, SCSI_SENSE_BUFFERSIZE, &sense_hdr);
> if (!ret)
> @@ -497,11 +523,14 @@ static int mode_select_handle_sense(stru
> err = SCSI_DH_RETRY;
> break;
> default:
> - sdev_printk(KERN_INFO, sdev,
> - "MODE_SELECT failed with sense %02x/%02x/%02x.\n",
> - sense_hdr.sense_key, sense_hdr.asc, sense_hdr.ascq);
> + break;
> }
>
> + RDAC_LOG(RDAC_LOG_FAILOVER, sdev, "array %s, ctlr %d, "
> + "MODE_SELECT returned with sense %02x/%02x/%02x",
> + (char *) h->ctlr->array_name, h->ctlr->index,
> + sense_hdr.sense_key, sense_hdr.asc, sense_hdr.ascq);
> +
> done:
> return err;
> }
> @@ -518,7 +547,9 @@ retry:
> if (!rq)
> goto done;
>
> - sdev_printk(KERN_INFO, sdev, "%s MODE_SELECT command.\n",
> + RDAC_LOG(RDAC_LOG_FAILOVER, sdev, "array %s, ctlr %d, "
> + "%s MODE_SELECT command",
> + (char *) h->ctlr->array_name, h->ctlr->index,
> (retry_cnt == RDAC_RETRY_COUNT) ? "queueing" : "retrying");
>
> err = blk_execute_rq(q, NULL, rq, 1);
> @@ -528,8 +559,12 @@ retry:
> if (err == SCSI_DH_RETRY && retry_cnt--)
> goto retry;
> }
> - if (err == SCSI_DH_OK)
> + if (err == SCSI_DH_OK) {
> h->state = RDAC_STATE_ACTIVE;
> + RDAC_LOG(RDAC_LOG_FAILOVER, sdev, "array %s, ctlr %d, "
> + "MODE_SELECT completed",
> + (char *) h->ctlr->array_name, h->ctlr->index);
> + }
>
> done:
> return err;
> @@ -567,6 +602,12 @@ static int rdac_check_sense(struct scsi_
> struct scsi_sense_hdr *sense_hdr)
> {
> struct rdac_dh_data *h = get_rdac_data(sdev);
> +
> + RDAC_LOG(RDAC_LOG_SENSE, sdev, "array %s, ctlr %d, "
> + "I/O returned with sense %02x/%02x/%02x",
> + (char *) h->ctlr->array_name, h->ctlr->index,
> + sense_hdr->sense_key, sense_hdr->asc, sense_hdr->ascq);
> +
> switch (sense_hdr->sense_key) {
> case NOT_READY:
> if (sense_hdr->asc == 0x04 && sense_hdr->ascq == 0x01)
>
>
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2009-09-08 15:36 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-09-04 3:42 [PATCH 3/3][RESUBMIT] scsi_dh_rdac: changes for rdac debug logging Moger, Babu
2009-09-05 15:19 ` James Bottomley
2009-09-05 19:43 ` Moger, Babu
2009-09-08 15:36 ` Chandra Seetharaman
-- strict thread matches above, loose matches on Subject: below --
2009-09-02 23:13 Moger, Babu
2009-08-28 14:23 Moger, Babu
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox