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 C0636C30653 for ; Thu, 27 Jun 2024 19:59:40 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 4DE7E6B009C; Thu, 27 Jun 2024 15:59:40 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 463FE6B00A6; Thu, 27 Jun 2024 15:59:40 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 26AA16B00A8; Thu, 27 Jun 2024 15:59:40 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0015.hostedemail.com [216.40.44.15]) by kanga.kvack.org (Postfix) with ESMTP id 0169B6B009C for ; Thu, 27 Jun 2024 15:59:39 -0400 (EDT) Received: from smtpin20.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay02.hostedemail.com (Postfix) with ESMTP id A7E01120EEB for ; Thu, 27 Jun 2024 19:59:39 +0000 (UTC) X-FDA: 82277733678.20.5E9B4D4 Received: from mgamail.intel.com (mgamail.intel.com [198.175.65.20]) by imf30.hostedemail.com (Postfix) with ESMTP id 6397A80014 for ; Thu, 27 Jun 2024 19:59:36 +0000 (UTC) Authentication-Results: imf30.hostedemail.com; dkim=pass header.d=intel.com header.s=Intel header.b="AP/TDaNT"; spf=pass (imf30.hostedemail.com: domain of lkp@intel.com designates 198.175.65.20 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=1719518368; 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=tEhsPpQCmj+wGbFcZI/NR1F0j+iCF4qKjIY7IEc7jiY=; b=2bEFNcrXW6Fc/mcU6ULrz+8H0IlAtySlYS1J6HUKpPLTpPGLU/Vnb/JX/AcUmYqv7d9zaX 2Gk2bDNu5rv3yEcg/woSIepKjsWwQXCYJJLpG2RACadlt8XEujsGR+WaDRma0irM4FBvYd Xm94KGm6uTjWPF8o7Cz5EpjtJglycIU= ARC-Authentication-Results: i=1; imf30.hostedemail.com; dkim=pass header.d=intel.com header.s=Intel header.b="AP/TDaNT"; spf=pass (imf30.hostedemail.com: domain of lkp@intel.com designates 198.175.65.20 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=1719518368; a=rsa-sha256; cv=none; b=ZRz1GeSlaNGvjBAmdPqWymx6k+ZdpRIznsoVMcsgt5XSnxzlVKkRObdyRt941/+rOlOhWX BCJKHS7BWg78XThXxYBh25kfwj6wvDK1oJIfWOb5wD5OYwKIj86fj1Y+p9epaUtMb/edDV J76/PBvJePmi/vAAlQOWA70U5tbM6GQ= DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1719518377; x=1751054377; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=3z165P0PXQq8QGPP1ffXeo3HZ7sQi3bxyPW8nz/fd7g=; b=AP/TDaNTocA/ggbf34DTVUKncxk2y0AOlsMlxC2Ylypb0qbZhjdV5UDV LGy4f8RUIkF7bFNa5F9f29BhNu3TjqnmW9OoxYfbIGT6cngi9o8lV5aN9 xmH3eH8F8dfQ/6HBidfCKY9vCOupprJN9O5oRiRTycAAvQGNiimJD5aMO QcoZM+3TmmK4H+ErWx2qoT8xQqc34ZsuqzdhUGHUNyA1w7P0YIyhoG9Jo XXoxqcRbt3pRCeWS4eDQhKF4c7oqMhVIBjzhEuob4mnoTEvNqP20T1osP Avo/GgClaUTrlhA0PWrgXkOTvL2jn8wk5M6iUP5GyN6TICKI/KEWhPfTC g==; X-CSE-ConnectionGUID: 6CjJB1ezQx+27prM3HunEA== X-CSE-MsgGUID: fXvedjh7RwGLj7geSHFKPg== X-IronPort-AV: E=McAfee;i="6700,10204,11116"; a="16498168" X-IronPort-AV: E=Sophos;i="6.09,167,1716274800"; d="scan'208";a="16498168" Received: from fmviesa001.fm.intel.com ([10.60.135.141]) by orvoesa112.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 27 Jun 2024 12:59:35 -0700 X-CSE-ConnectionGUID: bvoQ2NTiQuC1WXL9mUacPQ== X-CSE-MsgGUID: FSNI026cTPSECSKwoi+KBg== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.09,167,1716274800"; d="scan'208";a="75699645" Received: from lkp-server01.sh.intel.com (HELO 68891e0c336b) ([10.239.97.150]) by fmviesa001.fm.intel.com with ESMTP; 27 Jun 2024 12:59:31 -0700 Received: from kbuild by 68891e0c336b with local (Exim 4.96) (envelope-from ) id 1sMvHF-000GUu-0E; Thu, 27 Jun 2024 19:59:29 +0000 Date: Fri, 28 Jun 2024 03:58:55 +0800 From: kernel test robot To: Kairui Song , linux-mm@kvack.org Cc: oe-kbuild-all@lists.linux.dev, Andrew Morton , Linux Memory Management List , Matthew Wilcox , Johannes Weiner , Roman Gushchin , Waiman Long , Shakeel Butt , Nhat Pham , Michal Hocko , Chengming Zhou , Qi Zheng , Muchun Song , Chris Li , Yosry Ahmed , "Huang, Ying" , Kairui Song Subject: Re: [PATCH 7/7] mm/list_lru: Simplify the list_lru walk callback function Message-ID: <202406280355.Qwjjjiug-lkp@intel.com> References: <20240624175313.47329-8-ryncsn@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20240624175313.47329-8-ryncsn@gmail.com> X-Rspam-User: X-Rspamd-Server: rspam01 X-Rspamd-Queue-Id: 6397A80014 X-Stat-Signature: rnn3bcag1bzawcaogyqr4675d1x497em X-HE-Tag: 1719518376-943565 X-HE-Meta: U2FsdGVkX19A9HA6DLhmBje2382U1970KRDgMaSCrTMoivDlIvI5M4J/5Co+EDORRrHfXH1786mKw2hZi6FvpdYQdI47p7Yht6J5CO/+u+U7W9z9vcFYyQVCG3hG6lCCtBzoVIw1qWALwb1v/cqhnBXX7YufDMxQIQ9Wwd8gAgJ1divLI7Adz5595Q6fQnv17S7aQ2v17TgX+FDAx+ue4V/JoZqCTd1rakQ6F7uJLbPuo0ulSH7EhUxT8DG3X3gBvKcZU3ZH1hLrGwNnBoaBb4DsXarSdhp18iO7dld6zwHaso7oQpb2l29zXGLb0WTa85sf7LUc0m/X7G2zNVD3OMmn3u7Ms4u5cCIfx2wKU8/jGddrBrGUE58fSGq1CPkf3HhsKzpVJdFvRjOJTPDXpF7zaJbp0hKMHajtq14Jk5CWLjx1XhRNPuW+teLEyh6fCHxdwgyyYWVjTiNJmuJlk2rsVSUJN8pW848vjynWFxyt0Pm2BJ4Mt43ZwwAV+DSKrqH1BEG3AhEUdIozM9S425e/OVBSh9NWMWkdjiiGE00zJjBi+1/lYymAY8eOX/Qe2bKCYRyz9YyOltB1so6eRr7Lduq0xTf8zSqrVA9RFA6YpN/htzyySh1ZgV/DtW8O6udTWTgTtMhDkhTfj1EyFbPp5XjXUbJWk0ixpXSzFKJXwY6a57DS/h1JxlwTEO0DmyVteoMM/V4DW8T+03gzUyGtzpnFsF1Mw1pE2CEqBF6lU1lIoeJqZMRAMYdzGZ7j4dCBILMkybBuA8Q9IbtC83gwdTjioBqEIe1qm6BtKqi0n85M2yZ+6gHWFFMYvZ9NispDeAjc6KdBnec+ACkexvusBKaSun5nx/BUZvEczVZOZRZ55J+zxuzJKsBsYzKtVd2O0UvSZ8Rf4t2Mse97KXvCgekuP44akABu0ZeesyJu+TiNOfZcH4EA1kxVtAW77l3JE/QZW9b+kQSovsv v4iz8EG+ 2/UfU6MtEIxHUL/jL8YVdwna/h7SGuKkE3jQo3QmMmWMD47sdpxqKb0Vz5dnLJf80GmxZnfo6v7idw70eRYEFq+S4pBH4Jn0ow8qXoMY0I2IL1NO42RRXKOYw/6Lqb6qKQBuUgzhoEbqep949SO/PtvJcHn9De8Ldp5O/KkeW5En/s1zMam5FuFQ2yXYKLfUwH7j2f5HmdYd9JPsRjD5E4iLuJmP5a+TNspGR1OpK0N0iOV/+K+rJaiEpo6f1UeipELQwcdn1OFCXRrA/0D7KsdD+8O2TVzNTMbEhgovZuCmFXyk0uAtYHW8jQfyNtYTy/2/u2K0WOtx/UDJ5C9XpTubVSaLTBrQEiHPVTP7K9NucUleKP8TXBmHOPyC3+alR8oHqaX+pCX3XpCeN3LlMITa6zoXdKeFJuSQDTOgXrPGDvAIB3dDvQvQz/lkhEbFLkTHEUBqTnEFmHp2Ru5wrm2/lUA== 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: Hi Kairui, kernel test robot noticed the following build warnings: [auto build test WARNING on akpm-mm/mm-everything] [also build test WARNING on staging/staging-testing staging/staging-next staging/staging-linus xfs-linux/for-next linus/master v6.10-rc5 next-20240626] [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/Kairui-Song/mm-swap-workingset-make-anon-workingset-nodes-memcg-aware/20240625-231015 base: https://git.kernel.org/pub/scm/linux/kernel/git/akpm/mm.git mm-everything patch link: https://lore.kernel.org/r/20240624175313.47329-8-ryncsn%40gmail.com patch subject: [PATCH 7/7] mm/list_lru: Simplify the list_lru walk callback function config: i386-buildonly-randconfig-001-20240628 (https://download.01.org/0day-ci/archive/20240628/202406280355.Qwjjjiug-lkp@intel.com/config) compiler: gcc-10 (Ubuntu 10.5.0-1ubuntu1) 10.5.0 reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240628/202406280355.Qwjjjiug-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/202406280355.Qwjjjiug-lkp@intel.com/ All warnings (new ones prefixed by >>): drivers/android/binder_alloc.c:1060: warning: Function parameter or struct member 'lru' not described in 'binder_alloc_free_page' >> drivers/android/binder_alloc.c:1060: warning: Excess function parameter 'lock' description in 'binder_alloc_free_page' vim +1060 drivers/android/binder_alloc.c 0c972a05cde66e Todd Kjos 2017-06-29 1046 f2517eb76f1f2f Sherry Yang 2017-08-23 1047 /** f2517eb76f1f2f Sherry Yang 2017-08-23 1048 * binder_alloc_free_page() - shrinker callback to free pages f2517eb76f1f2f Sherry Yang 2017-08-23 1049 * @item: item to free f2517eb76f1f2f Sherry Yang 2017-08-23 1050 * @lock: lock protecting the item f2517eb76f1f2f Sherry Yang 2017-08-23 1051 * @cb_arg: callback argument f2517eb76f1f2f Sherry Yang 2017-08-23 1052 * f2517eb76f1f2f Sherry Yang 2017-08-23 1053 * Called from list_lru_walk() in binder_shrink_scan() to free f2517eb76f1f2f Sherry Yang 2017-08-23 1054 * up pages when the system is under memory pressure. f2517eb76f1f2f Sherry Yang 2017-08-23 1055 */ f2517eb76f1f2f Sherry Yang 2017-08-23 1056 enum lru_status binder_alloc_free_page(struct list_head *item, f2517eb76f1f2f Sherry Yang 2017-08-23 1057 struct list_lru_one *lru, f2517eb76f1f2f Sherry Yang 2017-08-23 1058 void *cb_arg) 5672291f7d5dc5 Kairui Song 2024-06-25 1059 __must_hold(&lru->lock) f2517eb76f1f2f Sherry Yang 2017-08-23 @1060 { e50f4e6cc9bfac Carlos Llamas 2023-12-01 1061 struct binder_lru_page *page = container_of(item, typeof(*page), lru); e50f4e6cc9bfac Carlos Llamas 2023-12-01 1062 struct binder_alloc *alloc = page->alloc; e50f4e6cc9bfac Carlos Llamas 2023-12-01 1063 struct mm_struct *mm = alloc->mm; a1b2289cef92ef Sherry Yang 2017-10-03 1064 struct vm_area_struct *vma; e50f4e6cc9bfac Carlos Llamas 2023-12-01 1065 struct page *page_to_free; df9aabead791d7 Carlos Llamas 2023-12-01 1066 unsigned long page_addr; f2517eb76f1f2f Sherry Yang 2017-08-23 1067 size_t index; f2517eb76f1f2f Sherry Yang 2017-08-23 1068 e50f4e6cc9bfac Carlos Llamas 2023-12-01 1069 if (!mmget_not_zero(mm)) e50f4e6cc9bfac Carlos Llamas 2023-12-01 1070 goto err_mmget; e50f4e6cc9bfac Carlos Llamas 2023-12-01 1071 if (!mmap_read_trylock(mm)) e50f4e6cc9bfac Carlos Llamas 2023-12-01 1072 goto err_mmap_read_lock_failed; 7710e2cca32e7f Carlos Llamas 2023-12-01 1073 if (!spin_trylock(&alloc->lock)) 7710e2cca32e7f Carlos Llamas 2023-12-01 1074 goto err_get_alloc_lock_failed; f2517eb76f1f2f Sherry Yang 2017-08-23 1075 if (!page->page_ptr) f2517eb76f1f2f Sherry Yang 2017-08-23 1076 goto err_page_already_freed; f2517eb76f1f2f Sherry Yang 2017-08-23 1077 f2517eb76f1f2f Sherry Yang 2017-08-23 1078 index = page - alloc->pages; df9aabead791d7 Carlos Llamas 2023-12-01 1079 page_addr = alloc->buffer + index * PAGE_SIZE; 5cec2d2e5839f9 Todd Kjos 2019-03-01 1080 3f489c2067c582 Carlos Llamas 2023-12-01 1081 vma = vma_lookup(mm, page_addr); 3f489c2067c582 Carlos Llamas 2023-12-01 1082 if (vma && vma != binder_alloc_get_vma(alloc)) 3f489c2067c582 Carlos Llamas 2023-12-01 1083 goto err_invalid_vma; a1b2289cef92ef Sherry Yang 2017-10-03 1084 e50f4e6cc9bfac Carlos Llamas 2023-12-01 1085 trace_binder_unmap_kernel_start(alloc, index); e50f4e6cc9bfac Carlos Llamas 2023-12-01 1086 e50f4e6cc9bfac Carlos Llamas 2023-12-01 1087 page_to_free = page->page_ptr; e50f4e6cc9bfac Carlos Llamas 2023-12-01 1088 page->page_ptr = NULL; e50f4e6cc9bfac Carlos Llamas 2023-12-01 1089 e50f4e6cc9bfac Carlos Llamas 2023-12-01 1090 trace_binder_unmap_kernel_end(alloc, index); a1b2289cef92ef Sherry Yang 2017-10-03 1091 a1b2289cef92ef Sherry Yang 2017-10-03 1092 list_lru_isolate(lru, item); 7710e2cca32e7f Carlos Llamas 2023-12-01 1093 spin_unlock(&alloc->lock); 5672291f7d5dc5 Kairui Song 2024-06-25 1094 spin_unlock(&lru->lock); f2517eb76f1f2f Sherry Yang 2017-08-23 1095 a1b2289cef92ef Sherry Yang 2017-10-03 1096 if (vma) { e41e164c3cdff6 Sherry Yang 2017-08-23 1097 trace_binder_unmap_user_start(alloc, index); e41e164c3cdff6 Sherry Yang 2017-08-23 1098 e9adcfecf572fc Mike Kravetz 2023-01-03 1099 zap_page_range_single(vma, page_addr, PAGE_SIZE, NULL); f2517eb76f1f2f Sherry Yang 2017-08-23 1100 e41e164c3cdff6 Sherry Yang 2017-08-23 1101 trace_binder_unmap_user_end(alloc, index); f2517eb76f1f2f Sherry Yang 2017-08-23 1102 } e50f4e6cc9bfac Carlos Llamas 2023-12-01 1103 d8ed45c5dcd455 Michel Lespinasse 2020-06-08 1104 mmap_read_unlock(mm); f867c771f98891 Tetsuo Handa 2020-07-17 1105 mmput_async(mm); e50f4e6cc9bfac Carlos Llamas 2023-12-01 1106 __free_page(page_to_free); e41e164c3cdff6 Sherry Yang 2017-08-23 1107 a1b2289cef92ef Sherry Yang 2017-10-03 1108 return LRU_REMOVED_RETRY; f2517eb76f1f2f Sherry Yang 2017-08-23 1109 3f489c2067c582 Carlos Llamas 2023-12-01 1110 err_invalid_vma: e50f4e6cc9bfac Carlos Llamas 2023-12-01 1111 err_page_already_freed: 7710e2cca32e7f Carlos Llamas 2023-12-01 1112 spin_unlock(&alloc->lock); 7710e2cca32e7f Carlos Llamas 2023-12-01 1113 err_get_alloc_lock_failed: 3f489c2067c582 Carlos Llamas 2023-12-01 1114 mmap_read_unlock(mm); 3e4e28c5a8f01e Michel Lespinasse 2020-06-08 1115 err_mmap_read_lock_failed: a1b2289cef92ef Sherry Yang 2017-10-03 1116 mmput_async(mm); a0c2baaf81bd53 Sherry Yang 2017-10-20 1117 err_mmget: f2517eb76f1f2f Sherry Yang 2017-08-23 1118 return LRU_SKIP; f2517eb76f1f2f Sherry Yang 2017-08-23 1119 } f2517eb76f1f2f Sherry Yang 2017-08-23 1120 -- 0-DAY CI Kernel Test Service https://github.com/intel/lkp-tests/wiki