From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mgamail.intel.com (mgamail.intel.com [192.198.163.18]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id EAFF3156242 for ; Mon, 2 Sep 2024 08:51:51 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=192.198.163.18 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1725267113; cv=none; b=FOfI8LLH0fMon8z+4m6Z82psod+/5o4VoRLG6ZTIYett2/c8z0Gb9k+UcV28HpkVKz57r6chGf0UXapAgd2pHizbQ1DU5KjoDYGVPKx6RHVTnZtZAQ5J9po0tsCzOb4Yow2Q67P7GvorTwVL4TzsCnt/DIod5ArU+aVSoYNJsYw= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1725267113; c=relaxed/simple; bh=IT4QuHWHrXqtNCswJJdgxgagZu3kUV+wSwcyDgWINbg=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=e4i0pQj89rfD4UF/U70Abn/LAyLkivEcKdBl6MhRvQ7hTyR8pUmCalEmBE++O1af8b18mwlIi5DqtXIXRMFzLB0Vp+AYN98DfGY5YYtjBv7aCSdZwz2TQ9rT53M7R1exU+6kChlRs8PiQUrGp8nxqdpmI3QIQNYsiEeB9meiR3g= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=intel.com; spf=pass smtp.mailfrom=intel.com; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b=Olg8aWxL; arc=none smtp.client-ip=192.198.163.18 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=intel.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=intel.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b="Olg8aWxL" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1725267112; x=1756803112; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=IT4QuHWHrXqtNCswJJdgxgagZu3kUV+wSwcyDgWINbg=; b=Olg8aWxLnX4PxNwQn5jcBsK46IyYzm+wrg+nMkJNhJ4NbebE/e9F3VzH 3V2aGNIC7R8PNqKeXzvtJJiTW9TpzH8V5I7zxEuWfMgz1M4ppcSoW8ra0 71EPW3s3frj6rcJ9iJq8EUPP0QJD1c5MhT5heQHeI1HawB3XfVoK2be+Y 6oO5G9K+OyhUCJC0Cnbwg4XRuh/P5lNgvVRw9o4yRocd0SZYE3jeWY4tg iX5V/FU/2OCQ2LChyCVzzkX0Af8V1uLoZVmVuwP2tLwKrE/UCR6vIgyZK mptiSDUYjoEIFKrYkQtL4CgQzSt4zBuYju+3CQchra+4xuGG/Kse1kPht w==; X-CSE-ConnectionGUID: HehejqCFQ8S1/YOGv6bGaA== X-CSE-MsgGUID: /E+DaR7RTU+cLW4hi7Iq4Q== X-IronPort-AV: E=McAfee;i="6700,10204,11182"; a="23350296" X-IronPort-AV: E=Sophos;i="6.10,195,1719903600"; d="scan'208";a="23350296" Received: from fmviesa008.fm.intel.com ([10.60.135.148]) by fmvoesa112.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 02 Sep 2024 01:51:51 -0700 X-CSE-ConnectionGUID: ikprLMwzQy2jVOMKIqk+gA== X-CSE-MsgGUID: 3xS1TwfyRCq2qzwqmm3ejg== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.10,195,1719903600"; d="scan'208";a="64552774" Received: from lkp-server01.sh.intel.com (HELO 9c6b1c7d3b50) ([10.239.97.150]) by fmviesa008.fm.intel.com with ESMTP; 02 Sep 2024 01:51:49 -0700 Received: from kbuild by 9c6b1c7d3b50 with local (Exim 4.96) (envelope-from ) id 1sl2mo-0005J0-2r; Mon, 02 Sep 2024 08:51:46 +0000 Date: Mon, 2 Sep 2024 16:51:14 +0800 From: kernel test robot To: Jens Wiklander , linux-kernel@vger.kernel.org, Ulf Hansson Cc: llvm@lists.linux.dev, oe-kbuild-all@lists.linux.dev, Linus Walleij , Arnd Bergmann , Greg Kroah-Hartman , Jens Wiklander , Dan Carpenter Subject: Re: [PATCH] rpmb: fix error path in rpmb_dev_register() Message-ID: <202409021647.u41JgxbX-lkp@intel.com> References: <20240830061404.2831708-1-jens.wiklander@linaro.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20240830061404.2831708-1-jens.wiklander@linaro.org> Hi Jens, kernel test robot noticed the following build warnings: [auto build test WARNING on next-20240829] [cannot apply to v6.11-rc5 v6.11-rc4 v6.11-rc3 linus/master v6.11-rc6] [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/Jens-Wiklander/rpmb-fix-error-path-in-rpmb_dev_register/20240830-141540 base: next-20240829 patch link: https://lore.kernel.org/r/20240830061404.2831708-1-jens.wiklander%40linaro.org patch subject: [PATCH] rpmb: fix error path in rpmb_dev_register() config: i386-randconfig-003-20240902 (https://download.01.org/0day-ci/archive/20240902/202409021647.u41JgxbX-lkp@intel.com/config) compiler: clang version 18.1.5 (https://github.com/llvm/llvm-project 617a15a9eac96088ae5e9134248d8236e34b91b1) reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240902/202409021647.u41JgxbX-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/202409021647.u41JgxbX-lkp@intel.com/ All warnings (new ones prefixed by >>): >> drivers/misc/rpmb-core.c:199:1: warning: unused label 'err_id_remove' [-Wunused-label] 199 | err_id_remove: | ^~~~~~~~~~~~~~ 1 warning generated. vim +/err_id_remove +199 drivers/misc/rpmb-core.c 1e9046e3a15460 Jens Wiklander 2024-08-14 146 1e9046e3a15460 Jens Wiklander 2024-08-14 147 /** 1e9046e3a15460 Jens Wiklander 2024-08-14 148 * rpmb_dev_register - register RPMB partition with the RPMB subsystem 1e9046e3a15460 Jens Wiklander 2024-08-14 149 * @dev: storage device of the rpmb device 1e9046e3a15460 Jens Wiklander 2024-08-14 150 * @descr: RPMB device description 1e9046e3a15460 Jens Wiklander 2024-08-14 151 * 1e9046e3a15460 Jens Wiklander 2024-08-14 152 * While registering the RPMB partition extract needed device information 1e9046e3a15460 Jens Wiklander 2024-08-14 153 * while needed resources are available. 1e9046e3a15460 Jens Wiklander 2024-08-14 154 * 1e9046e3a15460 Jens Wiklander 2024-08-14 155 * Returns: a pointer to a 'struct rpmb_dev' or an ERR_PTR on failure 1e9046e3a15460 Jens Wiklander 2024-08-14 156 */ 1e9046e3a15460 Jens Wiklander 2024-08-14 157 struct rpmb_dev *rpmb_dev_register(struct device *dev, 1e9046e3a15460 Jens Wiklander 2024-08-14 158 struct rpmb_descr *descr) 1e9046e3a15460 Jens Wiklander 2024-08-14 159 { 1e9046e3a15460 Jens Wiklander 2024-08-14 160 struct rpmb_dev *rdev; 1e9046e3a15460 Jens Wiklander 2024-08-14 161 int ret; 1e9046e3a15460 Jens Wiklander 2024-08-14 162 1e9046e3a15460 Jens Wiklander 2024-08-14 163 if (!dev || !descr || !descr->route_frames || !descr->dev_id || 1e9046e3a15460 Jens Wiklander 2024-08-14 164 !descr->dev_id_len) 1e9046e3a15460 Jens Wiklander 2024-08-14 165 return ERR_PTR(-EINVAL); 1e9046e3a15460 Jens Wiklander 2024-08-14 166 1e9046e3a15460 Jens Wiklander 2024-08-14 167 rdev = kzalloc(sizeof(*rdev), GFP_KERNEL); 1e9046e3a15460 Jens Wiklander 2024-08-14 168 if (!rdev) 1e9046e3a15460 Jens Wiklander 2024-08-14 169 return ERR_PTR(-ENOMEM); 1e9046e3a15460 Jens Wiklander 2024-08-14 170 rdev->descr = *descr; 1e9046e3a15460 Jens Wiklander 2024-08-14 171 rdev->descr.dev_id = kmemdup(descr->dev_id, descr->dev_id_len, 1e9046e3a15460 Jens Wiklander 2024-08-14 172 GFP_KERNEL); 1e9046e3a15460 Jens Wiklander 2024-08-14 173 if (!rdev->descr.dev_id) { 1e9046e3a15460 Jens Wiklander 2024-08-14 174 ret = -ENOMEM; 1e9046e3a15460 Jens Wiklander 2024-08-14 175 goto err_free_rdev; 1e9046e3a15460 Jens Wiklander 2024-08-14 176 } 1e9046e3a15460 Jens Wiklander 2024-08-14 177 1e9046e3a15460 Jens Wiklander 2024-08-14 178 mutex_lock(&rpmb_mutex); 1e9046e3a15460 Jens Wiklander 2024-08-14 179 ret = ida_simple_get(&rpmb_ida, 0, 0, GFP_KERNEL); 1e9046e3a15460 Jens Wiklander 2024-08-14 180 mutex_unlock(&rpmb_mutex); 1e9046e3a15460 Jens Wiklander 2024-08-14 181 if (ret < 0) 1e9046e3a15460 Jens Wiklander 2024-08-14 182 goto err_free_dev_id; 1e9046e3a15460 Jens Wiklander 2024-08-14 183 rdev->id = ret; 1e9046e3a15460 Jens Wiklander 2024-08-14 184 1e9046e3a15460 Jens Wiklander 2024-08-14 185 dev_set_name(&rdev->dev, "rpmb%d", rdev->id); 1e9046e3a15460 Jens Wiklander 2024-08-14 186 rdev->dev.class = &rpmb_class; 1e9046e3a15460 Jens Wiklander 2024-08-14 187 rdev->dev.parent = dev; 1e9046e3a15460 Jens Wiklander 2024-08-14 188 1e9046e3a15460 Jens Wiklander 2024-08-14 189 ret = device_register(&rdev->dev); d1c997b32ba265 Jens Wiklander 2024-08-30 190 if (ret) { d1c997b32ba265 Jens Wiklander 2024-08-30 191 put_device(&rdev->dev); d1c997b32ba265 Jens Wiklander 2024-08-30 192 return ERR_PTR(ret); d1c997b32ba265 Jens Wiklander 2024-08-30 193 } 1e9046e3a15460 Jens Wiklander 2024-08-14 194 1e9046e3a15460 Jens Wiklander 2024-08-14 195 dev_dbg(&rdev->dev, "registered device\n"); 1e9046e3a15460 Jens Wiklander 2024-08-14 196 1e9046e3a15460 Jens Wiklander 2024-08-14 197 return rdev; 1e9046e3a15460 Jens Wiklander 2024-08-14 198 1e9046e3a15460 Jens Wiklander 2024-08-14 @199 err_id_remove: 1e9046e3a15460 Jens Wiklander 2024-08-14 200 mutex_lock(&rpmb_mutex); 1e9046e3a15460 Jens Wiklander 2024-08-14 201 ida_simple_remove(&rpmb_ida, rdev->id); 1e9046e3a15460 Jens Wiklander 2024-08-14 202 mutex_unlock(&rpmb_mutex); 1e9046e3a15460 Jens Wiklander 2024-08-14 203 err_free_dev_id: 1e9046e3a15460 Jens Wiklander 2024-08-14 204 kfree(rdev->descr.dev_id); 1e9046e3a15460 Jens Wiklander 2024-08-14 205 err_free_rdev: 1e9046e3a15460 Jens Wiklander 2024-08-14 206 kfree(rdev); 1e9046e3a15460 Jens Wiklander 2024-08-14 207 return ERR_PTR(ret); 1e9046e3a15460 Jens Wiklander 2024-08-14 208 } 1e9046e3a15460 Jens Wiklander 2024-08-14 209 EXPORT_SYMBOL_GPL(rpmb_dev_register); 1e9046e3a15460 Jens Wiklander 2024-08-14 210 -- 0-DAY CI Kernel Test Service https://github.com/intel/lkp-tests/wiki