Re: do discovery through SW transports too

2013-06-11 Thread Mike Christie
On 6/9/13 9:54 AM, Or Gerlitz wrote: usr]# make Hey, Did you figure this out? Support for that changed. You need to do make from the top level dir, because of dependencies on other stuff. Once that stuff is built you can do make from the usr dir to build just the usr stuff. -- You

Re: [RFC_V5 PATCH 1/3] scsi_transport_iscsi: Add flash node mgmt support

2013-06-11 Thread Mike Christie
On 6/10/13 10:47 PM, Vikas Chaudhary wrote: -Original Message- From: Eddie Wai w...@broadcom.com Date: Wednesday 5 June 2013 7:00 AM To: shyam_i...@dell.com shyam_i...@dell.com Cc: open-iscsi@googlegroups.com open-iscsi@googlegroups.com, Vikas vikas.chaudh...@qlogic.com, Lalit

Re: [RFC_V5 PATCH 1/3] scsi_transport_iscsi: Add flash node mgmt support

2013-06-11 Thread Eddie Wai
On Tue, 2013-06-11 at 12:25 -0500, Mike Christie wrote: On 6/10/13 10:47 PM, Vikas Chaudhary wrote: -Original Message- From: Eddie Wai w...@broadcom.com Date: Wednesday 5 June 2013 7:00 AM To: shyam_i...@dell.com shyam_i...@dell.com Cc: open-iscsi@googlegroups.com

Re: [RFC_V5 PATCH 1/3] scsi_transport_iscsi: Add flash node mgmt support

2013-06-11 Thread Mike Christie
On 06/11/2013 12:43 PM, Eddie Wai wrote: On Tue, 2013-06-11 at 12:25 -0500, Mike Christie wrote: On 6/10/13 10:47 PM, Vikas Chaudhary wrote: -Original Message- From: Eddie Wai w...@broadcom.com Date: Wednesday 5 June 2013 7:00 AM To: shyam_i...@dell.com shyam_i...@dell.com Cc:

Re: [RFC_V5 PATCH 1/3] scsi_transport_iscsi: Add flash node mgmt support

2013-06-11 Thread Eddie Wai
On Tue, 2013-06-11 at 13:26 -0500, Mike Christie wrote: On 06/11/2013 12:43 PM, Eddie Wai wrote: On Tue, 2013-06-11 at 12:25 -0500, Mike Christie wrote: On 6/10/13 10:47 PM, Vikas Chaudhary wrote: -Original Message- From: Eddie Wai w...@broadcom.com Date: Wednesday 5

Re: do discovery through SW transports too

2013-06-11 Thread Or Gerlitz
On 06/06/2013 18:01, Mike Christie wrote: However, above I am not talking about that or doing discovery over a normal session in general. I was just trying to get clarification for what you wanted. I was not sure if there was some new iser spec stuff that I missed and you wanted to implement.

Re: do discovery through SW transports too

2013-06-11 Thread Or Gerlitz
On 06/06/2013 18:01, Mike Christie wrote: I think everything should be there. I thought I worked on iser when doing the work too. You need the attached kernel patch. In the other mails I think I said you need a change to the userspace iser code, but ignore that. In the common iscsi code I did a

iscsi initiaor, obsolete volumes remain zombie

2013-06-11 Thread Eli Shagam
Hi, I am new to iscsi. The iscsi in normal environment works fine. Both target and initiators are running on ubuntu. When a new lun is added to a target, it automatically appears on the initiators. But when a lun is removed on the target side, it is remain a zombie, on the initiator. Is there

rhel6.3 - iscsid's ability to reconnect after server restart, or service restart (open-iscsi 2.0-872.41)

2013-06-11 Thread torgeir . wulfsberg
Hi! rhel6.3 (unpatched) iscsistart version 2.0-872.41.el6 iscsid version 2.0-872.41.el6 iscsi-initiator-utils-6.2.0.872-41.el6.x86_64 We use rhel6.3 as diskless clients (iscsi) against our NetApp. Connecting through iBFT with iPXE at BIOS level. Adjusting

do the undoable - survive network disconnections under heavy load

2013-06-11 Thread uri . milman
Hey, I've a system which uses open-iscsi and mpp driver, to connect an external storage. The system reads data from all disks, and writes to some of them. Kernel is 2.6.27, but latest open-iscsi and mpp. When the storage is down, the system cannot umount some of the disks, and the application

iSCSI and multipath failover

2013-06-11 Thread Bubuli Nayak
Hello experts, I have learnt from MIke and others comment that multipath failover would be driven by nop timout + nop interval + replacement_timeout seconds. My question is what is the impact I set replacement_timeout to 0. I know if NOPOUT interval is low , more frequently iSCSI initiator would

do discovery through SW transports too

2013-06-11 Thread Or Gerlitz
Hi Mike, There are iser environments where we might need to do iscsi discovery over rdma connection, that is establish iser connection and then carry the discovery session over it. I was a bit away for the developments in the initiator over the last months so would need some crash help here.

iSCSI and multipath failover

2013-06-11 Thread bubuli nayak
Hello experts, I have learnt from MIke and others comment that multipath failover would be driven by nop timout + nop interval + replacement_timeout seconds. My question is what is the impact I set replacement_timeout to 0. I know if NOPOUT interval is low , more frequently iSCSI initiator

Re: iSCSI and multipath failover

2013-06-11 Thread Mike Christie
On 06/07/2013 12:55 AM, Bubuli Nayak wrote: Hello experts, I have learnt from MIke and others comment that multipath failover would be driven by nop timout + nop interval + replacement_timeout seconds. My question is what is the impact I set replacement_timeout to 0. I know if NOPOUT

[PATCH] iscsid: fix order of setting uid/gid and drop supplementary groups

2013-06-11 Thread Chris Leech
If using the user and group ID settings together the existing order of calling setuid first will almost always cause the setgid call to fail, assuming the new effective user id does not have the CAP_SETGID capability. The effective group ID needs to change first. While we're at it, if iscsid is

Re: iSCSI and multipath failover

2013-06-11 Thread Bubuli Nayak
Thanks Mike. I have few follow-up question please. Why is this default replacement timeout so high. If I understand correctly in a multipath environment it is perfectly alright to set it to 0 without any side effect. The other question is, how early iSCSI can detect session error , be it with

[PATCH 2/7] actor: simplify actor_check

2013-06-11 Thread Chris Leech
From: Adam Jackson a...@redhat.com Now that it's static we know it can't ever be called when poll_in_progress is true. Signed-off-by: Adam Jackson a...@redhat.com --- usr/actor.c | 15 --- 1 file changed, 4 insertions(+), 11 deletions(-) diff --git a/usr/actor.c b/usr/actor.c index

[PATCH 4/7] actor: Remove ACTOR_TICKS_10MS()

2013-06-11 Thread Chris Leech
From: Adam Jackson a...@redhat.com Again, just obfuscation. Signed-off-by: Adam Jackson a...@redhat.com --- usr/actor.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/usr/actor.c b/usr/actor.c index 9c2f449..e4e8a24 100644 --- a/usr/actor.c +++ b/usr/actor.c @@ -38,7

iscsid: stop 250ms actor poll when idle

2013-06-11 Thread Chris Leech
iscsid doesn't need to wake up every 250ms, we can drive the polling of the actor callbacks only when they are needed. Passing on the prep patches as they were sent to me by Adam Jackson, but I've reworked the final event loop changes to keep the time keeping for scheduled actors from falling

[PATCH 3/7] actor: s/ACTOR_TICKS/actor_jiffies/

2013-06-11 Thread Chris Leech
From: Adam Jackson a...@redhat.com No reason to obscure this. Signed-off-by: Adam Jackson a...@redhat.com --- usr/actor.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/usr/actor.c b/usr/actor.c index fb90924..9c2f449 100644 --- a/usr/actor.c +++ b/usr/actor.c @@

[PATCH 6/7] actor: Simplify actor_poll a little

2013-06-11 Thread Chris Leech
From: Adam Jackson a...@redhat.com Finally, a functionality change. Having expanded ACTOR_MAX_LOOPS it's now obvious that we were basically only doing actor_check() on every other call. Also, the commentary is just wrong on many levels, we'd only get called about every 250ms anyway. Just call

[PATCH 7/7] actor: Don't wake up poll() on a timeout if we don't need to

2013-06-11 Thread Chris Leech
Have actor_poll take an indication from the outer event loop of how many ticks of ACTOR_RESOLUTION have passed, and return how many ticks into the future it would like to be called again. Then in event_loop, try and keep the poll timeouts as long as possible without delaying actors or screwing up

[PATCH 5/7] actor: Unobfuscate ACTOR_MAX_LOOPS

2013-06-11 Thread Chris Leech
From: Adam Jackson a...@redhat.com Signed-off-by: Adam Jackson a...@redhat.com --- usr/actor.c | 2 +- usr/actor.h | 1 - 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/usr/actor.c b/usr/actor.c index e4e8a24..c084176 100644 --- a/usr/actor.c +++ b/usr/actor.c @@ -240,7 +240,7 @@

[PATCH 1/7] actor: Mark actor_check static

2013-06-11 Thread Chris Leech
From: Adam Jackson a...@redhat.com Signed-off-by: Adam Jackson a...@redhat.com --- usr/actor.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/usr/actor.c b/usr/actor.c index b8f8e61..880bf81 100644 --- a/usr/actor.c +++ b/usr/actor.c @@ -199,7 +199,7 @@