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 4FE5AC77B76 for ; Mon, 17 Apr 2023 15:29:17 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id CDFE78E0002; Mon, 17 Apr 2023 11:29:16 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id C8F8D8E0001; Mon, 17 Apr 2023 11:29:16 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id B57468E0002; Mon, 17 Apr 2023 11:29:16 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0012.hostedemail.com [216.40.44.12]) by kanga.kvack.org (Postfix) with ESMTP id A48818E0001 for ; Mon, 17 Apr 2023 11:29:16 -0400 (EDT) Received: from smtpin24.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay07.hostedemail.com (Postfix) with ESMTP id 535A2160444 for ; Mon, 17 Apr 2023 15:29:16 +0000 (UTC) X-FDA: 80691266712.24.AE4CFD6 Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by imf11.hostedemail.com (Postfix) with ESMTP id 822E34001F for ; Mon, 17 Apr 2023 15:29:13 +0000 (UTC) Authentication-Results: imf11.hostedemail.com; dkim=pass header.d=intel.com header.s=Intel header.b=AQE5G4nJ; spf=pass (imf11.hostedemail.com: domain of lkp@intel.com designates 134.134.136.65 as permitted sender) smtp.mailfrom=lkp@intel.com; dmarc=pass (policy=none) header.from=intel.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1681745354; 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: in-reply-to:in-reply-to:references:references:dkim-signature; bh=bpMcL7YfWv1yTAlRX9uT4bifYgKIN74+yXf1PoizNcI=; b=gQUkFD6punoJaFtkN8O2KEuLE/SCZLrokuO7XMo6BN78ZjEFC0iarImqcJq4xzKaNfaIAH W28mKXasMkB0dnqm+nn1hm7CBijrrGsmxNdV6WQbyrU4aZWD5CL5MEnjwl3BrhZBNjqmEQ Xb7W3E8iojdLBUpVEqTzGUAEa4Ep48c= ARC-Authentication-Results: i=1; imf11.hostedemail.com; dkim=pass header.d=intel.com header.s=Intel header.b=AQE5G4nJ; spf=pass (imf11.hostedemail.com: domain of lkp@intel.com designates 134.134.136.65 as permitted sender) smtp.mailfrom=lkp@intel.com; dmarc=pass (policy=none) header.from=intel.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1681745354; a=rsa-sha256; cv=none; b=zB7KIhjvAF0pxs+KjJTA3QPZBa/rDvRf5UQCClsEt6wvZYYEPjR8Avn5Tnn3ZHxv/E9qpF KmGD55rayHy4y1Xx8cVqN2nMl2bgovZumqEe1+CSSx6Pb4bhwFt79iHCDsHfvk+gwlxxpU 4po4fU9308ORvbUICivcYiJT7AubZxU= DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1681745353; x=1713281353; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=mETGBZwVEG/Xrc1N4yHAc/0CcMbvk6722ZuBpaiz+VI=; b=AQE5G4nJ4kARMg2k6BHBYiMfUI+dxzMUnmCh+Ja1kIAWrxlmlzPYWVVW z17tavyat2huGeXCbJgB2gnckdQeL0riJ+A9+409DSweXSG/rlFPv5bWu ZD/EHBP0cwvoctPYk3YBx91JpSHYFX1S1mNrUVvbKoqOW+wkqS4KyKP8y 7bIgst/Bn9ByyLiTIWckPen2MJf6B+hpzW3dPUuOFUvqSiCueN82sTDVm irxuxlkiVTC6zUrN704Yh5AAMeS5bVi81XI96bH237W3dJHC+HEBIDowT ctnvj4T+P+KuFNu4YqTzHlhOajgJOJgzKzHR6iSweb5fXpqVKChstqKTZ g==; X-IronPort-AV: E=McAfee;i="6600,9927,10683"; a="347665468" X-IronPort-AV: E=Sophos;i="5.99,204,1677571200"; d="scan'208";a="347665468" Received: from orsmga007.jf.intel.com ([10.7.209.58]) by orsmga103.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 17 Apr 2023 08:29:11 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6600,9927,10683"; a="684193562" X-IronPort-AV: E=Sophos;i="5.99,204,1677571200"; d="scan'208";a="684193562" Received: from lkp-server01.sh.intel.com (HELO b613635ddfff) ([10.239.97.150]) by orsmga007.jf.intel.com with ESMTP; 17 Apr 2023 08:28:54 -0700 Received: from kbuild by b613635ddfff with local (Exim 4.96) (envelope-from ) id 1poQmi-000cXN-1I; Mon, 17 Apr 2023 15:28:52 +0000 Date: Mon, 17 Apr 2023 23:28:46 +0800 From: kernel test robot To: Muhammad Usama Anjum , Peter Xu , David Hildenbrand , Andrew Morton , =?utf-8?B?TWljaGHFgiBNaXJvc8WCYXc=?= , Andrei Vagin , Danylo Mocherniuk , Paul Gofman , Cyrill Gorcunov , Mike Rapoport , Nadav Amit Cc: oe-kbuild-all@lists.linux.dev, Linux Memory Management List , Alexander Viro , Shuah Khan , Christian Brauner , Yang Shi , Vlastimil Babka , "Liam R . Howlett" , Yun Zhou , Suren Baghdasaryan , Alex Sierra , Muhammad Usama Anjum , Matthew Wilcox , Pasha Tatashin , Axel Rasmussen , "Gustavo A . R . Silva" , Dan Williams , linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org, linux-kselftest@vger.kernel.org, Greg KH Subject: Re: [PATCH v13 2/5] fs/proc/task_mmu: Implement IOCTL to get and optionally clear info about PTEs Message-ID: <202304172319.CjlksY4s-lkp@intel.com> References: <20230417125630.1146906-3-usama.anjum@collabora.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20230417125630.1146906-3-usama.anjum@collabora.com> X-Stat-Signature: bbuox749cu9cuhkqo7m9n3ftgnpdo6nh X-Rspam-User: X-Rspamd-Queue-Id: 822E34001F X-Rspamd-Server: rspam06 X-HE-Tag: 1681745353-234835 X-HE-Meta: U2FsdGVkX19Ojzsu7PX8evl8xorrIDKwGYObuSCZCuY6uFcMRM6BW0wTHMO7j19NVVix9+xxTW9KtRoYPQwJmhVgPXZaJwpAEatTWvYw8zqi/GcoBg9Y07tX/LOr//+ZZDEP/dCwYSISBWRB4ecvf/5qaU8oVUCE8kpzEGB9agMK5AoZlZ5whQSfTvibVaCfQ/uSRvKeAqRZf0E4MR7SZ3Mo16PALtd+lzf89eI419gWJfksX/LlULZWVYl8MdKpiR53Ssv+PEdQ06rRVSlT1cRmZV6qT8M9IGIzxtOPcZht9aQBX3na8IDhno8EkVNojl3jV8LoZ0sjd0CEDZFyfMmm6oGqVkxusAkJR3DVr8OqyEvbw0FuZUYfKt9GKdfp3GAdUr2FnmxQ8PzS8pkbZmwu2KSKeACYovfYkBPjlaL5O+Usol7QNlYl1mO6OD5ms8WfLrXa4f/G9Ecpef/YOstvDYiVjj/ZQKDVc06CFW1jHimtDyUTEyEcmGU4/x9zLE4dJVUcRtXUcRiEw80H8d/e2H3CeZ3KFu4m9Tzf+L5MQYOZipbLAG8yowGcO6kAHcvWPDE9EQjG+D/49RNoKEY/POoNU9zdPw1HfnbuyBi8IOCQR2753VfJjF4ZOnVY5xJkyILc/l1nBR7czukrWoqPY2c8nYUAg4Wo1simdgoPq5In6OE7qKMAGPYdNOufs+4boRPDuJbonnuRJY9KqGFNrY5FrUV2vjqbeO1kOvkkFcgPxXWJfpkp44zqp2Qv3MQIv/MkKDovDdoJwvuAxF6FEXQlJtybiK1/z5Pz6mD9pnGbyNM/qo2caRVOTIvxJRlrq31m85zmt0M+NUAJLMLwKPhiFhfgmibmBUiZUs2LMT2rxqsYp4zUbLIuLo1CYlP+dvjk+t/KcCVR5Diw5rBCjwNH+WPa6UNJwOkOlkX6vt9yY3jBTW4dSgqLfyzoF3Z17AHKIedtIsGs3oc uZzSzHIv /GPiapw/AJvxADRzqm+537MMLh8Mj4ShGDT2u3dXcCvo4naGehMme2EzTpBCaBkrupwBrSghw2I2UzPNdgu24WS1rOY3zg4rApeJPIIqUzxQWsTeOMwmvKZh79d5oz3YhhZ812mzizzrLozf5LOKKpHie+nyjOhjQdbuA3ljRf5NNM4Hoaa0m+2GTJVw1Z/Ixto6AHCS4VR9v16vdPNA4281bTfRJrKcmCpuyhft/aV5dOqxE2sIFZ7z55hwZlk3uo+60rY3c9ywyG49cuWukiafvD5qUVx152/6slSyfQdpXkEBOEbce0h6qDZAaCmoYg4zUzQNvro2LX464ARIqh8U1UgB+sVYt7YaazQvYAUKjcGq5KYg7a1f3K21KU3iTkiWJuQhieoZEMVs9fn4JwaN99SvuSzYGAwKgDXRjAZX2LueiVvfNcJ6lGXFpfA0oi03CJJfHQ27TPwvoJ0/OzvktiIbdBQy+6BYVCNsQES2TNRaiMCBn8jiXxg== 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: Hi Muhammad, kernel test robot noticed the following build warnings: [auto build test WARNING on akpm-mm/mm-everything] [cannot apply to linus/master v6.3-rc7] [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/Muhammad-Usama-Anjum/userfaultfd-UFFD_FEATURE_WP_ASYNC/20230417-210005 base: https://git.kernel.org/pub/scm/linux/kernel/git/akpm/mm.git mm-everything patch link: https://lore.kernel.org/r/20230417125630.1146906-3-usama.anjum%40collabora.com patch subject: [PATCH v13 2/5] fs/proc/task_mmu: Implement IOCTL to get and optionally clear info about PTEs config: i386-randconfig-a003-20230417 (https://download.01.org/0day-ci/archive/20230417/202304172319.CjlksY4s-lkp@intel.com/config) compiler: gcc-11 (Debian 11.3.0-8) 11.3.0 reproduce (this is a W=1 build): # https://github.com/intel-lab-lkp/linux/commit/2e7e6f5bd3b9c28493f9871282fe8f14e2263962 git remote add linux-review https://github.com/intel-lab-lkp/linux git fetch --no-tags linux-review Muhammad-Usama-Anjum/userfaultfd-UFFD_FEATURE_WP_ASYNC/20230417-210005 git checkout 2e7e6f5bd3b9c28493f9871282fe8f14e2263962 # save the config file mkdir build_dir && cp config build_dir/.config make W=1 O=build_dir ARCH=i386 olddefconfig make W=1 O=build_dir ARCH=i386 SHELL=/bin/bash fs/proc/ If you fix the issue, kindly add following tag where applicable | Reported-by: kernel test robot | Link: https://lore.kernel.org/oe-kbuild-all/202304172319.CjlksY4s-lkp@intel.com/ All warnings (new ones prefixed by >>): fs/proc/task_mmu.c: In function 'pagemap_scan_pte_hole': fs/proc/task_mmu.c:2059:14: error: invalid use of undefined type 'struct pagemap_scan_private' 2059 | if (p->max_pages && p->found_pages + n_pages > p->max_pages) | ^~ fs/proc/task_mmu.c:2059:30: error: invalid use of undefined type 'struct pagemap_scan_private' 2059 | if (p->max_pages && p->found_pages + n_pages > p->max_pages) | ^~ fs/proc/task_mmu.c:2059:57: error: invalid use of undefined type 'struct pagemap_scan_private' 2059 | if (p->max_pages && p->found_pages + n_pages > p->max_pages) | ^~ fs/proc/task_mmu.c:2060:28: error: invalid use of undefined type 'struct pagemap_scan_private' 2060 | n_pages = p->max_pages - p->found_pages; | ^~ fs/proc/task_mmu.c:2060:43: error: invalid use of undefined type 'struct pagemap_scan_private' 2060 | n_pages = p->max_pages - p->found_pages; | ^~ fs/proc/task_mmu.c:2062:15: error: implicit declaration of function 'pagemap_scan_output' [-Werror=implicit-function-declaration] 2062 | ret = pagemap_scan_output(false, vma->vm_file, false, false, p, addr, | ^~~~~~~~~~~~~~~~~~~ fs/proc/task_mmu.c: At top level: fs/proc/task_mmu.c:2068:22: error: 'pagemap_scan_test_walk' undeclared here (not in a function); did you mean 'pagemap_scan_pte_hole'? 2068 | .test_walk = pagemap_scan_test_walk, | ^~~~~~~~~~~~~~~~~~~~~~ | pagemap_scan_pte_hole fs/proc/task_mmu.c:2069:22: error: 'pagemap_scan_pmd_entry' undeclared here (not in a function); did you mean 'pagemap_scan_pte_hole'? 2069 | .pmd_entry = pagemap_scan_pmd_entry, | ^~~~~~~~~~~~~~~~~~~~~~ | pagemap_scan_pte_hole fs/proc/task_mmu.c: In function 'pagemap_scan_args_valid': fs/proc/task_mmu.c:2080:27: error: 'PM_SCAN_OPS' undeclared (first use in this function); did you mean 'PM_SCAN_OP_WP'? 2080 | if (arg->flags & ~PM_SCAN_OPS) | ^~~~~~~~~~~ | PM_SCAN_OP_WP fs/proc/task_mmu.c:2080:27: note: each undeclared identifier is reported only once for each function it appears in fs/proc/task_mmu.c:2085:35: error: 'PM_SCAN_BITS_ALL' undeclared (first use in this function); did you mean 'CPU_BITS_ALL'? 2085 | arg->return_mask) & ~PM_SCAN_BITS_ALL) | ^~~~~~~~~~~~~~~~ | CPU_BITS_ALL fs/proc/task_mmu.c:2106:14: error: implicit declaration of function 'PM_SCAN_DO_UFFD_WP'; did you mean 'PM_SCAN_OP_WP'? [-Werror=implicit-function-declaration] 2106 | if (!PM_SCAN_DO_UFFD_WP(arg)) | ^~~~~~~~~~~~~~~~~~ | PM_SCAN_OP_WP fs/proc/task_mmu.c: In function 'do_pagemap_scan': fs/proc/task_mmu.c:2123:37: error: storage size of 'p' isn't known 2123 | struct pagemap_scan_private p; | ^ fs/proc/task_mmu.c:2148:21: error: 'PAGEMAP_WALK_SIZE' undeclared (first use in this function); did you mean 'PAGE_TABLE_SIZE'? 2148 | p.vec_len = PAGEMAP_WALK_SIZE >> PAGE_SHIFT; | ^~~~~~~~~~~~~~~~~ | PAGE_TABLE_SIZE In file included from include/linux/kernel.h:27, from arch/x86/include/asm/percpu.h:27, from arch/x86/include/asm/preempt.h:6, from include/linux/preempt.h:78, from include/linux/spinlock.h:56, from include/linux/mmzone.h:8, from include/linux/gfp.h:7, from include/linux/mm.h:7, from include/linux/pagewalk.h:5, from fs/proc/task_mmu.c:2: include/linux/minmax.h:36:9: error: first argument to '__builtin_choose_expr' not a constant 36 | __builtin_choose_expr(__safe_cmp(x, y), \ | ^~~~~~~~~~~~~~~~~~~~~ include/linux/minmax.h:67:25: note: in expansion of macro '__careful_cmp' 67 | #define min(x, y) __careful_cmp(x, y, <) | ^~~~~~~~~~~~~ fs/proc/task_mmu.c:2173:29: note: in expansion of macro 'min' 2173 | p.vec_len = min(p.vec_len, empty_slots); | ^~~ fs/proc/task_mmu.c:2175:63: error: 'PAGEMAP_WALK_MASK' undeclared (first use in this function) 2175 | walk_end = (walk_start + PAGEMAP_WALK_SIZE) & PAGEMAP_WALK_MASK; | ^~~~~~~~~~~~~~~~~ fs/proc/task_mmu.c:2186:53: error: 'PM_SCAN_FOUND_MAX_PAGES' undeclared (first use in this function) 2186 | if (ret && ret != -ENOSPC && ret != PM_SCAN_FOUND_MAX_PAGES) | ^~~~~~~~~~~~~~~~~~~~~~~ >> fs/proc/task_mmu.c:2123:37: warning: unused variable 'p' [-Wunused-variable] 2123 | struct pagemap_scan_private p; | ^ fs/proc/task_mmu.c: At top level: fs/proc/task_mmu.c:2240:27: error: 'pagemap_read' undeclared here (not in a function); did you mean 'filemap_read'? 2240 | .read = pagemap_read, | ^~~~~~~~~~~~ | filemap_read fs/proc/task_mmu.c:2241:27: error: 'pagemap_open' undeclared here (not in a function); did you mean 'pid_maps_open'? 2241 | .open = pagemap_open, | ^~~~~~~~~~~~ | pid_maps_open fs/proc/task_mmu.c:2242:27: error: 'pagemap_release' undeclared here (not in a function); did you mean 'proc_map_release'? 2242 | .release = pagemap_release, | ^~~~~~~~~~~~~~~ | proc_map_release fs/proc/task_mmu.c:2246:2: error: #endif without #if 2246 | #endif /* CONFIG_PROC_PAGE_MONITOR */ | ^~~~~ cc1: some warnings being treated as errors vim +/p +2123 fs/proc/task_mmu.c 2115 2116 static long do_pagemap_scan(struct mm_struct *mm, 2117 struct pm_scan_arg __user *uarg) 2118 { 2119 unsigned long start, end, walk_start, walk_end; 2120 unsigned long empty_slots, vec_index = 0; 2121 struct mmu_notifier_range range; 2122 struct page_region __user *vec; > 2123 struct pagemap_scan_private p; 2124 struct pm_scan_arg arg; 2125 int ret = 0; 2126 2127 if (copy_from_user(&arg, uarg, sizeof(arg))) 2128 return -EFAULT; 2129 2130 start = untagged_addr((unsigned long)arg.start); 2131 vec = (struct page_region *)untagged_addr((unsigned long)arg.vec); 2132 2133 ret = pagemap_scan_args_valid(&arg, start, vec); 2134 if (ret) 2135 return ret; 2136 2137 end = start + arg.len; 2138 p.max_pages = arg.max_pages; 2139 p.found_pages = 0; 2140 p.flags = arg.flags; 2141 p.required_mask = arg.required_mask; 2142 p.anyof_mask = arg.anyof_mask; 2143 p.excluded_mask = arg.excluded_mask; 2144 p.return_mask = arg.return_mask; 2145 p.cur.len = 0; 2146 p.cur.start = 0; 2147 p.vec = NULL; 2148 p.vec_len = PAGEMAP_WALK_SIZE >> PAGE_SHIFT; 2149 2150 /* 2151 * Allocate smaller buffer to get output from inside the page walk 2152 * functions and walk page range in PAGEMAP_WALK_SIZE size chunks. As 2153 * we want to return output to user in compact form where no two 2154 * consecutive regions should be continuous and have the same flags. 2155 * So store the latest element in p.cur between different walks and 2156 * store the p.cur at the end of the walk to the user buffer. 2157 */ 2158 p.vec = kmalloc_array(p.vec_len, sizeof(*p.vec), GFP_KERNEL); 2159 if (!p.vec) 2160 return -ENOMEM; 2161 2162 if (p.flags & PM_SCAN_OP_WP) { 2163 mmu_notifier_range_init(&range, MMU_NOTIFY_PROTECTION_VMA, 0, 2164 vma->vm_mm, start, end); 2165 mmu_notifier_invalidate_range_start(&range); 2166 } 2167 2168 walk_start = walk_end = start; 2169 while (walk_end < end && !ret) { 2170 p.vec_index = 0; 2171 2172 empty_slots = arg.vec_len - vec_index; 2173 p.vec_len = min(p.vec_len, empty_slots); 2174 2175 walk_end = (walk_start + PAGEMAP_WALK_SIZE) & PAGEMAP_WALK_MASK; 2176 if (walk_end > end) 2177 walk_end = end; 2178 2179 ret = mmap_read_lock_killable(mm); 2180 if (ret) 2181 goto free_data; 2182 ret = walk_page_range(mm, walk_start, walk_end, 2183 &pagemap_scan_ops, &p); 2184 mmap_read_unlock(mm); 2185 2186 if (ret && ret != -ENOSPC && ret != PM_SCAN_FOUND_MAX_PAGES) 2187 goto free_data; 2188 2189 walk_start = walk_end; 2190 if (p.vec_index) { 2191 if (copy_to_user(&vec[vec_index], p.vec, 2192 p.vec_index * sizeof(*p.vec))) { 2193 /* 2194 * Return error even though the OP succeeded 2195 */ 2196 ret = -EFAULT; 2197 goto free_data; 2198 } 2199 vec_index += p.vec_index; 2200 } 2201 } 2202 2203 if (p.flags & PM_SCAN_OP_WP) { 2204 mmu_notifier_invalidate_range_end(&range); 2205 flush_tlb_mm_range(mm, start, end, PAGE_SHIFT, false); 2206 } 2207 2208 if (p.cur.len) { 2209 if (copy_to_user(&vec[vec_index], &p.cur, sizeof(*p.vec))) { 2210 ret = -EFAULT; 2211 goto free_data; 2212 } 2213 vec_index++; 2214 } 2215 2216 ret = vec_index; 2217 2218 free_data: 2219 kfree(p.vec); 2220 return ret; 2221 } 2222 -- 0-DAY CI Kernel Test Service https://github.com/intel/lkp-tests