Prhub

#34767 [CPU] Fix native KV hash compilation in Xeon image

原始 PR 作者 sagearc 合并时间 2026-09-10 12:51 文件变更 1 提交数 4 评论 0 代码增减 +1 / -0

执行摘要

为 Xeon Docker 镜像添加 libssl-dev 依赖,修复原生 KV 哈希扩展编译失败问题。

PR body 明确指出:“The xeon image fails when sglang first compiles its native KV hash extension because openssl/sha.h is missing.” 这是一个明确的构建失败问题,需要添加缺失的依赖来修复。

该 PR 是一个非常简单的依赖修复,价值明确,风险极低。对于关注 CPU 部署和 Docker 镜像构建的工程师来说,值得快速了解其内容,但无需深入精读其技术实现。

讨论亮点

无评论讨论。PR 由 mingfeima 直接批准合并,表明这是一个简单、明确且无争议的修复。

实现拆解

  1. 变更入口:修改 Docker 构建文件 docker/xeon.Dockerfile。
  2. 核心逻辑:在 apt-get install 命令的软件包列表中,新增 libssl-dev 一行。该包提供了 OpenSSL 开发头文件,其中包含了编译 KV 哈希扩展所需的 openssl/sha.h。
  3. 配套改动:无其他代码、测试或配置的配套改动。变更仅涉及 Docker 镜像构建层面。
文件 模块 状态 重要度
docker/xeon.Dockerfile Docker 构建 modified 2.24

分析完成后,这里会展示 LLM 生成的相对完整源码片段和详细注释。

评论区精华

没有提炼出高价值讨论线程

当前评论区没有形成足够清晰的争议点或结论,后续有更多讨论时会体现在这里。

风险与影响

风险极低。添加一个标准的开发依赖包,几乎不会引入回归、性能或安全问题。唯一潜在风险是 libssl-dev 包版本与系统其他组件的兼容性,但在 Docker 构建的隔离环境中,这通常由包管理器处理。

影响范围明确且有限。修复仅影响使用 xeon.Dockerfile 构建的 Docker 镜像,确保该镜像内的 SGLang 原生 KV 哈希扩展能够成功编译。对于依赖该镜像进行 CPU 推理的用户,这是一个必要的修复,否则他们的部署将失败。

Docker 镜像依赖变更

关联 Issue

未识别关联 Issue

当前没有检测到明确关联的 Issue 链接,后续同步到相关引用后会出现在这里。

完整报告

参与讨论