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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 8B4A2C4332F for ; Sun, 3 Apr 2022 17:22:45 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1359657AbiDCRYh (ORCPT ); Sun, 3 Apr 2022 13:24:37 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:50566 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1359654AbiDCRYg (ORCPT ); Sun, 3 Apr 2022 13:24:36 -0400 Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 56FA939836; Sun, 3 Apr 2022 10:22:42 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1649006562; x=1680542562; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=Kg3nT9zXXv061+GqAw9adWjDJuUK/w2Dr4majqksh4k=; b=Z0eZ59Fp7LPrRDbZMeKQdugdoFWL1UzSdHs0uI3Kl0mInG5l6+65iFog jnAYMbh0fypUYxk2KXOGTRh8R1V3vgeQA9qN8APWaajePoooBtLjQgUZK ELGwXWuZfl12h4nN2g+zvtXkq/g11P4q5njB6uHfuOUR4aZM8g7U2ZMsK lhUynb5EofZQvPqz8IRYFF82BAQoxPM4G8UbcbEtPIW7wHEblBwaGWB9X ZJGOQ/bN1Sd92gLPfI7zPiujOEu5V4lMjZzSVLTbcOvn0U3cvUIydEEG7 99E+GoYgYcIDdV4RrbNiEqvUqDcwukT2KskkIMfqw8QGYE3jY6ssCb+GV w==; X-IronPort-AV: E=McAfee;i="6200,9189,10306"; a="257980051" X-IronPort-AV: E=Sophos;i="5.90,231,1643702400"; d="scan'208";a="257980051" Received: from fmsmga002.fm.intel.com ([10.253.24.26]) by fmsmga102.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 03 Apr 2022 10:22:41 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.90,231,1643702400"; d="scan'208";a="651261140" Received: from lkp-server02.sh.intel.com (HELO a44fdfb70b94) ([10.239.97.151]) by fmsmga002.fm.intel.com with ESMTP; 03 Apr 2022 10:22:37 -0700 Received: from kbuild by a44fdfb70b94 with local (Exim 4.95) (envelope-from ) id 1nb3vx-00018E-AT; Sun, 03 Apr 2022 17:22:37 +0000 Date: Mon, 4 Apr 2022 01:22:18 +0800 From: kernel test robot To: Bui Quang Minh , cgroups@vger.kernel.org Cc: llvm@lists.linux.dev, kbuild-all@lists.01.org, Bui Quang Minh , Tejun Heo , Zefan Li , Johannes Weiner , Alexei Starovoitov , Daniel Borkmann , Andrii Nakryiko , Martin KaFai Lau , Song Liu , Yonghong Song , John Fastabend , KP Singh , linux-kernel@vger.kernel.org, netdev@vger.kernel.org, bpf@vger.kernel.org Subject: Re: [PATCH] cgroup: Kill the parent controller when its last child is killed Message-ID: <202204040103.yTRTggqu-lkp@intel.com> References: <20220403135717.8294-1-minhquangbui99@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20220403135717.8294-1-minhquangbui99@gmail.com> User-Agent: Mutt/1.10.1 (2018-07-13) Precedence: bulk List-ID: X-Mailing-List: bpf@vger.kernel.org Hi Bui, Thank you for the patch! Yet something to improve: [auto build test ERROR on tj-cgroup/for-next] [also build test ERROR on v5.17 next-20220401] [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] url: https://github.com/intel-lab-lkp/linux/commits/Bui-Quang-Minh/cgroup-Kill-the-parent-controller-when-its-last-child-is-killed/20220403-215911 base: https://git.kernel.org/pub/scm/linux/kernel/git/tj/cgroup.git for-next config: arm-mxs_defconfig (https://download.01.org/0day-ci/archive/20220404/202204040103.yTRTggqu-lkp@intel.com/config) compiler: clang version 15.0.0 (https://github.com/llvm/llvm-project c4a1b07d0979e7ff20d7d541af666d822d66b566) 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 # install arm cross compiling tool for clang build # apt-get install binutils-arm-linux-gnueabi # https://github.com/intel-lab-lkp/linux/commit/2bc22feae8a913c7f371bc79ef9967122d8d326c git remote add linux-review https://github.com/intel-lab-lkp/linux git fetch --no-tags linux-review Bui-Quang-Minh/cgroup-Kill-the-parent-controller-when-its-last-child-is-killed/20220403-215911 git checkout 2bc22feae8a913c7f371bc79ef9967122d8d326c # save the config file to linux build tree mkdir build_dir COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=arm SHELL=/bin/bash If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot All errors (new ones prefixed by >>): >> kernel/cgroup/cgroup.c:5169:39: error: no member named 'refcnt' in 'struct cgroup_bpf' if (!percpu_ref_is_dying(&cgrp->bpf.refcnt)) ~~~~~~~~~ ^ 1 error generated. vim +5169 kernel/cgroup/cgroup.c 5148 5149 static void css_release_work_fn(struct work_struct *work) 5150 { 5151 struct cgroup_subsys_state *css = 5152 container_of(work, struct cgroup_subsys_state, destroy_work); 5153 struct cgroup_subsys *ss = css->ss; 5154 struct cgroup *cgrp = css->cgroup; 5155 struct cgroup *parent = cgroup_parent(cgrp); 5156 5157 mutex_lock(&cgroup_mutex); 5158 5159 css->flags |= CSS_RELEASED; 5160 list_del_rcu(&css->sibling); 5161 5162 /* 5163 * If parent doesn't have any children, start killing it. 5164 * And don't kill the default root. 5165 */ 5166 if (parent && list_empty(&parent->self.children) && 5167 parent != &cgrp_dfl_root.cgrp && 5168 !percpu_ref_is_dying(&parent->self.refcnt)) { > 5169 if (!percpu_ref_is_dying(&cgrp->bpf.refcnt)) 5170 cgroup_bpf_offline(parent); 5171 percpu_ref_kill(&parent->self.refcnt); 5172 } 5173 5174 if (ss) { 5175 /* css release path */ 5176 if (!list_empty(&css->rstat_css_node)) { 5177 cgroup_rstat_flush(cgrp); 5178 list_del_rcu(&css->rstat_css_node); 5179 } 5180 5181 cgroup_idr_replace(&ss->css_idr, NULL, css->id); 5182 if (ss->css_released) 5183 ss->css_released(css); 5184 } else { 5185 struct cgroup *tcgrp; 5186 5187 /* cgroup release path */ 5188 TRACE_CGROUP_PATH(release, cgrp); 5189 5190 cgroup_rstat_flush(cgrp); 5191 5192 spin_lock_irq(&css_set_lock); 5193 for (tcgrp = cgroup_parent(cgrp); tcgrp; 5194 tcgrp = cgroup_parent(tcgrp)) 5195 tcgrp->nr_dying_descendants--; 5196 spin_unlock_irq(&css_set_lock); 5197 5198 /* 5199 * There are two control paths which try to determine 5200 * cgroup from dentry without going through kernfs - 5201 * cgroupstats_build() and css_tryget_online_from_dir(). 5202 * Those are supported by RCU protecting clearing of 5203 * cgrp->kn->priv backpointer. 5204 */ 5205 if (cgrp->kn) 5206 RCU_INIT_POINTER(*(void __rcu __force **)&cgrp->kn->priv, 5207 NULL); 5208 } 5209 5210 mutex_unlock(&cgroup_mutex); 5211 5212 INIT_RCU_WORK(&css->destroy_rwork, css_free_rwork_fn); 5213 queue_rcu_work(cgroup_destroy_wq, &css->destroy_rwork); 5214 } 5215 -- 0-DAY CI Kernel Test Service https://01.org/lkp