Skip to content

Fix multi-threaded application fail when memory pool is enabled#606

Merged
okuta merged 6 commits intocupy:masterfrom
kmaehashi:fix-mt-mempool
Nov 2, 2017
Merged

Fix multi-threaded application fail when memory pool is enabled#606
okuta merged 6 commits intocupy:masterfrom
kmaehashi:fix-mt-mempool

Conversation

@kmaehashi
Copy link
Copy Markdown
Member

This fixes #72.

I use memGetInfo() Runtime API, which seems to be most harmless, to explicitly bind the context to the current thread.
(I first tried getDevice(), but seems that it does not create a context)

@okuta okuta added cat:bug Bugs to-be-backported Pull-requests to be backported to stable branch labels Oct 21, 2017
@okuta okuta added this to the v4.0.0b1 milestone Oct 21, 2017
@okuta okuta self-assigned this Oct 21, 2017
@okuta
Copy link
Copy Markdown
Member

okuta commented Oct 31, 2017

LGTM!

@okuta okuta merged commit 61da6f5 into cupy:master Nov 2, 2017
okuta added a commit to okuta/cupy that referenced this pull request Nov 2, 2017
Fix multi-threaded application fail when memory pool is enabled
@kmaehashi kmaehashi deleted the fix-mt-mempool branch December 7, 2017 10:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cat:bug Bugs to-be-backported Pull-requests to be backported to stable branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Using CuPy with memory pool from multi-threaded application

2 participants