Re: [PATCH 0/2] Add MaxQueueCmd

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

 



On Tue, 13 Aug 2013 17:44:15 +0300
Or Gerlitz <ogerlitz@xxxxxxxxxxxx> wrote:

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

Oops, I fixed some checkscript warnings and 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



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

  Powered by Linux