已合并

fix mem_pool stuck #898

纪涛创建于 4月15日从taojiovvo/MindIE-LLM_6913： dev 合入到 Ascend/MindIE-LLM： dev

fix mem_pool stuck #898

已合并

纪涛创建于 4月15日

从taojiovvo/MindIE-LLM_6913 dev合入到Ascend/MindIE-LLM dev

纪涛成员

4月15日

合入背景

请描述为什么要做这个PR内的改动。
如涉及，请关联前序PR或同特性/需求下的其他PR。
如果是修复之前PR引入的问题，请关联引入问题的PR。
注意：Fixes #ISSUE ID会自动关闭issue，如问题部分解决请不要使用Fixes，可以用Fix part of #ISSUE ID替代.

修改内容

fix mem_pool stuck
A2、A3混部使用mookcake等池化时，会出现跑任务卡住，定位原因为c++与python交互gil锁问题

资料变更

请确认是否涉及资料变更。如涉及，需要在PR中体现，并简要说明修改内容。如不涉及，需填写“不涉及”。

接口变更

请确认是否涉及跨代码仓或者客户面可见的接口变更。如涉及，需要详细说明接口以及对应的变更内容，同时需要在资料中体现。如不涉及，需填写“不涉及”。

测试结果

请说明测试场景，测试方法以及测试结果。
测试用例设计时需考虑硬件、部署方式、功能、性能、精度、显存等维度。

CheckList

PR提交人对以下CheckList自检项进行全量自检，自检通过或不涉及，均修改 [ ] 为 [x]。

代码注释完备
正确记录错误日志
进行了返回值校验 (禁止使用void屏蔽安全函数、自研函数返回值；考虑接口的异常场景；调用底层组件接口时，需要进行返回值校验)
进行了空指针校验
若存在资源申请，使用后资源被正确的释放了
若涉及多线程场景，考虑了并发场景，不存在死锁问题
按照代码仓中提供的格式模板，使用clang-format工具格式化代码
符合Ascend社区的编码规范。C++ 语言编程指导 | C++ 语言安全编程指导

Pull Request已成功合入, 合并人@ascend-robot

（感谢纪涛的贡献）

ascend-robot成员

4月15日评论：

Thanks for your pull-request.
The full list of commands accepted by me can be found at here。
You can get sig-info at here

PR Approval Progress

✅ Congratulations! All modules have met the lgtm and approve requirements.

Module Approval Details

module	lgtm status	approve status
repo-Ascend/MindIE-LLM	✅ 吴铭泾, hw-zhoutianyang (2/2)	✅ 吴铭泾 (1/1)

💡 Tip:

Committer can comment /approve or /lgtm

Commenting /approve implies both code review (lgtm) and intent to merge (approve)

CLA Signature Pass

taojiovvo, thanks for your pull request. All authors of the commits have signed the CLA. 👍

ascend-robot成员

4月15日添加了label：ascend-cla/yes

ascend-robot成员

4月15日添加了label：needs-issue

纪涛成员

4月15日评论：

compile

ascend-robot成员

4月15日评论：

Linking Issue Notice

@taojiovvo , the pull request must be linked to at least one issue.
If an issue has already been linked, but the needs-issue label remains, you can remove the label by commenting /check-issue .

ascend-robot成员

4月15日评论：

问题/功能描述

修复了Python/C++混合编程环境中，全局解释器锁（GIL）管理不当导致的潜在运行时错误或死锁问题。通过移除冗余的GIL获取调用，确保了多线程环境下Python对象访问的正确性和安全性。

修改方案描述

在MemPool::LookUp方法中，移除了使用pybind11的py::gil_scoped_acquire获取GIL的冗余代码行。该函数内部已通过PyGILState_Ensure()和PyGILState_Release()手动管理GIL状态，重复获取可能导致冲突。修改后，GIL管理逻辑更加清晰正确，消除了潜在风险。

ascend-robot成员

4月15日评论：

✅ 流水线任务触发成功(流水线编号: 4928)，正在执行，请稍候。任务链接点此跳转 ...

ascend-robot成员

4月15日添加了label：ci-pipeline-running

ascend-robot成员

4月15日评论：

Ascend docs pipeline is running...

ascend-robot成员

4月15日添加了label：docs-ci-pipeline-running

ascend-robot成员

4月15日评论：

Ascend docs pipeline is running...

纪涛成员

4月15日关联了issue：[Bug]: 池化服务 gpqa数据集测试服务挂死

ascend-robot成员

4月15日删除了label：needs-issue

ascend-robot成员

4月15日评论：

✅ 跳过 docs ci 检查，没有需要检查的文档文件

ascend-robot成员

4月15日删除了label：docs-ci-pipeline-running

ascend-robot成员

4月15日添加了label：docs-ci-pipeline-success

ascend-robot成员

4月15日评论：

✅ 跳过 docs ci 检查，没有需要检查的文档文件

ascend-robot成员

4月15日删除了label：ci-pipeline-running

ascend-robot成员

4月15日添加了label：ci-pipeline-passed

ascend-robot成员

4月15日评论：

流水线 PR-pipeline_MindIE-LLM_gitcode#4928 已完成

阶段	任务名	状态	详情
编译构建	Build_ops_x86_A3	✅	>>>
	Build_ops_arm_A3	✅	>>>
	Build_ops_x86_A2	✅	>>>
	Build_ops_arm_A2	✅	>>>
	Build_linux_x86_abi1_atb	✅	>>>
	Build_linux_x86_abi1_llm	✅	>>>
	Build_linux_arm_abi1_atb	✅	>>>
	Build_linux_arm_abi1_llm	✅	>>>
	Build_linux_arm_abi0_atb	✅	>>>
	Build_linux_arm_abi0_llm	✅	>>>
恶意代码检查	Antipoison	✅	>>>
编码安全与规范检查	pre-commit	✅	>>>
开源片段检查	SCA	✅	>>>
开发者测试	UT_linux_arm_mindie-llm	✅	>>>
开发者测试	UT_linux_arm_atb-models	✅	>>>
流水线	PR-pipeline_MindIE-LLM_gitcode	✅	>>>

纪涛成员

4月15日评论：

/approve

ascend-robot成员

4月15日评论：

Review Code Warning

@taojiovvo , You can't review code by yourself, please contact the committers. 😲

hw-zhoutianyang

4月15日评论：

/lgtm

吴铭泾成员

4月15日评论：

/approve

ascend-robot成员

4月15日添加了label：approvedlgtm

纪涛成员

4月15日评论：

/merge

ascend-robot成员

4月15日添加了label：keeper_approved

ascend-robot成员

4月15日评论：

Review Guide

This pull-request passes review.
Committers who wrote a comment of /approve are: 吴铭泾.
Reviewers who wrote a comment of /lgtm are: 吴铭泾, hw-zhoutianyang.

ascend-robot成员

4月15日合入了pull request