已合并
fix(hccl): rename HCCL_COMM_HCCL_QOS_CONFIG_NOT_SET to HCCL_COMM_QOS_CONFIG_NOT_SET #33141
wuyouqi1创建于 4月3日
fix(hccl): rename HCCL_COMM_HCCL_QOS_CONFIG_NOT_SET to HCCL_COMM_QOS_CONFIG_NOT_SET #33141
已合并
Pull Request已成功合入, 合并人@ascend-robot
（感谢 wuyouqi1 的贡献）
ascend-robot
4月3日 评论：
4月3日 评论：
ascend-robot
4月3日 评论：
4月3日 评论：
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/pytorch | ✅ 李伟, ffmh (2/2) | ✅ 李伟 (1/1) |
💡 Tip:
- Committer can comment
/approveor/lgtm- Commenting
/approveimplies both code review (lgtm) and intent to merge (approve)
CLA Signature Pass
weixin_47897441, thanks for your pull request. All authors of the commits have signed the CLA. 👍
4月3日 添加了label：ascend-cla/yes
third_party/hccl/inc/hccl/hccl_types.h
@@ -127,7 +127,7 @@ const uint32_t HCCL_COMM_DEFAULT_OP_EXPANSION_MODE = 0;
127127const uint32_t HCCL_COMM_TRAFFIC_CLASS_CONFIG_NOT_SET = 0xffffffff;
128128const uint32_t HCCL_COMM_SERVICE_LEVEL_CONFIG_NOT_SET = 0xffffffff;
129129const int32_t HCCL_COMM_EXECTIMEOUT_CONFIG_NOT_SET = 0xffffffff;
130-const uint32_t HCCL_COMM_HCCL_QOS_CONFIG_NOT_SET = 0xffffffff;
命名规范/拼写错误: 第130行将常量名从'HCCL_COMM_HCCL_QOS_CONFIG_NOT_SET'修改为'HCCL_COMM_QOS_CONFIG_NOT_SET'，移除了冗余的'HCCL_'前缀。这个修改是正确的，因为它与上下文中其他类似常量的命名模式保持一致（例如HCCL_COMM_TRAFFIC_CLASS_CONFIG_NOT_SET、HCCL_COMM_SERVICE_LEVEL_CONFIG_NOT_SET）。然而，需要检查代码库中所有引用旧常量名的地方是否都已相应更新，以避免编译错误或逻辑错误。
问题类型: 命名规范/拼写错误
文件路径:
third_party/hccl/inc/hccl/hccl_types.h行号: 130
问题代码:
const uint32_t HCCL_COMM_QOS_CONFIG_NOT_SET = 0xffffffff;
修改建议:
1. 确认本次修改是重命名而非新增常量。2. 在代码库中全局搜索'HCCL_COMM_HCCL_QOS_CONFIG_NOT_SET'，确保所有引用点都已更新为新的常量名'HCCL_COMM_QOS_CONFIG_NOT_SET'。3. 检查是否有头文件包含顺序或条件编译可能导致某些编译单元看到不同的定义。
---
此评论由代码审查工具自动生成
此评论由代码审查工具自动生成
wuyouqi1
4月8日 评论：
4月8日 评论：
所有引用点都已更新为新的常量名'HCCL_COMM_QOS_CONFIG_NOT_SET'
ascend-robot
4月3日 评论：
4月3日 评论：
问题/功能描述
修复HCCL通信配置初始化函数中一个常量名称拼写错误，以及头文件中对应常量的命名不一致问题。该错误导致hcclQos字段被错误地初始化为一个不存在的常量值，可能影响通信服务质量（QoS）配置的默认行为。同时，头文件中该常量的命名存在冗余前缀，与同类常量命名风格不一致，降低了代码的可读性和维护性。
修改方案描述
将HcclCommConfigInit函数中config->hcclQos字段的初始化值从错误的HCCL_COMM_HCCL_QOS_CONFIG_NOT_SET更正为正确的HCCL_COMM_QOS_CONFIG_NOT_SET。同时，在头文件hccl_types.h中将该常量定义重命名为HCCL_COMM_QOS_CONFIG_NOT_SET，使其与结构体成员名及其他类似常量命名保持一致。此修改确保了通信配置结构体初始化的正确性，并提升了代码规范性。
wuyouqi1
4月3日 评论：
4月3日 评论：
compile
4月3日 添加了label：ci-pipeline-running
此处折叠了5条事件消息 查看更多
4月7日 添加了label：ci-pipeline-passed
ascend-robot
4月7日 评论：
4月7日 评论：
流水线 PR-pipeline_pytorch#14308 （重试第1次）已完成
| 阶段 | 任务名 | 状态 | 详情 |
|---|---|---|---|
| 编译构建 | Build_X86 | ✅ | >>> |
| Build_ARM | ✅ | >>> | |
| Build_LibTorch_x86 | ✅ | >>> | |
| Build_X86_torchair | 🛑 | >>> | |
| Build_ARM_torchair | 🛑 | >>> | |
| Build_ARM_inductor | 🛑 | >>> | |
| 恶意代码检查 | Antipoison | ✅ | >>> |
| 编码安全与规范检查 | CodeCheck | ✅ | >>> |
| check_error | ✅ | >>> | |
| 开源片段检查 | SCA | ✅ | >>> |
| 开发者测试 | UT_X86_Part_01 | 🛑 | >>> |
| UT_X86_Part_02 | 🛑 | >>> | |
| UT_ARM_A3_Part_01 | 🛑 | >>> | |
| UT_ARM_A3_Part_02 | 🛑 | >>> | |
| UT_DIST_X86_Part_01 | 🛑 | >>> | |
| UT_DIST_X86_Part_02 | 🛑 | >>> | |
| UT_DIST_X86_Part_03 | 🛑 | >>> | |
| UT_DIST_X86_Part_04 | 🛑 | >>> | |
| UT_inductor_Part_01 | 🛑 | >>> | |
| UT_inductor_Part_02 | 🛑 | >>> | |
| UT_inductor_Part_03 | 🛑 | >>> | |
| UT_inductor_Part_04 | 🛑 | >>> | |
| UT_ARM_A2_Part_01 | ✅ | >>> | |
| UT_ARM_A2_Part_02 | ✅ | >>> | |
| 流水线 | PR-pipeline_pytorch | ✅ | >>> |
4月7日 修改了pull request 的描述
ffmh
4月8日 评论：
4月8日 评论：
/lgtm
4月8日 解决了最后一个问题
4月8日 添加了label：approvedlgtm
ascend-robot
4月8日 评论：
4月8日 评论：
4月8日 合入了pull request
【合入来源】
【修改方案】
【资料变更】
不涉及
【接口变更】
不涉及
【功能验证】
【CheckList】