Skip to content

feat(fetch-content): record a profiler marker for each fetched file - #1083

Draft
jcristau wants to merge 2 commits into
taskcluster:mainfrom
jcristau:fetch-content-markers
Draft

jcristau wants to merge 2 commits into
taskcluster:mainfrom
jcristau:fetch-content-markers

Conversation

@jcristau

Copy link
Copy Markdown
Contributor

When MOZ_RESOURCE_MONITOR_FORWARD is set, append a marker covering the download and extraction of each fetch to it, using the forwarding protocol of mozsystemmonitor, so that a resource monitor started by run-task's startup hook shows them in its profile.

When MOZ_RESOURCE_MONITOR_FORWARD is set, append a marker covering the
download and extraction of each fetch to it, using the forwarding
protocol of mozsystemmonitor, so that a resource monitor started by
run-task's startup hook shows them in its profile.

Fetches run in parallel threads, and other processes may forward to the
same file, so open it the way mozsystemmonitor does: on Windows,
O_APPEND isn't atomic, so use a handle that may only append instead.
@jcristau
jcristau force-pushed the fetch-content-markers branch from a2bb9c0 to 25c9c9a Compare October 29, 2026 19:07

This branch has not been deployed

No deployments
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.

1 participant