New issue
Advanced search Search tips

Issue 646186 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Oct 2016
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 3
Type: Bug



Sign in to add a comment

[Sheriff] Some mus_ws_unittests crashes on Linux dbg

Project Member Reported by lazyboy@chromium.org, Sep 12 2016

Issue description

Hi, seems https://codereview.chromium.org/2329543002 is causing a lot of mus_ws_unittests to fail:

https://build.chromium.org/p/chromium.linux/builders/Linux%20Tests%20%28dbg%29%281%29%2832%29/builds/33286

relevant parts:
[32650:32650:0912/155014:4585063712:ERROR:window_tree.cc(1331)] Not implemented reached in virtual void ui::ws::WindowTree::OnWindowInputEventAck(uint32_t, mojom::EventResult): Wrong event acked. event_id=0, event_ack_id_=24847302
Received signal 11 SEGV_MAPERR 000000000004
#0 0x0000f73092a4 base::debug::StackTrace::StackTrace()
#1 0x0000f7308dcb base::debug::(anonymous namespace)::StackDumpSignalHandler()
#2 0x0000f76e4420 ([vdso]+0x41f)
#3 0x0000f74da03f base::Thread::Stop()
#4 0x00000836757f ui::GpuMain::~GpuMain()
#5 0x00000859bc5f ui::ws::GpuServiceProxy::~GpuServiceProxy()
#6 0x00000859bda3 ui::ws::GpuServiceProxy::~GpuServiceProxy()
#7 0x0000080e46a9 std::default_delete<>::operator()()
#8 0x00000857552d std::unique_ptr<>::reset()
#9 0x000008571d69 std::unique_ptr<>::~unique_ptr()
#10 0x00000856b374 ui::ws::WindowServer::~WindowServer()
#11 0x00000856bca3 ui::ws::WindowServer::~WindowServer()
#12 0x0000080e6ff9 std::default_delete<>::operator()()
#13 0x00000810688d std::unique_ptr<>::reset()
#14 0x000008103987 ui::ws::test::WindowServerTestHelper::~WindowServerTestHelper()
#15 0x000008103c8b ui::ws::test::WindowEventTargetingHelper::~WindowEventTargetingHelper()
#16 0x0000081c03d3 ui::ws::test::WindowTreeTest::~WindowTreeTest()
#17 0x0000081c021b ui::ws::test::WindowTreeTest_EventAck_Test::~WindowTreeTest_EventAck_Test()
#18 0x0000081c0263 ui::ws::test::WindowTreeTest_EventAck_Test::~WindowTreeTest_EventAck_Test()
#19 0x000008377092 base::RefCountedThreadSafe<>::DeleteInternal()
#20 0x000008670b90 testing::internal::HandleSehExceptionsInMethodIfSupported<>()
#21 0x000008663c03 testing::internal::HandleExceptionsInMethodIfSupported<>()
#22 0x0000086565a0 testing::TestInfo::Run()
#23 0x000008656d00 testing::TestCase::Run()
#24 0x00000865e65f testing::internal::UnitTestImpl::RunAllTests()
#25 0x000008673190 testing::internal::HandleSehExceptionsInMethodIfSupported<>()
#26 0x000008665973 testing::internal::HandleExceptionsInMethodIfSupported<>()
#27 0x00000865e268 testing::UnitTest::Run()
#28 0x0000085b6e6d RUN_ALL_TESTS()
#29 0x0000085b5761 base::TestSuite::Run()
#30 0x0000081942ee _ZN4base8internal13FunctorTraitsIMN4mojo17AssociatedBindingIN2ui5mojom13WindowManagerEEEFvvEvE6InvokeIPS7_JEEEvS9_OT_DpOT0_
#31 0x0000081941bb _ZN4base8internal12InvokeHelperILb0EvE8MakeItSoIRKMN4mojo17AssociatedBindingIN2ui5mojom13WindowManagerEEEFvvEJPS9_EEEvOT_DpOT0_
#32 0x0000081c5fce _ZN4base8internal7InvokerINS0_9BindStateIMNS_9TestSuiteEFivEJNS0_17UnretainedWrapperIS3_EEEEEFivEE7RunImplIRKS5_RKSt5tupleIJS7_EEJLj0EEEEiOT_OT0_NS_13IndexSequenceIJXspT1_EEEE
#33 0x0000081c5d71 _ZN4base8internal7InvokerINS0_9BindStateIMNS_9TestSuiteEFivEJNS0_17UnretainedWrapperIS3_EEEEEFivEE3RunEPNS0_13BindStateBaseE
#34 0x00000819447e base::Callback<>::Run()
#35 0x0000085b92b9 base::(anonymous namespace)::LaunchUnitTestsInternal()
#36 0x0000085b90a3 base::LaunchUnitTests()
#37 0x0000081c5bbe MasterProcessMain()
#38 0x0000084f514b main
#39 0x0000f34a7533 __libc_start_main
#40 0x00000808c431 <unknown>
  gs: 00000063  fs: 00000000  es: 0000002b  ds: 0000002b
 edi: 089edae4 esi: 089edd58 ebp: ffdc0d58 esp: ffdc0b60
 ebx: 089f2edc edx: 09258301 ecx: 00000000 eax: 00000000
 trp: 0000000e err: 00000004  ip: f74da03f  cs: 00000023
 efl: 00010202 usp: ffdc0b60  ss: 0000002b
[end of stack trace]
 
Project Member

Comment 1 by bugdroid1@chromium.org, Sep 12 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/1de77ecc00dc3cad6363919340d80e06f8b920bc

commit 1de77ecc00dc3cad6363919340d80e06f8b920bc
Author: lazyboy <lazyboy@chromium.org>
Date: Mon Sep 12 23:41:04 2016

Revert of services/ui: Use new gpu::GpuInit to do the GPU initialization. (patchset #3 id:40001 of https://codereview.chromium.org/2329543002/ )

Reason for revert:
Lots of mus_ws_unittests started failing on Linux Tests dbg and Linux Tests, e.g.:
https://build.chromium.org/p/chromium.linux/builders/Linux%20Tests%20%28dbg%29%281%29%2832%29/builds/33286
BUG= 646186 

Original issue's description:
> services/ui: Use new gpu::GpuInit to do the GPU initialization.
>
> BUG= 643746 
>
> Committed: https://crrev.com/8b2d8a7287c3a02a2398aded54429d5b2f1b411a
> Cr-Commit-Position: refs/heads/master@{#418062}

TBR=fsamuel@chromium.org,sky@chromium.org,sadrul@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG= 643746 

Review-Url: https://codereview.chromium.org/2338483003
Cr-Commit-Position: refs/heads/master@{#418105}

[modify] https://crrev.com/1de77ecc00dc3cad6363919340d80e06f8b920bc/services/ui/gpu/BUILD.gn
[delete] https://crrev.com/7d8fc6b4365fa14e80b83213fa7ea5f77fd43f9b/services/ui/gpu/gpu_main.cc
[delete] https://crrev.com/7d8fc6b4365fa14e80b83213fa7ea5f77fd43f9b/services/ui/gpu/gpu_main.h
[modify] https://crrev.com/1de77ecc00dc3cad6363919340d80e06f8b920bc/services/ui/gpu/gpu_service_internal.cc
[modify] https://crrev.com/1de77ecc00dc3cad6363919340d80e06f8b920bc/services/ui/gpu/gpu_service_internal.h
[modify] https://crrev.com/1de77ecc00dc3cad6363919340d80e06f8b920bc/services/ui/ws/DEPS
[modify] https://crrev.com/1de77ecc00dc3cad6363919340d80e06f8b920bc/services/ui/ws/gpu_service_proxy.cc
[modify] https://crrev.com/1de77ecc00dc3cad6363919340d80e06f8b920bc/services/ui/ws/gpu_service_proxy.h

Status: Fixed (was: Assigned)
The culprit CL landed with a fix.

Sign in to add a comment