From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-lf0-f70.google.com (mail-lf0-f70.google.com [209.85.215.70]) by kanga.kvack.org (Postfix) with ESMTP id 7F9EF6B03C7 for ; Wed, 21 Dec 2016 13:13:50 -0500 (EST) Received: by mail-lf0-f70.google.com with SMTP id d16so8793665lfb.7 for ; Wed, 21 Dec 2016 10:13:50 -0800 (PST) Received: from asavdk3.altibox.net (asavdk3.altibox.net. [109.247.116.14]) by mx.google.com with ESMTPS id m77si2243868lfm.80.2016.12.21.10.13.48 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Wed, 21 Dec 2016 10:13:48 -0800 (PST) Date: Wed, 21 Dec 2016 19:13:46 +0100 From: Sam Ravnborg Subject: Re: [RFC PATCH 02/14] sparc64: add new fields to mmu context for shared context support Message-ID: <20161221181346.GC3311@ravnborg.org> References: <1481913337-9331-1-git-send-email-mike.kravetz@oracle.com> <1481913337-9331-3-git-send-email-mike.kravetz@oracle.com> <20161217073813.GB23567@ravnborg.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Sender: owner-linux-mm@kvack.org List-ID: To: Mike Kravetz Cc: sparclinux@vger.kernel.org, linux-mm@kvack.org, linux-kernel@vger.kernel.org, "David S . Miller" , Bob Picco , Nitin Gupta , Vijay Kumar , Julian Calaby , Adam Buchbinder , "Kirill A . Shutemov" , Michal Hocko , Andrew Morton On Sun, Dec 18, 2016 at 03:45:31PM -0800, Mike Kravetz wrote: > On 12/16/2016 11:38 PM, Sam Ravnborg wrote: > > Hi Mike > > > >> diff --git a/arch/sparc/include/asm/mmu_context_64.h b/arch/sparc/include/asm/mmu_context_64.h > >> index b84be67..d031799 100644 > >> --- a/arch/sparc/include/asm/mmu_context_64.h > >> +++ b/arch/sparc/include/asm/mmu_context_64.h > >> @@ -35,15 +35,15 @@ void __tsb_context_switch(unsigned long pgd_pa, > >> static inline void tsb_context_switch(struct mm_struct *mm) > >> { > >> __tsb_context_switch(__pa(mm->pgd), > >> - &mm->context.tsb_block[0], > >> + &mm->context.tsb_block[MM_TSB_BASE], > >> #if defined(CONFIG_HUGETLB_PAGE) || defined(CONFIG_TRANSPARENT_HUGEPAGE) > >> - (mm->context.tsb_block[1].tsb ? > >> - &mm->context.tsb_block[1] : > >> + (mm->context.tsb_block[MM_TSB_HUGE].tsb ? > >> + &mm->context.tsb_block[MM_TSB_HUGE] : > >> NULL) > >> #else > >> NULL > >> #endif > >> - , __pa(&mm->context.tsb_descr[0])); > >> + , __pa(&mm->context.tsb_descr[MM_TSB_BASE])); > >> } > >> > > This is a nice cleanup that has nothing to do with your series. > > Could you submit this as a separate patch so we can get it applied. > > > > This is the only place left where the array index for tsb_block > > and tsb_descr uses hardcoded values. And it would be good to get > > rid of these. > > Sure, I will submit a separate cleanup patch for this. > > However, do note that in my series if CONFIG_SHARED_MMU_CTX is defined, > then MM_TSB_HUGE_SHARED is index 0, instead of MM_TSB_BASE being 0 in > the case where CONFIG_SHARED_MMU_CTX is not defined. This may seem > 'strange' and the obvious question would be 'why not put CONFIG_SHARED_MMU_CTX > at the end of the existing array (index 2)?'. The reason is that tsb_descr > array can not have any 'holes' when passed to the hypervisor. Since there > will always be a MM_TSB_BASE tsb, with MM_TSB_HUGE_SHARED before and > MM_TSB_HUGE after MM_TSB_BASE, few tricks are necessary to ensure no holes > are in the array passed to the hypervisor. So this is the explanation for the strange changes of the constants. Add a similar explanation to the code to help the next reader. Sam -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org