From nobody Thu Oct 2 18:17:41 2025 Received: from m16.mail.163.com (m16.mail.163.com [220.197.31.4]) (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 5BB9D28DC4; Fri, 12 Sep 2025 16:25:58 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=220.197.31.4 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1757694365; cv=none; b=rz4IZpUN5k2ldejZPd7FaCVX9SxnN4OYKoMLzqTxnZKddhgqkkYlfvhBtsvR/hp9ySip4gSnlVTS9aWrZ3I1NTUKsVVGlW5zXegZLMBV9btF1r+a1qpycp1vqAcoW1YZ4MVZXo91OG5+Vqk1ZdFtHLdU1LlOD/BLWU3yBBgu9rk= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1757694365; c=relaxed/simple; bh=CLmOWvcLwNcfZGNz2hJQssd3FoLO9Ae1v3BxpzWPCcc=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=JaUo+XK4q26fDhXhVlm5P2C5+KJISUQ8eGQIVsVFeXYhcqgbp62yqJZkACLDAQy1YRINjxVzT/Q1ubqKsV5QvGV64SoI2QZz2G2DY8n+hDOg/nwChaC7PaVPDT6fND8Mgv0rEUdTAT1WP9iSJ+dpyL1xVFXknZ4H+nHuYs1HJ84= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=163.com; spf=pass smtp.mailfrom=163.com; dkim=pass (1024-bit key) header.d=163.com header.i=@163.com header.b=RxizZYP3; arc=none smtp.client-ip=220.197.31.4 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=163.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=163.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=163.com header.i=@163.com header.b="RxizZYP3" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=163.com; s=s110527; h=From:To:Subject:Date:Message-ID:MIME-Version; bh=6I 8DMxtKR+qlvVeFZpv4d8AtVgPm89XYSRwMvApOEvw=; b=RxizZYP3YvD+LKoo4u Fc0dmp+TS8qX9geX6RYkROkxbIQMcxSFc7KgjFccyDjbsDcBIezVqZmiC/OT7epc JdhGbReEzx0yunyD7sMG5LTMFuvWhfxmM8D9EErvgmHwAt54Uddsd1Pnq1DDw30a yrXBQB9o9mMZ4U4Bp5HG4QO+M= Received: from MS-CMFLBWVCLQRG.localdomain (unknown []) by gzga-smtp-mtada-g0-4 (Coremail) with SMTP id _____wD33ztuScRoSUwPAw--.12617S2; Sat, 13 Sep 2025 00:25:19 +0800 (CST) From: GuangFei Luo To: gregkh@linuxfoundation.org Cc: rafael@kernel.org, dan.carpenter@linaro.org, lenb@kernel.org, linux-acpi@vger.kernel.org, linux-kernel@vger.kernel.org, linux-pm@vger.kernel.org, lkp@intel.com, luogf2025@163.com, sre@kernel.org, stable@vger.kernel.org Subject: [PATCH v3] ACPI: battery: prevent sysfs_add_battery re-entry on rapid events Date: Sat, 13 Sep 2025 00:25:16 +0800 Message-ID: <20250912162516.692861-1-luogf2025@163.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <2025091242-waltz-gruffly-0936@gregkh> References: <2025091242-waltz-gruffly-0936@gregkh> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-CM-TRANSID: _____wD33ztuScRoSUwPAw--.12617S2 X-Coremail-Antispam: 1Uf129KBjvJXoWxtr4fJFyrWFW5JrW8ZrW8WFg_yoWxCw4rpa 1rCayUtrW8GF48JwsruF4jgryrWan0qr9rWr95Grn2ka9rur1DArWIqFyUXF47GrykC3yx ZF1kt3Z5tr18Ww7anT9S1TB71UUUUU7qnTZGkaVYY2UrUUUUjbIjqfuFe4nvWSU5nxnvy2 9KBjDUYxBIdaVFxhVjvjDU0xZFpf9x07U04iUUUUUU= X-CM-SenderInfo: poxrwwisqskqqrwthudrp/1tbizRPGmWjENYfNQwABsQ Content-Type: text/plain; charset="utf-8" Hi Greg, Thanks for your review and suggestion. I've updated sysfs_add_battery() to address your comment. The locking is now applied explicitly inside the function to prevent re-entry issues, while keeping the function self-contained for all call sites. Patch version: v3 Thanks, GuangFei v3: - Modified the earlier approach: since sysfs_add_battery() is invoked from multiple places, the most reliable way is to add the lock inside the function itself. - sysfs_remove_battery() had a similar race issue in the past, which was fixed by adding a lock as well. Reference: https://lore.kernel.org/all/9c921c22a7f33397a6774d7fa076db9b6a0fd669 .1312318300.git.len.brown@intel.com/ v2: - Fix missing mutex_unlock in acpi_battery_update() (Reported-by: kernel test robot) v1: When removing and reinserting the laptop battery, ACPI can trigger two notifications in quick succession: - ACPI_BATTERY_NOTIFY_STATUS (0x80) - ACPI_BATTERY_NOTIFY_INFO (0x81) Both notifications call acpi_battery_update(). Because the events happen very close in time, sysfs_add_battery() can be re-entered before battery->bat is set, causing a duplicate sysfs entry error. This patch ensures that sysfs_add_battery() is not re-entered when battery->bat is already non-NULL, preventing the duplicate sysfs creation and stabilizing battery hotplug handling. [ 476.117945] sysfs: cannot create duplicate filename '/devices/LNXSYSTM:0= 0/LNXSYBUS:00/PNP0C0A:00/power_supply/BAT1' [ 476.118896] CPU: 1 UID: 0 PID: 185 Comm: kworker/1:4 Kdump: loaded Not t= ainted 6.12.38+deb13-amd64 #1 Debian 6.12.38-1 [ 476.118903] Hardware name: Gateway NV44 /SJV40-MV = , BIOS V1.3121 04/08/2009 [ 476.118906] Workqueue: kacpi_notify acpi_os_execute_deferred [ 476.118917] Call Trace: [ 476.118922] [ 476.118929] dump_stack_lvl+0x5d/0x80 [ 476.118938] sysfs_warn_dup.cold+0x17/0x23 [ 476.118943] sysfs_create_dir_ns+0xce/0xe0 [ 476.118952] kobject_add_internal+0xba/0x250 [ 476.118959] kobject_add+0x96/0xc0 [ 476.118964] ? get_device_parent+0xde/0x1e0 [ 476.118970] device_add+0xe2/0x870 [ 476.118975] __power_supply_register.part.0+0x20f/0x3f0 [ 476.118981] ? wake_up_q+0x4e/0x90 [ 476.118990] sysfs_add_battery+0xa4/0x1d0 [battery] [ 476.118998] acpi_battery_update+0x19e/0x290 [battery] [ 476.119002] acpi_battery_notify+0x50/0x120 [battery] [ 476.119006] acpi_ev_notify_dispatch+0x49/0x70 [ 476.119012] acpi_os_execute_deferred+0x1a/0x30 [ 476.119015] process_one_work+0x177/0x330 [ 476.119022] worker_thread+0x251/0x390 [ 476.119026] ? __pfx_worker_thread+0x10/0x10 [ 476.119030] kthread+0xd2/0x100 [ 476.119033] ? __pfx_kthread+0x10/0x10 [ 476.119035] ret_from_fork+0x34/0x50 [ 476.119040] ? __pfx_kthread+0x10/0x10 [ 476.119042] ret_from_fork_asm+0x1a/0x30 [ 476.119049] [ 476.142552] kobject: kobject_add_internal failed for BAT1 with -EEXIST, = don't try to register things with the same name in the same directory. [ 476.415022] ata1.00: unexpected _GTF length (8) [ 476.428076] sd 0:0:0:0: [sda] Starting disk [ 476.835035] ata1.00: unexpected _GTF length (8) [ 476.839720] ata1.00: configured for UDMA/133 [ 491.328831] sysfs: cannot create duplicate filename '/devices/LNXSYSTM:0= 0/LNXSYBUS:00/PNP0C0A:00/power_supply/BAT1' [ 491.329720] CPU: 1 UID: 0 PID: 185 Comm: kworker/1:4 Kdump: loaded Not t= ainted 6.12.38+deb13-amd64 #1 Debian 6.12.38-1 [ 491.329727] Hardware name: Gateway NV44 /SJV40-MV = , BIOS V1.3121 04/08/2009 [ 491.329731] Workqueue: kacpi_notify acpi_os_execute_deferred [ 491.329741] Call Trace: [ 491.329745] [ 491.329751] dump_stack_lvl+0x5d/0x80 [ 491.329758] sysfs_warn_dup.cold+0x17/0x23 [ 491.329762] sysfs_create_dir_ns+0xce/0xe0 [ 491.329770] kobject_add_internal+0xba/0x250 [ 491.329775] kobject_add+0x96/0xc0 [ 491.329779] ? get_device_parent+0xde/0x1e0 [ 491.329784] device_add+0xe2/0x870 [ 491.329790] __power_supply_register.part.0+0x20f/0x3f0 [ 491.329797] sysfs_add_battery+0xa4/0x1d0 [battery] [ 491.329805] acpi_battery_update+0x19e/0x290 [battery] [ 491.329809] acpi_battery_notify+0x50/0x120 [battery] [ 491.329812] acpi_ev_notify_dispatch+0x49/0x70 [ 491.329817] acpi_os_execute_deferred+0x1a/0x30 [ 491.329820] process_one_work+0x177/0x330 [ 491.329826] worker_thread+0x251/0x390 [ 491.329830] ? __pfx_worker_thread+0x10/0x10 [ 491.329833] kthread+0xd2/0x100 [ 491.329836] ? __pfx_kthread+0x10/0x10 [ 491.329838] ret_from_fork+0x34/0x50 [ 491.329842] ? __pfx_kthread+0x10/0x10 [ 491.329844] ret_from_fork_asm+0x1a/0x30 [ 491.329850] [ 491.329855] kobject: kobject_add_internal failed for BAT1 with -EEXIST, = don't try to register things with the same name in the same directory. Reported-by: kernel test robot Closes: https://lore.kernel.org/r/202509101620.yI0HZ5gT-lkp@intel.com/ Reported-by: Dan Carpenter Closes: https://lore.kernel.org/r/202509101620.yI0HZ5gT-lkp@intel.com/ Fixes: 508df92d1f8d ("ACPI: battery: register power_supply subdevice only w= hen battery is present") Signed-off-by: GuangFei Luo Cc: stable@vger.kernel.org --- drivers/acpi/battery.c | 16 +++++++++++----- 1 file changed, 11 insertions(+), 5 deletions(-) diff --git a/drivers/acpi/battery.c b/drivers/acpi/battery.c index 6905b56bf3e4..f6d4a8b39a9c 100644 --- a/drivers/acpi/battery.c +++ b/drivers/acpi/battery.c @@ -850,6 +850,12 @@ static void __exit battery_hook_exit(void) =20 static int sysfs_add_battery(struct acpi_battery *battery) { + mutex_lock(&battery->sysfs_lock); + if (battery->bat) { + mutex_unlock(&battery->sysfs_lock); + return 0; + } + struct power_supply_config psy_cfg =3D { .drv_data =3D battery, .attr_grp =3D acpi_battery_groups, @@ -896,9 +902,11 @@ static int sysfs_add_battery(struct acpi_battery *batt= ery) int result =3D PTR_ERR(battery->bat); =20 battery->bat =3D NULL; + mutex_unlock(&battery->sysfs_lock); return result; } battery_hook_add_battery(battery); + mutex_unlock(&battery->sysfs_lock); return 0; } =20 @@ -1026,11 +1034,9 @@ static int acpi_battery_update(struct acpi_battery *= battery, bool resume) return result; acpi_battery_quirks(battery); =20 - if (!battery->bat) { - result =3D sysfs_add_battery(battery); - if (result) - return result; - } + result =3D sysfs_add_battery(battery); + if (result) + return result; =20 /* * Wakeup the system if battery is critical low --=20 2.43.0