halide / Halide

a language for fast, portable data-parallel computation
https://halide-lang.org
Other
5.86k stars 1.07k forks source link

Backport #8259 to release/17.x #8270

Closed steven-johnson closed 3 months ago

steven-johnson commented 3 months ago
steven-johnson commented 3 months ago

Looks like there are Vulkan failures -- any clues? Are there more fixes I missed?

derek-gerstmann commented 3 months ago

Sorry! Yes, I saw this as well, but haven't been able to reproduce this on my Linux machine. Still investigating.

derek-gerstmann commented 3 months ago

We should also grab: https://github.com/halide/Halide/pull/8130

steven-johnson commented 3 months ago

PTAL -- lots of conflicts in the cherry pick so hopefully I did it right

derek-gerstmann commented 3 months ago

It looks like a few things from main got left behind in src/runtime/internal. I'll add those to this PR.

derek-gerstmann commented 3 months ago

I inspected buildbot-4 and it only had v1.2 of the Vulkan SDK installed, but the build target we're using is generating code for SDK v1.3. I've upgraded that machine, but the other linux workers will likely need to get upgraded too.

steven-johnson commented 3 months ago

the other linux workers will likely need to get upgraded too.

Tell me the process to upgrade and I'll handle it

derek-gerstmann commented 3 months ago

The packaging has changed since we originally setup the machines, and now everything is bundled under the vulkan-sdk for Ubuntu:

https://vulkan.lunarg.com/doc/view/latest/linux/getting_started_ubuntu.html

I just needed:

wget -qO - http://packages.lunarg.com/lunarg-signing-key-pub.asc | sudo apt-key add -
sudo wget -qO /etc/apt/sources.list.d/lunarg-vulkan-focal.list http://packages.lunarg.com/vulkan/lunarg-vulkan-focal.list
sudo apt update
sudo apt install vulkan-sdk

That should make vulkaninfo report SDK v1.3.x