linux-kernel - Re: [PATCH v6 0/4] uacce: driver fixes for memory leaks and state management

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <828e0dcd-ae17-448b-ba33-97603031fc60@huawei.com>
Date: Tue, 6 Jan 2026 10:38:46 +0800
From: huangchenghai <huangchenghai2@...wei.com>
To: <gregkh@...uxfoundation.org>, <zhangfei.gao@...aro.org>,
	<wangzhou1@...ilicon.com>
CC: <linux-kernel@...r.kernel.org>, <linux-crypto@...r.kernel.org>,
	<fanghao11@...wei.com>, <shenyang39@...wei.com>, <liulongfang@...wei.com>,
	<qianweili@...wei.com>, <linwenkai6@...ilicon.com>
Subject: Re: [PATCH v6 0/4] uacce: driver fixes for memory leaks and state
 management

Kindly ping for this fix.


Cheers,
Chenghai

在 2025/12/2 14:12, Chenghai Huang 写道:
> This patch series addresses several issues in the uacce:
> 1.Fix cdev handling in the cleanup path.
> 2.Fix sysfs file creation conditions.
> 3.Add error reporting for unsupported mremap operations.
> 4.Ensuring safe queue release with proper state management.
>
> ---
> Changes in v6:
> - In patch 1, if cdev_device_add() fails, it will automatically free the cdev, however,
>    we need to set uacce->cdev to NULL to prevent cdev_device_del() from being called.
> - Link to v5: https://lore.kernel.org/all/20251111093536.3729-1-huangchenghai2@huawei.com/
>
> Changes in v5:
> - There is no memory leak issue when cdev_device_add fails, but it is necessary
>    to check a flag to avoid calling cdev_device_del during abnormal exit.
> - Link to v4: https://lore.kernel.org/all/20251022021149.1771168-1-huangchenghai2@huawei.com/
>
> Changes in v4:
> - Revert the interception of sysfs creation for isolate_strategy.
> - Link to v3: https://lore.kernel.org/all/20251021135003.786588-1-huangchenghai2@huawei.com/
>
> Changes in v3:
> - Move the checks for the 'isolate_strategy_show' and
>    'isolate_strategy_store' functions to their respective call sites.
> - Use kobject_put to release the cdev memory instead of modifying
>    cdev to be a static structure member.
> - Link to v2: https://lore.kernel.org/all/20250916144811.1799687-1-huangchenghai2@huawei.com/
>
> Changes in v2:
> - Use cdev_init to allocate cdev memory to ensure that memory leaks
>    are avoided.
> - Supplement the reason for intercepting the remapping operation.
> - Add "cc: stable@...r.kernel.org" to paths with fixed.
> - Link to v1: https://lore.kernel.org/all/20250822103904.3776304-1-huangchenghai2@huawei.com/
>
> Chenghai Huang (2):
>    uacce: fix isolate sysfs check condition
>    uacce: ensure safe queue release with state management
>
> Wenkai Lin (1):
>    uacce: fix cdev handling in the cleanup path
>
> Yang Shen (1):
>    uacce: implement mremap in uacce_vm_ops to return -EPERM
>
>   drivers/misc/uacce/uacce.c | 48 +++++++++++++++++++++++++++++++-------
>   1 file changed, 40 insertions(+), 8 deletions(-)
>