[<prev] [next>] [day] [month] [year] [list]
Message-Id: <20181012.110220.321284613911888246.davem@davemloft.net>
Date: Fri, 12 Oct 2018 11:02:20 -0700 (PDT)
From: David Miller <davem@...emloft.net>
To: kirill@...temov.name
Cc: joel@...lfernandes.org, linux-kernel@...r.kernel.org,
kernel-team@...roid.com, minchan@...nel.org, pantin@...gle.com,
hughd@...gle.com, lokeshgidra@...gle.com, dancol@...gle.com,
mhocko@...nel.org, akpm@...ux-foundation.org,
aryabinin@...tuozzo.com, luto@...nel.org, bp@...en8.de,
catalin.marinas@....com, chris@...kel.net,
dave.hansen@...ux.intel.com, elfring@...rs.sourceforge.net,
fenghua.yu@...el.com, geert@...ux-m68k.org, gxt@....edu.cn,
deller@....de, mingo@...hat.com, jejb@...isc-linux.org,
jdike@...toit.com, jonas@...thpole.se, Julia.Lawall@...6.fr,
kasan-dev@...glegroups.com, kvmarm@...ts.cs.columbia.edu,
lftan@...era.com, linux-alpha@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org,
linux-hexagon@...r.kernel.org, linux-ia64@...r.kernel.org,
linux-m68k@...ts.linux-m68k.org, linux-mips@...ux-mips.org,
linux-mm@...ck.org, linux-parisc@...r.kernel.org,
linuxppc-dev@...ts.ozlabs.org, linux-riscv@...ts.infradead.org,
linux-s390@...r.kernel.org, linux-sh@...r.kernel.org,
linux-snps-arc@...ts.infradead.org, linux-um@...ts.infradead.org,
linux-xtensa@...ux-xtensa.org, jcmvbkbc@...il.com,
nios2-dev@...ts.rocketboards.org, openrisc@...ts.librecores.org,
peterz@...radead.org, richard@....at
Subject: Re: [PATCH v2 2/2] mm: speed up mremap by 500x on large regions
From: "Kirill A. Shutemov" <kirill@...temov.name>
Date: Fri, 12 Oct 2018 14:30:56 +0300
> I looked into the code more and noticed move_pte() helper called from
> move_ptes(). It changes PTE entry to suite new address.
>
> It is only defined in non-trivial way on Sparc. I don't know much about
> Sparc and it's hard for me to say if the optimization will break anything
> there.
>
> I think it worth to disable the optimization if __HAVE_ARCH_MOVE_PTE is
> defined. Or make architectures state explicitely that the optimization is
> safe.
What sparc is doing in move_pte() is flushing the data-cache
(synchronously) if the virtual address color of the mapping changes.
Hope this helps.
Powered by blists - more mailing lists