[lkml]   [2012]   [Jun]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH 2/3] zsmalloc: add generic path and remove x86 dependency
On 06/25/2012 12:19 PM, Greg Kroah-Hartman wrote:
> On Mon, Jun 25, 2012 at 12:10:57PM -0500, Seth Jennings wrote:
>> On 06/25/2012 11:59 AM, Greg Kroah-Hartman wrote:
>>> On Mon, Jun 25, 2012 at 11:14:37AM -0500, Seth Jennings wrote:
>>>> This patch adds generic pages mapping methods that
>>>> work on all archs in the absence of support for
>>>> local_tlb_flush_kernel_range() advertised by the
>>> Is this #define something that other arches define now? Or is this
>>> something new that you are adding here?
>> Something new I'm adding.
> Ah, ok.
>> The precedent for this approach is the __HAVE_ARCH_* defines
>> that let the arch independent stuff know if a generic
>> function needs to be defined or if there is an arch specific
>> function.
>> You can "grep -R __HAVE_ARCH_* arch/x86/" to see the ones
>> that already exist.
>> I guess I should have called it
> You need to get the mm developers to agree with this before I can take
> it.
> But, why even depend on this? Can't you either live without it

The whole point of the patch is _not_ to depend on it. It
just performs worse without it. We could just rip out all
the the page table assisted page mapping, but, for the
arches that have support for it, we'd be degrading
performance in exchange for portability. Why choose when we
can have both?

> , or just implement it for all arches somehow?

It can be implemented for some arches and already is for
some (MIPS, ARM, at least). But for some arches, I imagine
this can't be implemented due to hardware limitations.

A benefit of this approach is the arches opt-in to the
optimized zsmalloc by implementing
local_tlb_flush_kernel_range() without having to change
anything in zsmalloc.


 \ /
  Last update: 2012-06-25 21:21    [W:0.068 / U:0.128 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site