Skip to content

Initialize padded sort buffers before CUB sort#8

Open
SRO-SA wants to merge 1 commit into
masterfrom
fix/issue-7-uninitialized-device-memory-cub-sort
Open

Initialize padded sort buffers before CUB sort#8
SRO-SA wants to merge 1 commit into
masterfrom
fix/issue-7-uninitialized-device-memory-cub-sort

Conversation

@SRO-SA

@SRO-SA SRO-SA commented Jun 24, 2026

Copy link
Copy Markdown
Collaborator

Fixes #7

Summary:

  • Zero-initializes padded open/close device buffers before they are used by the parser sort path.
  • Zero-initializes the temporary depth buffer before map/scan/sort operations.
  • Applies the same fix to both standard JSON and JSON Lines parser paths.
  • Keeps the sort range limited to the real populated open/close count.

Testing:

  • I could not run CUDA or compute-sanitizer locally because my laptop does not have CUDA access.
  • I reviewed the affected code paths manually.
  • Please rerun the initcheck reproducer from the issue to confirm the uninitialized device-memory reads are resolved.

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.

Uninitialized device-memory read passed into CUB radix sort

1 participant