Skip to content

Add plugin activity log file functionality#588

Merged
quic-seaswara merged 1 commit intoqualcomm:mainfrom
parth-07:PrimitivePluginActLog
Feb 26, 2026
Merged

Add plugin activity log file functionality#588
quic-seaswara merged 1 commit intoqualcomm:mainfrom
parth-07:PrimitivePluginActLog

Conversation

@parth-07
Copy link
Contributor

@parth-07 parth-07 commented Nov 13, 2025

This commit adds primitive support for plugin activity log file functionality.
It adds the option '--plugin-activity-log ' to the linker. With this option,
the linker emits a JSON plugin activity log file that contains the
ordered records of relevent plugin actions for later analysis and debugging. Currently,
only the following operations are fully supported: RemoveSymbol, UpdateChunks,
and updateLinkStat. The support for remaining operations will be added
in follow-up patches.

One key difference between the plugin operation information in the map-file
and the plugin activity log file is that in the map-file, the plugin
operations are separated as per operation type and the plugin. On the
other hand, in the plugin activity log file, we have a single array of
the operations by all the plugins. This allows to determine the total
ordering of the operations that can be important for various analysis.

Copy link
Contributor

@quic-seaswara quic-seaswara left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is going in the right direction.

It would be useful to combine just not the activity of adding and removing chunks, we might need linker state records and also the various operations it tries to invoke via the LinkerWrapper.

Can we also generate one file per plugin ?

@partaror partaror force-pushed the PrimitivePluginActLog branch from a088a55 to bcff6f5 Compare November 24, 2025 08:56
@parth-07 parth-07 force-pushed the PrimitivePluginActLog branch from bcff6f5 to dd570ce Compare November 25, 2025 18:34
@parth-07
Copy link
Contributor Author

It would be useful to combine just not the activity of adding and removing chunks, we might need linker state records and also the various operations it tries to invoke via the LinkerWrapper.

Yes, totally agreed. This PR only adds the primitive framework for recording and printing the JSON file. Follow-up patches will extend the support for more relevant plugin operations.

Can we also generate one file per plugin ?

I do not think we should generate plugin actions per plugin as we plan to add a utility tool that users can use to filter plugin activities of a particular plugin. Please let me know your thoughts on this.

@parth-07 parth-07 force-pushed the PrimitivePluginActLog branch from dd570ce to d79d8c7 Compare November 25, 2025 18:39
@partaror partaror marked this pull request as ready for review November 25, 2025 18:41
Copy link
Contributor

@quic-seaswara quic-seaswara left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As usual, pls document :)

A script to read the JSON log file will be useful.

@parth-07
Copy link
Contributor Author

As usual, pls document :)

Can we please document this once the functionality is close to complete? This PR only adds primitive support of this feature.

@parth-07 parth-07 force-pushed the PrimitivePluginActLog branch 3 times, most recently from 3c950f7 to 193d0c9 Compare December 1, 2025 15:52
@partaror partaror force-pushed the PrimitivePluginActLog branch from 193d0c9 to ddb164f Compare December 2, 2025 04:21
@parth-07 parth-07 force-pushed the PrimitivePluginActLog branch from ddb164f to d400a49 Compare January 6, 2026 13:24
@partaror partaror force-pushed the PrimitivePluginActLog branch 2 times, most recently from 2f44045 to cc80ad7 Compare January 12, 2026 16:24
@parth-07 parth-07 force-pushed the PrimitivePluginActLog branch from 6930420 to 288dfb9 Compare February 3, 2026 14:10
@parth-07 parth-07 force-pushed the PrimitivePluginActLog branch 2 times, most recently from 2a287e9 to c631cba Compare February 12, 2026 11:47
This commit adds primitive support for plugin activity log file functionality.
It adds the option '--plugin-activity-log <file>' to the linker. With this option,
the linker emits a JSON plugin activity log file that contains the
ordered records of relevent plugin actions for later analysis and debugging. Currently,
only the following operations are fully supported: RemoveSymbol, UpdateChunks,
and updateLinkStat. The support for remaining operations will be added
in follow-up patches.

One key difference between the plugin operation information in the map-file
and the plugin activity log file is that in the map-file, the plugin
operations are separated as per operation type and the plugin. On the
other hand, in the plugin activity log file, we have a single array of
the operations by all the plugins. This allows to determine the total
ordering of the operations that can be important for various analysis.

Signed-off-by: Parth Arora <partaror@qti.qualcomm.com>
@quic-seaswara quic-seaswara merged commit 1e7bfd0 into qualcomm:main Feb 26, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants