Re: [PATCH 0/2] Add MaxQueueCmd

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 13/08/2013 02:55, FUJITA Tomonori wrote:
On Thu,  8 Aug 2013 16:03:34 +0300
Shlomo Pongratz <shlomop@xxxxxxxxxxxx> wrote:

This patch series does two things.

The first patch is a restructuring patch that makes it easy to configure tgtd local parameters via tgtadm.
Local parmeters in this context are parameters that are not sent to the initiator.
Currently there is one such parameter i.e. MaxXmitDataSegmentLength.

The second patch makes use of the first one in order to add a new parameter named MaxQueueCmd.
The value of this parameter varies between 1 and 512, and is default value is set to 128 in order to keep backward compatibility.
The need for the 2nd patch raised when running tgt against optimized iser initiators,
where the hard-coded value of 128 in-flight command become a bottleneck for the maximal number of IOPS.
Once addressed we were able to get decent improvement.


Shlomo Pongratz (2):
   Enable more iscsi local params
   Add MaxQueueCmd iscsi local parameter

  usr/iscsi/iscsi_if.h  |    5 ++++-
  usr/iscsi/iscsid.c    |   26 +++++++++++++++-----------
  usr/iscsi/iscsid.h    |    5 +++++
  usr/iscsi/iser.c      |   15 ++++++++-------
  usr/iscsi/iser_text.c |    4 ++--
  usr/iscsi/param.c     |    8 ++++++--
  usr/iscsi/session.c   |    2 ++
  usr/iscsi/target.c    |    4 +++-
  8 files changed, 45 insertions(+), 24 deletions(-)
Looks reasonable. Applied, thanks.

Hi Tomo,

I don't see the 2nd patch on your git, only the 1st one, was that somehow forgotten?

Or.


5040543 iscsi: Enable more iscsi local params
e7f651f iscsi: Fix leak of task->data
1e3be54 Add the possibility for building a debian package
3664a39 Allow recursive includes in /etc/tgt/conf.d
2559a86 tgt 1.0.38

--
To unsubscribe from this list: send the line "unsubscribe stgt" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux SCSI]     [Linux RAID]     [Linux Clusters]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]

  Powered by Linux