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 Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id 5BFACCE8D52 for ; Thu, 19 Sep 2024 07:12:25 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id E95BA6B0085; Thu, 19 Sep 2024 03:12:24 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id E459E6B0088; Thu, 19 Sep 2024 03:12:24 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id D0D6D6B0089; Thu, 19 Sep 2024 03:12:24 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0013.hostedemail.com [216.40.44.13]) by kanga.kvack.org (Postfix) with ESMTP id B05B36B0085 for ; Thu, 19 Sep 2024 03:12:24 -0400 (EDT) Received: from smtpin23.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay10.hostedemail.com (Postfix) with ESMTP id 53F20C0AFD for ; Thu, 19 Sep 2024 07:12:24 +0000 (UTC) X-FDA: 82580619408.23.02FEF8E Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by imf19.hostedemail.com (Postfix) with ESMTP id 945AA1A0014 for ; Thu, 19 Sep 2024 07:12:21 +0000 (UTC) Authentication-Results: imf19.hostedemail.com; dkim=none; spf=pass (imf19.hostedemail.com: domain of anshuman.khandual@arm.com designates 217.140.110.172 as permitted sender) smtp.mailfrom=anshuman.khandual@arm.com; dmarc=pass (policy=none) header.from=arm.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1726729885; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=VfvmTILgdFtcSVTD5VawbOnNN+7L2DqOz2QZKDFkEeE=; b=lDJs7R/igFwFV6wbhkEp69FSq37Ge7zT1+0W+BKeLDOzYNAxEJsiBXVB8PSHeRaxwywlys W4lVyKUAGqsj68HhJNLN+uX36wAqNjcUgMBDkh7u6doHrZoZV/+FsDNFMDW74zEoDzQ71S mJY6O+xSeqbDVYUjcapZA7omhSadw8s= ARC-Authentication-Results: i=1; imf19.hostedemail.com; dkim=none; spf=pass (imf19.hostedemail.com: domain of anshuman.khandual@arm.com designates 217.140.110.172 as permitted sender) smtp.mailfrom=anshuman.khandual@arm.com; dmarc=pass (policy=none) header.from=arm.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1726729885; a=rsa-sha256; cv=none; b=17aP7xMFnnHE6nedbdtPrfCP6AqI4L50yv7rjdQGpONaHkNJPWg0sEBxGSXNx1YO846RHA QUUXXDIwg3+ZThx/uKvPS5qzEMBPyLCf2CvgDfzp30TjyeZa8PusmoM1DuwFYG9WaNMkDp TeYQ42K5Yp5OA6KahiUjmxLD9o4b3Xc= 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 190C81007; Thu, 19 Sep 2024 00:12:50 -0700 (PDT) Received: from [10.163.34.169] (unknown [10.163.34.169]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 109FE3F64C; Thu, 19 Sep 2024 00:12:11 -0700 (PDT) Message-ID: <6191a730-1a0f-476e-8041-a0a51094b6b3@arm.com> Date: Thu, 19 Sep 2024 12:42:08 +0530 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH V2 4/7] mm: Use pmdp_get() for accessing PMD entries To: kernel test robot , linux-mm@kvack.org Cc: oe-kbuild-all@lists.linux.dev, Andrew Morton , David Hildenbrand , Ryan Roberts , "Mike Rapoport (IBM)" , Arnd Bergmann , x86@kernel.org, linux-m68k@lists.linux-m68k.org, linux-fsdevel@vger.kernel.org, kasan-dev@googlegroups.com, linux-kernel@vger.kernel.org, linux-perf-users@vger.kernel.org, Dimitri Sivanich , Muchun Song , Andrey Ryabinin , Miaohe Lin , Naoya Horiguchi , Pasha Tatashin , Dennis Zhou , Tejun Heo , Christoph Lameter , Uladzislau Rezki , Christoph Hellwig References: <20240917073117.1531207-5-anshuman.khandual@arm.com> <202409190244.JcrD4CwD-lkp@intel.com> Content-Language: en-US From: Anshuman Khandual In-Reply-To: <202409190244.JcrD4CwD-lkp@intel.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-Rspamd-Server: rspam03 X-Rspam-User: X-Rspamd-Queue-Id: 945AA1A0014 X-Stat-Signature: 56ydduk7k5jgbsb6z4861zpwxzmdbnru X-HE-Tag: 1726729941-374036 X-HE-Meta: U2FsdGVkX1+2o5ARrMbOLQfYFqrwnM1CUyB0m/scXBP0BdX0azNR0DY+CkyBzCrFfyDAg+2XOvDec1x0b3mGj57bLRJ1kzwrjlflq5ryRIpC+p5XEpbo/GnkswoG3HrL90lRZ7VcEeBkyq5nhFQwQ6oMt/n9EWqoejx3R87YIFvlhrh3Jg5juGYfvZ8Pi/G9jSux3A+x1gpIwFODHW7ZjG9wz5qBD2FHKcOJ3v9X5kEpmCe71XmkQ1pWMbtbF85DpfxE4Q8SkzUwqQ1BC8eN3fIOxc7DK1yVFcF6fb6l90c2mgGLoh8yxIzDkzIXWOwuQnL1xQgez+AYWug0k9YjeidvNRPxovrVN2zj8xL3xDjvGiAR+j84mGjLbtMpSTV2JRfUUVCzCKuLEJqqUpqZiVCiKA9BuxP2ohAi3IGZvND4HGEnb1mXBQhQxrVhso6YFVAFtCSUdDlte5Dn89bJWghRbVI3BBEHIkUEg+b/VwLdKSs4SWGQhDCVxBy6EI5MRfOg0LjhmgdjxSuyKlg1OBjfITGa1wa8VGsouFtkGZsPRSXhNKuYfC1NKAA3tfh6Pq0m904jz8qSJRPChCOa3F4IxX2BD1Uy8OpEC/63bnPP0DbgOx/kK16zhjwn7kia/bih1sNVtZEigRyl1CN7omTmfIxlK2obsnXJ1npFZdSW4Ocoy0iEHW1wjU7Au5KhlKDMoMJaiMzKKDgK1Z1YD2rFScSGLZCi9YK0BhmqKb7hK7XZ/EYhEGhDQFLm9+Lsq97MkYo8419vLANf8VpvnskB9wzOl4XMBrw/Usie7D7SFZ+iuTwff/WL4LzuyVucX9cPIs24JLyU0OjNwFPYZmqljn+Q2B+f2qI/sAv01wx6RyqFkANpietRe4i4D9DBLEeYIDz+G1dMRWTnFTRXUARBIgo4cU92FAQtymCZhNSJtylZ/0ppqceMl3tnEGfKJ8xbdf3HAjGvvQ4DX5B GFP3rKrT 6k3BUVNN2I+OepGrKrBF4cPM59aUdJrYsFmv/kTfg4IutnNhZ/6fX0r6JuZIKkRahw/xQmMvjygEuuVL4N7fYC1/I9GrQ3RhgUtrphMmCXr8qn/RkxE1Bjvgn48m2LuWci8cKFqyG+sI/w1MMLxxRTbJ02hPlE9lvD0M+BqltorxY6BSRMm/+U1INQgXIokzZ5xPYC654MauARnDwrBFO9UwHmZZJmtghoBAuX1QwUfj+FQrsoFg4WN8Er1v3jKrwC38HAGxrSDbPrs5g8axfn2tXcpsMbZjQ7P+l3TJg14jrhBxmdyDWnH5Lbl/DvFOZ2Nd+26csk7p8rAaWotwDJZCHct338YDA3yfFZCYtu+yaR/8Ky6rDNQUDATz6eI/9JLgFIYRFE2spbDm341Zrcj/s7t3RonP2q0M+C1pgc8H2aW+JS8h6xjH3xBlkPEGvMpIUfAEYlrI9sC5B/4xrFPpPPoqEAzs6wHEk X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: On 9/19/24 00:37, kernel test robot wrote: > Hi Anshuman, > > kernel test robot noticed the following build errors: > > [auto build test ERROR on char-misc/char-misc-testing] > [also build test ERROR on char-misc/char-misc-next char-misc/char-misc-linus brauner-vfs/vfs.all dennis-percpu/for-next linus/master v6.11] > [cannot apply to akpm-mm/mm-everything next-20240918] > [If your patch is applied to the wrong git tree, kindly drop us a note. > And when submitting patch, we suggest to use '--base' as documented in > https://git-scm.com/docs/git-format-patch#_base_tree_information] > > url: https://github.com/intel-lab-lkp/linux/commits/Anshuman-Khandual/m68k-mm-Change-pmd_val/20240917-153331 > base: char-misc/char-misc-testing > patch link: https://lore.kernel.org/r/20240917073117.1531207-5-anshuman.khandual%40arm.com > patch subject: [PATCH V2 4/7] mm: Use pmdp_get() for accessing PMD entries > config: openrisc-allnoconfig (https://download.01.org/0day-ci/archive/20240919/202409190244.JcrD4CwD-lkp@intel.com/config) > compiler: or1k-linux-gcc (GCC) 14.1.0 > reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240919/202409190244.JcrD4CwD-lkp@intel.com/reproduce) > > If you fix the issue in a separate patch/commit (i.e. not just a new version of > the same patch/commit), kindly add following tags > | Reported-by: kernel test robot > | Closes: https://lore.kernel.org/oe-kbuild-all/202409190244.JcrD4CwD-lkp@intel.com/ > > All errors (new ones prefixed by >>): > > In file included from include/asm-generic/bug.h:22, > from arch/openrisc/include/asm/bug.h:5, > from include/linux/bug.h:5, > from include/linux/mmdebug.h:5, > from include/linux/mm.h:6, > from include/linux/pagemap.h:8, > from mm/pgtable-generic.c:10: > mm/pgtable-generic.c: In function 'pmd_clear_bad': >>> arch/openrisc/include/asm/pgtable.h:369:36: error: lvalue required as unary '&' operand > 369 | __FILE__, __LINE__, &(e), pgd_val(e)) > | ^ > include/linux/printk.h:437:33: note: in definition of macro 'printk_index_wrap' > 437 | _p_func(_fmt, ##__VA_ARGS__); \ > | ^~~~~~~~~~~ > arch/openrisc/include/asm/pgtable.h:368:9: note: in expansion of macro 'printk' > 368 | printk(KERN_ERR "%s:%d: bad pgd %p(%08lx).\n", \ > | ^~~~~~ > include/asm-generic/pgtable-nop4d.h:25:50: note: in expansion of macro 'pgd_ERROR' > 25 | #define p4d_ERROR(p4d) (pgd_ERROR((p4d).pgd)) > | ^~~~~~~~~ > include/asm-generic/pgtable-nopud.h:32:50: note: in expansion of macro 'p4d_ERROR' > 32 | #define pud_ERROR(pud) (p4d_ERROR((pud).p4d)) > | ^~~~~~~~~ > include/asm-generic/pgtable-nopmd.h:36:50: note: in expansion of macro 'pud_ERROR' > 36 | #define pmd_ERROR(pmd) (pud_ERROR((pmd).pud)) > | ^~~~~~~~~ > mm/pgtable-generic.c:54:9: note: in expansion of macro 'pmd_ERROR' > 54 | pmd_ERROR(pmdp_get(pmd)); > | ^~~~~~~~~ > > > vim +369 arch/openrisc/include/asm/pgtable.h > > 61e85e367535a7 Jonas Bonn 2011-06-04 363 > 61e85e367535a7 Jonas Bonn 2011-06-04 364 #define pte_ERROR(e) \ > 61e85e367535a7 Jonas Bonn 2011-06-04 365 printk(KERN_ERR "%s:%d: bad pte %p(%08lx).\n", \ > 61e85e367535a7 Jonas Bonn 2011-06-04 366 __FILE__, __LINE__, &(e), pte_val(e)) > 61e85e367535a7 Jonas Bonn 2011-06-04 367 #define pgd_ERROR(e) \ > 61e85e367535a7 Jonas Bonn 2011-06-04 368 printk(KERN_ERR "%s:%d: bad pgd %p(%08lx).\n", \ > 61e85e367535a7 Jonas Bonn 2011-06-04 @369 __FILE__, __LINE__, &(e), pgd_val(e)) > 61e85e367535a7 Jonas Bonn 2011-06-04 370 > This build failure can be fixed with dropping address output from pxd_ERROR() helpers as is being done for the x86 platform. Similar fix is also required for the UM architecture as well. diff --git a/arch/openrisc/include/asm/pgtable.h b/arch/openrisc/include/asm/pgtable.h index 60c6ce7ff2dc..831efb71ab54 100644 --- a/arch/openrisc/include/asm/pgtable.h +++ b/arch/openrisc/include/asm/pgtable.h @@ -362,11 +362,11 @@ static inline unsigned long pmd_page_vaddr(pmd_t pmd) #define pfn_pte(pfn, prot) __pte((((pfn) << PAGE_SHIFT)) | pgprot_val(prot)) #define pte_ERROR(e) \ - printk(KERN_ERR "%s:%d: bad pte %p(%08lx).\n", \ - __FILE__, __LINE__, &(e), pte_val(e)) + printk(KERN_ERR "%s:%d: bad pte (%08lx).\n", \ + __FILE__, __LINE__, pte_val(e)) #define pgd_ERROR(e) \ - printk(KERN_ERR "%s:%d: bad pgd %p(%08lx).\n", \ - __FILE__, __LINE__, &(e), pgd_val(e)) + printk(KERN_ERR "%s:%d: bad pgd (%08lx).\n", \ + __FILE__, __LINE__, pgd_val(e)) extern pgd_t swapper_pg_dir[PTRS_PER_PGD]; /* defined in head.S */