New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 801135 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Jan 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug



Sign in to add a comment

DUT rebooted during graphics_Gbm on nyan_big and peach_pit

Project Member Reported by hiroh@chromium.org, Jan 11 2018

Issue description

DUT always reboots during graphics_Gbm in last few CQs.

Any clear errors aren't shown in the log.


 

Comment 1 by hiroh@chromium.org, Jan 11 2018

client log is attached.
client.0.DEBUG
16.8 KB Download

Comment 2 by hiroh@chromium.org, Jan 11 2018

Cc: marc...@chromium.org hiroh@chromium.org diand...@chromium.org ahass...@chromium.org
Components: OS>Kernel>Graphics
Owner: gurcheta...@chromium.org

Comment 3 by hiroh@chromium.org, Jan 11 2018

Labels: -Pri-0 Pri-2
I submitted the CL to move grahics_Gbm out of bvt-inline.

Comment 4 by hiroh@chromium.org, Jan 11 2018

Labels: -Pri-2 Pri-1
The peach_pit and nyan_big failures happened when various vgem related CLs were in the CQ:

https://luci-milo.appspot.com/buildbot/chromeos/nyan_big-paladin/4078

the gbmtest uses vgem, hence the various failures.  These changes are now out of the CQ.  Moving graphics_Gbm out of bvt-inline would not the correct course of action in this case.
Looks like <https://chromium-review.googlesource.com/c/chromiumos/third_party/autotest/+/861691> was marked -1.

Can we make sure that the vgem CLs are marked -1 until this gets resolved?
OK, marking -1 on:

* https://chromium-review.googlesource.com/c/chromiumos/third_party/kernel/+/858263
  BACKPORT: CHROMIUM: drm/vgem: Add short circuit for reimporting our GEMs
  kernel 3.14

* https://chromium-review.googlesource.com/c/chromiumos/third_party/kernel/+/858490
  BACKPORT: CHROMIUM: drm/vgem: Add short circuit for reimporting our GEMs
  kernel 3.10

* https://chromium-review.googlesource.com/c/chromiumos/third_party/kernel/+/858599
  BACKPORT: CHROMIUM: drm/vgem: Add short circuit for reimporting our GEMs
  kernel 3.8

Other vgem changes int the kernels depend on those ones so it should keep the others out.  Even though 3.14 devices didn't fail, still seems best to mark them all -1.  

---

The -1 verified should get cleared by a rebase, or I'm happy to remove the -1 as needed.

===

Note that I _didn't_ mark this as -1 even though it's vgem related:

* https://chromium-review.googlesource.com/c/chromiumos/third_party/kernel/+/860737
  CHROMIUM: add case of CONFIG_ARM64 in drm_clflush_sg
  kernel 3.18


Comment 8 by hiroh@chromium.org, Jan 12 2018

Thank you for locating. I abandoned the CL to move graphics_Gbm out of bvt-inline.
Status: Verified (was: Untriaged)
Fixed the issues with the vgem patches.

Sign in to add a comment