Bug 225660 - [BMalloc] Temporarily disable bmalloc in debug builds while examining 76167814
Summary: [BMalloc] Temporarily disable bmalloc in debug builds while examining 76167814
Status: RESOLVED WONTFIX
Alias: None
Product: WebKit
Classification: Unclassified
Component: bmalloc (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Dean Jackson
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2021-05-11 10:22 PDT by Dean Jackson
Modified: 2021-12-12 08:43 PST (History)
3 users (show)

See Also:


Attachments
Patch (1.80 KB, patch)
2021-05-11 10:30 PDT, Dean Jackson
ews-feeder: commit-queue-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Dean Jackson 2021-05-11 10:22:07 PDT
[BMalloc] Temporarily disable bmalloc in debug builds while examining 76167814
Comment 1 Radar WebKit Bug Importer 2021-05-11 10:24:29 PDT
<rdar://problem/77852400>
Comment 2 Geoffrey Garen 2021-05-11 10:26:58 PDT
Kinda sketchy to lose debug testing coverage for bmalloc and gigacage. Maybe release coverage is sufficient? Maybe.
Comment 3 Dean Jackson 2021-05-11 10:30:11 PDT
Created attachment 428292 [details]
Patch
Comment 4 Dean Jackson 2021-05-11 10:32:50 PDT
(In reply to Geoffrey Garen from comment #2)
> Kinda sketchy to lose debug testing coverage for bmalloc and gigacage. Maybe
> release coverage is sufficient? Maybe.

The original bug hasn't been touched in weeks, which is more sketchy :)
Comment 5 Dean Jackson 2021-05-11 10:33:39 PDT
(In reply to Dean Jackson from comment #4)
> (In reply to Geoffrey Garen from comment #2)
> > Kinda sketchy to lose debug testing coverage for bmalloc and gigacage. Maybe
> > release coverage is sufficient? Maybe.
> 
> The original bug hasn't been touched in weeks, which is more sketchy :)

Sorry, I meant months.
Comment 6 Chris Dumez 2021-05-11 11:03:08 PDT
Comment on attachment 428292 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=428292&action=review

> Source/bmalloc/ChangeLog:3
> +        [BMalloc] Temporarily disable bmalloc in debug builds while examining 76167814

Why can't people use the environment variables to disable bmalloc when debugging? Disabling BMalloc for everybody in debug seems aggressive.
Comment 7 Dean Jackson 2021-05-11 11:05:05 PDT
(In reply to Chris Dumez from comment #6)
> Comment on attachment 428292 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=428292&action=review
> 
> > Source/bmalloc/ChangeLog:3
> > +        [BMalloc] Temporarily disable bmalloc in debug builds while examining 76167814
> 
> Why can't people use the environment variables to disable bmalloc when
> debugging? Disabling BMalloc for everybody in debug seems aggressive.

Because it is difficult to do this for the simulator, and people will have to remember to undo the workaround. Seems better to track this officially.

Well, the best would be to address the real bug, which is almost certainly in the toolchain :)
Comment 8 Chris Dumez 2021-05-11 11:08:08 PDT
We're loosing useful test coverage on bots and local machines just for the purpose of making the debugger work. It doesn't seem like a good trade-offs. We want to be running something as close as possible to the configuration we ship.
Comment 9 Dean Jackson 2021-05-12 09:06:46 PDT
(In reply to Chris Dumez from comment #8)
> We're loosing useful test coverage on bots and local machines just for the
> purpose of making the debugger work. It doesn't seem like a good trade-offs.
> We want to be running something as close as possible to the configuration we
> ship.

We don't ship debug builds.

But to be clear, I don't want to do this. I want the actual bug to be examined - it's been more than a month and nothing has happened.

I should remove the r? flag because I don't expect anyone will want to land this, but it serves as a patch that people can apply to get things working locally.