[ESIMD]Fix test crashes after driver update #15732
Conversation
| sycl::image<2> Img(srcY, image_channel_order::rgba, | ||
| image_channel_type::unsigned_int32, | ||
| range<2>{width / sizeof(uint4), height}); | ||
| image_channel_type::unsigned_int8, |
There was a problem hiding this comment.
do you mind explaining the fix? thanks
There was a problem hiding this comment.
I am not an expert in image and image accessors so I have only rough ideas what went wrong: We created a buffer of uint8 with size of IMAGE_WIDTHxIMAGE_HEIGHTxsizeof(int) i.e. you can think of image of IMAGE_WIDTHxIMAGE_HEIGHT where every pixel is an int i.e. 4 bytes and here we creating an image that will use this buffer, we say that we have for channels (rgba parameter) and every channel is int32 which is incorrect since buffer was allocated with uint8 for every channel. Later we reading it as vector of uint8 and it looks like some internal mapping inside an image accessor leads it to read memory beyond buffer memory causing crash on newer gpu driver
BTW, we have bunch of other tests using images that are either crashing or failing that I will try to address it in follow up PR if I manage to fix it on our side (I am not sure whom to blame yet)
There was a problem hiding this comment.
ah ok i see thanks, yeah there's definitely a type mismatch with srcY
2 participants
No description provided.