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

increase safety memory for autosuggestion #72

Merged
merged 1 commit into from
Oct 27, 2017

Conversation

psychocrypt
Copy link
Collaborator

  • increase safty memory from 64 to 128 MiB
  • NVIDIA: increase lmem reserve per thread to 1kiB

This should avoid that the autosuggestion create a config which use to much memory.

- increase safty memory from 64 to 128 MiB
- NVIDIA: increase lmem reserve per thread to 1kiB
@fireice-uk
Copy link
Owner

@psychocrypt Is this going to fix crashes experienced by @JerichoJones in #72

@psychocrypt
Copy link
Collaborator Author

@fireice-uk The link to @JerichoJones issue is wrong. But we had a lot of issues in the past where we hit the memory wall during the runtime.
The reason is in the most cases the dynamic usage of lmem (register outsourced to the gpu main memory). Be more conservative with the memory should create more stable configurations.

Copy link
Owner

@fireice-uk fireice-uk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will merge wen CI passes

@fireice-uk fireice-uk merged commit a7fb89b into fireice-uk:dev Oct 27, 2017
@psychocrypt psychocrypt deleted the topic-increaseSavetyMemory branch October 30, 2017 17:26
gnagel pushed a commit to gnagel/xmr-stak that referenced this pull request Mar 23, 2019
…yMemory

increase safety memory for autosuggestion
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants