searching for reviewer(gerald)
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 554916
Push 37899 by malexandru@mozilla.com at Wed, 28 Oct 2020 21:47:27 +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 554879
Push 37899 by malexandru@mozilla.com at Wed, 28 Oct 2020 21:47:27 +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 554664
Push 37897 by ncsoregi@mozilla.com at Tue, 27 Oct 2020 21:20:52 +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 554663
Push 37897 by ncsoregi@mozilla.com at Tue, 27 Oct 2020 21:20:52 +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 554662
Push 37897 by ncsoregi@mozilla.com at Tue, 27 Oct 2020 21:20:52 +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 554571
Push 37894 by dluca@mozilla.com at Tue, 27 Oct 2020 04:41:26 +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 554570
Push 37894 by dluca@mozilla.com at Tue, 27 Oct 2020 04:41:26 +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 554569
Push 37894 by dluca@mozilla.com at Tue, 27 Oct 2020 04:41:26 +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 553904
Push 37885 by apavel@mozilla.com at Thu, 22 Oct 2020 14:54:09 +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 553899
Push 37885 by apavel@mozilla.com at Thu, 22 Oct 2020 14:54:09 +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 552801
Push 37858 by btara@mozilla.com at Tue, 13 Oct 2020 21:43:51 +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 552553
Push 37850 by ncsoregi@mozilla.com at Sat, 10 Oct 2020 09:37:08 +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 551716
Push 37840 by rmaries@mozilla.com at Wed, 07 Oct 2020 09:42:23 +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 551715
Push 37840 by rmaries@mozilla.com at Wed, 07 Oct 2020 09:42:23 +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 551537
Push 37837 by nbeleuzu@mozilla.com at Mon, 05 Oct 2020 21:58:09 +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 550916
Push 37824 by csabou@mozilla.com at Thu, 01 Oct 2020 09:40:20 +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 550357
Push 37812 by cbrindusan@mozilla.com at Fri, 25 Sep 2020 21:47:43 +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 550355
Push 37812 by cbrindusan@mozilla.com at Fri, 25 Sep 2020 21:47:43 +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 549535
Push 37801 by malexandru@mozilla.com at Tue, 22 Sep 2020 03:47:42 +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
1aad6539812fd0e9c6697bf62c1c6389fdc9b2d6: Bug 1665945 - Fix compilation. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Sat, 19 Sep 2020 01:04:27 +0000 - rev 549360
Push 37795 by ccoroiu@mozilla.com at Sat, 19 Sep 2020 09:34:53 +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 545284
Push 37712 by btara@mozilla.com at Wed, 19 Aug 2020 21:28:29 +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 544430
Push 37694 by dluca@mozilla.com at Wed, 12 Aug 2020 21:49:48 +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 544233
Push 37691 by malexandru@mozilla.com at Tue, 11 Aug 2020 21:47:38 +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 542693
Push 37654 by csabou@mozilla.com at Fri, 31 Jul 2020 04:11:13 +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 542692
Push 37654 by csabou@mozilla.com at Fri, 31 Jul 2020 04:11:13 +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 542691
Push 37654 by csabou@mozilla.com at Fri, 31 Jul 2020 04:11:13 +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 541536
Push 37625 by csabou@mozilla.com at Wed, 22 Jul 2020 04:32:41 +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 541535
Push 37625 by csabou@mozilla.com at Wed, 22 Jul 2020 04:32:41 +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 540596
Push 37605 by btara@mozilla.com at Thu, 16 Jul 2020 03:30:51 +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 538355
Push 37561 by ncsoregi@mozilla.com at Thu, 02 Jul 2020 09:46:06 +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 538136
Push 37558 by dluca@mozilla.com at Wed, 01 Jul 2020 09:30:12 +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 538067
Push 37557 by abutkovits@mozilla.com at Wed, 01 Jul 2020 03:27: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 538003
Push 37556 by abutkovits@mozilla.com at Tue, 30 Jun 2020 21:45:13 +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 538002
Push 37556 by abutkovits@mozilla.com at Tue, 30 Jun 2020 21:45:13 +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 537983
Push 37556 by abutkovits@mozilla.com at Tue, 30 Jun 2020 21:45:13 +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 537982
Push 37556 by abutkovits@mozilla.com at Tue, 30 Jun 2020 21:45:13 +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 537859
Push 37554 by nbeleuzu@mozilla.com at Tue, 30 Jun 2020 10:00:42 +0000
Bug 1647911 - Use text markers for payload presented markers. r=gerald Differential Revision: https://phabricator.services.mozilla.com/D80946
74444d8237d1342d94a5a0b094b550112ef7881d: Bug 1463035 - Remove MOZ_SIGNAL_TRAMPOLINE. r=gerald
Mike Hommey <mh+mozilla@glandium.org> - Fri, 26 Jun 2020 03:48:49 +0000 - rev 537551
Push 37544 by dluca@mozilla.com at Fri, 26 Jun 2020 09:45:40 +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 537537
Push 37544 by dluca@mozilla.com at Fri, 26 Jun 2020 09:45:40 +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 537422
Push 37542 by abutkovits@mozilla.com at Thu, 25 Jun 2020 21:45:10 +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 537420
Push 37542 by abutkovits@mozilla.com at Thu, 25 Jun 2020 21:45:10 +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 537419
Push 37542 by abutkovits@mozilla.com at Thu, 25 Jun 2020 21:45:10 +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 536434
Push 37520 by dluca@mozilla.com at Fri, 19 Jun 2020 04:04:08 +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 536331
Push 37520 by dluca@mozilla.com at Fri, 19 Jun 2020 04:04:08 +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 535685
Push 37508 by btara@mozilla.com at Mon, 15 Jun 2020 21:48:38 +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 535684
Push 37508 by btara@mozilla.com at Mon, 15 Jun 2020 21:48:38 +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
c2f99008a2a00e208a20b51e9586ff9cbc054095: Bug 1626918 - Add categories for real-time media tracing. r=gerald
Paul Adenot <paul@paul.cx> - Fri, 12 Jun 2020 13:13:53 +0000 - rev 535469
Push 37501 by nbeleuzu@mozilla.com at Sat, 13 Jun 2020 03:21:52 +0000
Bug 1626918 - Add categories for real-time media tracing. r=gerald Differential Revision: https://phabricator.services.mozilla.com/D78509
9f3bca4deddef935d54eee48342273736b0a4e64: Bug 1626918 - Start and stop audio callback tracing when the profiler starts and stops. r=gerald
Paul Adenot <paul@paul.cx> - Fri, 12 Jun 2020 13:13:35 +0000 - rev 535468
Push 37501 by nbeleuzu@mozilla.com at Sat, 13 Jun 2020 03:21:52 +0000
Bug 1626918 - Start and stop audio callback tracing when the profiler starts and stops. r=gerald Differential Revision: https://phabricator.services.mozilla.com/D78508
48f24a4445a1d9e2d9cc60dd2028f2d383d07b8b: Bug 1626918 - Add two missing constructors to TracingMarkerPayload. r=gerald
Paul Adenot <paul@paul.cx> - Fri, 12 Jun 2020 13:13:28 +0000 - rev 535467
Push 37501 by nbeleuzu@mozilla.com at Sat, 13 Jun 2020 03:21:52 +0000
Bug 1626918 - Add two missing constructors to TracingMarkerPayload. r=gerald Differential Revision: https://phabricator.services.mozilla.com/D78507
fca3a809b80526eb38ba5462c7d6b8ed5d62ab11: Bug 1626918 - Teach AsyncLogger to output profiler markers. r=gerald
Paul Adenot <paul@paul.cx> - Fri, 12 Jun 2020 13:12:59 +0000 - rev 535463
Push 37501 by nbeleuzu@mozilla.com at Sat, 13 Jun 2020 03:21:52 +0000
Bug 1626918 - Teach AsyncLogger to output profiler markers. r=gerald Differential Revision: https://phabricator.services.mozilla.com/D78501