From: "Aneesh Kumar K.V" <aneesh.kumar@linux.vnet.ibm.com>
To: benh@kernel.crashing.org, paulus@samba.org, mpe@ellerman.id.au,
Anton Blanchard <anton@ozlabs.org>
Cc: linuxppc-dev@lists.ozlabs.org,
"Aneesh Kumar K.V" <aneesh.kumar@linux.vnet.ibm.com>
Subject: [RFC PATCH 00/17] Remove slot tracking from linux page table
Date: Wed, 2 Aug 2017 11:09:59 +0530 [thread overview]
Message-ID: <20170802054016.8927-1-aneesh.kumar@linux.vnet.ibm.com> (raw)
Hi,
This patch series removes hash pte slot tracking in linux page table.
This free up 4 bits from linux page table and brings the hash and radix
linux page table closer. The series also attempt remove __real_pte_t
because without slot tracking 4k subpage and 64k page , pte formats
are similar.
However not tracking slot implies we search the hash group during invalidate
and updatepp operations. That involves searching max 16 slots to find the
matching hash page table entry. W.r.t subpages, since we don't track the
validity of slots, when invalidating 64K page, we ends up calling invalidate
for all subpages irrespective of whether we have taken a subpage
fault or not.
W.r.t THP, we skip the above and still track slots in level deposited page table.
The patch series do have an impact, hence i am sending this as an RFC series
before doing further measurements with kvm. On baremetal a kernel build gives.
Without patch:
/usr/bin/time -p make vmlinux modules > /dev/null
real 270.70
user 280.23
sys 57.99
With patch
/usr/bin/time -p make vmlinux modules > /dev/null
real 272.97
user 281.32
sys 61.46
That is 6% impact on system time: The real time impact is within the runtime
variance.
Let me know if you think we should continue with this approach.
-aneesh
Aneesh Kumar K.V (17):
powerpc/mm: Update native_hpte_find to return hash pte
powerpc/pseries: Update hpte find helper to take hash value
powerpc/ps3/mm: Add helper for finding hash pte slot using hash value
powerpc/mm: Add hash invalidate callback
powerpc/mm: use hash_invalidate for __kernel_map_pages()
powerpc/mm: Switch flush_hash_range to not use slot
powerpc/mm: Add hash updatepp callback
powerpc/mm/hash: Don't track hash pte slot number in linux page table.
powerpc/mm: Remove unused flag arg in global_invalidates
powerpc/mm: Add new firmware feature HASH API
powerpc/kvm/hash: Implement HASH_REMOVE hcall
powerpc/kvm/hash: Implement HASH_PROTECT hcall
powerpc/kvm/hash: Implement HASH_BULK_REMOVE hcall
powerpc/mm/pseries: Use HASH_PROTECT hcall in guest
powerpc/mm/pseries: Use HASH_REMOVE hcall in guest
powerpc/mm/pseries: Move slot based bulk remove to helper
powerpc/mm/pseries: Use HASH_BULK_REMOVE hcall in guest
arch/powerpc/include/asm/book3s/64/hash-4k.h | 16 +-
arch/powerpc/include/asm/book3s/64/hash-64k.h | 44 +--
arch/powerpc/include/asm/book3s/64/hash.h | 5 +-
arch/powerpc/include/asm/book3s/64/mmu-hash.h | 12 +
arch/powerpc/include/asm/book3s/64/pgtable.h | 26 --
arch/powerpc/include/asm/book3s/64/tlbflush-hash.h | 3 +-
arch/powerpc/include/asm/firmware.h | 3 +-
arch/powerpc/include/asm/hvcall.h | 5 +-
arch/powerpc/include/asm/pgtable-be-types.h | 10 -
arch/powerpc/include/asm/pgtable-types.h | 9 -
arch/powerpc/include/asm/plpar_wrappers.h | 23 ++
arch/powerpc/kvm/book3s_hv.c | 3 +
arch/powerpc/kvm/book3s_hv_rm_mmu.c | 306 ++++++++++++++++++---
arch/powerpc/kvm/book3s_hv_rmhandlers.S | 4 +
arch/powerpc/kvm/powerpc.c | 4 +
arch/powerpc/mm/dump_linuxpagetables.c | 10 -
arch/powerpc/mm/hash64_4k.c | 9 +-
arch/powerpc/mm/hash64_64k.c | 108 ++------
arch/powerpc/mm/hash_native_64.c | 172 ++++++++----
arch/powerpc/mm/hash_utils_64.c | 65 +----
arch/powerpc/mm/hugetlbpage-hash64.c | 13 +-
arch/powerpc/mm/tlb_hash64.c | 9 +-
arch/powerpc/platforms/ps3/htab.c | 88 ++++++
arch/powerpc/platforms/pseries/firmware.c | 1 +
arch/powerpc/platforms/pseries/lpar.c | 193 ++++++++++---
include/uapi/linux/kvm.h | 1 +
26 files changed, 736 insertions(+), 406 deletions(-)
--
2.13.3
next reply other threads:[~2017-08-02 5:40 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-08-02 5:39 Aneesh Kumar K.V [this message]
2017-08-02 5:40 ` [RFC PATCH 01/17] powerpc/mm: Update native_hpte_find to return hash pte Aneesh Kumar K.V
2017-08-02 5:40 ` [RFC PATCH 02/17] powerpc/pseries: Update hpte find helper to take hash value Aneesh Kumar K.V
2017-08-02 5:40 ` [RFC PATCH 03/17] powerpc/ps3/mm: Add helper for finding hash pte slot using " Aneesh Kumar K.V
2017-08-02 5:40 ` [RFC PATCH 04/17] powerpc/mm: Add hash invalidate callback Aneesh Kumar K.V
2017-08-02 5:40 ` [RFC PATCH 05/17] powerpc/mm: use hash_invalidate for __kernel_map_pages() Aneesh Kumar K.V
2017-08-02 5:40 ` [RFC PATCH 06/17] powerpc/mm: Switch flush_hash_range to not use slot Aneesh Kumar K.V
2017-08-02 5:40 ` [RFC PATCH 07/17] powerpc/mm: Add hash updatepp callback Aneesh Kumar K.V
2017-08-02 5:40 ` [RFC PATCH 08/17] powerpc/mm/hash: Don't track hash pte slot number in linux page table Aneesh Kumar K.V
2017-08-02 5:40 ` [RFC PATCH 09/17] powerpc/mm: Remove unused flag arg in global_invalidates Aneesh Kumar K.V
2017-08-02 5:40 ` [RFC PATCH 10/17] powerpc/mm: Add new firmware feature HASH API Aneesh Kumar K.V
2017-08-02 5:40 ` [RFC PATCH 11/17] powerpc/kvm/hash: Implement HASH_REMOVE hcall Aneesh Kumar K.V
2017-08-02 5:40 ` [RFC PATCH 12/17] powerpc/kvm/hash: Implement HASH_PROTECT hcall Aneesh Kumar K.V
2017-08-02 5:40 ` [RFC PATCH 13/17] powerpc/kvm/hash: Implement HASH_BULK_REMOVE hcall Aneesh Kumar K.V
2017-08-02 5:40 ` [RFC PATCH 14/17] powerpc/mm/pseries: Use HASH_PROTECT hcall in guest Aneesh Kumar K.V
2017-08-02 5:40 ` [RFC PATCH 15/17] powerpc/mm/pseries: Use HASH_REMOVE " Aneesh Kumar K.V
2017-08-02 5:40 ` [RFC PATCH 16/17] powerpc/mm/pseries: Move slot based bulk remove to helper Aneesh Kumar K.V
2017-08-02 12:50 ` Segher Boessenkool
2017-08-02 5:40 ` [RFC PATCH 17/17] powerpc/mm/pseries: Use HASH_BULK_REMOVE hcall in guest Aneesh Kumar K.V
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=20170802054016.8927-1-aneesh.kumar@linux.vnet.ibm.com \
--to=aneesh.kumar@linux.vnet.ibm.com \
--cc=anton@ozlabs.org \
--cc=benh@kernel.crashing.org \
--cc=linuxppc-dev@lists.ozlabs.org \
--cc=mpe@ellerman.id.au \
--cc=paulus@samba.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;
as well as URLs for NNTP newsgroup(s).