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.

Sign up for GitHub

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

Jump to bottom

Issues with the code of SUM Reduction #9

Open

xuxy09 opened this issue Sep 11, 2022 · 2 comments

xuxy09 commented Sep 11, 2022

cuda_programming/03_sum_reduction/diverged/sumReduction.cu

Line 75 in 8711be0

sumReduction<<<1, TB_SIZE>>> (d_v_r, d_v_r);

Shouldn't the second call be <<<1, GRID_SIZE>>> instead of <<<1, TB_SIZE>>>? I think GRID_SIZE is the number of partial sums.

101001000 commented Feb 3, 2023

I had the same thoughts

weiminruan commented Nov 19, 2024

I have the same issue, i think it should be <<<1, GRID_SIZE>>>, and for the later version <<<1, GRID_SIZE/2>>> for preprocessing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment