Alex Brown

Results 11 comments of Alex Brown

Looks like the rhel8 build has a new problem with the path. Also the cs9 build says py36 is missing.

I think it's failing cause normal current working directory is build instead of project root. This patch should fix it: [cipath.txt](https://github.com/ROCmSoftwarePlatform/Tensile/files/8807202/cipath.txt) .

A number of those test failures are also caused by current rocm build on CI, unrelated to changes in this PR. Send me a message and I can try and...

> Sorry last comment fell into a github commit so almost invisible: "Is this available on all OS and container controls? Why not get CI to tell you the limit,...

Yes this is related to #1631 . This logic file contains an error: https://github.com/ROCmSoftwarePlatform/rocBLAS/blob/develop/library/src/blas3/Tensile/Logic/asm_full/aldebaran/aldebaran_Cijk_Ailk_Bljk_DB_GB.yaml The first kernel in that file fails to generate. Rocblas build is not reporting any errors...

> Is there any document which explains how this works? It is difficult to understand the behavior from asm code. Uploading to the corresponding ticket or any way is OK....

> User should be able to know required workspace size from getHerursitic api. If the user given workspace size is not enough for real stream-K function, should we just return...

> Suggest adding a comment or implement this path to ensure stream-K works with backward pass in the future. I don't see any code that needs to be changed in...

> The purpose of GSU=0 still keep GSU codes is we want to reduce the number of kernels. @KKyang Did this impact anything if without this change? Yes, we need...

> Or If GSU=0 is only used for StreamK case, please reject solution for GSU=0 + StreamK=False. Added a reject condition to require either GSU or StreamK to be enabled