On 02/03/2010 10:08 AM, Or Gerlitz wrote:
Mike Christie wrote:
So in the end the only lock we would have in the io path is the per task one
and what you were thinking the patch buy us? less cpu usage? more iops? did you
had some proof-of-concept testbed that yielded this under the patch?
More iops was the goal. It seemed like too many people were waiting on
the session lock (iscsi_queueocmmand, iscsi_xmit_data and
iscsi_complete_pdu were). I did not really get to test the patch. The
patch works worse then the original code due to the reordering bug I
added when I removed the session lock use from iscsi_data_xmit (this was
the queue_map reordering issue I meantioned in the other mail).
--
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?hl=en.
