From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-alma10-1.taild15c8.ts.net [100.103.45.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 9F49B3EBF37; Thu, 18 Jun 2026 15:15:29 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=100.103.45.18 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781795730; cv=none; b=LQdPaIloS1cDiCVqC3fbisPQqjJfxv2bvZT9yF4+1/pnOrV0XhgSLUqkvANnv4j++2uVGWLx4Im6088FWmTBudPPVkp4GrE3HJwijY21YfVg+9MVvAAMi3wkQbaP9qHpuv2tihxvsoFpbVcG5b75OGkDpPii1q0Wu2OC+NYFeh8= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781795730; c=relaxed/simple; bh=RrzjsI1WUKmi96R0Jjhfss4SKJGUgjQ7X7/YuTliBO4=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=aY2UfbalGjb2ttDt2knlXd7xDFU1SoouLmqBAs3dk0PTXjQOtRgoyVWU3BPDfY98SZSemJaM8ZFzfBDo2AwcNR7z7nJKIqQjjKWUmp/OaiNjMTkXeAydF7B468KcQYMdYjqT9kIUpI8vnQEZN3ft2T3iQEshK7xJ7uASU/+Iupo= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=dI+RUT9n; arc=none smtp.client-ip=100.103.45.18 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="dI+RUT9n" Received: by smtp.kernel.org (Postfix) with ESMTPSA id A8F441F00A3A; Thu, 18 Jun 2026 15:15:28 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1781795729; bh=L6MPeEDmuSnl5VMNAu0xLTOn+1aNO6rM35POBLTCeUQ=; h=From:To:Cc:Subject:Date:In-Reply-To:References; b=dI+RUT9n/kBduhaMUwfp6OoNLIRxDVx5T8EF7kM1soOL/rbfsd6Zn28rFN7/ygolT c1qGMswsswVH5tOv8ABOK9qfTXdByovJvsDl/2xqvfF22lv/d4It+54XKOi6ttF+WL FZUpmghRC+N+cASLC6W94VloT8KsV6XrTEpLrs8atEdCm4HDfr3pllObp8QvDag2Vx Jp8bvBySMQgw2MYmm3YL//7gNLNgC0RN7X0RBOPsIx8Qtbs47M2SjSxrqmRKNugWxA IeU+aFuRZEb5M10P/pHwDKCSY7xTQphMqHjEOmS05cRuZ8djouNmPCR/nuzjPSEwGL VqDf2ca6RX0qQ== From: SeongJae Park To: Cc: SeongJae Park , "# 5 . 18 . x" , Andrew Morton , Jiapeng Chong , damon@lists.linux.dev, linux-kernel@vger.kernel.org, linux-mm@kvack.org Subject: [RFC PATCH v1.2 01/11] mm/damon/sysfs: kobject_del() target (normal), context and kdamond dirs Date: Thu, 18 Jun 2026 08:15:05 -0700 Message-ID: <20260618151517.5366-2-sj@kernel.org> X-Mailer: git-send-email 2.47.3 In-Reply-To: <20260618151517.5366-1-sj@kernel.org> References: <20260618151517.5366-1-sj@kernel.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit On CONFIG_DEBUG_KOBJECT_RELEASE enabled kernel, lack of kobject_del() could cause directories creation failures due to the name conflicts. Fix those issues for normal creation paths of target, context and kdamond directories, and error paths of context and kdamond directories by adding kobject_del() calls. Note that this fix for target directories is not complete since it has a similar issue in the damon_sysfs_targets_add_dirs() error path. Because the normal path issue and the error path issue are introduced by different commits, this commit is fixing only the normal path issue. A commit for the error path will be added next. Fixes: c951cd3b8901 ("mm/damon: implement a minimal stub for sysfs-based DAMON interface") Cc: # 5.18.x Signed-off-by: SeongJae Park --- mm/damon/sysfs.c | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/mm/damon/sysfs.c b/mm/damon/sysfs.c index 2e95e3bac774d..dba1c67fc188f 100644 --- a/mm/damon/sysfs.c +++ b/mm/damon/sysfs.c @@ -333,6 +333,7 @@ static void damon_sysfs_targets_rm_dirs(struct damon_sysfs_targets *targets) for (i = 0; i < targets->nr; i++) { damon_sysfs_target_rm_dirs(targets_arr[i]); + kobject_del(&targets_arr[i]->kobj); kobject_put(&targets_arr[i]->kobj); } targets->nr = 0; @@ -1642,6 +1643,7 @@ static void damon_sysfs_contexts_rm_dirs(struct damon_sysfs_contexts *contexts) for (i = 0; i < contexts->nr; i++) { damon_sysfs_context_rm_dirs(contexts_arr[i]); + kobject_del(&contexts_arr[i]->kobj); kobject_put(&contexts_arr[i]->kobj); } contexts->nr = 0; @@ -1680,13 +1682,15 @@ static int damon_sysfs_contexts_add_dirs(struct damon_sysfs_contexts *contexts, err = damon_sysfs_context_add_dirs(context); if (err) - goto out; + goto del_out; contexts_arr[i] = context; contexts->nr++; } return 0; +del_out: + kobject_del(&context->kobj); out: damon_sysfs_contexts_rm_dirs(contexts); kobject_put(&context->kobj); @@ -2501,6 +2505,7 @@ static void damon_sysfs_kdamonds_rm_dirs(struct damon_sysfs_kdamonds *kdamonds) for (i = 0; i < kdamonds->nr; i++) { damon_sysfs_kdamond_rm_dirs(kdamonds_arr[i]); + kobject_del(&kdamonds_arr[i]->kobj); kobject_put(&kdamonds_arr[i]->kobj); } kdamonds->nr = 0; @@ -2555,13 +2560,15 @@ static int damon_sysfs_kdamonds_add_dirs(struct damon_sysfs_kdamonds *kdamonds, err = damon_sysfs_kdamond_add_dirs(kdamond); if (err) - goto out; + goto del_out; kdamonds_arr[i] = kdamond; kdamonds->nr++; } return 0; +del_out: + kobject_del(&kdamond->kobj); out: damon_sysfs_kdamonds_rm_dirs(kdamonds); kobject_put(&kdamond->kobj); -- 2.47.3