searching for reviewer(gerald)
c24b430f397bcb4ec51f2a837feb725d24f533cf: Bug 1674234 - Add a profiler marker for creating the skeleton UI, r=gerald.
Florian Quèze <florian@queze.net> - Mon, 02 Nov 2020 19:45:22 +0000 - rev 621778
Push 14639 by ffxbld-merge at Mon, 16 Nov 2020 09:54:46 +0000
Bug 1674234 - Add a profiler marker for creating the skeleton UI, r=gerald. Differential Revision: https://phabricator.services.mozilla.com/D95219
2684003a37f9525cef54f1c7c4f1084f81e51218: Bug 1674234 - Add a profiler marker for creating the skeleton UI, r=gerald.
Florian Quèze <florian@queze.net> - Fri, 30 Oct 2020 20:58:14 +0000 - rev 621571
Push 14639 by ffxbld-merge at Mon, 16 Nov 2020 09:54:46 +0000
Bug 1674234 - Add a profiler marker for creating the skeleton UI, r=gerald. Differential Revision: https://phabricator.services.mozilla.com/D95219
c386129a2f645e354a64d154ccdecd252e2943e5: Bug 1673723 - Test marker schema from xpcshell tests; r=gerald,julienw
Greg Tatum <gtatum@mozilla.com> - Wed, 28 Oct 2020 14:08:59 +0000 - rev 621195
Push 14639 by ffxbld-merge at Mon, 16 Nov 2020 09:54:46 +0000
Bug 1673723 - Test marker schema from xpcshell tests; r=gerald,julienw Moving markers definitions to where they are used means that it's harder to unit test them from gtests. xpcshell tests have the benefits of exercising the markers in a more realistic setting. However, we still need to test the output of the schema. This patch adds a new capability to do a deep equality check of the schema results. The utility function attempts to make the output more readable by humans, and suggest how to fix errors. This will hopefully make it easy for users to update and maintain their markers. Differential Revision: https://phabricator.services.mozilla.com/D94890
2f48d38b9bc922b6c61d7de5049a4eac78f24ac5: Bug 1673623 - Initialize the baseprofiler in xpcshell tests; r=gerald
Greg Tatum <gtatum@mozilla.com> - Tue, 27 Oct 2020 21:42:20 +0000 - rev 621158
Push 14639 by ffxbld-merge at Mon, 16 Nov 2020 09:54:46 +0000
Bug 1673623 - Initialize the baseprofiler in xpcshell tests; r=gerald The baseprofiler is required to be in place first for features that are only implemented in the baseprofiler, such as the MarkerThreadId::MainThread(). This patch adds the initialization, and upates the ProfilerIOInterposeObserver to use baseprofiler-only features. The issue before, was that the base profiler was not initialized, and the mozilla::baseprofiler::profiler_main_thread_id() was incorrectly reporting that the main thread index was 0. No tests were changed in this patch, but it does have coverage with: tools/profiler/tests/xpcshell/test_feature_fileioall.js This test would fail with just the interposer changes. Differential Revision: https://phabricator.services.mozilla.com/D94839
a824b0b49ed6e6f2eb5983fb82c76cd20ce56600: Bug 1671701 - Add more comprehensive tests for FileIO markers; r=gerald,julienw
Greg Tatum <gtatum@mozilla.com> - Tue, 27 Oct 2020 14:04:15 +0000 - rev 620943
Push 14639 by ffxbld-merge at Mon, 16 Nov 2020 09:54:46 +0000
Bug 1671701 - Add more comprehensive tests for FileIO markers; r=gerald,julienw This re-works our tests to run all of the branches in the interposer. There is a bit of a risk that this won't pass on all platforms as there is an allow list of known operations. However, it's currently only limited to macOS and Linux. Please note the placement of utility functions in shared-head.js if they were generally useful beyond the xpcshell tests, and in xpcshell/head.js if the functions were only useful for the specific FileIO tests. Differential Revision: https://phabricator.services.mozilla.com/D93850
98aa4a9932170e6e48516544cacfd70905f3a621: Bug 1671701 - Remove FileIOMarkerPayload; r=gerald
Greg Tatum <gtatum@mozilla.com> - Tue, 27 Oct 2020 14:04:12 +0000 - rev 620942
Push 14639 by ffxbld-merge at Mon, 16 Nov 2020 09:54:46 +0000
Bug 1671701 - Remove FileIOMarkerPayload; r=gerald Differential Revision: https://phabricator.services.mozilla.com/D93849
8d8f9a7db856867528c8a3681d805a834d2a258a: Bug 1671701 - Migrated FileIO markers to the Markers 2.0 API; r=gerald,julienw
Greg Tatum <gtatum@mozilla.com> - Tue, 27 Oct 2020 14:04:04 +0000 - rev 620941
Push 14639 by ffxbld-merge at Mon, 16 Nov 2020 09:54:46 +0000
Bug 1671701 - Migrated FileIO markers to the Markers 2.0 API; r=gerald,julienw This commit uses the new Markers 2.0 API for FileIO Markers. I had to create another option for the MarkerStack class in order to conditionally capture a backtrace inside of the Macro. Otherwise the macro invocation failed. Differential Revision: https://phabricator.services.mozilla.com/D93848
b4b6ec163792fdb6a23cca53bf666d6a7b34f747: Bug 1671701 - Add more comprehensive tests for FileIO markers; r=gerald,julienw
Greg Tatum <gtatum@mozilla.com> - Mon, 26 Oct 2020 20:27:21 +0000 - rev 620850
Push 14639 by ffxbld-merge at Mon, 16 Nov 2020 09:54:46 +0000
Bug 1671701 - Add more comprehensive tests for FileIO markers; r=gerald,julienw This re-works our tests to run all of the branches in the interposer. There is a bit of a risk that this won't pass on all platforms as there is an allow list of known operations. However, it's currently only limited to macOS and Linux. Please note the placement of utility functions in shared-head.js if they were generally useful beyond the xpcshell tests, and in xpcshell/head.js if the functions were only useful for the specific FileIO tests. Differential Revision: https://phabricator.services.mozilla.com/D93850
a343865ccfe0da9dda67992af9c7bba8e630705a: Bug 1671701 - Remove FileIOMarkerPayload; r=gerald
Greg Tatum <gtatum@mozilla.com> - Mon, 26 Oct 2020 20:12:29 +0000 - rev 620849
Push 14639 by ffxbld-merge at Mon, 16 Nov 2020 09:54:46 +0000
Bug 1671701 - Remove FileIOMarkerPayload; r=gerald Differential Revision: https://phabricator.services.mozilla.com/D93849
e6f882892feac3ac5dff7bdf87b3073fa8a372c9: Bug 1671701 - Migrated FileIO markers to the Markers 2.0 API; r=gerald,julienw
Greg Tatum <gtatum@mozilla.com> - Mon, 26 Oct 2020 20:12:22 +0000 - rev 620848
Push 14639 by ffxbld-merge at Mon, 16 Nov 2020 09:54:46 +0000
Bug 1671701 - Migrated FileIO markers to the Markers 2.0 API; r=gerald,julienw This commit uses the new Markers 2.0 API for FileIO Markers. I had to create another option for the MarkerStack class in order to conditionally capture a backtrace inside of the Macro. Otherwise the macro invocation failed. Differential Revision: https://phabricator.services.mozilla.com/D93848
0189678a564bfbfda6b6dbee621f8c6327fe56d6: Bug 1672251 - Allow capturing profiles of tests that trigger private browsing, r=gerald.
Florian Quèze <florian@queze.net> - Wed, 21 Oct 2020 18:49:38 +0000 - rev 620182
Push 14639 by ffxbld-merge at Mon, 16 Nov 2020 09:54:46 +0000
Bug 1672251 - Allow capturing profiles of tests that trigger private browsing, r=gerald. This allows `./mach test <test path and name> --profiler` to work even for tests that use private browsing. Differential Revision: https://phabricator.services.mozilla.com/D94152
e268681dd7b3ee3ea415f94e07f7c5b864fc1f52: Bug 1672256 - Record thread creation markers for threads created off main thread, r=gerald.
Florian Quèze <florian@queze.net> - Wed, 21 Oct 2020 18:23:16 +0000 - rev 620177
Push 14639 by ffxbld-merge at Mon, 16 Nov 2020 09:54:46 +0000
Bug 1672256 - Record thread creation markers for threads created off main thread, r=gerald. Differential Revision: https://phabricator.services.mozilla.com/D94158
71405c00ab2a66e57ffa08c0413a9d7f947e1405: Bug 1670786 - Add profiler markers with stacks of new thread creations, r=gerald.
Florian Quèze <florian@queze.net> - Tue, 13 Oct 2020 11:04:57 +0000 - rev 618602
Push 14302 by ffxbld-merge at Mon, 19 Oct 2020 16:06:48 +0000
Bug 1670786 - Add profiler markers with stacks of new thread creations, r=gerald. Differential Revision: https://phabricator.services.mozilla.com/D93264
76816622157c99d3168daabeb422ef19786840e1: Bug 1646719 - P1: Add more debugging logs r=gerald
Chun-Min Chang <chun.m.chang@gmail.com> - Sun, 04 Oct 2020 07:19:18 +0000 - rev 618354
Push 14302 by ffxbld-merge at Mon, 19 Oct 2020 16:06:48 +0000
Bug 1646719 - P1: Add more debugging logs r=gerald Add some logs that can help us to check how many cache-streams are suspended and the working status in `MediaCache::Update` Differential Revision: https://phabricator.services.mozilla.com/D92124
fdc241290ef22afefd8f87be597b4681dce16f8e: Bug 1665810 - Add an end-to-end mochitest for DOMEvent markers r=gerald
Greg Tatum <gtatum@mozilla.com> - Tue, 06 Oct 2020 13:57:50 +0000 - rev 617517
Push 14302 by ffxbld-merge at Mon, 19 Oct 2020 16:06:48 +0000
Bug 1665810 - Add an end-to-end mochitest for DOMEvent markers r=gerald I created a new test file for testing markers in the parent process. It can be re-used to test a variety of different markers and their payloads to ensure they are properly being created, and with relevant information. The idea here is that this tests the entire pipeline, and excercises the code as an end user of the profiler would. Differential Revision: https://phabricator.services.mozilla.com/D92457
410bdec8b73ef3b3aef2856f948a0bbf7288963e: Bug 1669266 - Upgrade the DOMEvent marker to not be of type "tracing" r=gerald
Greg Tatum <gtatum@mozilla.com> - Tue, 06 Oct 2020 13:57:46 +0000 - rev 617516
Push 14302 by ffxbld-merge at Mon, 19 Oct 2020 16:06:48 +0000
Bug 1669266 - Upgrade the DOMEvent marker to not be of type "tracing" r=gerald This is part of the Markers 2.0 work. This payload proved to be a bit ambiguous when moving to the new marker schema, so it requires an upgrader. The test is included as the following commit. Differential Revision: https://phabricator.services.mozilla.com/D92456
ff9593c6b768566a025b982f83e335c566108a1b: Bug 1524625 - DLL loads during early startup should show profiler markers, r=gerald.
Florian Quèze <florian@queze.net> - Mon, 05 Oct 2020 13:47:27 +0000 - rev 617338
Push 14302 by ffxbld-merge at Mon, 19 Oct 2020 16:06:48 +0000
Bug 1524625 - DLL loads during early startup should show profiler markers, r=gerald. Differential Revision: https://phabricator.services.mozilla.com/D92340
17e142537e65afcad3665af0d158b18f3e227cb9: Bug 1668056 - Bailout profiler markers should be text markers. r=jandem,gerald
Florian Quèze <florian@queze.net> - Wed, 30 Sep 2020 12:19:54 +0000 - rev 616717
Push 14302 by ffxbld-merge at Mon, 19 Oct 2020 16:06:48 +0000
Bug 1668056 - Bailout profiler markers should be text markers. r=jandem,gerald Differential Revision: https://phabricator.services.mozilla.com/D91775
d1c2e8215817a5025f982b559989e8310d4a78a8: Bug 1666617 - Allow creating MarkerInnerWindowId with a Maybe<uint64_t>. r=gerald
Markus Stange <mstange.moz@gmail.com> - Wed, 23 Sep 2020 00:04:17 +0000 - rev 616158
Push 14302 by ffxbld-merge at Mon, 19 Oct 2020 16:06:48 +0000
Bug 1666617 - Allow creating MarkerInnerWindowId with a Maybe<uint64_t>. r=gerald Depends on D91078 Differential Revision: https://phabricator.services.mozilla.com/D91079
8688720f9916ced0df9fc84305c92f245725472f: Bug 1667406 - add Renderer to the default thread filter, r=gerald.
Florian Quèze <florian@queze.net> - Fri, 25 Sep 2020 15:16:34 +0000 - rev 616156
Push 14302 by ffxbld-merge at Mon, 19 Oct 2020 16:06:48 +0000
Bug 1667406 - add Renderer to the default thread filter, r=gerald. Differential Revision: https://phabricator.services.mozilla.com/D91447
4df42e22a94f6db026bd554dbf1f44a7c46c9d05: Bug 1666246 - Allow duplicate ThreadIds in the profiler's registered thread list. r=gerald
Markus Stange <mstange.moz@gmail.com> - Mon, 21 Sep 2020 23:36:12 +0000 - rev 615336
Push 14302 by ffxbld-merge at Mon, 19 Oct 2020 16:06:48 +0000
Bug 1666246 - Allow duplicate ThreadIds in the profiler's registered thread list. r=gerald This patch makes us trust the TLS whenever we try to determine whether the current thread is already registered. It also removes assertions that assume that thread IDs can never be re-used without a proper unregistration of the old thread. Differential Revision: https://phabricator.services.mozilla.com/D90915
fe10fb7d3f3be59080d44226fb630bee503567c0: Bug 1666246 - Allow duplicate ThreadIds in the profiler's registered thread list. r=gerald, a=RyanVM DEVEDITION_82_0b2_BUILD1 DEVEDITION_82_0b2_RELEASE FIREFOX_82_0b2_BUILD1 FIREFOX_82_0b2_RELEASE
Markus Stange <mstange.moz@gmail.com> - Mon, 21 Sep 2020 23:36:12 +0000 - rev 614719
Push 13972 by ryanvm@gmail.com at Tue, 22 Sep 2020 18:37:49 +0000
Bug 1666246 - Allow duplicate ThreadIds in the profiler's registered thread list. r=gerald, a=RyanVM This patch makes us trust the TLS whenever we try to determine whether the current thread is already registered. It also removes assertions that assume that thread IDs can never be re-used without a proper unregistration of the old thread. Differential Revision: https://phabricator.services.mozilla.com/D90915
1aad6539812fd0e9c6697bf62c1c6389fdc9b2d6: Bug 1665945 - Fix compilation. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Sat, 19 Sep 2020 01:04:27 +0000 - rev 614651
Push 13943 by ffxbld-merge at Mon, 21 Sep 2020 13:41:08 +0000
Bug 1665945 - Fix compilation. r=gerald Fix ""error: member access into incomplete type 'mozilla::layers::IGPUVideoSurfaceManager" build bustage with --disable-accessibility" we don't want to fully declare the class in the header as it would require to leak most of gfx headers. Differential Revision: https://phabricator.services.mozilla.com/D90776
5dfd6570eddba5aaa1da1cb6e2ad31839c07834c: Bug 1659771 - Remove obsolete nsIProfiler.AddMarker API, r=gerald.
Florian Quèze <florian@queze.net> - Tue, 18 Aug 2020 22:52:55 +0000 - rev 609933
Push 13553 by ffxbld-merge at Mon, 24 Aug 2020 12:51:36 +0000
Bug 1659771 - Remove obsolete nsIProfiler.AddMarker API, r=gerald. Differential Revision: https://phabricator.services.mozilla.com/D87470
8ddec38d3444d14bad19bb5a7146d4536c354eaa: Bug 1658708 - Do not create a MarkerTiming when profiler is not active. r=gerald
Nazım Can Altınova <canaltinova@gmail.com> - Wed, 12 Aug 2020 14:38:44 +0000 - rev 609079
Push 13553 by ffxbld-merge at Mon, 24 Aug 2020 12:51:36 +0000
Bug 1658708 - Do not create a MarkerTiming when profiler is not active. r=gerald Differential Revision: https://phabricator.services.mozilla.com/D86807
e3d865b6d393400f7161450ef548e19f765447a9: Bug 1553421 - Add request method to network markers r=necko-reviewers,gerald,valentin
Michael Hoffmann <brennan.brisad@gmail.com> - Mon, 10 Aug 2020 06:33:22 +0000 - rev 608881
Push 13553 by ffxbld-merge at Mon, 24 Aug 2020 12:51:36 +0000
Bug 1553421 - Add request method to network markers r=necko-reviewers,gerald,valentin Differential Revision: https://phabricator.services.mozilla.com/D85878
552a50d291d8af13290f40abffc870500a69fc36: Bug 1640969 - Update profiler gtests for startTime and endTime in markers; r=gerald
Gerald Squelart <gsquelart@mozilla.com> - Thu, 30 Jul 2020 08:14:29 +0000 - rev 607342
Push 13553 by ffxbld-merge at Mon, 24 Aug 2020 12:51:36 +0000
Bug 1640969 - Update profiler gtests for startTime and endTime in markers; r=gerald Differential Revision: https://phabricator.services.mozilla.com/D84595
702818e2e923100961f4660e217e8b5fd0c404d4: Bug 1640969 - Remove startTime and endTime from most marker payloads; r=gerald
Gerald Squelart <gsquelart@mozilla.com> - Thu, 30 Jul 2020 08:14:27 +0000 - rev 607341
Push 13553 by ffxbld-merge at Mon, 24 Aug 2020 12:51:36 +0000
Bug 1640969 - Remove startTime and endTime from most marker payloads; r=gerald Differential Revision: https://phabricator.services.mozilla.com/D84346
b481d63c4ce15925bf289527de574ef288c65c5b: Bug 1640969 - Add Instant, Interval, IntervalStart, IntervalEnd markers to the profiler; r=gerald
Gerald Squelart <gsquelart@mozilla.com> - Thu, 30 Jul 2020 22:21:10 +0000 - rev 607340
Push 13553 by ffxbld-merge at Mon, 24 Aug 2020 12:51:36 +0000
Bug 1640969 - Add Instant, Interval, IntervalStart, IntervalEnd markers to the profiler; r=gerald This patch is the first step to our so-called "Markers 2.0" work. It's attempting to consolidate some ad-hoc practices in the profiler into more solid practices. This is the first step to getting the Gecko profiler to produce a format that has the timing information applied to the serialized marker tuple, rather than a mix in the tuple and the payload. Differential Revision: https://phabricator.services.mozilla.com/D78990
6c83e21dd2959fa7802bdd74d1968a6200bd0d0b: Bug 1650281 - P2: Make sure `gCombinedSizes` won't be underflowed r=gerald
Chun-Min Chang <chun.m.chang@gmail.com> - Tue, 21 Jul 2020 23:39:14 +0000 - rev 605922
Push 13434 by ffxbld-merge at Mon, 27 Jul 2020 16:59:11 +0000
Bug 1650281 - P2: Make sure `gCombinedSizes` won't be underflowed r=gerald In any case, `gCombinedSizes` should be larger than or equal to the buffer within `MemoryClockCache`. Differential Revision: https://phabricator.services.mozilla.com/D84274
3c239f976c21784dccc3a6b4a502c0901ae909fd: Bug 1650281 - P1: Widen `gCombinedSizes` once the buffers grow r=gerald
Chun-Min Chang <chun.m.chang@gmail.com> - Tue, 21 Jul 2020 23:38:57 +0000 - rev 605921
Push 13434 by ffxbld-merge at Mon, 27 Jul 2020 16:59:11 +0000
Bug 1650281 - P1: Widen `gCombinedSizes` once the buffers grow r=gerald The `gCombinedSizes` need to be enlarged once the inner buffer within `MemoryBlockCache` grows. Otherwise, when the `MemoryBlockCache` is released, subtracting the buffer-size of the `MemoryBlockCache` from `gCombinedSizes` lead to a underflow. Differential Revision: https://phabricator.services.mozilla.com/D84273
f06874791ca47bdd5afe504f57ca377b5ecd5d4f: Bug 1642772: name processes in the profiler with eTLD+1 r=gerald,dveditz
Randell Jesup <rjesup@wgate.com> - Wed, 15 Jul 2020 18:00:54 +0000 - rev 604982
Push 13434 by ffxbld-merge at Mon, 27 Jul 2020 16:59:11 +0000
Bug 1642772: name processes in the profiler with eTLD+1 r=gerald,dveditz Differential Revision: https://phabricator.services.mozilla.com/D77926
6ce9479deb886000294debdc5406ed8fcf883ffd: Bug 1649670 - Fix "control reaches end of non-void function" in ProfilerMarkerPayload.cpp; r=gerald
Jim Porter <jporter@mozilla.com> - Thu, 02 Jul 2020 00:08:56 +0000 - rev 602741
Push 13434 by ffxbld-merge at Mon, 27 Jul 2020 16:59:11 +0000
Bug 1649670 - Fix "control reaches end of non-void function" in ProfilerMarkerPayload.cpp; r=gerald Differential Revision: https://phabricator.services.mozilla.com/D81884
4647ddd16fbdb2b18370d27f5b0f2b0a235dd48f: Bug 1649653 - fix compilation on non-profiler platforms. r=gerald
Cameron Kaiser <spectre@floodgap.com> - Wed, 01 Jul 2020 03:03:56 +0000 - rev 602522
Push 13434 by ffxbld-merge at Mon, 27 Jul 2020 16:59:11 +0000
Bug 1649653 - fix compilation on non-profiler platforms. r=gerald Differential Revision: https://phabricator.services.mozilla.com/D81803
866a65ae320df3677e5a74fa9addffa0487f50ee: Bug 1613496 - Add IPC profiler markers to track when messages are processed by the IO thread queues r=gerald,froydnj
Jim Porter <jporter@mozilla.com> - Tue, 30 Jun 2020 18:39:42 +0000 - rev 602453
Push 13434 by ffxbld-merge at Mon, 27 Jul 2020 16:59:11 +0000
Bug 1613496 - Add IPC profiler markers to track when messages are processed by the IO thread queues r=gerald,froydnj This adds 3 new profiler markers for each IPC message: * One just before the first byte is sent over the IPC channel * One just after the last byte is sent over the IPC channel * One just after the last byte is received from the IPC channel With the already-existing IPC markers (for when SendXXX and RecvXXX are called), this allows us to calculate the following statistics: * Send thread latency * IPC send duration * IPC recv latency * Recv thread latency For more information on how this is presented in the UI, see: <https://github.com/firefox-devtools/profiler/pull/2535>. Differential Revision: https://phabricator.services.mozilla.com/D70790
05b31b2e850fabf71a873a02a2a17c95fead7d9d: Bug 1649233 - When profiling, record the theoretical budget with a single payload. r=gerald
Paul Adenot <paul@paul.cx> - Tue, 30 Jun 2020 14:25:27 +0000 - rev 602389
Push 13434 by ffxbld-merge at Mon, 27 Jul 2020 16:59:11 +0000
Bug 1649233 - When profiling, record the theoretical budget with a single payload. r=gerald Differential Revision: https://phabricator.services.mozilla.com/D81593
91bf772a2223becd92cf89c6ca2420f24a7d8c8e: Bug 1649233 - Add a new ProfilerMarkerPayload that is just a time duration. r=gerald,canaltinova
Paul Adenot <paul@paul.cx> - Tue, 30 Jun 2020 14:25:09 +0000 - rev 602388
Push 13434 by ffxbld-merge at Mon, 27 Jul 2020 16:59:11 +0000
Bug 1649233 - Add a new ProfilerMarkerPayload that is just a time duration. r=gerald,canaltinova Differential Revision: https://phabricator.services.mozilla.com/D81592
399e54895a750c8671383ecde660a6dca4f3b17e: Bug 1649233 - When profiling, record the theoretical budget with a single payload. r=gerald
Paul Adenot <paul@paul.cx> - Tue, 30 Jun 2020 09:05:00 +0000 - rev 602369
Push 13434 by ffxbld-merge at Mon, 27 Jul 2020 16:59:11 +0000
Bug 1649233 - When profiling, record the theoretical budget with a single payload. r=gerald Differential Revision: https://phabricator.services.mozilla.com/D81593
3955edb371cfe118d6895836ba2d6c2c2c2680bd: Bug 1649233 - Add a new ProfilerMarkerPayload that is just a time duration. r=gerald,canaltinova
Paul Adenot <paul@paul.cx> - Tue, 30 Jun 2020 12:10:00 +0000 - rev 602368
Push 13434 by ffxbld-merge at Mon, 27 Jul 2020 16:59:11 +0000
Bug 1649233 - Add a new ProfilerMarkerPayload that is just a time duration. r=gerald,canaltinova Differential Revision: https://phabricator.services.mozilla.com/D81592
897a601bef1cac60d63e602369ccafe2ec6d45cc: Bug 1647911 - Use text markers for payload presented markers. r=gerald
Markus Stange <mstange.moz@gmail.com> - Mon, 29 Jun 2020 22:41:20 +0000 - rev 602245
Push 13434 by ffxbld-merge at Mon, 27 Jul 2020 16:59:11 +0000
Bug 1647911 - Use text markers for payload presented markers. r=gerald Differential Revision: https://phabricator.services.mozilla.com/D80946
002beecf7a10859d138d17ef7e436f90f9632c0b: Bug 1649653 - fix compilation on non-profiler platforms. r=gerald, a=RyanVM
Cameron Kaiser <spectre@floodgap.com> - Wed, 01 Jul 2020 03:03:56 +0000 - rev 601892
Push 13342 by ryanvm@gmail.com at Thu, 02 Jul 2020 16:51:46 +0000
Bug 1649653 - fix compilation on non-profiler platforms. r=gerald, a=RyanVM Differential Revision: https://phabricator.services.mozilla.com/D81803
74444d8237d1342d94a5a0b094b550112ef7881d: Bug 1463035 - Remove MOZ_SIGNAL_TRAMPOLINE. r=gerald
Mike Hommey <mh+mozilla@glandium.org> - Fri, 26 Jun 2020 03:48:49 +0000 - rev 601636
Push 13310 by ffxbld-merge at Mon, 29 Jun 2020 14:50:06 +0000
Bug 1463035 - Remove MOZ_SIGNAL_TRAMPOLINE. r=gerald This was previously r=jchen and landed, but was backed out because some Android tests were running on an unrealistically old Linux kernel. These tests have been retired, so this can reland. Differential Revision: https://phabricator.services.mozilla.com/D81278
7c27fcb3b1832e5f83a6c50d74e1d14f666ae0cd: Bug 1648654 - Remove AutoObjectMapperFaultyLib. r=gerald
Mike Hommey <mh+mozilla@glandium.org> - Fri, 26 Jun 2020 01:26:56 +0000 - rev 601622
Push 13310 by ffxbld-merge at Mon, 29 Jun 2020 14:50:06 +0000
Bug 1648654 - Remove AutoObjectMapperFaultyLib. r=gerald Bug 1486524 disabled the features from the custom linker that required AutoObjectMapperFaultyLib. We can now rely on AutoObjectMapperPOSIX instead. Differential Revision: https://phabricator.services.mozilla.com/D81264
7556d4f93b29fa049ce14d614d4d1a478e9f80a6: Bug 1624993 - Transfer all the Android markers to platform side while capturing the profile r=gerald
Nazım Can Altınova <canaltinova@gmail.com> - Thu, 25 Jun 2020 14:29:29 +0000 - rev 601507
Push 13310 by ffxbld-merge at Mon, 29 Jun 2020 14:50:06 +0000
Bug 1624993 - Transfer all the Android markers to platform side while capturing the profile r=gerald Differential Revision: https://phabricator.services.mozilla.com/D79124
ecd086541b97f9904a4c18410839807f192e25b8: Bug 1624993 - Add StoreMarker function to unify the marker insertion points. r=gerald
Nazım Can Altınova <canaltinova@gmail.com> - Thu, 25 Jun 2020 14:28:54 +0000 - rev 601505
Push 13310 by ffxbld-merge at Mon, 29 Jun 2020 14:50:06 +0000
Bug 1624993 - Add StoreMarker function to unify the marker insertion points. r=gerald Differential Revision: https://phabricator.services.mozilla.com/D79423
4ab0751c4913405858b8df5e7cb5fb516c93b52c: Bug 1624993 - Add PutObjects method inside the ProfilerBuffer class as well. r=gerald
Nazım Can Altınova <canaltinova@gmail.com> - Thu, 25 Jun 2020 14:28:41 +0000 - rev 601504
Push 13310 by ffxbld-merge at Mon, 29 Jun 2020 14:50:06 +0000
Bug 1624993 - Add PutObjects method inside the ProfilerBuffer class as well. r=gerald Differential Revision: https://phabricator.services.mozilla.com/D79122
8be12844b53840e42fa3e2f46b46b0b806bb4631: Bug 1646453: Add MSCOM channel hook to enable profiler markers for COM IPC; r=Jamie,gerald
Aaron Klotz <aklotz@mozilla.com> - Fri, 19 Jun 2020 00:40:18 +0000 - rev 600519
Push 13310 by ffxbld-merge at Mon, 29 Jun 2020 14:50:06 +0000
Bug 1646453: Add MSCOM channel hook to enable profiler markers for COM IPC; r=Jamie,gerald To intercept COM IPC, we provide an `IChannelHook` interface to `CoRegisterChannelHook`, which gives us notifications about COM IPC that we can use to insert profiler markers. Note that `IChannelHook` is not documented on MSDN, however it is defined in the SDK header files. When the profiler is available, once XPCOM is up: * If the profiler is active, we immediately register the channel hook; * Otherwise we register an observer and hold off on registering the hook until the profiler is started, at which point we register the hook and remove the observer. Differential Revision: https://phabricator.services.mozilla.com/D80053
c7984ec2a543e2beb6965d8af7d7f59fcb3a078e: Bug 1646453: Add MSCOM channel hook to enable profiler markers for COM IPC; r=Jamie,gerald
Aaron Klotz <aklotz@mozilla.com> - Thu, 18 Jun 2020 17:17:24 +0000 - rev 600416
Push 13310 by ffxbld-merge at Mon, 29 Jun 2020 14:50:06 +0000
Bug 1646453: Add MSCOM channel hook to enable profiler markers for COM IPC; r=Jamie,gerald To intercept COM IPC, we provide an `IChannelHook` interface to `CoRegisterChannelHook`, which gives us notifications about COM IPC that we can use to insert profiler markers. Note that `IChannelHook` is not documented on MSDN, however it is defined in the SDK header files. When the profiler is available, once XPCOM is up: * If the profiler is active, we immediately register the channel hook; * Otherwise we register an observer and hold off on registering the hook until the profiler is started, at which point we register the hook and remove the observer. Differential Revision: https://phabricator.services.mozilla.com/D80053
1b97fdd559460d14d9566c5d047e5df32d5cff16: Bug 1645469 - Add profiler subcategories for SpiderMonkey JITs. r=jandem,gerald
Ted Campbell <tcampbell@mozilla.com> - Mon, 15 Jun 2020 14:56:50 +0000 - rev 599770
Push 13310 by ffxbld-merge at Mon, 29 Jun 2020 14:50:06 +0000
Bug 1645469 - Add profiler subcategories for SpiderMonkey JITs. r=jandem,gerald In the JIT frame sampler, we apply the appropriate category in addition to the "implementation" field. For JS frames (IS_JS_FRAME), we identify as either BaselineInterpreter or Interpreter. Note that JS_Other still applies to various places we enter SpiderMonkey outside of RunScript. Differential Revision: https://phabricator.services.mozilla.com/D79524
7c59c362deccc5a74943b28cecf40484ae088184: Bug 1645464 - Use a shared ProfilingCategoryList.h r=gerald,froydnj
Ted Campbell <tcampbell@mozilla.com> - Mon, 15 Jun 2020 13:59:55 +0000 - rev 599769
Push 13310 by ffxbld-merge at Mon, 29 Jun 2020 14:50:06 +0000
Bug 1645464 - Use a shared ProfilingCategoryList.h r=gerald,froydnj Replace the duplicate lists in mozglue/baseprofiler/public and js/public with a shared list. Add this list to both moz.build files so it is published twice which simplifies supporting different standalone configurations. Differential Revision: https://phabricator.services.mozilla.com/D79520