Issue with --local command line parameter. about sycl-bench

[DitiD]

Hi, we would like to know if there are any solutions for the above mentioned issue. Thanks.

[illuhad]

I cannot reproduce the issue with hipSYCL. Your output indicates that somehow a block size of 0 enters the benchmark. This value is derived from the local size. I had a quick look at the code paths, and I don't understand how this could happen - it does not for me.
There's an assert that checks that the block size is non-zero. Can you check what happens when compiling with debug assertions enabled?

[DitiD]

Hi, we are not working with hipSYCL. The issue that we are facing is occurring during runtime. The test case is failing to execute when we are not passing the local parameter (as in, when it is taking the value of local parameter as 256 by default).

Command being used to execute - ./blocked_transform --device=gpu

However, it is working fine when we are explicitly defining the local parameter to 256 during runtime.

Command being used to execute - ./blocked_transform --device=gpu --local=256

We are not sure as to why this issue is occurring.

Thanks.

[illuhad]

Hi, we are not working with hipSYCL. The issue that we are facing is occurring during runtime.

I'm aware of this. But I don't have an installation of the DPC++ SYCL implementation with CUDA backend here. I'm just saying I cannot reproduce this with my setup. And I don't understand why DPC++ or hipSYCL would behave differently here anyway. The error does not seem to be related to SYCL specific functionality.

The test case is failing to execute when we are not passing the local parameter (as in, when it is taking the value of local parameter as 256 by default).
Command being used to execute - ./blocked_transform --device=gpu
However, it is working fine when we are explicitly defining the local parameter to 256 during runtime.
Command being used to execute - ./blocked_transform --device=gpu --local=256

I understood this. As I've said I cannot reproduce here. Command line option handling is the same for DPC++ and hipSYCL. For further investigation into the issue, I asked you the following:

There's an assert that checks that the block size is non-zero. Can you check what happens when compiling with debug assertions enabled?

i.e. make sure that the NDEBUG macro is not set when building.

[DitiD]

Hi, as suggested, I've added the following in the blocked_transform.cpp code and I've rebuilt it again.

#include<assert.h>
#define NDEBUG

It seems that by default, the value of local size is being taken as 1024 (please see attached screenshot below).

However, when I am defining '--local' to be either 256 (default value) or 1024 explicitly, it is working fine.

Command being used:

./blocked_transform --device=gpu --local=256
./blocked_transform --device=gpu --local=1024

Could this be a bug in the code?
Thanks.

[DitiD]

Hi, is there any update regarding this issue? Thanks.