Closed Bug 1884293 Opened 7 months ago Closed 7 months ago

Disable PHC on aarch64 release channel

Categories

(Core :: Memory Allocator, task)

ARM64
Unspecified
task

Tracking

()

RESOLVED FIXED
125 Branch
Tracking Status
firefox125 --- fixed

People

(Reporter: pbone, Assigned: pbone)

References

(Blocks 1 open bug)

Details

Attachments

(1 file, 1 obsolete file)

We want to roll out PHC to more desktops, but let's exclude aarch64 for now because it has a performance problem (Bug 1874022). PHC will still be enabled for Nightly on aarch64.

Assignee: nobody → pbone
Status: NEW → ASSIGNED
Pushed by pbone@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/fe38c21e5d5b Disable PHC on aarch64 release channel r=gcp
Status: ASSIGNED → RESOLVED
Closed: 7 months ago
Resolution: --- → FIXED
Target Milestone: --- → 125 Branch
Attachment #9390668 - Flags: approval-mozilla-beta?

Uplift Approval Request

  • Code covered by automated testing: yes
  • Risk associated with taking this patch: Low risk
  • Needs manual QE test: no
  • Explanation of risk level: Change is to build settings only and will disable the PHC configuration for aarch64 builds in Late beta or later.
  • Is Android affected?: no
  • String changes made/needed: None
  • Fix verified in Nightly: no
  • User impact if declined: Delay in PHC rollout. We'd like to roll out PHC to 100% of the x86_64 desktop population, but we need to disable it for aarch64 first.
  • Steps to reproduce for manual QE testing: None
Attachment #9390668 - Flags: approval-mozilla-release?

Comment on attachment 9390668 [details]
Bug 1884293 - Disable PHC on aarch64 release channel

we are out of betas but leaving open in case of an RC respin or for the planned dot release

Attachment #9390668 - Flags: approval-mozilla-beta?
Regressions: 1885003

Comment on attachment 9390668 [details]
Bug 1884293 - Disable PHC on aarch64 release channel

Hi Dianna, I want to cancel the uplift request. Is removing the ? from approval-mozilla-release enough?

Thanks.

Flags: needinfo?(dsmith)
Attachment #9390668 - Flags: approval-mozilla-release?

A patch has been attached on this bug, which was already closed. Filing a separate bug will ensure better tracking. If this was not by mistake and further action is needed, please alert the appropriate party. (Or: if the patch doesn't change behavior -- e.g. landing a test case, or fixing a typo -- then feel free to disregard this message)

Attachment #9390668 - Flags: approval-mozilla-release?
Attachment #9390668 - Attachment is obsolete: true
Attachment #9390668 - Flags: approval-mozilla-release? → approval-mozilla-release-

Yes, you should also abandon any extra phab revision. I just did this now, so you're all set!

Flags: needinfo?(dsmith)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: