Erez Zilber wrote:
> On Thu, Jul 30, 2009 at 6:37 AM, Mike Christie<micha...@cs.wisc.edu> 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 
open-iscsi.org patches. That will take some time.

Thanks.

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"open-iscsi" group.
To post to this group, send email to open-iscsi@googlegroups.com
To unsubscribe from this group, send email to 
open-iscsi+unsubscr...@googlegroups.com
For more options, visit this group at http://groups.google.com/group/open-iscsi
-~----------~----~----~----~------~----~------~--~---

Reply via email to