deepseek-ai / deepseek-ai/DeepEP

[Question] Why not check the CQE when ibgda_poll_cq

Open
#558 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
Cuda
Stars
10.1k
Forks
1.4k
Avg merge
4d 1h
Merged PRs (30d)
2

Description

Hello,

We noticed that there is no CQE check in the DeepEP implemented ibgda_poll_cq, while nvshmem has the check as the following. Could some error be ignored, introducing some data corruption or hung?

if (unlikely(opcode == MLX5_CQE_REQ_ERR)) {
ibgda_mlx5_err_cqe_t *cqe_err = (ibgda_mlx5_err_cqe_t *)cqe64;
*error = cqe_err->syndrome;
status = -1;
}

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.