Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756694AbdCUHRd (ORCPT ); Tue, 21 Mar 2017 03:17:33 -0400 Received: from mail-wr0-f193.google.com ([209.85.128.193]:33939 "EHLO mail-wr0-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755241AbdCUHRb (ORCPT ); Tue, 21 Mar 2017 03:17:31 -0400 Date: Tue, 21 Mar 2017 08:17:25 +0100 From: Ingo Molnar To: Thomas Garnier Cc: Thomas Gleixner , Ingo Molnar , "H . Peter Anvin" , Andrey Ryabinin , Alexander Potapenko , Dmitry Vyukov , Kees Cook , Andrew Morton , Dave Hansen , Borislav Petkov , Hugh Dickins , Xiao Guangrong , Matthew Wilcox , Boris Ostrovsky , Andy Lutomirski , Paul Gortmaker , Michal Hocko , zijun_hu , Chris Wilson , x86@kernel.org, linux-kernel@vger.kernel.org, kasan-dev@googlegroups.com, linux-mm@kvack.org, kernel-hardening@lists.openwall.com, richard.weiyang@gmail.com Subject: Re: [PATCH tip v2] x86/mm: Correct fixmap header usage on adaptable MODULES_END Message-ID: <20170321071725.GA15782@gmail.com> References: <20170320194024.60749-1-thgarnie@google.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170320194024.60749-1-thgarnie@google.com> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2399 Lines: 69 * Thomas Garnier wrote: > This patch removes fixmap headers on non-x86 code introduced by the > adaptable MODULE_END change. It is also removed in the 32-bit pgtable > header. Instead, it is added by default in the pgtable generic header > for both architectures. > > Signed-off-by: Thomas Garnier > --- > arch/x86/include/asm/pgtable.h | 1 + > arch/x86/include/asm/pgtable_32.h | 1 - > arch/x86/kernel/module.c | 1 - > arch/x86/mm/dump_pagetables.c | 1 - > arch/x86/mm/kasan_init_64.c | 1 - > mm/vmalloc.c | 4 ---- > 6 files changed, 1 insertion(+), 8 deletions(-) So I already have v1 and there's no explanation about the changes from v1 to v2. The interdiff between v1 and v2 is below, it only affects x86, presumably it's done to simplify the header usage slightly: instead of including fixmap.h in both pgtable_32/64.h it's only included in the common pgtable.h file. That's a sensible cleanup of the original patch and I'd rather not rebase it (as tip:x86/mm has other changes as well), so could I've applied the delta cleanup on top of the existing changes, with its own changelog. Thanks, Ingo ============> diff --git a/arch/x86/include/asm/pgtable.h b/arch/x86/include/asm/pgtable.h index 84f6ec4d47ec..9f6809545269 100644 --- a/arch/x86/include/asm/pgtable.h +++ b/arch/x86/include/asm/pgtable.h @@ -601,6 +601,7 @@ pte_t *populate_extra_pte(unsigned long vaddr); #include #include #include +#include static inline int pte_none(pte_t pte) { diff --git a/arch/x86/include/asm/pgtable_32.h b/arch/x86/include/asm/pgtable_32.h index fbc73360aea0..bfab55675c16 100644 --- a/arch/x86/include/asm/pgtable_32.h +++ b/arch/x86/include/asm/pgtable_32.h @@ -14,7 +14,6 @@ */ #ifndef __ASSEMBLY__ #include -#include #include #include diff --git a/arch/x86/include/asm/pgtable_64.h b/arch/x86/include/asm/pgtable_64.h index 13709cf74ab6..1a4bc71534d4 100644 --- a/arch/x86/include/asm/pgtable_64.h +++ b/arch/x86/include/asm/pgtable_64.h @@ -13,7 +13,6 @@ #include #include #include -#include extern pud_t level3_kernel_pgt[512]; extern pud_t level3_ident_pgt[512];