[libc-commits] [PATCH] D148191: [libc][wip] Draft of concurrent GPU RPC

[Jon Chesterfield via Phabricator via libc-commits]

JonChesterfield added a comment.

Two dubious compile time assumptions in this draft.

One is that wavefront==64 is not a thing. Changing that on the GPU side is calling a macro, the tedious thing is Loader.cpp will need to query the hardware to see what the expected wavesize is, allocate space accordingly, and deal with the server type varying based on it in some reasonable fashion.

Second is that mutual exclusion within a process involves a locks bitmap. This shouldn't be in shared/fine grain/pinned memory as the other process never accesses it. That's presently implemented with static globals. Changing the number of slots to a runtime value (perhaps based on the number of compute units) involves allocating GPU memory in the loader and passing that along with with fine grain memory, which is a bigger API change than I wanted to make here.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D148191/new/

https://reviews.llvm.org/D148191