-
Notifications
You must be signed in to change notification settings - Fork 4
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
SpGEMM bad_alloc Error #31
base: main
Are you sure you want to change the base?
Conversation
// Allocate memory for the host matrix data | ||
h_C.m_data.resize(d_C.rows * d_C.cols); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can see it here, what is the size of d_C.rows * d_C.cols
? I think this is
h_C.m_data.resize(d_C.rows * d_C.cols); | ||
|
||
// Copy matrix data from device to host | ||
cudaMemcpy(h_C.m_data.data(), d_C.m_data_ptr, sizeof(type_t) * d_C.rows * d_C.cols, cudaMemcpyDeviceToHost); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same issue as above.
examples/spgemm/thread_mapped.cu
Outdated
csc_t<index_t, offset_t, type_t> csc(mtx.load(parameters.filename)); | ||
|
||
// Output matrix. | ||
matrix_t<type_t> C(csr.rows, csc.cols); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same issue as above.
… and a column of B per thread with the stride size of TILE_SIZE
@Floruaaa666 I see that this is a work in progress, let me know when you'd like me to review for merge after its been cleaned up. |
My SpGEMM approach uses the inner product (CSR*CSC = dense format) and I used a few input matrices from SS to test. For some of the input matrices, the program had an error saying:
The matrices with this error: