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

Issue 742676 link

Starred by 2 users

Issue metadata

Status: Archived
Owner: ----
Closed: Mar 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Task



Sign in to add a comment

Consider adding a 32-bit-userspace PreCQ bot

Project Member Reported by jorgelo@chromium.org, Jul 14 2017

Issue description

VMTests on PreCQ are only executed on x86_64. This misses failures that only happen on 32-bit userspaces (like all our ARM boards). 32-vs-64 bit failures are common in lower-level software.
 
Cc: lhchavez@chromium.org dgreid@chromium.org

Comment 2 by xixuan@chromium.org, Jul 14 2017

I note that VMTest of the affected boards doesn't fail. So will a vm-test pre-cq be able to catch such bugs?

Comment 3 by ihf@chromium.org, Jul 14 2017

AFAIK we are not planning to run ARM VMs for performance reasons. This leaves the proposal at adding a 32 bit Intel userspace, which is not used in any product. The intent of the VM is to get some cheap/scalable but somewhat dirty coverage. I am worried that supporting a non-product makes this an expensive proposal. I would accept this failure as a limitation (dirty) part of the VM testing approach.
That's fair. I'm just usually worried about things that don't fail in Pre-CQ but fail in the CQ.

To Xixuan's point, this was a system call that was succeeding in 64-bit userspaces and failed in 32-bit userspaces -- we would need to run 32-bit VMTests for things to fail.
Labels: -Pri-3 Pri-2
Re#3: I agree with ihf@ here 
- arm VMs are too slow for anything usable.
- We should probably re-enable a 32bit VMTest board both in pre-cq and cq (we should avoid pre-cq targets that don't have a cq equivalent, because they're more likely to break)

Comment 6 by xixuan@chromium.org, Jul 17 2017

Labels: Chase-Pending
Cc: dgarr...@chromium.org
Labels: -Chase-Pending
We don't have working support for vmtest on arm platforms. This is probably a good idea, but we need product support for this.
Status: Archived (was: Untriaged)
This bug is very old, is Untriaged, and has no owner.  If it is still relevant, reopen as Untriaged or open a new bug

Sign in to add a comment