Skip to content

Conversation

@stemann
Copy link
Contributor

@stemann stemann commented Dec 15, 2023

Similar rebuild as #7802 - need for v8.2.1 mentioned in #7561 (comment)

@stemann
Copy link
Contributor Author

stemann commented Dec 15, 2023

@maleadt Can I bother you with this one too?

@maleadt
Copy link
Member

maleadt commented Dec 18, 2023

need for v8.2.1 mentioned in #7561 (comment)

I don't see that explained there; why is there a dependency on a specific patch version? I would assume that both TensorRT versions would be compatible with CUDNN 8.2.4, no?

@stemann
Copy link
Contributor Author

stemann commented Dec 18, 2023

CUDNN v8.2.1 is used as the build version for the TensorRT recipe v8.0.3 (and v8.0.1) - it would probably work just fine to replace the build version with v8.2.4 - I just remember putting quite a bit of effort into getting the right patch versions...

There was some form of argument for v8.2.1 back in #4483 (comment)

@maleadt
Copy link
Member

maleadt commented Dec 18, 2023

CUDNN v8.2.1 is used as the build version for the TensorRT recipe v8.0.3 (and v8.0.1) - it would probably work just fine to replace the build version with v8.2.4

Maybe try using 8.2.4 first? If it doesn't work, let me know here and I'll upload the 8.2.1 binaries.

@stemann
Copy link
Contributor Author

stemann commented Dec 18, 2023

Right - good idea.

No current amount of searching can conjure up a raison-d'être for v8.2.1 :-)

@stemann
Copy link
Contributor Author

stemann commented Dec 18, 2023

... err - I don't think we strictly need v8.2.1, but I think I figured out that the reason for v8.2.1 was to have the exact same versions as Nvidia provides for those old Jetson 4.6 distributions (the v8.2.4 aarch64-linux-gnu CUDNN binaries are from https://developer.download.nvidia.com/compute/cuda/repos/ubuntu1804/arm64 rather than from the APT sources @ https://repo.download.nvidia.com/jetson/).

@ViralBShah ViralBShah closed this Feb 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants