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=-10.5 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,MENTIONS_GIT_HOSTING, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,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 64F54C433E1 for ; Sat, 1 Aug 2020 06:18:34 +0000 (UTC) Received: from ml01.01.org (ml01.01.org [198.145.21.10]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 39A2720725 for ; Sat, 1 Aug 2020 06:18:34 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 39A2720725 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=intel.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-nvdimm-bounces@lists.01.org Received: from ml01.vlan13.01.org (localhost [IPv6:::1]) by ml01.01.org (Postfix) with ESMTP id 04C8D1297090B; Fri, 31 Jul 2020 23:18:34 -0700 (PDT) Received-SPF: Pass (mailfrom) identity=mailfrom; client-ip=134.134.136.126; helo=mga18.intel.com; envelope-from=lkp@intel.com; receiver= Received: from mga18.intel.com (mga18.intel.com [134.134.136.126]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id EBD4E1296C722; Fri, 31 Jul 2020 23:18:31 -0700 (PDT) IronPort-SDR: hKgWTCYCxygPsatgZxrJ2cx5ci9or8YMvhP2OzYXCPOTQH+I24jgRek6W84fomt/QDxs1jr+Vj kNg152nGJMvA== X-IronPort-AV: E=McAfee;i="6000,8403,9699"; a="139472461" X-IronPort-AV: E=Sophos;i="5.75,420,1589266800"; d="gz'50?scan'50,208,50";a="139472461" X-Amp-Result: UNKNOWN X-Amp-Original-Verdict: FILE UNKNOWN X-Amp-File-Uploaded: False Received: from orsmga005.jf.intel.com ([10.7.209.41]) by orsmga106.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 31 Jul 2020 23:18:30 -0700 IronPort-SDR: 1N1pJo/BGwcS5rmwpT0YPe1jggo5XRW+dtbFLeNfsTVl+LU8hSYnW88onORbE5EszHn5qOWWB4 KM9Fae43l/xA== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.75,420,1589266800"; d="gz'50?scan'50,208,50";a="465882780" Received: from lkp-server01.sh.intel.com (HELO e21119890065) ([10.239.97.150]) by orsmga005.jf.intel.com with ESMTP; 31 Jul 2020 23:18:26 -0700 Received: from kbuild by e21119890065 with local (Exim 4.92) (envelope-from ) id 1k1kqf-0000L6-G2; Sat, 01 Aug 2020 06:18:25 +0000 Date: Sat, 1 Aug 2020 14:18:12 +0800 From: kernel test robot To: Dan Williams , akpm@linux-foundation.org Subject: Re: [PATCH v3 21/23] device-dax: Add an 'align' attribute Message-ID: <202008011403.PtFkHpqE%lkp@intel.com> References: <159625241660.3040297.3801913809845542130.stgit@dwillia2-desk3.amr.corp.intel.com> MIME-Version: 1.0 In-Reply-To: <159625241660.3040297.3801913809845542130.stgit@dwillia2-desk3.amr.corp.intel.com> User-Agent: Mutt/1.10.1 (2018-07-13) Message-ID-Hash: G6U4YT3H6U753NH7EA6RKM3VXUGZ7QVJ X-Message-ID-Hash: G6U4YT3H6U753NH7EA6RKM3VXUGZ7QVJ X-MailFrom: lkp@intel.com X-Mailman-Rule-Hits: nonmember-moderation X-Mailman-Rule-Misses: dmarc-mitigation; no-senders; approved; emergency; loop; banned-address; member-moderation Content-Disposition: inline X-Content-Filtered-By: Mailman/MimeDel 3.1.1 CC: kbuild-all@lists.01.org, Joao Martins , peterz@infradead.org, dave.hansen@linux.intel.com, ard.biesheuvel@linaro.org, linux-mm@kvack.org, linux-nvdimm@lists.01.org, linux-kernel@vger.kernel.org, linux-acpi@vger.kernel.org X-Mailman-Version: 3.1.1 Precedence: list List-Id: "Linux-nvdimm developer list." Archived-At: List-Archive: List-Help: List-Post: List-Subscribe: List-Unsubscribe: Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Hi Dan, Thank you for the patch! Yet something to improve: [auto build test ERROR on 01830e6c042e8eb6eb202e05d7df8057135b4c26] url: https://github.com/0day-ci/linux/commits/Dan-Williams/device-dax-Support-sub-dividing-soft-reserved-ranges/20200801-114823 base: 01830e6c042e8eb6eb202e05d7df8057135b4c26 config: arm-randconfig-r006-20200731 (attached as .config) compiler: arm-linux-gnueabi-gcc (GCC) 9.3.0 reproduce (this is a W=1 build): wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # save the attached .config to linux build tree COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=arm If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot All errors (new ones prefixed by >>): drivers/dax/bus.c: In function 'do_id_store': drivers/dax/bus.c:94:27: warning: suggest braces around empty body in an 'else' statement [-Wempty-body] 94 | /* nothing to remove */; | ^ drivers/dax/bus.c:99:29: warning: suggest braces around empty body in an 'else' statement [-Wempty-body] 99 | /* dax_id already added */; | ^ In file included from include/linux/percpu-refcount.h:54, from include/linux/memremap.h:6, from drivers/dax/bus.c:3: drivers/dax/bus.c: In function 'dev_dax_shrink': include/linux/kernel.h:850:29: warning: comparison of distinct pointer types lacks a cast 850 | (!!(sizeof((typeof(x) *)1 == (typeof(y) *)1))) | ^~ include/linux/kernel.h:864:4: note: in expansion of macro '__typecheck' 864 | (__typecheck(x, y) && __no_side_effects(x, y)) | ^~~~~~~~~~~ include/linux/kernel.h:874:24: note: in expansion of macro '__safe_cmp' 874 | __builtin_choose_expr(__safe_cmp(x, y), \ | ^~~~~~~~~~ include/linux/kernel.h:883:19: note: in expansion of macro '__careful_cmp' 883 | #define min(x, y) __careful_cmp(x, y, <) | ^~~~~~~~~~~~~ drivers/dax/bus.c:881:12: note: in expansion of macro 'min' 881 | shrink = min(to_shrink, range_len(range)); | ^~~ In file included from include/linux/device.h:15, from drivers/dax/bus.c:4: drivers/dax/bus.c: In function 'dev_dax_validate_align': drivers/dax/bus.c:1062:16: warning: format '%llu' expects argument of type 'long long unsigned int', but argument 6 has type 'resource_size_t' {aka 'unsigned int'} [-Wformat=] 1062 | dev_dbg(dev, "%s: align %u invalid for size %llu\n", | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ include/linux/dev_printk.h:19:22: note: in definition of macro 'dev_fmt' 19 | #define dev_fmt(fmt) fmt | ^~~ drivers/dax/bus.c:1062:3: note: in expansion of macro 'dev_dbg' 1062 | dev_dbg(dev, "%s: align %u invalid for size %llu\n", | ^~~~~~~ drivers/dax/bus.c:1062:50: note: format string is defined here 1062 | dev_dbg(dev, "%s: align %u invalid for size %llu\n", | ~~~^ | | | long long unsigned int | %u In file included from include/linux/device.h:15, from drivers/dax/bus.c:4: drivers/dax/bus.c:1071:17: warning: format '%ld' expects argument of type 'long int', but argument 6 has type 'ssize_t' {aka 'int'} [-Wformat=] 1071 | dev_dbg(dev, "%s: align %u invalid for range %ld\n", | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ include/linux/dev_printk.h:19:22: note: in definition of macro 'dev_fmt' 19 | #define dev_fmt(fmt) fmt | ^~~ drivers/dax/bus.c:1071:4: note: in expansion of macro 'dev_dbg' 1071 | dev_dbg(dev, "%s: align %u invalid for range %ld\n", | ^~~~~~~ drivers/dax/bus.c:1071:51: note: format string is defined here 1071 | dev_dbg(dev, "%s: align %u invalid for range %ld\n", | ~~^ | | | long int | %d >> drivers/dax/bus.c:1082:7: error: 'PMD_SIZE' undeclared (first use in this function); did you mean 'PUD_SIZE'? 1082 | case PMD_SIZE: | ^~~~~~~~ | PUD_SIZE drivers/dax/bus.c:1082:7: note: each undeclared identifier is reported only once for each function it appears in In file included from include/linux/compiler_types.h:65, from : include/linux/compiler_attributes.h:214:41: warning: attribute 'fallthrough' not preceding a case label or default label 214 | # define fallthrough __attribute__((__fallthrough__)) | ^~~~~~~~~~~~~ drivers/dax/bus.c:1081:3: note: in expansion of macro 'fallthrough' 1081 | fallthrough; | ^~~~~~~~~~~ vim +1082 drivers/dax/bus.c 1050 1051 static ssize_t dev_dax_validate_align(struct dev_dax *dev_dax) 1052 { 1053 resource_size_t dev_size = dev_dax_size(dev_dax); 1054 struct device *dev = &dev_dax->dev; 1055 ssize_t rc, i; 1056 1057 if (dev->driver) 1058 return -EBUSY; 1059 1060 rc = -EINVAL; 1061 if (dev_size > 0 && !alloc_is_aligned(dev_dax, dev_size)) { 1062 dev_dbg(dev, "%s: align %u invalid for size %llu\n", 1063 __func__, dev_dax->align, dev_size); 1064 return rc; 1065 } 1066 1067 for (i = 0; i < dev_dax->nr_range; i++) { 1068 size_t len = range_len(&dev_dax->ranges[i].range); 1069 1070 if (!alloc_is_aligned(dev_dax, len)) { 1071 dev_dbg(dev, "%s: align %u invalid for range %ld\n", 1072 __func__, dev_dax->align, i); 1073 return rc; 1074 } 1075 } 1076 1077 switch (dev_dax->align) { 1078 case PUD_SIZE: 1079 if (!IS_ENABLED(CONFIG_HAVE_ARCH_TRANSPARENT_HUGEPAGE_PUD)) 1080 break; 1081 fallthrough; > 1082 case PMD_SIZE: 1083 if (!has_transparent_hugepage()) 1084 break; 1085 fallthrough; 1086 case PAGE_SIZE: 1087 rc = 0; 1088 break; 1089 } 1090 1091 return rc; 1092 } 1093 --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org _______________________________________________ Linux-nvdimm mailing list -- linux-nvdimm@lists.01.org To unsubscribe send an email to linux-nvdimm-leave@lists.01.org