From: Ingo Molnar <mingo@kernel.org>
To: Mikulas Patocka <mpatocka@redhat.com>
Cc: Mike Snitzer <snitzer@redhat.com>,
Thomas Gleixner <tglx@linutronix.de>,
Dan Williams <dan.j.williams@intel.com>,
device-mapper development <dm-devel@redhat.com>,
X86 ML <x86@kernel.org>,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH v3 RESEND] x86: optimize memcpy_flushcache
Date: Mon, 10 Sep 2018 15:18:00 +0200 [thread overview]
Message-ID: <20180910131800.GA41487@gmail.com> (raw)
In-Reply-To: <alpine.LRH.2.02.1808081720460.24747@file01.intranet.prod.int.rdu2.redhat.com>
* Mikulas Patocka <mpatocka@redhat.com> wrote:
> Here I resend it:
>
>
> From: Mikulas Patocka <mpatocka@redhat.com>
> Subject: [PATCH] x86: optimize memcpy_flushcache
>
> I use memcpy_flushcache in my persistent memory driver for metadata
> updates, there are many 8-byte and 16-byte updates and it turns out that
> the overhead of memcpy_flushcache causes 2% performance degradation
> compared to "movnti" instruction explicitly coded using inline assembler.
>
> The tests were done on a Skylake processor with persistent memory emulated
> using the "memmap" kernel parameter. dd was used to copy data to the
> dm-writecache target.
>
> This patch recognizes memcpy_flushcache calls with constant short length
> and turns them into inline assembler - so that I don't have to use inline
> assembler in the driver.
>
> Signed-off-by: Mikulas Patocka <mpatocka@redhat.com>
>
> ---
> arch/x86/include/asm/string_64.h | 20 +++++++++++++++++++-
> arch/x86/lib/usercopy_64.c | 4 ++--
> 2 files changed, 21 insertions(+), 3 deletions(-)
Applied to tip:x86/asm, thanks!
I'll push it out later today after some testing.
Thanks,
Ingo
next prev parent reply other threads:[~2018-09-10 13:18 UTC|newest]
Thread overview: 66+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-05-19 5:25 [patch 0/4] dm-writecache patches Mikulas Patocka
2018-05-19 5:25 ` [patch 1/4] x86: optimize memcpy_flushcache Mikulas Patocka
2018-05-19 14:21 ` Dan Williams
2018-05-24 18:20 ` [PATCH v2] " Mike Snitzer
2018-06-18 13:23 ` [PATCH v2 RESEND] " Mike Snitzer
2018-06-21 14:31 ` Ingo Molnar
2018-06-22 1:19 ` Mikulas Patocka
2018-06-22 1:30 ` Ingo Molnar
2018-08-08 21:22 ` [PATCH v3 " Mikulas Patocka
2018-09-10 13:18 ` Ingo Molnar [this message]
2018-09-11 6:22 ` [tip:x86/asm] x86/asm: Optimize memcpy_flushcache() tip-bot for Mikulas Patocka
2018-05-19 5:25 ` [patch 2/4] swait: export the symbols __prepare_to_swait and __finish_swait Mikulas Patocka
2018-05-22 6:34 ` Christoph Hellwig
2018-05-22 18:52 ` Mike Snitzer
2018-05-23 9:21 ` Peter Zijlstra
2018-05-23 15:10 ` Mike Snitzer
2018-05-23 18:10 ` [PATCH v2] swait: export " Mike Snitzer
2018-05-23 20:38 ` Mikulas Patocka
2018-05-23 21:51 ` Mike Snitzer
2018-05-24 14:10 ` Peter Zijlstra
2018-05-24 15:09 ` Mike Snitzer
2018-05-19 5:25 ` [patch 3/4] dm-writecache Mikulas Patocka
2018-05-22 6:37 ` Christoph Hellwig
2018-05-19 5:25 ` [patch 4/4] dm-writecache: use new API for flushing Mikulas Patocka
[not found] ` <20180519052635.567438191-59tzFEB+POEAvxtiuMwx3w@public.gmane.org>
2018-05-22 6:39 ` [dm-devel] " Christoph Hellwig
[not found] ` <20180522063946.GB8054-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org>
2018-05-22 18:41 ` Mike Snitzer
[not found] ` <20180522184103.GA25826-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2018-05-22 19:00 ` Dan Williams
[not found] ` <CAPcyv4g395RccdEPF=sCu5RdCH9zwD+GvOfSp7-F0Oh64CeDgA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2018-05-22 19:19 ` Mike Snitzer
[not found] ` <20180522191942.GB25904-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2018-05-22 19:27 ` Dan Williams
[not found] ` <CAPcyv4hWswV=VCfB7KatoW_zc-kUUju2jD45N-Gsg4sW-XFe-A-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2018-05-22 20:52 ` Mike Snitzer
[not found] ` <20180522205214.GA26259-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2018-05-22 22:53 ` [dm-devel] " Jeff Moyer
[not found] ` <x49r2m3nvyk.fsf-RRHT56Q3PSP4kTEheFKJxxDDeQx5vsVwAInAS/Ez/D0@public.gmane.org>
2018-05-23 20:57 ` Mikulas Patocka
2018-05-28 13:52 ` Mikulas Patocka
[not found] ` <alpine.LRH.2.02.1805280943460.16538-Hpncn10jQN4oNljnaZt3ZvA+iT7yCHsGwRM8/txMwJMAicBL8TP8PQ@public.gmane.org>
2018-05-28 17:41 ` Dan Williams
[not found] ` <CAPcyv4gxCNqqG2G3R6f=kH3zNbByhgE2R-trJWurH49-p+51Gg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2018-05-30 13:42 ` [dm-devel] " Jeff Moyer
[not found] ` <x49h8mpmf8v.fsf-RRHT56Q3PSP4kTEheFKJxxDDeQx5vsVwAInAS/Ez/D0@public.gmane.org>
2018-05-30 13:51 ` Mikulas Patocka
2018-05-30 13:52 ` Jeff Moyer
2018-05-24 8:15 ` Mikulas Patocka
2018-05-25 3:12 ` Dan Williams
2018-05-25 6:17 ` Mikulas Patocka
[not found] ` <alpine.LRH.2.02.1805250213270.13894-Hpncn10jQN4oNljnaZt3ZvA+iT7yCHsGwRM8/txMwJMAicBL8TP8PQ@public.gmane.org>
2018-05-25 12:51 ` Mike Snitzer
[not found] ` <20180525125126.GA9275-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2018-05-25 15:57 ` Dan Williams
[not found] ` <CAPcyv4j-v5HGBg9JMBXt7UUyw=K_bGC7t5tQGfXM9G4qjOpFjQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2018-05-26 7:02 ` Mikulas Patocka
[not found] ` <alpine.LRH.2.02.1805260246250.15978-Hpncn10jQN4oNljnaZt3ZvA+iT7yCHsGwRM8/txMwJMAicBL8TP8PQ@public.gmane.org>
2018-05-26 15:26 ` Dan Williams
[not found] ` <CAPcyv4jwLsCNvNhgvv2DDSZuQsQXYEpKUU_5w1pTnyKj-uDBbQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2018-05-28 13:32 ` Mikulas Patocka
[not found] ` <alpine.LRH.2.02.1805280922190.13486-Hpncn10jQN4oNljnaZt3ZvA+iT7yCHsGwRM8/txMwJMAicBL8TP8PQ@public.gmane.org>
2018-05-28 18:14 ` Dan Williams
[not found] ` <CAPcyv4jXV4aZ6wqqjVPL8J4LxwTtXnzQCLZ0Wk5oB4_0uxpg7g-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2018-05-30 13:07 ` Mikulas Patocka
[not found] ` <alpine.LRH.2.02.1805300903220.1090-Hpncn10jQN4oNljnaZt3ZvA+iT7yCHsGwRM8/txMwJMAicBL8TP8PQ@public.gmane.org>
2018-05-30 13:16 ` Mike Snitzer
[not found] ` <20180530131623.GB2106-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2018-05-30 13:21 ` Mikulas Patocka
[not found] ` <alpine.LRH.2.02.1805300919070.11327-Hpncn10jQN4oNljnaZt3ZvA+iT7yCHsGwRM8/txMwJMAicBL8TP8PQ@public.gmane.org>
2018-05-30 13:26 ` Mike Snitzer
[not found] ` <20180530132647.GB5157-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2018-05-30 13:33 ` Mikulas Patocka
[not found] ` <alpine.LRH.2.02.1805300929500.11327-Hpncn10jQN4oNljnaZt3ZvA+iT7yCHsGwRM8/txMwJMAicBL8TP8PQ@public.gmane.org>
2018-05-30 13:54 ` Mike Snitzer
[not found] ` <20180530135421.GA81788-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2018-05-30 14:09 ` Mikulas Patocka
[not found] ` <alpine.LRH.2.02.1805300959460.16696-Hpncn10jQN4oNljnaZt3ZvA+iT7yCHsGwRM8/txMwJMAicBL8TP8PQ@public.gmane.org>
2018-05-30 14:21 ` Mike Snitzer
[not found] ` <20180530142104.GA5416-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2018-05-30 14:46 ` Mikulas Patocka
[not found] ` <alpine.LRH.2.02.1805301023380.21890-Hpncn10jQN4oNljnaZt3ZvA+iT7yCHsGwRM8/txMwJMAicBL8TP8PQ@public.gmane.org>
2018-05-31 3:42 ` Mike Snitzer
[not found] ` <20180531034216.GB88973-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2018-06-03 15:03 ` Mikulas Patocka
2018-05-31 3:39 ` Mike Snitzer
[not found] ` <20180531033919.GA88973-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2018-05-31 8:16 ` Mikulas Patocka
[not found] ` <alpine.LRH.2.02.1805310412240.30096-Hpncn10jQN4oNljnaZt3ZvA+iT7yCHsGwRM8/txMwJMAicBL8TP8PQ@public.gmane.org>
2018-05-31 12:09 ` Mike Snitzer
2018-05-30 15:58 ` Dan Williams
[not found] ` <CAPcyv4j3pDQ3YMK5Eije0dgGw+1CQYazPJ-zvqfvGF-9O_gVJw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2018-05-30 22:39 ` Dan Williams
[not found] ` <CAPcyv4gD-ZaXyV1SFAndobUYJOGLkx9rOyb5zNbzJXB_kVCzoA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2018-05-31 8:19 ` Mikulas Patocka
[not found] ` <alpine.LRH.2.02.1805310416540.30096-Hpncn10jQN4oNljnaZt3ZvA+iT7yCHsGwRM8/txMwJMAicBL8TP8PQ@public.gmane.org>
2018-05-31 14:51 ` Dan Williams
[not found] ` <CAPcyv4gzFbdBbUhPJ6W4mTJWAkvqVw_4FVhVdePEJzc28Sg-Sg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2018-05-31 15:31 ` Mikulas Patocka
[not found] ` <alpine.LRH.2.02.1805311127001.31059-Hpncn10jQN4oNljnaZt3ZvA+iT7yCHsGwRM8/txMwJMAicBL8TP8PQ@public.gmane.org>
2018-05-31 16:39 ` Dan Williams
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=20180910131800.GA41487@gmail.com \
--to=mingo@kernel.org \
--cc=dan.j.williams@intel.com \
--cc=dm-devel@redhat.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mpatocka@redhat.com \
--cc=snitzer@redhat.com \
--cc=tglx@linutronix.de \
--cc=x86@kernel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox