Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

525 series #130

Merged
merged 1 commit into from
Nov 11, 2022
Merged

525 series #130

merged 1 commit into from
Nov 11, 2022

Conversation

qumaciel
Copy link
Contributor

Introduce the new 525 series:

Noticeable changes:

  • GSP firmware now have multiple files
  • There is a new (undocumented) file libnvidia-api.so.1

Significant change:

  • I've changed how we handle the libnvidia-egl-wayland and libnvidia-egl-gbm versions. I suggest to keep the recent versions first, so we avoid going through all the if-statements until we get to the end. I think it is less prone to errors as the versions grow with time.

I don't know whether you'll prefer revert this, but I think, at least, we should split the logic for egl-wayland and egl-gbm versions, since they have separated release cycles.

@ptr1337
Copy link
Contributor

ptr1337 commented Nov 10, 2022

Seems good to me.
But im wondering, why not using the latest egl-wayland version? The latest is the 1.1.11 Release

@qumaciel
Copy link
Contributor Author

But im wondering, why not using the latest egl-wayland version? The latest is the 1.1.11 Release

I was also expecting the 1.1.11, but that is what was shipped with the installer. :S

@aufkrawall
Copy link

Thanks. Works with the proprietary kernel module, but not the open one:

patching file src/nvidia-modeset/Makefile
Hunk #1 FAILED at 95.
1 out of 1 hunk FAILED -- saving rejects to file src/nvidia-modeset/Makefile.rej
patching file src/nvidia/Makefile
Hunk #1 FAILED at 119.
1 out of 1 hunk FAILED -- saving rejects to file src/nvidia/Makefile.rej
==> FEHLER: Ein Fehler geschah in prepare().
    Breche ab...

@qumaciel
Copy link
Contributor Author

Thanks. Works with the proprietary kernel module, but not the open one:

Could you test whether commenting this line in PKGBUILD
patch -Np1 --no-backup-if-mismatch -i "$srcdir"/nvidia-open-gcc-ibt-sls.diff
would work?

According to NVIDIA, this issue was solved, so, in principle, we don't need this patch anymore.

If not, I'll have a look tomorrow.

@ptr1337
Copy link
Contributor

ptr1337 commented Nov 10, 2022

The patch can be dropped. It has been fixed. But just for open modules. For the closed one it will probably take some more time.
Also I think you can update the EGL versions, since they "should" be compatible each other.

One thing which I notice that the nvidia-all is way smaller then the official one from arch, with changed pkgver:

Package (14)             Old Version   New Version  Net Change 

egl-wayland              2:1.1.11-2.1                 -0,09 MiB
lib32-nvidia-utils       525.53-2                   -164,39 MiB
lib32-opencl-nvidia      525.53-2                    -75,79 MiB
nvidia-dkms              525.53-2                    -68,34 MiB
nvidia-settings          525.53-1                     -1,62 MiB
nvidia-utils             525.53-2                   -650,71 MiB
opencl-nvidia            525.53-2                    -75,46 MiB
lib32-nvidia-utils-tkg                 525.53-226    164,35 MiB
lib32-opencl-nvidia-tkg                525.53-226     75,79 MiB
nvidia-dkms-tkg                        525.53-226     68,34 MiB
nvidia-egl-wayland-tkg                 525.53-226      0,07 MiB
nvidia-settings-tkg                    525.53-226      1,64 MiB
nvidia-utils-tkg                       525.53-226    572,77 MiB
opencl-nvidia-tkg                      525.53-226     75,46 MiB

There is mainly a difference between nvidia-utils and nvidia-utils-tkg from about 80MB.

@Tk-Glitch
Copy link
Member

I'm fine with the changes. Thanks qumaciel, I was too slow yet again 🐸 ❤️

@ptr1337 80MB seems large. I'll check that out.

@Tk-Glitch Tk-Glitch merged commit 8016370 into Frogging-Family:master Nov 11, 2022
@qumaciel qumaciel deleted the 525.53 branch November 11, 2022 11:50
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.

4 participants