From: Vladislav Bolkhovitin <vst-d+Crzxg7Rs0@public.gmane.org>
To: Bart Van Assche <bvanassche-HInyCGIudOg@public.gmane.org>
Cc: linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
Roland Dreier <rolandd-FYB4Gu1CFyUAvxtiuMwx3w@public.gmane.org>,
David Dillow <dave-i1Mk8JYDVaaSihdK6806/g@public.gmane.org>,
Ralph Campbell
<ralph.campbell-h88ZbnxC6KDQT0dZR+AlfA@public.gmane.org>
Subject: Re: [PATCH] IB/srp: use multiple CPU cores more effectively
Date: Mon, 02 Aug 2010 17:08:06 +0400 [thread overview]
Message-ID: <4C56C336.4040009@vlnb.net> (raw)
In-Reply-To: <201008021015.40472.bvanassche-HInyCGIudOg@public.gmane.org>
Bart Van Assche, on 08/02/2010 12:15 PM wrote:
> SRP I/O with small block sizes causes a high CPU load. Processing IB
> completions on the context of a kernel thread instead of in interrupt context
> allows to process up to 25% more I/O operations per second. This patch does
> add a kernel parameter 'thread' that allows to specify whether to process IB
> completions in interrupt context or in kernel thread context. Also, the IB
> receive notification processing loop is rewritten as proposed earlier by Ralph
> Campbell (see also https://patchwork.kernel.org/patch/89426/). As the
> measurement results below show, rewriting the IB receive notification
> processing loop did not have a measurable impact on performance. Processing
> IB receive notifications in thread context however does have a measurable
> impact: workloads with I/O depth one are processed at most 10% slower and
> workloads with larger I/O depths are processed up to 25% faster.
>
> block size number of IOPS IOPS IOPS
> in bytes threads without with with
> ($bs) ($numjobs) this patch thread=n thread=y
> 512 1 25,400 25,400 23,100
> 512 128 122,000 122,000 153,000
> 4096 1 25,000 25,000 22,700
> 4096 128 122,000 121,000 157,000
> 65536 1 14,300 14,400 13,600
> 65536 4 36,700 36,700 36,600
> 524288 1 3,470 3,430 3,420
> 524288 4 5,020 5,020 4,990
>
> performance test used to gather the above results:
> fio --bs=${bs} --ioengine=sg --buffered=0 --size=128M --rw=read \
> --thread --numjobs=${numjobs} --loops=100 --group_reporting \
> --gtod_reduce=1 --name=${dev} --filename=${dev}
> other ib_srp kernel module parameters: srp_sg_tablesize=128
How about results of "dd Xflags=direct" in different modes to find out
the lowest latency the driver can process 512 and 4K packets? Sorry, I
don't trust fio, when it comes to precise latency measurements.
Vlad
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2010-08-02 13:08 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-08-02 8:15 [PATCH] IB/srp: use multiple CPU cores more effectively Bart Van Assche
[not found] ` <201008021015.40472.bvanassche-HInyCGIudOg@public.gmane.org>
2010-08-02 13:08 ` Vladislav Bolkhovitin [this message]
[not found] ` <4C56C336.4040009-d+Crzxg7Rs0@public.gmane.org>
2010-08-02 15:57 ` Bart Van Assche
[not found] ` <AANLkTinBTv5SZJ_H9C15CWZ5hYGFe38840zy78+N-wbO-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2010-08-02 18:16 ` Vladislav Bolkhovitin
[not found] ` <4C570B7F.2010306-d+Crzxg7Rs0@public.gmane.org>
2010-08-02 18:36 ` David Dillow
[not found] ` <1280774209.2451.10.camel-FqX9LgGZnHWDB2HL1qBt2PIbXMQ5te18@public.gmane.org>
2010-08-02 18:40 ` Bart Van Assche
[not found] ` <AANLkTikYEvQfbWGLMZGZ_c+ggy0hAkiS9RAsBmGVKDDA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2010-08-02 19:07 ` Vladislav Bolkhovitin
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=4C56C336.4040009@vlnb.net \
--to=vst-d+crzxg7rs0@public.gmane.org \
--cc=bvanassche-HInyCGIudOg@public.gmane.org \
--cc=dave-i1Mk8JYDVaaSihdK6806/g@public.gmane.org \
--cc=linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=ralph.campbell-h88ZbnxC6KDQT0dZR+AlfA@public.gmane.org \
--cc=rolandd-FYB4Gu1CFyUAvxtiuMwx3w@public.gmane.org \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.