Skip to content

hexagon: use DIRID 13 in libggml-htp.inf for modern InfVerif#22306

Open
mengshengwu wants to merge 1 commit intoggml-org:masterfrom
qualcomm:hexagon/fix-inf-dirid-13
Open

hexagon: use DIRID 13 in libggml-htp.inf for modern InfVerif#22306
mengshengwu wants to merge 1 commit intoggml-org:masterfrom
qualcomm:hexagon/fix-inf-dirid-13

Conversation

@mengshengwu
Copy link
Copy Markdown
Contributor

Overview

libggml-htp.inf is used by the WoS build to produce and sign libggml-htp.cat for the HTP skel .so files (see ggml/src/ggml-hexagon/CMakeLists.txt). When the signed package was submitted to the Microsoft signing team, InfVerif rejected it because:

  • DIRID 6 historically referred to a legacy/root directory and is no longer a valid DestinationDirs value under the current InfVerif rules (Windows 11 build >= 10.0.27871).
  • The recommended destination for file-copy-only / extension driver packages is DIRID 13, i.e. %DriverStore%\FileRepository\<inf>_<hash>\.

This is also consistent with how Qualcomm's own qcadsprpc driver is deployed on WoS — htp-drv.cpp already resolves libcdsprpc.dll out of \SystemRoot\System32\DriverStore\FileRepository\qcadsprpc....

Verification

  • With DIRID 13, InfVerif accepts the package and the MS signing pipeline succeeds (DIRID 6 was previously rejected).
  • The WoS build still produces and signs libggml-htp.cat without issue.
  • Already used the signed package (installed via the updated inf) to run inference on WoS end-to-end — FastRPC successfully resolves libggml-htp-vXX.so from the DriverStore location at runtime, so the move does not break the skel lookup path.

Requirements

@mengshengwu mengshengwu requested a review from a team as a code owner April 24, 2026 05:21
@github-actions github-actions Bot added ggml changes relating to the ggml tensor library for machine learning Hexagon labels Apr 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ggml changes relating to the ggml tensor library for machine learning Hexagon

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant