From mboxrd@z Thu Jan 1 00:00:00 1970 From: rppt@kernel.org Subject: Re: [PATCH v2 12/13] um: add support for folded p4d page tables Date: Fri, 01 Nov 2019 11:14:28 +0100 Message-ID: <20191101101432.7998A2086D@mail.kernel.org> References: <1572597584-6390-1-git-send-email-rppt@kernel.org> <1572597584-6390-13-git-send-email-rppt@kernel.org> <1435962204.69872.1572600009245.JavaMail.zimbra@nod.at> Mime-Version: 1.0 Content-Transfer-Encoding: quoted-printable Return-path: DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1572603273; bh=yZctnhE8ydugwCZWCgTxz8UucAyxJoh5W9y/O7iyOQI=; h=Date:In-Reply-To:References:Subject:To:CC:From:From; b=slfnJUOu2R2Lq4bGn6qHIg4mJZZaAFU2VJUMCjoUXPPOJfiDzb//RzSVYyEL2uHjj ep3oLIUO4xnaRN/mSzvsENH6nNaEDBZefMsHcOqj4xfzKSd10py5bZROhZBLLGfuaC JHM+4ZGhc7H/dGVowx+QbrvRxbthnHAE+gl6UkqI= In-Reply-To: <1435962204.69872.1572600009245.JavaMail.zimbra@nod.at> Sender: linux-kernel-owner@vger.kernel.org List-ID: Content-Type: text/plain; charset="iso-8859-1" To: Richard Weinberger Cc: linux-mm , Andrew Morton , anton ivanov , Arnd Bergmann , davem , Geert Uytterhoeven , Greentime Hu , Greg Ungerer , Helge Deller , "James E.J. Bottomley" , Jeff Dike , "Kirill A. Shutemov" , torvalds , Mark Salter , Matt Turner , Michal Simek , Peter Rosin , Rolf Eike Beer , Russell King , Sam Creasey , Vin ,linux-kernel ,linux-m68k ,linux-parisc ,linux-um ,sparclinux ,Mike Rapoport From: Mike Rapoport Message-ID: On November 1, 2019 10:20:09 AM GMT+01:00, Richard Weinberger wrote: >----- Urspr=C3=BCngliche Mail ----- >> Von: "Mike Rapoport" > >[=2E=2E=2E] > >> #define pte_page(x) pfn_to_page(pte_pfn(x)) >> diff --git a/arch/um/kernel/mem=2Ec b/arch/um/kernel/mem=2Ec >> index 417ff64=2E=2E6fd17bc 100644 >> --- a/arch/um/kernel/mem=2Ec >> +++ b/arch/um/kernel/mem=2Ec >> @@ -92,10 +92,26 @@ static void __init one_md_table_init(pud_t *pud) >> #endif >> } >>=20 >> +static void __init one_pud_table_init(p4d_t *p4d) >> +{ >> +#if CONFIG_PGTABLE_LEVELS > 3 > >Isn't this dead code? > >For uml we have: >config PGTABLE_LEVELS > int > default 3 if 3_LEVEL_PGTABLES > default 2 It's kinda a provision for 4 levels support in UML :) I can drop this in the next respin, no problem=2E >Thanks, >//richard --=20 Sincerely yours, Mike