On 09/18/2015 05:38 AM, adheer.chandravan...@qlogic.com wrote:
> From: Adheer Chandravanshi <adheer.chandravan...@qlogic.com>
> Mike,
> This is patchset v3 to add ping support in iscsiuio.
> Please review and apply following patches to open-iscsi.git tree at your 
> earliest convenience.
> Changes with respect to v2 patchset:
>  * Corrected the logic for ping status message
>  * Change the transport callout name to exec_ping 
> Adheer Chandravanshi (2):
>   iscsid: Changes to support ping through iscsiuio
>   iscsiuio: Add ping support through iscsiuio

Do you distro people have any comments?

My only concern is that if this is the first command run for the iface,
the ping will fail. The user has to retry. I thought it might cause
confusion. For normal session and discovery session login, we retry the
same error for the user.

The options on the table are:

1. Add the apply op command. The user will have to run this before
running ping.

2. Add the apply op, and add a udev rule so when the module is loaded we
can have it run automatically.

If we go this route, do distro people want us to get the rule in udev or
carry it in the iscsi package and install it.

3. Have the ping command retry like is done with normal/discovery
session login.

You received this message because you are subscribed to the Google Groups 
"open-iscsi" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to open-iscsi+unsubscr...@googlegroups.com.
To post to this group, send email to open-iscsi@googlegroups.com.
Visit this group at http://groups.google.com/group/open-iscsi.
For more options, visit https://groups.google.com/d/optout.

Reply via email to