-
-
Notifications
You must be signed in to change notification settings - Fork 495
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
DietPi-Software | Kodi: Assure min 128M gpu_mem on noninteractive installs #3173
Comments
@glogiotatidis You as well used the RPi Buster image with v6.25, right? Or was it Stretch? However from the error log I am not 100% sure, which finally caused the crash, at least looks like it has some audio issues. |
Yes it's Same HW, same setup. I don't have altered any audio settings. It defaults to Forcing HDMI audio, or disabling audio altogether do not fix the issue. Enabled Kodi debug logs and tried installing a few packages mentioned like udisks2 and upower, nothing |
Fix: It was the default GPU memory split! Kodi was installed non-interactivelly with Can we have memory split conf values in |
@glogiotatidis |
Found the issue: https://github.com/MichaIng/DietPi/blob/master/dietpi/dietpi-software#L12192-L12197
Hmm since Kodi requires AFAIK min 128 MiB, we should force this (if lower value found) in every case, then ask user if recommended value 320 MiB should be applied. Although on RPi4 256 MiB should be more then sufficient since several GPU features have been moved over to system memory with that model. However this is another topic. I am not keen to add a non-effective dietpi.txt setting since the effective value can be changed in config.txt already: https://github.com/MichaIng/DietPi/blob/master/config.txt#L59-L61 However either as above, strictly required values should be applied in every case on install, and probably we should add some hint to dietpi.txt or online docs (getting started thread) that device specific boot configs (config.txt on RPi) can be used additionally to apply values from first boot on. |
Neat, thanks for the quick debug session @MichaIng |
Okay, the minimum 128 MiB, required for Kodi to start, is now applied automatically: 309688a |
Creating a bug report/issue
Required Information
Additional Information (if applicable)
Steps to reproduce
$ startkodi
Expected behaviour
Actual behaviour
Extra details
Kodi crash log
The text was updated successfully, but these errors were encountered: