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.

Sign up for GitHub

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

Jump to bottom

160MB GPU RAM cause RPI Zero 2 to hang #175

Closed
Benik3 opened this issue Sep 10, 2023 · 1 comment · Fixed by #176 or #178
Closed

160MB GPU RAM cause RPI Zero 2 to hang #175

Benik3 opened this issue Sep 10, 2023 · 1 comment · Fixed by #176 or #178
Labels
bug Something isn't working

Comments

@Benik3
Copy link

Benik3 commented Sep 10, 2023

What happened

The installation script sets GPU memory to 160MB, which cause RPi Zero 2W to hang during boot.
128MB is OK.

What did you expect to happen

Boot RPi Zero 2 properly

How to reproduce

fetch latest master
sudo make install
reboot

Additional information

No response
@Benik3 Benik3 added the bug Something isn't working label Sep 10, 2023
@mryel00 mryel00 changed the title 160MB GPU RAM cause RPI Zero to hang 160MB GPU RAM cause RPI Zero 2 to hang Sep 10, 2023
@mryel00
Copy link
Member

mryel00 commented Sep 10, 2023

Nice catch, I didn't yet had such a problem and I already installed it quite a few times on the pi zero 2w. But you are right and this is a bug inside the code. To be precise it's here and comes from not using else if.

I updated the title as you reported it with a pi zero 2 and we don't recommend to use crowsnest on a pi zero 1.

@mryel00 mryel00 linked a pull request Sep 10, 2023 that will close this issue
fix: fix wrong gpumem calculation #176
Merged
@mryel00 mryel00 mentioned this issue Sep 12, 2023
Merged
KwadFan pushed a commit that referenced this issue Sep 12, 2023
@mryel00
fix: fix wrong syntax in gpumem (#177)
dacc0d9 
Fix syntax error made in #176
Fixes #175
@KwadFan KwadFan mentioned this issue Sep 12, 2023
Merged
mryel00 added a commit that referenced this issue Sep 12, 2023
@KwadFan @mryel00
Merge develop branch (#178)
3e847d1 
* fix: fix wrong gpumem calculation (#176)

Signed-off-by: Patrick Gehrsitz <[email protected]>

* fix: fix wrong syntax in gpumem (#177)

Fix syntax error made in #176
Fixes #175

---------

Signed-off-by: Patrick Gehrsitz <[email protected]>
Co-authored-by: Patrick Gehrsitz <[email protected]>
This was referenced Sep 13, 2023
Merged
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: fix wrong gpumem calculation
Merge develop branch mainsail-crew/crowsnest
2 participants
@Benik3 @mryel00