Revert "[heap] Add notes for preempting/finishing jobs" [v8/v8 : main]
0 views
Skip to first unread message
Omer Katz (Gerrit)
Dec 17, 2025, 9:16:53 AM (11 days ago) Dec 17
to Rubber Stamper, Michael Lippautz, V8 LUCI CQ, AyeAye, Hannes Payer, oilpan-r...@chromium.org, mlippau...@chromium.org
Attention needed from Michael Lippautz
Omer Katz voted and added 1 comment![]( "Open in Gerrit")
Votes added by Omer Katz
| Auto-Submit | +1 |
| Commit-Queue | +1 |
Related details
Attention is currently required from:
- Michael Lippautz
Submit Requirements:
Code-Owners
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Rubber Stamper (Gerrit)
Dec 17, 2025, 9:17:12 AM (11 days ago) Dec 17
to Omer Katz, Michael Lippautz, V8 LUCI CQ, AyeAye, Hannes Payer, oilpan-r...@chromium.org, mlippau...@chromium.org
Attention needed from Michael Lippautz
Rubber Stamper voted![]( "Open in Gerrit")
| Bot-Commit | +1 |
| Commit-Queue | +2 |
Related details
Attention is currently required from:
- Michael Lippautz
Submit Requirements:
Code-Owners
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Michael Lippautz (Gerrit)
Dec 17, 2025, 9:18:50 AM (11 days ago) Dec 17
to Omer Katz, Rubber Stamper, V8 LUCI CQ, AyeAye, Hannes Payer, oilpan-r...@chromium.org, mlippau...@chromium.org
Attention needed from Omer Katz
Michael Lippautz voted Code-Review+1![]( "Open in Gerrit")
Attention is currently required from:
- Omer Katz
Submit Requirements:
Code-Owners
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
V8 LUCI CQ (Gerrit)
Dec 17, 2025, 9:56:14 AM (11 days ago) Dec 17
to Omer Katz, Michael Lippautz, Rubber Stamper, AyeAye, Hannes Payer, oilpan-r...@chromium.org, mlippau...@chromium.org
V8 LUCI CQ submitted the change![]( "Open in Gerrit")
Change information
Commit message:
Revert "[heap] Add notes for preempting/finishing jobs"
This reverts commit d61897cc21fe643b7378f538464dae866fb4841b.
Reason for revert: Speculative revert due to crashes in cppgc starting at the revision in which this CL landed (e.g. crbug.com/469686890)
Original change's description:
> [heap] Add notes for preempting/finishing jobs
>
> Help distinguish on a trace which concurrent jobs end because of
> preemption and which end because they run out of work.
> Notes are added only to jobs that can yield.
>
> Bug: 42204085
> Change-Id: If3064174cc6ccfda50230b906589f456983e46a9
> Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/7261997
> Reviewed-by: Michael Lippautz <mlip...@chromium.org>
> Commit-Queue: Omer Katz <omer...@chromium.org>
> Cr-Commit-Position: refs/heads/main@{#104328}
Bug: 42204085
Change-Id: I3b68b5863d9d69ed3f47964b134e289f40856c3a
Auto-Submit: Omer Katz <omer...@chromium.org>
Commit-Queue: Rubber Stamper <rubber-...@appspot.gserviceaccount.com>
Bot-Commit: Rubber Stamper <rubber-...@appspot.gserviceaccount.com>
Commit-Queue: Omer Katz <omer...@chromium.org>
Reviewed-by: Michael Lippautz <mlip...@chromium.org>
Cr-Commit-Position: refs/heads/main@{#104373}
Files:
- M src/heap/array-buffer-sweeper.cc
- M src/heap/concurrent-marking.cc
- M src/heap/cppgc/concurrent-marker.cc
- M src/heap/cppgc/stats-collector.h
- M src/heap/cppgc/sweeper.cc
- M src/heap/scavenger.cc
- M src/heap/sweeper.cc
- M src/heap/sweeper.h
Change size: S
Delta: 8 files changed, 3 insertions(+), 34 deletions(-)
Branch: refs/heads/main
Submit Requirements:
Code-Review: +1 by Michael Lippautz, Bot-Commit+1 by Rubber Stamper
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Reply all
Reply to author
Forward
0 new messages