Erez Zilber wrote:
> On Thu, Jul 30, 2009 at 6:37 AM, Mike Christie<> wrote:
>> On 07/26/2009 08:48 AM, Erez Zilber wrote:
>>> I've attached a new version. I hope it's better. Whenever possible,
>>> there's a dbg statement before&  after. For example, if we free the
>>> conn object, I can't put a dbg call after it (because conn is already
>>> NULL). If you still see specific things that need to be fixed, let me
>>> know.
>> Thanks for the work on this.
>> How about the attached.
>> - I added a ":" between the function name and debug output.
>> - Removed some extra newlines
>> - Tried to add dbg statements at the top and end of functions that can
>> take a long time or fail in odd ways because they call into the scsi
>> layer like the scanning, blocking, target removal, etc. For functions
>> like allocation, adding, destroying and freeing I tried to just add a
>> dbg statement at the top of end of the function.
>> The patch was made over the linux-2.6-iscsi tree iscsi branch.
> Looks good.

Ok. Merged in linux-2.6-iscsi. I have to do some reworking to the patches. That will take some time.


You received this message because you are subscribed to the Google Groups 
"open-iscsi" group.
To post to this group, send email to
To unsubscribe from this group, send email to
For more options, visit this group at

Reply via email to