From: Seth Jennings <sjenning@linux.vnet.ibm.com>
To: linuxppc-dev@lists.ozlabs.org,
Benjamin Herrenschmidt <benh@kernel.crashing.org>
Cc: Minchan Kim <minchan@kernel.org>,
Robert Jennings <rcj@linux.vnet.ibm.com>,
Nitin Gupta <ngupta@vflare.org>
Subject: CPU-local TLB flushing
Date: Mon, 18 Jun 2012 15:55:33 -0500 [thread overview]
Message-ID: <4FDF95C5.3090903@linux.vnet.ibm.com> (raw)
This is a continuation of a thread a few months ago:
https://lists.ozlabs.org/pipermail/linuxppc-dev/2012-February/095775.html
zsmalloc is now in the staging tree and there are patches
on lkml to convert the x86 only tlb flushing code to arch
independent code.
https://lkml.org/lkml/2012/5/14/55
A quick back story, zsmalloc does some pte/tlb manipulation
to quickly map a pair of pages into a single VM area. It
does this with interrupts disabled in a preallocated per-cpu
VM area, which means the mapping only exists in the TLB of
the cpu that does the mapping and, therefore, only needs to
be flushed on that same cpu during the unmapping process.
Right now, zsmalloc uses __flush_tlb_one() on x86 to do a
cpu-local single entry tlb flush. Afaict, there is no such
call on ppc64.
The patch replaces that x86 call with a call to a new function,
local_unmap_kernel_range(), which is exactly the same as
unmap_kernel_range() in mm/vmalloc.c except that it calls
local_flush_tlb_kernel_range() instead of
flush_tlb_kernel_range().
A few archs support local_flush_tlb_kernel_range() already and
another patch in the patchset above, introduces this function
for x86; basically a wrapper for __flush_tlb_single().
For PPC_STD_MMU_64, it looked like all the tlb flushing
functions were just stubs, so I just added a stub for
local_flush_tlb_kernel_range(). This was stable running
a single threaded application, bound to one cpu, but crashes
with even two threads.
With local_flush_tlb_kernel_range() being a stub,
the new function local_unmap_kernel_range() is exactly
the same as unmap_kernel_range() since
local_flush_tlb_kernel_range() and flush_tlb_kernel_range()
are both stubs on ppc64.
My knowledge of the ppc64 hashing tlb design is almost
nothing, but it seems like this should work, albeit slowly
since it would be a global flush rather than cpu-local.
I was wondering if anyone could tell me why this doesn't work,
and what needs to be done to make it work.
Thanks in advance for any help! Let me know if you need
clarification on something.
--
Seth
next reply other threads:[~2012-06-18 20:55 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-06-18 20:55 Seth Jennings [this message]
2012-06-19 0:47 ` CPU-local TLB flushing Michael Ellerman
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=4FDF95C5.3090903@linux.vnet.ibm.com \
--to=sjenning@linux.vnet.ibm.com \
--cc=benh@kernel.crashing.org \
--cc=linuxppc-dev@lists.ozlabs.org \
--cc=minchan@kernel.org \
--cc=ngupta@vflare.org \
--cc=rcj@linux.vnet.ibm.com \
/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).