-
Notifications
You must be signed in to change notification settings - Fork 2.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
TRACE: Add transport tracing to linux example apps #16893
TRACE: Add transport tracing to linux example apps #16893
Conversation
PR #16893: Size comparison from fbcaa81 to 8fb2b48 Increases above 0.2%:
Increases (1 build for linux)
Decreases (1 build for linux)
Full report (22 builds for cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
|
8fb2b48
to
1649e7b
Compare
PR #16893: Size comparison from 193b85c to 1649e7b Increases above 0.2%:
Increases (1 build for linux)
Decreases (1 build for linux)
Full report (22 builds for cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
|
/rebase |
- Add tracing to the linux example platform for linux apps. - Separate message tracing from pigweed trace, to support handling the trace messages without enabling full pigweed tracing. - Enable just the transport trace by default for linux.
1649e7b
to
0bc99d7
Compare
PR #16893: Size comparison from 1e032b6 to 0bc99d7 Increases above 0.2%:
Increases (9 builds for linux)
Decreases (2 builds for linux)
Full report (31 builds for cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
PR #16893: Size comparison from 1e032b6 to 1e05d6b Increases above 0.2%:
Increases (24 builds for cc13x2_26x2, cyw30739, esp32, k32w, linux, nrfconnect, p6, telink)
Decreases (13 builds for cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, nrfconnect)
Full report (31 builds for cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
|
* TRACE: Add transport tracing to linux example apps - Add tracing to the linux example platform for linux apps. - Separate message tracing from pigweed trace, to support handling the trace messages without enabling full pigweed tracing. - Enable just the transport trace by default for linux. * Fix trace failure on standalone ack * Restyled by clang-format Co-authored-by: Tennessee Carmel-Veilleux <[email protected]> Co-authored-by: Restyled.io <[email protected]>
* TRACE: Add transport tracing to linux example apps - Add tracing to the linux example platform for linux apps. - Separate message tracing from pigweed trace, to support handling the trace messages without enabling full pigweed tracing. - Enable just the transport trace by default for linux. * Fix trace failure on standalone ack * Restyled by clang-format Co-authored-by: Tennessee Carmel-Veilleux <[email protected]> Co-authored-by: Restyled.io <[email protected]>
* TRACE: Add transport tracing to linux example apps - Add tracing to the linux example platform for linux apps. - Separate message tracing from pigweed trace, to support handling the trace messages without enabling full pigweed tracing. - Enable just the transport trace by default for linux. * Fix trace failure on standalone ack * Restyled by clang-format Co-authored-by: Tennessee Carmel-Veilleux <[email protected]> Co-authored-by: Restyled.io <[email protected]>
Problem
Message tracing allows capture of the secure messages which can be used for test automation. These traces should be enabled by default for linux and mac and enabled at runtime using compile time flags. #16846
Change overview
Testing
--trace_log 1
and--trace_file trace.log
chip_enable_transport_trace
was disabled and the trace command arguments were no longer availablechip_enable_transport_pw_trace
was enabled.