-
Notifications
You must be signed in to change notification settings - Fork 327
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
App crash while tracing with gapid #2332
Comments
Hi, Is there any update on this issue? I am also getting this crash with an appliation for GearVR with Oculus VR framework.
|
What device are you trying to trace on? Can you collect the output of |
I am tracing on Galaxy S7. I am tracing the app at https://developer.oculus.com/downloads/package/oculus-mobile-sdk/. In travel. Will upload the logs as soon as I get a chance. |
I am attaching the log for Rel 1.21. Also attaching the log with the ASSERT commented out on master branch. |
This should be fixed at head, if you are able to try and build that |
Andrew which commit fixed this. I did try with head last week and the issue
was there with head also. I don't see any checkin in last week which sounds
like would have fixed this issue. Please let me know and I will build and
try again.
…On Wed, Dec 12, 2018 at 9:51 AM Andrew Woloszyn ***@***.***> wrote:
This should be fixed at head, if you are able to try and build that
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#2332 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AFaS3zU9bxG44ysKjoR5eqkMXk_5N6B6ks5u4UIogaJpZM4YDTW0>
.
|
Tried compiling head but getting following error:
https://docs.bazel.build/versions/master/skylark/lib/CcToolchainInfo.html#strip_executable says not to sue this. Will test once I figure out how to compile this. |
What version of bazel are you using? I think on Windows you need to use 0.20.0. On other OSs 0.18 is sufficient, though. |
I am using 0.19.2. Will upgrade to 0.20.0 and check. |
With 0.20 I keep getting following error. I will try 0.21 bazel. Hopefully it has latest go.
|
With 0.21 bazel, I am getting some other errors. I will park it for now check this later.
|
I don't know if 0.21 will work. 0.20 should work. I think you may just need a |
Also, please ensure |
@pmuetschard head compiles with your suggestion. @AWoloszyn I see same crash with head also. Attaching the log with head. |
The interesting line being: |
11-28 14:13:55.126 21632 21632 F DEBUG : #5 pc 001d5293 /data/app/com.google.android.gapid.armeabi-ftkbx446RfJ049uYBuVAsA==/lib/arm/libgapii.so It would be great to know what function this was in, then we could get some idea of what was causing this. Since you are able to build: If you make the following change:
Here: https://github.com/google/gapid/blob/master/core/cc/log.h#L50 That will print out the names of the functions that are being called. |
Attaching the log with suggested change:
|
From the log messages it appears the crash is in @rishi-rranjan thank you so much for the quick updates, all the logs and patience. It would be very helpful if you could run a debug build of gapid, so that the native stack trace printed by Android would contain the symbol names. To get a debug build, use |
@pmuetschard Please find the log attached with |
Thank you so much! The relevant bits are:
|
@pmuetschard Has this been fixed on master? |
Any update on this issue? |
@pmuetschard any update on this issue? |
@pmuetschard I am trying to see if master branch has this fixed but I keep getting following error with master branch. Any guidance on this?
|
You likely have to set your JAVA_HOME environment variable. This recently was made a new requirement of bazel. |
@AWoloszyn Thanks for the help. With JAVA_HOME I am able to compile the master branch but with Oculus mobile VR, I still see the same crash. |
If you have a working build, would you be able to try |
Manually merged change. With this change crash is not seen but When I open the trace, it gives an error After closing and reopening, I can see the log so we can close this now.
|
Tried with 1.5.0 on Oculus Go and it worked. Closing this. |
Thank you for the update! |
Hi, I was tracing my app with gapid, My app crashed with this log:
[./gapil/runtime/cc/slice.inc:182] Assert: <index < count()>: slice index out of bounds
The text was updated successfully, but these errors were encountered: