We can do that, but it will require the user to update their driver (no Blender change is required). We are working to get open-sourced HIP-RT integrated in Blender 4.3 and eliminate any driver…
The issue is now resolved in the latest HIP-RT SDK. I think it is too late to push it to Blender 4.2 . I expect it to become available in Blender 4.3.
easy-encription is no longer needed. Thanks for the feedback. We'll update the HIP-RT and let you know.
@brecht
HIP-RT has been updated and now the changes are public: https://github.com/GPUOpen-LibrariesAndSDKs/HIPRT/releases
Let me know if there is any pending task from AMD side.
It seems the leak is somewhere in the hiprt runtime. I will keep you posted.
@salipour Have you had a chance to look at this? Can you reproduce the issue?
Sort of. Frame one was using around under 6 GB and by frame 50, the usage had increased to 8 GB. Is it an issue…
Ah yes, technically we don't even need the .lib with the current implementation. We should add a comment about this though.
I wasn't sure if it would be best to patch hiprt to not include…
I forgot the C++20 issue. @salipour we could also use help to make HIPRT build with C++17. That's the version used by the VFX reference platform (https://vfxplatform.com/) that Blender and other…
- The trick of renaming the library to get rid of the version number in it, doesn't work on windows, there's always a .lib and .dll pair, and the .lib will have full name of the dll embedded in…
I gotta admit i'm a bit confused this got put up for review given the state it is in.
Well, that was the advice, to quickly put something together and make a WIP pull request. Anyway, let me…
It's not clear to me which build options should be enabled. There were a bunch listed in the original code, but which ones do we actually want to set, and how?
Let me check with HIPRT team…
Sorry for asking here, but expected to be in blender 4.2 release? or 4.3+
4.2
This is what AMD officially supports: https://rocm.docs.amd.com/projects/install-on-linux/en/docs-6.1.1/reference/system-requirements.html
@brecht I made changes according to the feedback. What's the next step to move forward with this pull request?
The main intention was to check if the dynamic library for hiprt exists before enabling it.