Ulrich Windl wrote:
> On 13 Aug 2009 at 11:55, Mike Christie wrote:
> 
>> some additions:
>>
>>
>> Mike Christie wrote:
>>> wait_on_commands()
>>> {
>>      while (check_restrictions(conn->task) &&
>>              session->state == LOGGED_IN)
>>              wait for a while
> 
> "wait for a while" always looks like a design flaw to me. Wait for what?
> 
For the xmit thread to catch up. Due to the asynchronous nature the
xmit thread is independent on the error handler thread, so the
eh thread has to be synchronized here until the xmit thread has processed
all outstanding commands.

Tricky, that. And the main reason why I didn't do it, but used the simpler
approach :-)

Cheers,

Hannes
-- 
Dr. Hannes Reinecke                   zSeries & Storage
[email protected]                          +49 911 74053 688
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg
GF: Markus Rex, HRB 16746 (AG Nürnberg)

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

Reply via email to