linux-rdma.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Bart Van Assche <Bart.VanAssche@wdc.com>
To: "roman.penyaev@profitbricks.com" <roman.penyaev@profitbricks.com>,
	"linux-block@vger.kernel.org" <linux-block@vger.kernel.org>,
	"linux-rdma@vger.kernel.org" <linux-rdma@vger.kernel.org>,
	"sagi@grimberg.me" <sagi@grimberg.me>
Cc: "hch@infradead.org" <hch@infradead.org>,
	"danil.kipnis@profitbricks.com" <danil.kipnis@profitbricks.com>,
	"jinpu.wang@profitbricks.com" <jinpu.wang@profitbricks.com>,
	"axboe@kernel.dk" <axboe@kernel.dk>,
	"ogerlitz@mellanox.com" <ogerlitz@mellanox.com>
Subject: Re: [PATCH 00/24] InfiniBand Transport (IBTRS) and Network Block Device (IBNBD)
Date: Mon, 5 Feb 2018 16:58:03 +0000	[thread overview]
Message-ID: <1517849882.3764.13.camel@wdc.com> (raw)
In-Reply-To: <23dcfda7-eac1-ed13-b24e-3586c284ee55@grimberg.me>

On Mon, 2018-02-05 at 14:16 +0200, Sagi Grimberg wrote:
> - Your latency measurements are surprisingly high for a null target
>    device (even for low end nvme device actually) regardless of the
>    transport implementation.
> 
> For example:
> - QD=1 read latency is 648.95 for ibnbd (I assume usecs right?) which is
>    fairly high. on nvme-rdma its 1058 us, which means over 1 millisecond
>    and even 1.254 ms for srp. Last time I tested nvme-rdma read QD=1
>    latency I got ~14 us. So something does not add up here. If this is
>    not some configuration issue, then we have serious bugs to handle..
> 
> - QD=16 the read latencies are > 10ms for null devices?! I'm having
>    troubles understanding how you were able to get such high latencies
>    (> 100 ms for QD>=100)
> 
> Can you share more information about your setup? It would really help
> us understand more.

I would also appreciate it if more information could be provided about the
measurement results. In addition to answering Sagi's questions, would it
be possible to share the fio job that was used for measuring latency? In
https://events.static.linuxfound.org/sites/events/files/slides/Copy%20of%20IBNBD-Vault-2017-5.pdf
I found the following:

iodepth=128
iodepth_batch_submit=128

If you want to keep the pipeline full I think that you need to set the
iodepth_batch_submit parameter to a value that is much lower than iodepth.
I think that setting iodepth_batch_submit equal to iodepth will yield
suboptimal IOPS results. Jens, please correct me if I got this wrong.

Thanks,

Bart.



  parent reply	other threads:[~2018-02-05 16:58 UTC|newest]

Thread overview: 77+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-02-02 14:08 [PATCH 00/24] InfiniBand Transport (IBTRS) and Network Block Device (IBNBD) Roman Pen
2018-02-02 14:08 ` [PATCH 02/24] ibtrs: private headers with IBTRS protocol structs and helpers Roman Pen
2018-02-02 14:08 ` [PATCH 05/24] ibtrs: client: main functionality Roman Pen
2018-02-02 16:54   ` Bart Van Assche
2018-02-05 13:27     ` Roman Penyaev
     [not found]       ` <CAJrWOzD0YXWGCpR+BTOAJ7HhuVG_cjuomBK7vpGeTYu3H+DE4w-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2018-02-05 14:14         ` Sagi Grimberg
     [not found]           ` <f0c64f7f-2870-0657-11ec-123a3e99b7a7-NQWnxTmZq1alnMjI0IkVqw@public.gmane.org>
2018-02-05 17:05             ` Roman Penyaev
2018-02-05 11:19   ` Sagi Grimberg
     [not found]     ` <c593c579-c5b9-8a3e-10dd-c7731b91b55a-NQWnxTmZq1alnMjI0IkVqw@public.gmane.org>
2018-02-05 14:19       ` Roman Penyaev
     [not found]         ` <CAJrWOzBRar_YjYJMYcmk7tr7o+L7TdrXLv78dL84zV2zvTDJGQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2018-02-05 16:24           ` Bart Van Assche
2018-02-02 14:08 ` [PATCH 06/24] ibtrs: client: statistics functions Roman Pen
2018-02-02 14:08 ` [PATCH 08/24] ibtrs: server: private header with server structs and functions Roman Pen
2018-02-02 14:08 ` [PATCH 10/24] ibtrs: server: statistics functions Roman Pen
2018-02-02 14:08 ` [PATCH 12/24] ibtrs: include client and server modules into kernel compilation Roman Pen
2018-02-02 14:08 ` [PATCH 13/24] ibtrs: a bit of documentation Roman Pen
     [not found] ` <20180202140904.2017-1-roman.penyaev-EIkl63zCoXaH+58JC4qpiA@public.gmane.org>
2018-02-02 14:08   ` [PATCH 01/24] ibtrs: public interface header to establish RDMA connections Roman Pen
2018-02-02 14:08   ` [PATCH 03/24] ibtrs: core: lib functions shared between client and server modules Roman Pen
2018-02-05 10:52     ` Sagi Grimberg
2018-02-06 12:01       ` Roman Penyaev
2018-02-06 16:10         ` Jason Gunthorpe
     [not found]           ` <20180206161005.GE10095-uk2M96/98Pc@public.gmane.org>
2018-02-07 10:34             ` Roman Penyaev
2018-02-02 14:08   ` [PATCH 04/24] ibtrs: client: private header with client structs and functions Roman Pen
     [not found]     ` <20180202140904.2017-5-roman.penyaev-EIkl63zCoXaH+58JC4qpiA@public.gmane.org>
2018-02-05 10:59       ` Sagi Grimberg
2018-02-06 12:23         ` Roman Penyaev
2018-02-02 14:08   ` [PATCH 07/24] ibtrs: client: sysfs interface functions Roman Pen
     [not found]     ` <20180202140904.2017-8-roman.penyaev-EIkl63zCoXaH+58JC4qpiA@public.gmane.org>
2018-02-05 11:20       ` Sagi Grimberg
2018-02-06 12:28         ` Roman Penyaev
2018-02-02 14:08   ` [PATCH 09/24] ibtrs: server: main functionality Roman Pen
     [not found]     ` <20180202140904.2017-10-roman.penyaev-EIkl63zCoXaH+58JC4qpiA@public.gmane.org>
2018-02-05 11:29       ` Sagi Grimberg
2018-02-06 12:46         ` Roman Penyaev
2018-02-02 14:08   ` [PATCH 11/24] ibtrs: server: sysfs interface functions Roman Pen
2018-02-02 14:08   ` [PATCH 14/24] ibnbd: private headers with IBNBD protocol structs and helpers Roman Pen
2018-02-02 14:08   ` [PATCH 15/24] ibnbd: client: private header with client structs and functions Roman Pen
2018-02-02 14:08   ` [PATCH 16/24] ibnbd: client: main functionality Roman Pen
     [not found]     ` <20180202140904.2017-17-roman.penyaev-EIkl63zCoXaH+58JC4qpiA@public.gmane.org>
2018-02-02 15:11       ` Jens Axboe
2018-02-05 12:54         ` Roman Penyaev
2018-02-02 14:08   ` [PATCH 17/24] ibnbd: client: sysfs interface functions Roman Pen
2018-02-02 14:09   ` [PATCH 20/24] ibnbd: server: functionality for IO submission to file or block dev Roman Pen
2018-02-02 14:09   ` [PATCH 21/24] ibnbd: server: sysfs interface functions Roman Pen
2018-02-02 14:09   ` [PATCH 23/24] ibnbd: a bit of documentation Roman Pen
2018-02-02 15:55     ` Bart Van Assche
2018-02-05 13:03       ` Roman Penyaev
2018-02-05 14:16         ` Sagi Grimberg
2018-02-02 17:05   ` [PATCH 00/24] InfiniBand Transport (IBTRS) and Network Block Device (IBNBD) Bart Van Assche
2018-02-05  8:56     ` Jinpu Wang
2018-02-05 11:36       ` Sagi Grimberg
     [not found]         ` <40a7fc35-f86c-1d9d-f057-e5822708fc32-NQWnxTmZq1alnMjI0IkVqw@public.gmane.org>
2018-02-05 13:38           ` Danil Kipnis
2018-02-05 14:17             ` Sagi Grimberg
2018-02-05 16:40               ` Danil Kipnis
2018-02-05 18:38                 ` Bart Van Assche
     [not found]                   ` <9abc87a6-cb47-d3bf-65c6-6f386401bdb9-Sjgp3cTcYWE@public.gmane.org>
2018-02-06  9:44                     ` Danil Kipnis
     [not found]                       ` <CAHg0Huxejb065yVRLquNUKDZHB=NhuCBOnkgE-Q4FW3_Y-A_rQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2018-02-06 15:35                         ` Bart Van Assche
2018-02-05 16:16       ` Bart Van Assche
     [not found]         ` <1517847408.3764.5.camel-Sjgp3cTcYWE@public.gmane.org>
2018-02-05 16:36           ` Jinpu Wang
2018-02-07 16:35         ` Christopher Lameter
2018-02-07 17:18           ` Roman Penyaev
2018-02-07 17:32             ` Bart Van Assche
     [not found]               ` <1518024719.2870.39.camel-Sjgp3cTcYWE@public.gmane.org>
2018-02-08 17:38                 ` Danil Kipnis
2018-02-08 18:09                   ` Bart Van Assche
2018-02-05 12:16   ` Sagi Grimberg
2018-02-05 12:30     ` Sagi Grimberg
     [not found]       ` <1270775c-950b-76f7-be4f-d5a3198395c2-NQWnxTmZq1alnMjI0IkVqw@public.gmane.org>
2018-02-07 13:06         ` Roman Penyaev
2018-02-05 16:58     ` Bart Van Assche [this message]
2018-02-05 17:16       ` Roman Penyaev
     [not found]         ` <CAJrWOzDwDAt-w08-zmhJHsgedTyGQ-Dp=vHTvmrdSz9st7npxQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2018-02-05 17:20           ` Bart Van Assche
2018-02-06 11:47             ` Roman Penyaev
     [not found]     ` <23dcfda7-eac1-ed13-b24e-3586c284ee55-NQWnxTmZq1alnMjI0IkVqw@public.gmane.org>
2018-02-06 13:12       ` Roman Penyaev
2018-02-06 16:01         ` Bart Van Assche
     [not found]           ` <1517932890.3934.30.camel-Sjgp3cTcYWE@public.gmane.org>
2018-02-07 12:57             ` Roman Penyaev
     [not found]               ` <CAJrWOzD3+tcA1cvSNOsN8t7uCOidJ2x6wsqqvFiEHdxW2th=sA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2018-02-07 16:35                 ` Bart Van Assche
2018-02-02 14:08 ` [PATCH 18/24] ibnbd: server: private header with server structs and functions Roman Pen
2018-02-02 14:08 ` [PATCH 19/24] ibnbd: server: main functionality Roman Pen
2018-02-02 14:09 ` [PATCH 22/24] ibnbd: include client and server modules into kernel compilation Roman Pen
2018-02-02 14:09 ` [PATCH 24/24] MAINTAINERS: Add maintainer for IBNBD/IBTRS modules Roman Pen
2018-02-02 16:07 ` [PATCH 00/24] InfiniBand Transport (IBTRS) and Network Block Device (IBNBD) Bart Van Assche
     [not found]   ` <1517587623.2675.8.camel-XdAiOPVOjttBDgjK7y7TUQ@public.gmane.org>
2018-02-02 16:40     ` Doug Ledford
     [not found]       ` <1517589637.3936.16.camel-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2018-02-05  8:45         ` Jinpu Wang

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1517849882.3764.13.camel@wdc.com \
    --to=bart.vanassche@wdc.com \
    --cc=axboe@kernel.dk \
    --cc=danil.kipnis@profitbricks.com \
    --cc=hch@infradead.org \
    --cc=jinpu.wang@profitbricks.com \
    --cc=linux-block@vger.kernel.org \
    --cc=linux-rdma@vger.kernel.org \
    --cc=ogerlitz@mellanox.com \
    --cc=roman.penyaev@profitbricks.com \
    --cc=sagi@grimberg.me \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).