Sahar A. Kashi salipour
  • Joined on 2022-02-10
Sahar A. Kashi commented on pull request blender/blender#121050 2024-08-09 16:42:08 +02:00
Cycles: Linux Support for HIP-RT

Hey sorry didn't mean for this change list to get reviewed. I was moving from Linux to Windows so, I pushed the changes to pick up on Windows. Should have pushed to a different branch!

Sahar A. Kashi commented on pull request blender/blender#121050 2024-08-09 16:38:35 +02:00
Cycles: Linux Support for HIP-RT

bvh_file and bvh_file_oro are loaded by hiprt64.dll. If moved to where rest of the precompiled kernels are, hiprt64 will not be able to load them.

Sahar A. Kashi commented on pull request blender/blender#121050 2024-08-08 21:35:43 +02:00
Cycles: Linux Support for HIP-RT

I am not sure where to move this part. I was falling back to the same logic as One API: 3acb5ae41b/intern/cycl

Sahar A. Kashi pushed to HIPRT_OPEN_SOURCE at salipour/AMD_HIPRT 2024-08-08 04:48:23 +02:00
a73c214129 harvest.cmake
Sahar A. Kashi commented on pull request blender/blender#121050 2024-08-07 18:47:27 +02:00
Cycles: Linux Support for HIP-RT

No WITH_CYCLES_HIP_BINARIES?

The WITH_CYCLES_HIP_BINARIES are taken care by the release configuration (in the blender_release.cmake).

The issue can be seen before building…

Sahar A. Kashi commented on issue blender/blender#125086 2024-08-06 22:16:56 +02:00
Cycles render test: Failing HIP-RT tests

@Alaska the device part of the code are disabled. So, the intersection returns false all the time and that's why part of the object is missing. I will fix and open a pull request.

Sahar A. Kashi commented on pull request blender/blender#121050 2024-08-06 20:39:54 +02:00
Cycles: Linux Support for HIP-RT

If you start with a clean Linux environment (as described in some earlier comment) and clean checkout, this should work:

  • Apply patch against the latest main branch
  • make deps -…
Sahar A. Kashi commented on pull request blender/blender#121050 2024-08-06 20:25:17 +02:00
Cycles: Linux Support for HIP-RT

Right now, this is how it is laid out: Cycles stand-alone: hiprt64.dll and shader binaries (hipfb files) are in the root directory Blender full build: hiprt64.dll and shader binaries are in…

Sahar A. Kashi commented on pull request blender/blender#121050 2024-08-02 18:52:58 +02:00
Cycles: Linux Support for HIP-RT

@Sergey I updated the install path as well. Please let me know of any more concerns. I am eager to get this pull request merged.

Sahar A. Kashi pushed to HIPRT_OPEN_SOURCE at salipour/AMD_HIPRT 2024-08-02 18:49:31 +02:00
de2114b13f Updated the install path for Cycles stand-alone
Sahar A. Kashi commented on pull request blender/blender#121050 2024-07-30 18:19:51 +02:00
Cycles: Linux Support for HIP-RT

@Sergey @LazyDodo any comment on the pending issues?

Sahar A. Kashi commented on pull request blender/blender#121050 2024-07-25 21:01:02 +02:00
Cycles: Linux Support for HIP-RT

It also hardcodes compilation to a specific structure of percompiled libraries folder, making it harder to fit into Linux distros which ship Blender and don't use our precompiled libraries.

@S…

Sahar A. Kashi commented on pull request blender/blender#121050 2024-07-25 20:32:40 +02:00
Cycles: Linux Support for HIP-RT

@Sergey @LazyDodo What do you think about the location of the couple binaries discussed here? They have to be seen by hiprt library. Cycle standalone? Linux first time compilation? Let me know…

Sahar A. Kashi pushed to HIPRT_OPEN_SOURCE at salipour/AMD_HIPRT 2024-07-24 22:52:19 +02:00
3febe21c63 Updated CMake to query HIP-RT version from hiprt.h
1458a565b3 Fixed formatting/style
Compare 2 commits »
Sahar A. Kashi commented on pull request blender/blender#121050 2024-07-23 19:38:23 +02:00
Cycles: Linux Support for HIP-RT

Anyway, I will do away with version.txt and use the header to get the version instead for both Windows and Linux.

Sahar A. Kashi commented on pull request blender/blender#121050 2024-07-23 18:43:40 +02:00
Cycles: Linux Support for HIP-RT

@Sergey how should Cycles standalone be handled. The binaries should go wherever hiprt.dll goes. That also means for Linux without precompiled binaries, these binaries should get built and copied…

Sahar A. Kashi commented on pull request blender/blender#121050 2024-07-23 18:33:45 +02:00
Cycles: Linux Support for HIP-RT
Sahar A. Kashi commented on pull request blender/blender#121050 2024-07-22 18:31:55 +02:00
Cycles: Linux Support for HIP-RT

These two files are loaded by hiprt64.dll, so they have to be copied somewhere that hiprt64.dll can find them.

Sahar A. Kashi commented on pull request blender/blender#121050 2024-07-22 18:29:35 +02:00
Cycles: Linux Support for HIP-RT

Oh! I misunderstood your earlier comment about version.txt not being detected. So, you meant version.txt didn't get copied into the lib folder. It worked for me on both Ubuntu and Windows. I will…

Sahar A. Kashi commented on pull request blender/blender#121050 2024-07-20 01:04:00 +02:00
Cycles: Linux Support for HIP-RT

@Sergey and @LazyDodo I think everything is in good shape. I updated the version file to point to the latest version of HIP-RT that removes the Python dependency and confirm dependencies build…