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.

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

Fix pigweed tokenizer port #26283

Merged

Conversation

marius-alex-tache
Copy link
Contributor

Issue introduced in: #25351

token was not used at all in HandleTokenizedLog. Encoded message should also contain the token.
The old implementation was using PW_TOKENIZE_TO_GLOBAL_HANDLER_WITH_PAYLOAD, which did the token copying behind the scenes.

Issue introduced in: project-chip#25351

Encoded message should also contain the token.

Signed-off-by: Marius Tache <marius.tache@nxp.com>
@github-actions
Copy link

PR #26283: Size comparison from 5a68477 to 671c9e7

Increases (1 build for cc32xx)
platform target config section 5a68477 671c9e7 change % change
cc32xx lock CC3235SF_LAUNCHXL .debug_info 19489249 19489251 2 0.0
Full report (1 build for cc32xx)
platform target config section 5a68477 671c9e7 change % change
cc32xx lock CC3235SF_LAUNCHXL 0 0 0 0.0
(read only) 601114 601114 0 0.0
(read/write) 204132 204132 0 0.0
.ARM.attributes 44 44 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 197544 197544 0 0.0
.comment 206 206 0 0.0
.data 1468 1468 0 0.0
.debug_abbrev 956756 956756 0 0.0
.debug_aranges 103416 103416 0 0.0
.debug_frame 349704 349704 0 0.0
.debug_info 19489249 19489251 2 0.0
.debug_line 2678035 2678035 0 0.0
.debug_line_str 513 513 0 0.0
.debug_loc 33340 33340 0 0.0
.debug_loclists 1501882 1501882 0 0.0
.debug_ranges 4984 4984 0 0.0
.debug_rnglists 96008 96008 0 0.0
.debug_str 3024877 3024877 0 0.0
.ramVecs 780 780 0 0.0
.resetVecs 64 64 0 0.0
.rodata 104170 104170 0 0.0
.shstrtab 265 265 0 0.0
.stack 2048 2048 0 0.0
.strtab 477531 477531 0 0.0
.symtab 285936 285936 0 0.0
.text 494824 494824 0 0.0

@bzbarsky-apple bzbarsky-apple merged commit 6211b74 into project-chip:master Apr 29, 2023
@GabrielCouturier GabrielCouturier deleted the fix/pw_tokenizer_token_not_included branch November 10, 2023 08:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants