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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id A6708C3DA4A for ; Tue, 30 Jul 2024 02:12:52 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:In-Reply-To:Content-Type: MIME-Version:References:Message-ID:Subject:Cc:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=D6/B1InhGAdC6d7xTYYpZAnaejq0sgMAYvqjcptskls=; b=HpDvYwEDrwq/fQnik7VyQy+HnI ENU2r6uXaav0OiqnZfEzZ3ElqcObzsQ2+YGtQ+Q8uabtD44bHghMBW23fKui7W4a3cCyhtk6fgFSQ dUBz1kyCb+w8TvAWiKfZzmkFriKQVxq18C7KCO9B5ksIWjZiQbMJq1G0oKXEG52pTG8S35b6RBb8E RmZ9qvoF3xhmO8Qd1NNMnR3O5n6Qau5EeSNPz/SJeq0EThoJtWPc6Ua8gx7YCFLD6HxWEi91Z6gyX aZOoZgMCRbJF5ZnXyXgy5VYGPUQkWiIoh945vY3Yg6LvAeJ2gpETBKTte/vGpOIXpCKtmL1uJj/AS LcI/6p0g==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sYcLx-0000000DJEO-0Ezs; Tue, 30 Jul 2024 02:12:41 +0000 Received: from dfw.source.kernel.org ([2604:1380:4641:c500::1]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1sYcLU-0000000DJ7V-3aU0 for linux-arm-kernel@lists.infradead.org; Tue, 30 Jul 2024 02:12:14 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by dfw.source.kernel.org (Postfix) with ESMTP id B0C4961BB0; Tue, 30 Jul 2024 02:12:11 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 71295C4AF07; Tue, 30 Jul 2024 02:12:10 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1722305531; bh=eA77p9u5560PhQ2+Qo/vv+lMkVA7visV3V6tX1tIebU=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=r+UNl4+P+aBuEwCi15lh1dI0u61rlCGxvYKiC8Zrfg70eL6OZ14UxHwX5V+nkWD4z GTFvLQSr4vhQWjMHSOW4qqkck+CW+Ua6QZTZVGlIr74fy8uIKQCAONj0BFE16P3Mjf PUMPQcm2k6Ctw0LU2whXelX1qpLnAA/Mmc6qofQwdBAQEcKPH0U69fx2wYOV4BGT2f xaO+O9X8TAf6iJM4cllGx1cbjKuKnW39T/oRbBP/PwPoYidP4yvYQ/xLzsaHaVetrO M+eNYMDFWibewkBkHuLOaWbXQ0sbtQMeKcV4b+vIzgNfje4gxnYr4f3dhxaboZrIpC MwD+65uq+A1bA== Date: Mon, 29 Jul 2024 19:12:08 -0700 From: Nathan Chancellor To: kernel test robot Cc: Baruch Siach , Christoph Hellwig , Marek Szyprowski , Catalin Marinas , Will Deacon , llvm@lists.linux.dev, oe-kbuild-all@lists.linux.dev, linux-s390@vger.kernel.org, Ramon Fried , Petr =?utf-8?B?VGVzYcWZw61r?= , Robin Murphy , linux-kernel@vger.kernel.org, iommu@lists.linux.dev, Elad Nachman , linuxppc-dev@lists.ozlabs.org, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH v3 2/3] dma-mapping: replace zone_dma_bits by zone_dma_limit Message-ID: <20240730021208.GA8272@thelio-3990X> References: <053fa4806a2c63efcde80caca473a8b670a2701c.1722249878.git.baruch@tkos.co.il> <202407300338.oaUo6jtB-lkp@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <202407300338.oaUo6jtB-lkp@intel.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240729_191213_022487_BE471DF1 X-CRM114-Status: GOOD ( 23.95 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Tue, Jul 30, 2024 at 04:20:51AM +0800, kernel test robot wrote: > Hi Baruch, > > kernel test robot noticed the following build warnings: > > [auto build test WARNING on arm64/for-next/core] > [also build test WARNING on powerpc/next powerpc/fixes s390/features linus/master v6.11-rc1 next-20240729] > [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/Baruch-Siach/dma-mapping-improve-DMA-zone-selection/20240729-211018 > base: https://git.kernel.org/pub/scm/linux/kernel/git/arm64/linux.git for-next/core > patch link: https://lore.kernel.org/r/053fa4806a2c63efcde80caca473a8b670a2701c.1722249878.git.baruch%40tkos.co.il > patch subject: [PATCH v3 2/3] dma-mapping: replace zone_dma_bits by zone_dma_limit > config: arm-allnoconfig (https://download.01.org/0day-ci/archive/20240730/202407300338.oaUo6jtB-lkp@intel.com/config) > compiler: clang version 20.0.0git (https://github.com/llvm/llvm-project ccae7b461be339e717d02f99ac857cf0bc7d17fc) > reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240730/202407300338.oaUo6jtB-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/202407300338.oaUo6jtB-lkp@intel.com/ > > All warnings (new ones prefixed by >>): > > In file included from kernel/dma/direct.c:7: > In file included from include/linux/memblock.h:12: > In file included from include/linux/mm.h:2253: > include/linux/vmstat.h:514:36: warning: arithmetic between different enumeration types ('enum node_stat_item' and 'enum lru_list') [-Wenum-enum-conversion] > 514 | return node_stat_name(NR_LRU_BASE + lru) + 3; // skip "nr_" > | ~~~~~~~~~~~ ^ ~~~ > >> kernel/dma/direct.c:23:46: warning: implicit conversion from 'unsigned long long' to 'phys_addr_t' (aka 'unsigned int') changes value from 18446744073709551615 to 4294967295 [-Wconstant-conversion] > 23 | phys_addr_t zone_dma_limit __ro_after_init = DMA_BIT_MASK(24); > | ~~~~~~~~~~~~~~ ^~~~~~~~~~~~~~~~ > include/linux/dma-mapping.h:77:40: note: expanded from macro 'DMA_BIT_MASK' > 77 | #define DMA_BIT_MASK(n) (((n) == 64) ? ~0ULL : ((1ULL<<(n))-1)) > | ^~~~~ > 2 warnings generated. FWIW, this is likely a false positive due to an issue in Clang with the control flow graph for global variables: https://github.com/ClangBuiltLinux/linux/issues/92 DMA_BIT_MASK() has been the biggest offender :/ If there is any way to refactor this code to avoid this, that would be great (as that has been one of our longest outstanding issues and getting it fixed in the compiler does not seem super easy at this point). Cheers, Nathan > vim +23 kernel/dma/direct.c > > > 7 #include > 8 #include > 9 #include > 10 #include > 11 #include > 12 #include > 13 #include > 14 #include > 15 #include > 16 #include "direct.h" > 17 > 18 /* > 19 * Most architectures use ZONE_DMA for the first 16 Megabytes, but some use > 20 * it for entirely different regions. In that case the arch code needs to > 21 * override the variable below for dma-direct to work properly. > 22 */ > > 23 phys_addr_t zone_dma_limit __ro_after_init = DMA_BIT_MASK(24); > 24 > > -- > 0-DAY CI Kernel Test Service > https://github.com/intel/lkp-tests/wiki >