Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

OpenCL AES: Don't bother using local memory on CPU devices #5615

Merged

Conversation

magnumripper
Copy link
Member

Not very important (makes little/no difference) but I like having the different code paths "in use" to catch future problems.

@magnumripper magnumripper merged commit 340f436 into openwall:bleeding-jumbo Dec 17, 2024
35 of 36 checks passed
@magnumripper magnumripper deleted the opencl-aes-cpu-nolocal branch December 17, 2024 22:05
claudioandre-br added a commit to openwall/john-packages that referenced this pull request Dec 18, 2024
Add raw-SHA512-free-opencl to the list of problematic formats.

Side effect of openwall/john#5613 and openwall/john#5615.

Document all formats that fail and therefore need to be disabled
during testing.

Signed-off-by: Claudio André <[email protected]>
claudioandre-br added a commit to openwall/john-packages that referenced this pull request Dec 25, 2024
Add raw-SHA512-free-opencl to the list of problematic formats.

Side effect of openwall/john#5613 and openwall/john#5615.

Document all formats that fail and therefore need to be disabled
during testing.

Signed-off-by: Claudio André <[email protected]>
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