Skip to content
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.

Sign up for GitHub

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

Jump to bottom

RPC: Support pw logging for linux rpc examples #20057

Merged
merged 1 commit into from
Jun 29, 2022
Merged

RPC: Support pw logging for linux rpc examples #20057

merged 1 commit into from
Jun 29, 2022

Conversation

rgoliver
Copy link

@rgoliver rgoliver commented Jun 28, 2022 •
edited
Loading

Problem

Linux logging currently uses printf, which does not get piped over the HDLC encoded socket to the test scripts for RPC builds.

Change overview

  • Add a config option CHIP_USE_PW_LOGGING, which uses pw log output instead of printf on linux. This allows the logs to correctly get HDLC encoded and be piped over the socket for RPC builds.
  • Enable the config option on the linux rpc examples.
  • Update console to decode these logs.

Testing

Viewed logs of the linux RPC lighting example, using the chip-console.

RPC: Support pw logging for linux rpc examples
34c2b0f 
Add a config option CHIP_USE_PW_LOGGING, which uses pw log output
instead of printf on linux. This allows the logs to correctly get
HDLC encoded and be piped over the socket for linux RPC builds.
@pullapprove pullapprove bot requested review from mspang and vijs June 28, 2022 19:01
@github-actions
Copy link

github-actions bot commented Jun 28, 2022 •
edited
Loading

PR #20057: Size comparison from 41e474d to 34c2b0f

Increases (5 builds for cc13x2_26x2, linux, nrfconnect, telink)
platform target config section 41e474d 34c2b0f change % change
cc13x2_26x2 all-clusters-app LP_CC2652R7 (read/write) 188076 188084 8 0.0
lock-ftd LP_CC2652R7 (read only) 667439 667447 8 0.0
.text 590976 590984 8 0.0
linux lighting-app debug+rpc (read only) 2547001 2547073 72 0.0
.rodata 213416 213544 128 0.1
.text 2163378 2163682 304 0.0
nrfconnect all-clusters-minimal-app nrf52840dk_nrf52840 (read/write) 1155447 1155463 16 0.0
text 799424 799428 4 0.0
telink light-switch-app tlsr9518adk80d text 564546 564548 2 0.0
Decreases (3 builds for cc13x2_26x2, linux)
platform target config section 41e474d 34c2b0f change % change
cc13x2_26x2 all-clusters-app LP_CC2652R7 (read only) 664907 664899 -8 -0.0
.text 576700 576692 -8 -0.0
lock-ftd LP_CC2652R7 (read/write) 175688 175680 -8 -0.0
linux lighting-app debug+rpc (read/write) 152952 152920 -32 -0.0
.got 4424 4392 -32 -0.7
Full report (39 builds for cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
platform target config section 41e474d 34c2b0f change % change
cc13x2_26x2 all-clusters-app LP_CC2652R7 (read only) 664907 664899 -8 -0.0
(read/write) 188076 188084 8 0.0
.bss 75876 75876 0 0.0
.data 3356 3356 0 0.0
.rodata 87891 87891 0 0.0
.text 576700 576692 -8 -0.0
all-clusters-minimal-app LP_CC2652R7 (read only) 653639 653639 0 0.0
(read/write) 198640 198640 0 0.0
.bss 75172 75172 0 0.0
.data 3356 3356 0 0.0
.rodata 91359 91359 0 0.0
.text 561964 561964 0 0.0
lock-ftd LP_CC2652R7 (read only) 667439 667447 8 0.0
(read/write) 175688 175680 -8 -0.0
.bss 72908 72908 0 0.0
.data 3280 3280 0 0.0
.rodata 75983 75983 0 0.0
.text 590976 590984 8 0.0
lock-mtd LP_CC2652R7 (read only) 616855 616855 0 0.0
(read/write) 146024 146024 0 0.0
.bss 68628 68628 0 0.0
.data 3280 3280 0 0.0
.rodata 75863 75863 0 0.0
.text 540504 540504 0 0.0
pump-app LP_CC2652R7 (read only) 677007 677007 0 0.0
(read/write) 166976 166976 0 0.0
.bss 72996 72996 0 0.0
.data 3280 3280 0 0.0
.rodata 88367 88367 0 0.0
.text 588156 588156 0 0.0
pump-controller-app LP_CC2652R7 (read only) 662919 662919 0 0.0
(read/write) 181184 181184 0 0.0
.bss 73116 73116 0 0.0
.data 3276 3276 0 0.0
.rodata 84215 84215 0 0.0
.text 578224 578224 0 0.0
shell LP_CC2652R7 (read only) 657362 657362 0 0.0
(read/write) 191124 191124 0 0.0
.bss 78180 78180 0 0.0
.data 3360 3360 0 0.0
.rodata 84650 84650 0 0.0
.text 572396 572396 0 0.0
cyw30739 light cyw930739m2evb_01 (read/write) 580446 580446 0 0.0
.app_xip_area 457456 457456 0 0.0
.bss 65952 65952 0 0.0
.data 716 716 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
lock cyw930739m2evb_01 (read/write) 582014 582014 0 0.0
.app_xip_area 458832 458832 0 0.0
.bss 66144 66144 0 0.0
.data 720 720 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
ota-requestor-no-progress-logging cyw930739m2evb_01 (read/write) 583494 583494 0 0.0
.app_xip_area 461352 461352 0 0.0
.bss 65160 65160 0 0.0
.data 660 660 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
efr32 lighting-app BRD4161A (read/write) 1081380 1081380 0 0.0
.bss 134756 134756 0 0.0
.data 2048 2048 0 0.0
.text 944556 944556 0 0.0
BRD4161A+rpc (read/write) 1135668 1135668 0 0.0
.bss 151444 151444 0 0.0
.data 2260 2260 0 0.0
.text 981944 981944 0 0.0
BRD4161A+rs911x (read/write) 943880 943880 0 0.0
.bss 140472 140472 0 0.0
.data 2048 2048 0 0.0
.text 801340 801340 0 0.0
lock-app BRD4161A+wf200 (read/write) 1121040 1121040 0 0.0
.bss 140920 140920 0 0.0
.data 2060 2060 0 0.0
.text 978040 978040 0 0.0
window-app BRD4161A (read/write) 1066660 1066660 0 0.0
.bss 134836 134836 0 0.0
.data 2076 2076 0 0.0
.text 929724 929724 0 0.0
esp32 all-clusters-app c3devkit (read only) 1018290 1018290 0 0.0
(read/write) 1486610 1486610 0 0.0
.dram0.bss 71864 71864 0 0.0
.dram0.data 14624 14624 0 0.0
.flash.rodata 214688 214688 0 0.0
.flash.text 1018290 1018290 0 0.0
.iram0.text 62902 62902 0 0.0
m5stack (read only) 1072459 1072459 0 0.0
(read/write) 488696 488696 0 0.0
.dram0.bss 77384 77384 0 0.0
.dram0.data 34144 34144 0 0.0
.flash.rodata 245172 245172 0 0.0
.flash.text 1067075 1067075 0 0.0
.iram0.text 123267 123267 0 0.0
k32w light k32w061+release (read/write) 659688 659688 0 0.0
.bss 71276 71276 0 0.0
.data 1992 1992 0 0.0
.text 580620 580620 0 0.0
lock k32w061+release (read/write) 685944 685944 0 0.0
.bss 71748 71748 0 0.0
.data 2004 2004 0 0.0
.text 606392 606392 0 0.0
linux all-clusters-app debug (read only) 2952905 2952905 0 0.0
(read/write) 178136 178136 0 0.0
.bss 84960 84960 0 0.0
.data 2048 2048 0 0.0
.data.rel.ro 84936 84936 0 0.0
.dynamic 608 608 0 0.0
.got 4536 4536 0 0.0
.init 27 27 0 0.0
.init_array 1040 1040 0 0.0
.rodata 263005 263005 0 0.0
.text 2512834 2512834 0 0.0
all-clusters-minimal-app debug (read only) 2807017 2807017 0 0.0
(read/write) 170072 170072 0 0.0
.bss 84288 84288 0 0.0
.data 2048 2048 0 0.0
.data.rel.ro 77576 77576 0 0.0
.dynamic 608 608 0 0.0
.got 4488 4488 0 0.0
.init 27 27 0 0.0
.init_array 1040 1040 0 0.0
.rodata 264669 264669 0 0.0
.text 2367458 2367458 0 0.0
bridge-app debug+rpc (read only) 2310481 2310481 0 0.0
(read/write) 148896 148896 0 0.0
.bss 72352 72352 0 0.0
.data 3824 3824 0 0.0
.data.rel.ro 66952 66952 0 0.0
.dynamic 608 608 0 0.0
.got 4392 4392 0 0.0
.init 27 27 0 0.0
.init_array 728 728 0 0.0
.rodata 197600 197600 0 0.0
.text 1951266 1951266 0 0.0
chip-tool debug (read only) 10171909 10171909 0 0.0
(read/write) 610824 610824 0 0.0
.bss 24384 24384 0 0.0
.data 1088 1088 0 0.0
.data.rel.ro 579056 579056 0 0.0
.dynamic 624 624 0 0.0
.got 5016 5016 0 0.0
.init 27 27 0 0.0
.init_array 640 640 0 0.0
.rodata 509029 509029 0 0.0
.text 8254149 8254149 0 0.0
lighting-app debug+rpc (read only) 2547001 2547073 72 0.0
(read/write) 152952 152920 -32 -0.0
.bss 72896 72896 0 0.0
.data 2096 2096 0 0.0
.data.rel.ro 72088 72088 0 0.0
.dynamic 608 608 0 0.0
.got 4424 4392 -32 -0.7
.init 27 27 0 0.0
.init_array 816 816 0 0.0
.rodata 213416 213544 128 0.1
.text 2163378 2163682 304 0.0
lock-app debug (read only) 2508233 2508233 0 0.0
(read/write) 147896 147896 0 0.0
.bss 71264 71264 0 0.0
.data 1712 1712 0 0.0
.data.rel.ro 69064 69064 0 0.0
.dynamic 608 608 0 0.0
.got 4424 4424 0 0.0
.init 27 27 0 0.0
.init_array 784 784 0 0.0
.rodata 228104 228104 0 0.0
.text 2115106 2115106 0 0.0
ota-provider-app debug (read only) 2317401 2317401 0 0.0
(read/write) 141736 141736 0 0.0
.bss 70944 70944 0 0.0
.data 1944 1944 0 0.0
.data.rel.ro 63064 63064 0 0.0
.dynamic 608 608 0 0.0
.got 4488 4488 0 0.0
.init 27 27 0 0.0
.init_array 672 672 0 0.0
.rodata 203224 203224 0 0.0
.text 1951522 1951522 0 0.0
ota-requestor-app debug (read only) 2434521 2434521 0 0.0
(read/write) 148608 148608 0 0.0
.bss 73280 73280 0 0.0
.data 2232 2232 0 0.0
.data.rel.ro 67240 67240 0 0.0
.dynamic 608 608 0 0.0
.got 4480 4480 0 0.0
.init 27 27 0 0.0
.init_array 728 728 0 0.0
.rodata 207232 207232 0 0.0
.text 2056258 2056258 0 0.0
shell debug (read only) 2542665 2542665 0 0.0
(read/write) 164488 164488 0 0.0
.bss 80904 80904 0 0.0
.data 1264 1264 0 0.0
.data.rel.ro 76672 76672 0 0.0
.dynamic 608 608 0 0.0
.got 4096 4096 0 0.0
.init 27 27 0 0.0
.init_array 920 920 0 0.0
.rodata 227122 227122 0 0.0
.text 2159250 2159250 0 0.0
tv-app debug (read only) 3096529 3096529 0 0.0
(read/write) 279240 279240 0 0.0
.bss 188648 188648 0 0.0
.data 4848 4848 0 0.0
.data.rel.ro 79312 79312 0 0.0
.dynamic 608 608 0 0.0
.got 4848 4848 0 0.0
.init 27 27 0 0.0
.init_array 952 952 0 0.0
.rodata 248768 248768 0 0.0
.text 2660002 2660002 0 0.0
tv-casting-app debug (read only) 5554449 5554449 0 0.0
(read/write) 185136 185136 0 0.0
.bss 73704 73704 0 0.0
.data 2416 2416 0 0.0
.data.rel.ro 102776 102776 0 0.0
.dynamic 608 608 0 0.0
.got 4720 4720 0 0.0
.init 27 27 0 0.0
.init_array 864 864 0 0.0
.rodata 343049 343049 0 0.0
.text 4934594 4934594 0 0.0
mbed lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2446672 2446672 0 0.0
.bss 215716 215716 0 0.0
.data 5872 5872 0 0.0
.text 1409316 1409316 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read/write) 1174595 1174595 0 0.0
bss 144650 144650 0 0.0
rodata 141284 141284 0 0.0
text 809720 809720 0 0.0
all-clusters-minimal-app nrf52840dk_nrf52840 (read/write) 1155447 1155463 16 0.0
bss 143886 143886 0 0.0
rodata 133216 133216 0 0.0
text 799424 799428 4 0.0
p6 all-clusters-app default (read/write) 2562568 2562568 0 0.0
.bss 150896 150896 0 0.0
.data 2776 2776 0 0.0
.text 1520832 1520832 0 0.0
all-clusters-minimal-app default (read/write) 2508416 2508416 0 0.0
.bss 150176 150176 0 0.0
.data 2776 2776 0 0.0
.text 1466680 1466680 0 0.0
light-app default (read/write) 2438584 2438584 0 0.0
.bss 142232 142232 0 0.0
.data 2592 2592 0 0.0
.text 1396848 1396848 0 0.0
lock-app default (read/write) 2464856 2464856 0 0.0
.bss 142080 142080 0 0.0
.data 2600 2600 0 0.0
.text 1423120 1423120 0 0.0
telink light-switch-app tlsr9518adk80d (read/write) 797712 797712 0 0.0
bss 72344 72344 0 0.0
noinit 40416 40416 0 0.0
text 564546 564548 2 0.0
lighting-app tlsr9518adk80d (read/write) 817780 817780 0 0.0
bss 73164 73164 0 0.0
noinit 40416 40416 0 0.0
text 581092 581092 0 0.0

@andy31415 andy31415 merged commit 06dcd23 into project-chip:master Jun 29, 2022
cpagravel pushed a commit to cpagravel/connectedhomeip that referenced this pull request Jul 2, 2022
@cpagravel
RPC: Support pw logging for linux rpc examples (project-chip#20057)
0e47be8 
Add a config option CHIP_USE_PW_LOGGING, which uses pw log output
instead of printf on linux. This allows the logs to correctly get
HDLC encoded and be piped over the socket for linux RPC builds.
@cpagravel cpagravel mentioned this pull request Jul 2, 2022
Merged
andy31415 pushed a commit that referenced this pull request Jul 3, 2022
@cpagravel @MtTsai @aBozowski
Cherry pick - Chef changes (06-27:07-01). RPC logging, Linux cross pl…
dc14d56 
…atform building. (#20248)

* Put software version string as <branch>:<commit_id> on esp32 while doing automation testing (#19943)

* RPC: Support pw logging for linux rpc examples (#20057)

Add a config option CHIP_USE_PW_LOGGING, which uses pw log output
instead of printf on linux. This allows the logs to correctly get
HDLC encoded and be piped over the socket for linux RPC builds.

* Chef - Continue instead of breaking on build error (#19988)

* Move chef .matter files back to /chef/devices (#20142)

* Remove matter files from root zzz

* Move .matter regen for chef

* Update bundle for changed .matter location

* Chef - Support cross platform Linux builds for arm64 (#20232)

* Chef - Use array for passing linux args to args.gni

* Chef - Add support for cross compiling linux on arm64

Co-authored-by: MtTsai <7930801+MtTsai@users.noreply.github.com>
Co-authored-by: rgoliver <rgoliver@google.com>
Co-authored-by: Austin Bozowski <bozowski@google.com>
rerasool pushed a commit to SiliconLabs/matter that referenced this pull request Sep 9, 2022
@cpagravel @MtTsai
@aBozowski @danielsilabs
Cherry pick - Chef changes (06-27:07-01). RPC logging, Linux cross pl…
f059d5d 
…atform building. (project-chip#20248)

* Put software version string as <branch>:<commit_id> on esp32 while doing automation testing (project-chip#19943)

* RPC: Support pw logging for linux rpc examples (project-chip#20057)

Add a config option CHIP_USE_PW_LOGGING, which uses pw log output
instead of printf on linux. This allows the logs to correctly get
HDLC encoded and be piped over the socket for linux RPC builds.

* Chef - Continue instead of breaking on build error (project-chip#19988)

* Move chef .matter files back to /chef/devices (project-chip#20142)

* Remove matter files from root zzz

* Move .matter regen for chef

* Update bundle for changed .matter location

* Chef - Support cross platform Linux builds for arm64 (project-chip#20232)

* Chef - Use array for passing linux args to args.gni

* Chef - Add support for cross compiling linux on arm64

Co-authored-by: MtTsai <7930801+MtTsai@users.noreply.github.com>
Co-authored-by: rgoliver <rgoliver@google.com>
Co-authored-by: Austin Bozowski <bozowski@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@anush-apple anush-apple Awaiting requested review from anush-apple

@arkq arkq Awaiting requested review from arkq

@Byungjoo-Lee Byungjoo-Lee Awaiting requested review from Byungjoo-Lee

@carol-apple carol-apple Awaiting requested review from carol-apple

@chrisdecenzo chrisdecenzo Awaiting requested review from chrisdecenzo

@chshu chshu Awaiting requested review from chshu

@chulspro chulspro Awaiting requested review from chulspro

@Damian-Nordic Damian-Nordic Awaiting requested review from Damian-Nordic

@dhrishi dhrishi Awaiting requested review from dhrishi

@electrocucaracha electrocucaracha Awaiting requested review from electrocucaracha

@erjiaqing erjiaqing Awaiting requested review from erjiaqing

@franck-apple franck-apple Awaiting requested review from franck-apple

@gjc13 gjc13 Awaiting requested review from gjc13

@harimau-qirex harimau-qirex Awaiting requested review from harimau-qirex

@harsha-rajendran harsha-rajendran Awaiting requested review from harsha-rajendran

@hawk248 hawk248 Awaiting requested review from hawk248

@isiu-apple isiu-apple Awaiting requested review from isiu-apple

@jelderton jelderton Awaiting requested review from jelderton

@jepenven-silabs jepenven-silabs Awaiting requested review from jepenven-silabs

@jmartinez-silabs jmartinez-silabs Awaiting requested review from jmartinez-silabs

@jtung-apple jtung-apple Awaiting requested review from jtung-apple

@lazarkov lazarkov Awaiting requested review from lazarkov

@LuDuda LuDuda Awaiting requested review from LuDuda

@mlepage-google mlepage-google Awaiting requested review from mlepage-google

@mrjerryjohns mrjerryjohns Awaiting requested review from mrjerryjohns

@msandstedt msandstedt Awaiting requested review from msandstedt

@mspang mspang Awaiting requested review from mspang

@robszewczyk robszewczyk Awaiting requested review from robszewczyk

@saurabhst saurabhst Awaiting requested review from saurabhst

@selissia selissia Awaiting requested review from selissia

@tcarmelveilleux tcarmelveilleux Awaiting requested review from tcarmelveilleux

@tecimovic tecimovic Awaiting requested review from tecimovic

@tehampson tehampson Awaiting requested review from tehampson

@vijs vijs Awaiting requested review from vijs

@andy31415 andy31415 Awaiting requested review from andy31415

@vivien-apple vivien-apple Awaiting requested review from vivien-apple

@wbschiller wbschiller Awaiting requested review from wbschiller

@woody-apple woody-apple Awaiting requested review from woody-apple

@xylophone21 xylophone21 Awaiting requested review from xylophone21

@yufengwangca yufengwangca Awaiting requested review from yufengwangca

@bzbarsky-apple bzbarsky-apple Awaiting requested review from bzbarsky-apple

Assignees
No one assigned
Labels
examples lib linux platform review - approved
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@rgoliver @andy31415 @bzbarsky-apple