Gitiles
Code Review Sign In
gerrit.openbmc.org / openbmc / phosphor-debug-collector / 938588b02e0fd2c808cd88ad25e0d1c752514492
commit938588b02e0fd2c808cd88ad25e0d1c752514492[log] [tgz]
authorJayanth Othayoth <ojayanth@gmail.com>Thu Feb 06 06:50:54 2025 -0600
committerPatrick Williams <patrick@stwcx.xyz>Thu Feb 06 12:39:03 2025 -0500
tree39f5e229a44082428c675335c2c799b84cd83a13
parent282c3c515b4135cac53b164316464140a6effbce [diff]
clang-tidy: disable readability-suspicious-call-ar

This commit disables the readability-suspicious-call-argument check
in Clang-Tidy configuration to avoid false positives errors

'''
  ../dump-extensions/openpower-dumps/dump_manager_system.cpp:94:18: error: 2nd argument 'size' (passed to 'dumpSize')
  looks like it might be swapped with the 3rd, 'dumpId' (passed to 'sourceId')
 [readability-suspicious-call-argument,-warnings-as-errors]
   94 |         upEntry->update(timeStamp, size, dumpId);
      |                  ^                 ~~~~  ~~~~~~
  ../dump-extensions/openpower-dumps/system_dump_entry.hpp:78:10: note: in the call to 'update', declared here
   78 |     void update(uint64_t timeStamp, uint64_t dumpSize, const uint32_t sourceId)
      |          ^                                   ~~~~~~~~                 ~~~~~~~~
'''

Change-Id: I39ff4d4002373cee8c1a17f2c416c563d259a74a
Signed-off-by: Jayanth Othayoth <ojayanth@gmail.com>
1 file changed
tree: 39f5e229a44082428c675335c2c799b84cd83a13
  1. dump-extensions/
  2. host-transport-extensions/
  3. subprojects/
  4. test/
  5. tools/
  6. .clang-format
  7. .clang-tidy
  8. .gitignore
  9. .shellcheck-ignore
  10. bmc_dump_entry.cpp
  11. bmc_dump_entry.hpp
  12. core_manager.cpp
  13. core_manager.hpp
  14. core_manager_main.cpp
  15. dump-extensions.hpp
  16. dump_entry.cpp
  17. dump_entry.hpp
  18. dump_manager.cpp
  19. dump_manager.hpp
  20. dump_manager_bmc.cpp
  21. dump_manager_bmc.hpp
  22. dump_manager_faultlog.cpp
  23. dump_manager_faultlog.hpp
  24. dump_manager_main.cpp
  25. dump_offload.cpp
  26. dump_offload.hpp
  27. dump_serialize.cpp
  28. dump_serialize.hpp
  29. dump_types.cpp.mako
  30. dump_types.hpp.mako
  31. dump_utils.cpp
  32. dump_utils.hpp
  33. elog_watch.cpp
  34. elog_watch.hpp
  35. example_dump_types.yaml
  36. example_errors_watch.yaml
  37. faultlog_dump_entry.cpp
  38. faultlog_dump_entry.hpp
  39. ffdc
  40. host_transport_exts.hpp
  41. LICENSE
  42. map_gen.py
  43. meson.build
  44. meson.options
  45. OWNERS
  46. ramoops_manager.cpp
  47. ramoops_manager.hpp
  48. ramoops_manager_main.cpp
  49. README.md
  50. watch.cpp
  51. watch.hpp
README.md

phosphor-debug-collector

Phosphor Debug Collector provides mechanisms to collect various log files and system parameters. Used to troubleshoot problems in OpenBMC based systems.

One such mechanism is dreport, a script that collects debug data and packages it into an archive file.

To Build

To build this package with meson, do the following steps:

    1. meson builddir
    2. ninja -C builddir

To clean the built files run ninja -C builddir clean.

To run unit tests

Tests can be run in the CI docker container, refer local-ci-build

or with an OpenBMC x86 sdk(see below for x86 steps).

meson -Dtests=enabled build
ninja -C build test