Skip to content

Fix CUDA loader format arguments#90

Open
fallintoplace wants to merge 1 commit into
NVIDIA:mainfrom
fallintoplace:fix/cuda-loader-format-args
Open

Fix CUDA loader format arguments#90
fallintoplace wants to merge 1 commit into
NVIDIA:mainfrom
fallintoplace:fix/cuda-loader-format-args

Conversation

@fallintoplace

Copy link
Copy Markdown

Summary

  • Pass the CUDA symbol name to the loader error message when cuGetProcAddress_v2 fails.
  • Pass the symbol name, not the CUDA result code, when a symbol resolves to a null function pointer.

Why

These error paths use %s in PyErr_Format, but the old calls supplied an integer result code in that slot. If the loader hit either path, error reporting could interpret that integer as a string pointer instead of producing the intended exception.

Validation

  • git diff --check
  • Full extension build not run locally: CUDA Toolkit / nvcc is not installed in this environment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant