74dc62460b8cdd5bfeac47bfe8e759fc04b55ef8 |
|
13-Oct-2015 |
Alex Vakulenko <avakulenko@google.com> |
system/core: Rename "chromeos" -> "brillo" in include paths and namespaces libchromeos is transitioning to libbrillo and chromeos namespaces and include directory is changing to brillo. Bug: 24872993 Change-Id: I797613a38c7444a113f12e38366a424388477276
/system/core/crash_reporter/udev_collector.cc
|
6c34c2576eb241f134431b2cc847a6c7d70ef012 |
|
20-Aug-2015 |
Steve Fung <stevefung@google.com> |
crash_reporter: Relicense as Apache 2. MODULE_LICENSE_BSD and NOTICE were removed as the license is no longer different from the rest of system/core. Bug: 23368756 Change-Id: I31e48eb7c3d153c7c5ec4f694c61474dedac2eeb
/system/core/crash_reporter/udev_collector.cc
|
129bea543b7572bfdf09c6a7b3cebbe3b57ce723 |
|
23-Jul-2015 |
Steve Fung <stevefung@google.com> |
crash_reporter: Fix header include paths Fix the header include paths to be relative to the crash_reporter base folder. Bug: 22873331 Change-Id: Icdd0495a79cd7679f38c54a84e189bfb3c9d7f1c
/system/core/crash_reporter/udev_collector.cc
|
6aa551ec416bac9ec96a6103e25be78221cf36a4 |
|
06-Mar-2015 |
Peter Qiu <zqiu@chromium.org> |
crash-reporter: add support for device coredump Copy the device coredump file to /var/spool/crash directory when udev event "devcoredump" is detected. Also include the last 50 lines of /var/log/messages and /var/log/net.log in the coredump file for debugging purpose. Only perform the collection if feedback is allowed (metrics are enabled) or the device is running a developer image. By default, crash-reporter will not upload device coredumps to the crash server. The user can enable device coredumps upload via a to-be-added crosh command, which should only be done at the direction of CrOS engineers. BUG=chromium:464872 TEST=USE="asan clang" FEATURES=test emerge-$BOARD crash-reporter Manual Test: 1. Trigger wifi firmware error on a Clapper running developer image by running following command: "echo 1 > /sys/kernel/debug/iwlwifi/0000\:01\:00.0/iwlmvm/fw_restart" 2. Verify there is a device coredump file in "/var/spool/crash/" "devcoredump_iwlwifi*.devcore", as well as the additional log file ".log" and the meta file ".meta". Change-Id: Ic4cf67d4b5715a6f422505f409276d1261b7d51f Reviewed-on: https://chromium-review.googlesource.com/257091 Reviewed-by: Zeping Qiu <zqiu@chromium.org> Commit-Queue: Zeping Qiu <zqiu@chromium.org> Tested-by: Zeping Qiu <zqiu@chromium.org> Reviewed-by: Mike Frysinger <vapier@chromium.org>
/system/core/crash_reporter/udev_collector.cc
|
c76413993650f794d82ac41f18658de95eed5b3e |
|
22-Sep-2014 |
Ben Chan <benchan@chromium.org> |
crash-reporter: Clean up angle brackets in template types. C++11 no longer has the angle bracket pitfall in template types. BUG=None TEST=`FEATURES=test emerge-$BOARD crash-reporter` Change-Id: I25410c458f730d8e0a38237b3711439262a1b774 Reviewed-on: https://chromium-review.googlesource.com/219194 Reviewed-by: Mike Frysinger <vapier@chromium.org> Commit-Queue: Ben Chan <benchan@chromium.org> Tested-by: Ben Chan <benchan@chromium.org>
/system/core/crash_reporter/udev_collector.cc
|
262d798b380a59ef691ede4c6a32aa71ff191295 |
|
18-Sep-2014 |
Ben Chan <benchan@chromium.org> |
crash-reporter: Replace NULL with nullptr. BUG=None TEST=`FEATURES=test emerge-$BOARD crash-reporter` Change-Id: If0804613ee0385752d01f7bbe01902ffef53bd94 Reviewed-on: https://chromium-review.googlesource.com/218870 Reviewed-by: Alex Vakulenko <avakulenko@chromium.org> Commit-Queue: Ben Chan <benchan@chromium.org> Tested-by: Ben Chan <benchan@chromium.org>
/system/core/crash_reporter/udev_collector.cc
|
ab6cc90503ca2db976a3cb9c9382a9da85c4b5a2 |
|
05-Sep-2014 |
Ben Chan <benchan@chromium.org> |
Update code to include base/files/file_util.h file_util.h was moved from base to base/files (https://codereview.chromium.org/468253002). This CL updates platform2 code to include base/files/file_util.h instead of base/file_util.h. BUG=chromium:411001 TEST=Trybot run on paladin, release, and chromiumos-sdk builders. Change-Id: I488925b54615e131e508a460dc1a27f88168f936 Reviewed-on: https://chromium-review.googlesource.com/216851 Reviewed-by: Alex Vakulenko <avakulenko@chromium.org> Tested-by: Ben Chan <benchan@chromium.org> Commit-Queue: Ben Chan <benchan@chromium.org>
/system/core/crash_reporter/udev_collector.cc
|
895fa5d99f4187d67eee807cf7812d1af9853b49 |
|
03-Sep-2014 |
Ben Chan <benchan@chromium.org> |
crash-reporter: Include base/macros.h instead of base/basictypes.h crash-reporter no longer uses the integer types from base/basictypes.h. It should simply include base/macros.h for the DISALLOW_COPY_AND_ASSIGN macro instead. This CL also marks several classes DISALLOW_COPY_AND_ASSIGN. BUG=None TEST=`FEATURES=test emerge-$BOARD crash-reporter` Change-Id: I02c90916fdda63c4d4134ed87b13382ed2fbab26 Reviewed-on: https://chromium-review.googlesource.com/216009 Reviewed-by: Alex Vakulenko <avakulenko@chromium.org> Commit-Queue: Ben Chan <benchan@chromium.org> Tested-by: Ben Chan <benchan@chromium.org>
/system/core/crash_reporter/udev_collector.cc
|
7e77690375bc8a896a8de318d69d515e67c7aefe |
|
18-Jun-2014 |
Ben Chan <benchan@chromium.org> |
crash-reporter: Fix coding style and cpplint issues. BUG=chromium:385849 TEST=`FEATURES=test emerge-$BOARD platform2` TEST=`cpplint.py --filter=--build/header_guard,-build/include_order *.h *.cc` Change-Id: I3c24dd9487df50cdb22fc1c7739c9e95e452afae Reviewed-on: https://chromium-review.googlesource.com/204487 Tested-by: Ben Chan <benchan@chromium.org> Reviewed-by: Mike Frysinger <vapier@chromium.org> Commit-Queue: Ben Chan <benchan@chromium.org>
/system/core/crash_reporter/udev_collector.cc
|
a557c1187ff19d422db2a9c951ecd8f7243e79bd |
|
06-Feb-2014 |
Mike Frysinger <vapier@chromium.org> |
update to libbase-242728 BUG=chromium:331128 TEST=`FEATURES=test emerge-x86-alex platform2` works TEST=`cbuildbot chromiumos-sdk` works CQ-DEPEND=CL:185131 Change-Id: Ia03a6ea7aaf6b4ee5d9c512ebf7080c0c28920f1 Reviewed-on: https://chromium-review.googlesource.com/185150 Reviewed-by: Ben Chan <benchan@chromium.org> Commit-Queue: Mike Frysinger <vapier@chromium.org> Tested-by: Mike Frysinger <vapier@chromium.org>
/system/core/crash_reporter/udev_collector.cc
|
9f90acaa4d420530d7b4ddd37112518df68e373a |
|
20-Feb-2013 |
Simon Que <sque@chromium.org> |
Replace FilePath declaration with file_path.h include Fixes a clang error. Also use "base::FilePath" since FilePath has been moved to the base namespace. BUG=chromium-os:39046 TEST=The following command passes: USE="chrome_internal" CFLAGS="-clang -print-cmdline" CXXFLAGS="-clang -print-cmdline" emerge-x86-alex crash-reporter Change-Id: I7da82093c5685b5b556cba971b1e1b14ac0f59c4 Signed-off-by: Simon Que <sque@chromium.org> Reviewed-on: https://gerrit.chromium.org/gerrit/43577 Reviewed-by: Ben Chan <benchan@chromium.org>
/system/core/crash_reporter/udev_collector.cc
|
20cb7ccc6d5e2f6cf6c91e945479ddfeec00a6e5 |
|
02-Nov-2012 |
Yufeng Shen <miletus@chromium.org> |
Construct signature and exec_name from udev rules Current all udev collected reports have the same signature and exec_name. This patch makes that different udev rules generate reports with different signatures and exec_name so that it is easier to search a specific kind of udev report on crash server. BUG=chromium-os:35169 TEST=Tested locally, with a fake udev rule ACTION=="change", SUBSYSTEM=="i2c", DRIVER=="cyapa", ENV{ERROR}=="1", RUN+="/sbin/crash_reporter --udev=SUBSYSTEM=i2c-cyapa:ACTION=change" and fake crash_reporter_logs.conf entry crash_reporter-udev-collection-change--i2c-cyapa:grep -C 20 cyapa /var/log/messages Generate a report log with meta data: " sig=crash_reporter-udev-collection-change--i2c-cyapa exec_name=udev-i2c-cyapa ver=3120.0.2012_10_31_1248 payload=/var/spool/crash/change__i2c_cyapa.20121102.125731.0.log.gz payload_size=6557 done=1 " Change-Id: I9f7905b69e786a2a13cb8a8df46eeba243157009 Reviewed-on: https://gerrit.chromium.org/gerrit/37262 Reviewed-by: Simon Que <sque@chromium.org> Reviewed-by: Michael Krebs <mkrebs@chromium.org> Commit-Ready: Yufeng Shen <miletus@chromium.org> Tested-by: Yufeng Shen <miletus@chromium.org>
/system/core/crash_reporter/udev_collector.cc
|
acc7938ede0e3c07a2d1809f82b174b51f7ab7c5 |
|
05-May-2012 |
Simon Que <sque@chromium.org> |
Create separate udev collector class Previously added a udev collector to CrashCollector. Now it's time to put it in its own class. BUG=chromium-os:30268 TEST=See "TEST=" under https://gerrit.chromium.org/gerrit/21498 Change-Id: I6a62826cb84ef7ccd42a512b00e127af6de3280d Signed-off-by: Simon Que <sque@chromium.org> Reviewed-on: https://gerrit.chromium.org/gerrit/25112 Reviewed-by: Michael Krebs <mkrebs@chromium.org>
/system/core/crash_reporter/udev_collector.cc
|