Bug 230889

Summary:

Fix rounding issue in run-jsc-stress-tests

Product:

WebKit

Reporter:

Geza Lore <glore>

Component:

New Bugs

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

aperez, webkit-bug-importer, ysuzuki

Priority:

Keywords:

InRadar

Version:

WebKit Nightly Build

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Patch	none

Description Geza Lore 2021-09-28 04:10:54 PDT

Fix rounding issue in run-jsc-stress-tests

Comment 1 Geza Lore 2021-09-28 04:17:51 PDT

Created attachment 439455 [details]
Patch

Comment 2 Geza Lore 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).

Comment 3 EWS 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].

Comment 4 Radar WebKit Bug Importer 2021-09-28 05:45:16 PDT

<rdar://problem/83619026>