-
Notifications
You must be signed in to change notification settings - Fork 146
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
Assertion `finder != proc_spaces.end()' failed. #305
Comments
@jiazhihao remind me where this app code is? Also, can you generate a backtrace? |
Attached please find the backtrace. The code are available at sapling:~/home/zhihao/legion/apps/cnn. You can compile the code in that sub folder. [0] Thread 16 (Thread 0x7f91facd8700 (LWP 30156)): |
This is an application bug. The mapper is returning an invalid target processor ID for one of the slices from a call to 'slice_task': proc = { I will fix the runtime to actually check for bad processor IDs, but the bug is going to be in your custom mapper. |
Just looking at the code in your mapper it looks like you need to replace this: gpus[idx] with this: gpus[idx % gpus.size()] |
Speaking of which, @streichler how hard would it be to add an interface to all the different kinds of realm types that support the 'exists' method to also support a 'valid' method? |
I see. That's why I cannot reproduce the failure if the application is given enough GPUs. |
Closing this since it is an application bug. |
I encountered the following assertion failure in the up-to-date master branch. The failure can be deterministically reproduced on n0000 under the following steps:
zhihao@n0000: cd /home/zhihao/legion/apps/cnn/
zhihao@n0000:/legion/apps/cnn$ ./cnn -ll:gpu 2 -ll:fsize 4000
cnn: /home/zhihao/legion/runtime//legion/runtime.cc:12348: Legion::AddressSpaceID Legion::Internal::Runtime::find_address_space(Legion::Processor) const: Assertion `finder != proc_spaces.end()' failed.
The text was updated successfully, but these errors were encountered: