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

Random freezes throughout the app (Android) #21799

Open
pavloburykh opened this issue Dec 11, 2024 · 13 comments
Open

Random freezes throughout the app (Android) #21799

pavloburykh opened this issue Dec 11, 2024 · 13 comments
Assignees
Milestone

Comments

@pavloburykh
Copy link
Contributor

pavloburykh commented Dec 11, 2024

At some some point Android Status app randomly freezes. It happens on different screens (i.e. chats, wallet etc)

Below you can see video and logs. Mind that attached logs are not related to the attached video.

In logs search for events within timeline approx from 12:26 (UTC+2).

logcat3.txt

telegram-cloud-document-2-5384273974106549770.mp4
@vkjr
Copy link
Contributor

vkjr commented Dec 11, 2024

@pavloburykh, could you please specify the build on which you encounter such behaviour?

@pavloburykh
Copy link
Contributor Author

@pavloburykh, could you please specify the build on which you encounter such behaviour?

unfortunately I cannot say for sure at which point issue has arised. It is difficult to identify cause it happens randomly and we do not have any reproduction steps.

@pavloburykh
Copy link
Contributor Author

@vkjr I have managed to reproduce the issue again in the build with enabled logs. Attaching video and logs below.

Time on device in video is UTC +2. Not sure about timezone in logs. So we need to look for logs around 20:39 (UTC +2)

https://drive.google.com/drive/folders/1k9JJMg-uFp4dsbKeN1iOmc0ySF2L0yhe?usp=sharing

telegram-cloud-document-2-5391336287615605117.1.mp4

@pavloburykh
Copy link
Contributor Author

@vkjr I have managed to reproduce the issue again in the build with enabled logs. Attaching video and logs below.

Time on device in video is UTC +2. Not sure about timezone in logs. So we need to look for logs around 20:39 (UTC +2)

https://drive.google.com/drive/folders/1k9JJMg-uFp4dsbKeN1iOmc0ySF2L0yhe?usp=sharing

@vkjr just in case, attaching logcat as well https://drive.google.com/file/d/1RPWUyn1QPMWxZLT1PBVQBuXmTAlTbJG6/view?usp=sharing

@Parveshdhull
Copy link
Member

Parveshdhull commented Dec 12, 2024

Thank you @pavloburykh for sharing the logs. Probably issue is in go side.

12-11 17:07:01.429 32724   525 F libc    : Fatal signal 6 (SIGABRT), code -6 (SI_TKILL) in tid 525 (Thread-15), pid 32724 (status.ethereum)
12-11 17:07:04.706 18836 18836 F DEBUG   : backtrace:
12-11 17:07:04.706 18836 18836 F DEBUG   :       #00 pc 000000000102a358  /data/app/~~-rF5cXFgqvhu_eSlwtArEA==/im.status.ethereum-ozKicltaKfMZQMn98YqXGA==/lib/arm64/libgojni.so

Another error probably related

12-11 20:36:58.199 18683     0 E Go      : fatal error: concurrent map iteration and map write

@vkjr
Copy link
Contributor

vkjr commented Dec 12, 2024

@Parveshdhull, thanks for analysing the log file! The last error record looks especially suspicious regarding to the freeze.
Btw, do you use any specific tools for analysing log files?

@Parveshdhull
Copy link
Member

Btw, do you use any specific tools for analysing log files?

Nothing special, just using vim search to locate crashes.

@vkjr
Copy link
Contributor

vkjr commented Dec 13, 2024

Current assumption is that waku error leads to freezes.
So as far as I understand we are waiting for this status-go PR to be merged. And then to be included in status-mobile, here is the PR for this.
@pavloburykh, plese correct me if I'm wrong 🙏

@pavloburykh
Copy link
Contributor Author

@pavloburykh, plese correct me if I'm wrong 🙏

@vkjr that's correct. Thank you for your investigation.

@vkjr
Copy link
Contributor

vkjr commented Dec 16, 2024

@pavloburykh, I just encountered this comment.
So it looks like needed waku changes are already in mobile, so this issue can be re-checked

@pavloburykh
Copy link
Contributor Author

@pavloburykh, I just encountered this comment. So it looks like needed waku changes are already in mobile, so this issue can be re-checked

@vkjr the fix has been merged into release branch but not in develop yet. Let's wait until the fix is in develop and then re-check and close this issue.

@vkjr
Copy link
Contributor

vkjr commented Dec 18, 2024

@pavloburykh, sure!
Did you notice any freezes on release build after the merge?

@pavloburykh
Copy link
Contributor Author

@pavloburykh, sure! Did you notice any freezes on release build after the merge?

Nope, everything is good so far.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: No status
Development

No branches or pull requests

3 participants