From: Mike Christie <michaelc@cs.wisc.edu>
To: Vikas Chaudhary <vikas.chaudhary@qlogic.com>
Cc: "jbottomley@parallels.com" <jbottomley@parallels.com>,
linux-scsi <linux-scsi@vger.kernel.org>,
Lalit Chandivade <lalit.chandivade@qlogic.com>,
Ravi Anand <ravi.anand@qlogic.com>,
Adheer Chandravanshi <adheer.chandravanshi@qlogic.com>
Subject: Re: [PATCH 4/8] scsi_transport_iscsi: Exporting new attrs for iscsi session and connection in sysfs
Date: Tue, 28 May 2013 11:02:58 -0500 [thread overview]
Message-ID: <51A4D532.8080705@cs.wisc.edu> (raw)
In-Reply-To: <3FC4AB8B47ECD546BCD4B361A64BEACD30EEFEC2@avmb3.qlogic.org>
On 05/28/2013 06:55 AM, Vikas Chaudhary wrote:
>
>
> -----Original Message-----
> From: Mike Christie <michaelc@cs.wisc.edu>
> Date: Tuesday 28 May 2013 2:27 PM
> To: Vikas <vikas.chaudhary@qlogic.com>
> Cc: "jbottomley@parallels.com" <jbottomley@parallels.com>, scsi
> <linux-scsi@vger.kernel.org>, Lalit Chandivade
> <lalit.chandivade@qlogic.com>, Ravi Anand <ravi.anand@qlogic.com>, Adheer
> Chandravanshi <adheer.chandravanshi@qlogic.com>
> Subject: Re: [PATCH 4/8] scsi_transport_iscsi: Exporting new attrs for
> iscsi session and connection in sysfs
>
>> On 05/09/2013 05:02 AM, vikas.chaudhary@qlogic.com wrote:
>>> + ISCSI_PARAM_DEF_TASKMGMT_TMO,
>>
>> We currently have:
>>
>> ISCSI_PARAM_ABORT_TMO,
>> ISCSI_PARAM_LU_RESET_TMO,
>> ISCSI_PARAM_HOST_RESET_TMO,
>>
>>
>> Does your card just have the one timeout for all types of tmfs?
>>
>> I think the interface to set timers for the different eh operations
>> should be generic.
>
> We have added TASKMGMT_TMO as our adapter sets single TMO for all task
> management commands.
> Let us know if its ok or not?
>
I was thinking maybe a better name to reflect that it is for all tmfs,
but I think it is ok.
next prev parent reply other threads:[~2013-05-28 16:03 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-05-09 10:02 [PATCH 0/8] qla4xxx: Updates for scsi "misc" branch vikas.chaudhary
2013-05-09 10:02 ` [PATCH 1/8] iscsi_transport: Remove net param enum numbers vikas.chaudhary
2013-05-09 10:02 ` [PATCH 2/8] iscsi_transport: Additional parameters for net settings vikas.chaudhary
2013-05-28 8:49 ` Mike Christie
2013-05-28 8:50 ` Mike Christie
2013-05-28 11:36 ` Vikas Chaudhary
2013-05-31 14:26 ` Mike Christie
2013-05-09 10:02 ` [PATCH 3/8] qla4xxx: " vikas.chaudhary
2013-05-09 10:02 ` [PATCH 4/8] scsi_transport_iscsi: Exporting new attrs for iscsi session and connection in sysfs vikas.chaudhary
2013-05-28 8:57 ` Mike Christie
2013-05-28 11:55 ` Vikas Chaudhary
2013-05-28 16:02 ` Mike Christie [this message]
2013-05-09 10:02 ` [PATCH 5/8] libiscsi: " vikas.chaudhary
2013-05-09 10:02 ` [PATCH 6/8] qla4xxx: " vikas.chaudhary
2013-05-09 10:02 ` [PATCH 7/8] qla4xxx: Add support to allow flashnode multi-session login vikas.chaudhary
2013-05-09 10:02 ` [PATCH 8/8] qla4xxx: Update driver version to 5.03.00-k10 vikas.chaudhary
2013-06-26 12:46 ` [PATCH 0/8] qla4xxx: Updates for scsi "misc" branch Vikas Chaudhary
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=51A4D532.8080705@cs.wisc.edu \
--to=michaelc@cs.wisc.edu \
--cc=adheer.chandravanshi@qlogic.com \
--cc=jbottomley@parallels.com \
--cc=lalit.chandivade@qlogic.com \
--cc=linux-scsi@vger.kernel.org \
--cc=ravi.anand@qlogic.com \
--cc=vikas.chaudhary@qlogic.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.