New issue
Advanced search Search tips

Issue 622339 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jun 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 3
Type: Bug

Blocking:
issue 82385
issue 621972



Sign in to add a comment

LNK1285 on CrWinASan(dll) bot

Project Member Reported by thakis@chromium.org, Jun 22 2016

Issue description

https://build.chromium.org/p/chromium.fyi/builders/CrWinAsan%28dll%29/builds/4405/steps/compile/logs/stdio

LINK : fatal error LNK1285: corrupt PDB file 'C:\b\build\slave\CrWinAsan_dll_\build\src\out\Release\base_win64.dll.pdb'; delete and rebuild

This has been going on for a while.

The win asan bots are the only bots building with debug info. Deleting the pdb file will probably get things unstuck, but filing this instead if someone wants to do some debugging what happened.
 

Comment 1 by r...@chromium.org, Jun 22 2016

I saw this while building LLVM, but http://llvm.org/viewvc/llvm-project?view=revision&revision=273294 made it go away for me.

I think this won't be the last time we emit corrupt codeview, so it might be worth automating the process of deleting corrupt PDBs on the TOT bots. =/

Comment 2 by r...@chromium.org, Jun 22 2016

Well, for better or worse this still reproduces with clang:

[163/163] Linking CXX executable bin\clang.exe
FAILED: bin/clang.exe
...
LINK : fatal error LNK1285: corrupt PDB file 'D:\src\llvm\build_debug\bin\clang.pdb'; delete and rebuild

Comment 3 by thakis@chromium.org, Jun 22 2016

Ok, so I guess this is just a known bug then, and the bot will green up once it's fixed.

Does this block clang rolls? the asan lkgr bots use pinned clang and build with debug info too, right?

Comment 4 by r...@chromium.org, Jun 22 2016

> Ok, so I guess this is just a known bug then, and the bot will green up once it's fixed.

The TOT bots don't do a full clobber, right? They just use a unique -D flag to invalidate the ninja cache? If so, the existing corrupt PDB will be there in place until we remove it, so the bot won't turn green by itself. We may want to take the time to really clobber PDBs, because we can expect LLVM will generate corrupt codeview from time to time.

> Does this block clang rolls? the asan lkgr bots use pinned clang and build with debug info too, right?

ASan uses pinned clang, so yes, I would consider this a blocker for a clang roll.

Comment 5 by thakis@chromium.org, Jun 22 2016

Blocking: 621972
adding blocker then. i wanted to roll today (it's been a month), guess i'll wait a bit.

It's very easy to temporarily make the bot a real clobber bot for a few builds once you think the bug's fixed. (we just have to add 'clobber' to the bot's recipe config)

Comment 6 by r...@chromium.org, Jun 22 2016

There was a remaining bug in large record splitting that I fixed in r273477. Can we clobber the asan builder and see if it goes away and stays away?
Project Member

Comment 9 by bugdroid1@chromium.org, Jun 22 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/infra.git/+/34a6ba6f15ad287e5cc819003cf700c75872c645

commit 34a6ba6f15ad287e5cc819003cf700c75872c645
Author: recipe-roller <recipe-roller@chromium.org>
Date: Wed Jun 22 23:35:40 2016

Roll recipe dependencies (trivial).

This is an automated CL created by the recipe roller. This CL rolls recipe
changes from upstream projects (e.g. depot_tools) into downstream projects
(e.g. tools/build).

More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug
(or complain)

build:
  https://crrev.com/f75591f68055d86863a3fce9f49534fb8ce86aa9 Deploy build90-b1 (Pixel C) on chromium.gpu.fyi waterfall. (kbr@chromium.org)
  https://crrev.com/73b661438ae50e81c0bdbb7e9d5fe52dae7e4db5 Make CrWinASan* bots true clobber bots. (thakis@chromium.org)

R=kbr@chromium.org,thakis@chromium.org
BUG=622411, 622339 

TBR=martiniss@chromium.org,phajdan.jr@chromium.org

Review-Url: https://codereview.chromium.org/2084303003

[modify] https://crrev.com/34a6ba6f15ad287e5cc819003cf700c75872c645/infra/config/recipes.cfg

Comment 10 by r...@chromium.org, Jun 23 2016

Owner: r...@chromium.org
Status: Fixed (was: Untriaged)
Looks like clobbering fixed it:
https://build.chromium.org/p/chromium.fyi/builders/CrWinAsan%28dll%29/builds/4409

I think this was probably caused by David Majnemer's r272960, which made us emit enum types. Enum types are often large, and LLVM's intrinsic enum was the root cause of the same LNK1285 error in LLVM. With r273453 and r273477, we now split up large list records, and things seem to work again.
Project Member

Comment 11 by bugdroid1@chromium.org, Jun 23 2016

Sign in to add a comment