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

Issue 594659 link

Starred by 2 users

Issue metadata

Status: Verified
Owner:
Last visit > 30 days ago
Closed: Mar 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

VMTests went from 40 minutes to 90 minutes on x86-alex-tot-chrome-pfq-informational

Project Member Reported by steve...@chromium.org, Mar 14 2016

Issue description

Starting here:

https://uberchromegw.corp.google.com/i/chromeos.chrome/builders/x86-alex-tot-chrome-pfq-informational/builds/19352

VMTests started taking ~ 90 minutes instead of ~40 minutes

Chrome range is:
380365 - 380382

 
Cc: ihf@chromium.org marc...@chromium.org
The crashes all appear to be in swrast_dri - is that Mesa? This might be a red herring, but did we make any recent changes to the mesa drivers or the VMTests that might be causing crashes on shutdown?

(And by recent I mean specifically around Wed March 9th)

Comment 5 by ihf@chromium.org, Mar 14 2016

Cc: x...@chromium.org
Owner: marc...@chromium.org
Yes, I think that is Mesa. Would be nice if there were symbols. Maybe removed a patch to aggressively. I'll dig some more, but that should be for Stephane.

Crash reason:  SIGSEGV
Crash address: 0x0
Process uptime: not available

Thread 0 (crashed)
 0  swrast_dri.so + 0x5cc116
    eip = 0x6e36e116   esp = 0x6d46dfe0   ebp = 0x6d46e178   ebx = 0x6f017a54
    esi = 0x7badd57c   edi = 0x7a3b0708   eax = 0x7bae3760   ecx = 0x00000000
    edx = 0x00000000   efl = 0x00210206
    Found by: given as instruction pointer in context
 1  swrast_dri.so + 0x5cb325
    eip = 0x6e36d325   esp = 0x6d46e180   ebp = 0x6d46e1e8
    Found by: previous frame's frame pointer
 2  swrast_dri.so + 0x5cad3b
    eip = 0x6e36cd3b   esp = 0x6d46e1f0   ebp = 0x6d46e218
    Found by: previous frame's frame pointer

Comment 6 by ihf@chromium.org, Mar 14 2016

Well, it still has the swrast patch
https://chromium-review.googlesource.com/#/c/331365/
Digging for symbols.

Comment 7 by ihf@chromium.org, Mar 14 2016

Nope, no symbolisation anywhere.

Comment 8 by ihf@chromium.org, Mar 14 2016

Many but not all login related tests crashing. I think logging in as guest may repro. Sample tests
login_LogoutProcessCleanup
login_CryptohomeIncognito
Status: Started (was: Assigned)
I was able to reproduce the crashes running:

./bin/cros_run_vm_test --board=x86-alex --results_dir=/tmp/vm_results_f1dcaac login_LogoutProcessCleanup

at both:

d4dcbd342dd54f55383daf8bc44b2c9d97fe0d0b - the last revsion in build 19352
f1dcaac3801ed8d0a8abe62f1496e61fbaf639a7 - the last revision in build 19351

Since 19352* was slow and had crashes, and 19351 was fast and did not have crashes, I can only surmise that this is related to Mesa or the VM, but I don't know how to verify whether something changed there? Both runs appear to be using Chrome OS R51-8043.0.0.


* https://uberchromegw.corp.google.com/i/chromeos.chrome/builders/x86-alex-tot-chrome-pfq-informational/builds/19352
Project Member

Comment 12 by bugdroid1@chromium.org, Mar 15 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/0b1086ca2be31dccd9d05faa547e2063b269a5d0

commit 0b1086ca2be31dccd9d05faa547e2063b269a5d0
Author: Stéphane Marchesin <marcheu@chromium.org>
Date: Tue Mar 15 01:34:31 2016

media-libs/mesa: Fix llvmpipe alignment restrictions

The code assumes that a struct on the stack is aligned to a multiple
of 16, which isn't the case on 32 bit platforms. So relax the
alignment constraints on the generated assembly.

BUG= chromium:594659 
TEST=run a 32 bit image in a VM

Change-Id: I5dde3c6488e6f8722a10ff1d9da7d52bce0d17ca
Reviewed-on: https://chromium-review.googlesource.com/332681
Reviewed-by: Ilja Friedel <ihf@chromium.org>
Tested-by: Stéphane Marchesin <marcheu@chromium.org>
Commit-Queue: Stéphane Marchesin <marcheu@chromium.org>

[modify] https://crrev.com/0b1086ca2be31dccd9d05faa547e2063b269a5d0/media-libs/mesa/mesa-11.3.0.ebuild
[modify] https://crrev.com/0b1086ca2be31dccd9d05faa547e2063b269a5d0/media-libs/mesa/mesa-9999.ebuild
[add] https://crrev.com/0b1086ca2be31dccd9d05faa547e2063b269a5d0/media-libs/mesa/files/10.5-lp_rast-Remove-alignment-constraints.patch
[add] https://crrev.com/0b1086ca2be31dccd9d05faa547e2063b269a5d0/media-libs/mesa/mesa-11.3.0-r1.ebuild

Status: Verified (was: Started)
32-bit x86 VM's seem happy now:
https://uberchromegw.corp.google.com/i/chromeos/builders/x86-alex-chrome-pfq


Sign in to add a comment