tp/ui: Add support for aflags#5210
Open
sashwinbalaji wants to merge 4 commits intodev/sashwinbalaji/aflags_devicefrom
Open
tp/ui: Add support for aflags#5210sashwinbalaji wants to merge 4 commits intodev/sashwinbalaji/aflags_devicefrom
sashwinbalaji wants to merge 4 commits intodev/sashwinbalaji/aflags_devicefrom
Conversation
🎨 Perfetto UI Builds
|
sashwinbalaji
force-pushed
the
dev/sashwinbalaji/aflags_tp
branch
from
07bf2bf to
dd3cd31
Compare
LalitMaganti
approved these changes
Mar 20, 2026
This data source captures snapshots of Android aconfig flags. It invokes 'aflags list --format proto' and parses the binary protobuf. The data source supports one-shot capture at the start of the trace or periodic polling. It uses an incremental state to only emit flags that have changed since the last snapshot. The trace format uses raw strings for flag metadata (package, name, etc.) to keep the implementation simple and avoid interning complexity, adding ~120KB per full system snapshot. Test: perfetto_unittests --gtest_filter='AndroidAflagsDataSourceTest.*' Test: record_android_trace with android.aflags data source Change-Id: Icff7ad647df9c332e1e9a1f94b5fc935b19f2a12
Change-Id: I905dbde109dc9d7d983a9b9d0d442dec8037106a
Trace Processor: - Added android_aflags table in android_tables.py. - Implemented parsing for AndroidAflags packets in AndroidProbesParser. - Added aflags.sql standard library module with android_aflags view. - Added diff tests in tests_aflags.py. UI: - Updated TraceInfoPage Android tab to display aflags data. - Added a new 'Android Aflags' section with a detailed grid. Test: diff_test_trace_processor.py --name-filter='AndroidAflags' Test: Manually verified UI with a trace containing aflags Change-Id: I1314b3017e23f94a66748b4e3e908c70beda715f
sashwinbalaji
force-pushed
the
dev/sashwinbalaji/aflags_device
branch
from
40a0d73 to
87e2efe
Compare
sashwinbalaji
force-pushed
the
dev/sashwinbalaji/aflags_tp
branch
from
dd3cd31 to
a0960ef
Compare
sashwinbalaji
force-pushed
the
dev/sashwinbalaji/aflags_device
branch
from
87e2efe to
cc4c3d8
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Trace Processor:
UI:
Bug: b/314981157
Test: diff_test_trace_processor.py --name-filter='AndroidAflags'
Test: Manually verified UI with a trace containing aflags