<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
  
</head>
<body bgcolor="#ffffff" text="#000066">
<br>
<blockquote cite="mid52wtlbmh49.fsf@cisco.com" type="cite">
  <pre wrap="">I think it makes more sense to handle this the same way I handled
max_sectors: make it a per-target parameter passed in when connecting
to the target.  We could make cmds_per_lun a similar parameter, but
are there likely to be any SRP targets that need this to be limited?
Also, what is max_targets?

  </pre>
</blockquote>
OK we can do the same way that you handled max_sectors.<br>
SRP targets may prefer a specific cmds_per_lun to reach max sequential
performance.<br>
max_targets == max_id<br>
<br>
<blockquote cite="mid52wtlbmh49.fsf@cisco.com" type="cite">
  <pre wrap="">  
    > + fix the bug of reuse the iu while it's still in_use

I think I see the bug: a send may complete and have its IU recycled
before the corresponding command is completed, and end up screwing
things up.  Is this right?
  </pre>
</blockquote>
Yes<br>
<blockquote cite="mid52wtlbmh49.fsf@cisco.com" type="cite">
  <pre wrap="">If so I would prefer to fix things in a slightly different way.
Rather than a TX ring, we should just keep a list of free IUs ready to
send and only add IUs to the end of the list when we're really done
with the IU.
  </pre>
</blockquote>
A free list of IUs is fine also<br>
<br>
Vu<br>
<br>
</body>
</html>