Bug 230889

Summary: Fix rounding issue in run-jsc-stress-tests
Product: WebKit Reporter: Geza Lore <glore>
Component: New BugsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: aperez, webkit-bug-importer, ysuzuki
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Geza Lore
Reported 2021-09-28 04:10:54 PDT
Fix rounding issue in run-jsc-stress-tests
Attachments
Patch (1.28 KB, patch)
2021-09-28 04:17 PDT, Geza Lore 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Geza Lore
Comment 1 2021-09-28 04:17:51 PDT
Created attachment 439455 [details] Patch
Geza Lore
Comment 2 2021-09-28 04:22:09 PDT
Sometimes ceil(N/K) == ceil(N/(K-1)), plus Ruby Array.slice gives you a nil if the start offset is past the end. This fixes the crashes arising when you did hit that rare combination of N (number of tests) and K (number of parallel processes).
EWS
Comment 3 2021-09-28 05:44:17 PDT
Committed r283164 (242212@main): <https://commits.webkit.org/242212@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 439455 [details].
Radar WebKit Bug Importer
Comment 4 2021-09-28 05:45:16 PDT
<rdar://problem/83619026>
Note You need to log in before you can comment on or make changes to this bug.