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. -- 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
