From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-11.2 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,NICE_REPLY_A, SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_1 autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 256E2C432BE for ; Tue, 31 Aug 2021 03:44:34 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 0032E6103D for ; Tue, 31 Aug 2021 03:44:33 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S239749AbhHaDp1 (ORCPT ); Mon, 30 Aug 2021 23:45:27 -0400 Received: from foss.arm.com ([217.140.110.172]:50670 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S239631AbhHaDpZ (ORCPT ); Mon, 30 Aug 2021 23:45:25 -0400 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id F1C021FB; Mon, 30 Aug 2021 20:44:30 -0700 (PDT) Received: from [10.163.72.217] (unknown [10.163.72.217]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 772CD3F5A1; Mon, 30 Aug 2021 20:44:29 -0700 (PDT) Subject: Re: [PATCH V2] arm64/mm: Drop To: Catalin Marinas Cc: linux-arm-kernel@lists.infradead.org, Will Deacon , linux-kernel@vger.kernel.org, Mark Rutland References: <1629457516-32306-1-git-send-email-anshuman.khandual@arm.com> <20210820183520.GC23080@arm.com> From: Anshuman Khandual Message-ID: <84b647b6-9cee-5aad-78f8-7bc253300534@arm.com> Date: Tue, 31 Aug 2021 09:15:30 +0530 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.10.0 MIME-Version: 1.0 In-Reply-To: <20210820183520.GC23080@arm.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 8/21/21 12:05 AM, Catalin Marinas wrote: > On Fri, Aug 20, 2021 at 04:35:16PM +0530, Anshuman Khandual wrote: >> PAGE_SHIFT (PAGE_SIZE and PAGE_MASK) which is derived from ARM64_PAGE_SHIFT >> should be moved into instead like in case for other platforms, >> and then subsequently can be just dropped off completely. > > These were moved to page-def.h as part of commit b6531456ba27 ("arm64: > factor out PAGE_* and CONT_* definitions") to avoid some circular header > dependencies. > >> diff --git a/arch/arm64/include/asm/memory.h b/arch/arm64/include/asm/memory.h >> index 824a3655dd93..649d26396f9e 100644 >> --- a/arch/arm64/include/asm/memory.h >> +++ b/arch/arm64/include/asm/memory.h >> @@ -12,7 +12,7 @@ >> >> #include >> #include >> -#include >> +#include > > In 5.14-rc3, asm/page.h still includes asm/memory.h. Dropping from does not seem to cause any problem, will change that. Afterwards build tested it across page sizes and also with some random configs. Is that circular dependency still present ? Also wondering why was included in to begin with ?