关于bank conflict章节的描述

[Yi-sir]

CUDATutorial/docs/09_optimize_reduce/02_bank_conflict/README.md

Line 49 in 969a8e8

0 号 warp 的 0 号线程访问 0 号 bank 的 0 号地址和 128 号地址，1 号 warp 的 0 号线程访问 1 号 bank 的 32 号地址和 128 号地址，2 号 warp 的 0 号线程访问 2 号 bank 的 64 号地址和 160 号地址，以此类推。

请问这里的描述是对的吗？看图理解感觉是：

0号warp的0号线程访问bank0的0和128地址；
0号warp的1号线程访问bank1的0和128地址；
1号warp的0号线程访问bank0的32和160地址；
2号warp的0号线程访问bank0的64和192地址。

请大佬赐教，不胜感激。

[Yi-sir]

CUDATutorial/docs/12_convolution/01_naive_conv/naive_conv.cu

Line 203 in 969a8e8

std::cout << "GPU: " << out_cpu[i] << " CPU: " << out[i] << std::endl;

顺便捉虫，这行变量大概是写反了

[AndSonder]

是的，这里当时这里写的时候应该是看错行了，你的理解是对的，后面我提 pr 改一下

[AndSonder]

CUDATutorial/docs/12_convolution/01_naive_conv/naive_conv.cu

Line 203 in 969a8e8

std::cout << "GPU: " << out_cpu[i] << " CPU: " << out[i] << std::endl;
顺便捉虫，这行变量大概是写反了

哈哈，后面我一块给修了

[Yi-sir]

感谢作者。还有点小问题：刚看了naive_conv，里面分配线程数时写死了16*16容易引起误会（毕竟其它部分代码都是根据参数计算的）。也许可以改成这样：

const int blockDim_x =
      (out_h * out_w / k) > 1024/k ? 1024/k : (out_h * out_w / k);
const int blockDim_y = k;

[AndSonder]

感谢作者。还有点小问题：刚看了naive_conv，里面分配线程数时写死了16*16容易引起误会（毕竟其它部分代码都是根据参数计算的）。也许可以改成这样：

const int blockDim_x =
(out_h * out_w / k) > 1024/k ? 1024/k : (out_h * out_w / k);
const int blockDim_y = k;

good idea! 方便的话也可以提一个 pr ～

[Yi-sir]

感谢作者。还有点小问题：刚看了naive_conv，里面分配线程数时写死了16*16容易引起误会（毕竟其它部分代码都是根据参数计算的）。也许可以改成这样：
const int blockDim_x =
(out_h * out_w / k) > 1024/k ? 1024/k : (out_h * out_w / k);
const int blockDim_y = k;

good idea! 方便的话也可以提一个 pr ～

好的，已提：#62