Closed Bug 777381 Opened 13 years ago Closed 13 years ago

B2G HAL SIGUSR2 interferes with profiling

Categories

(Core :: Gecko Profiler, defect)

x86
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla17

People

(Reporter: BenWa, Assigned: jrmuizel)

Details

Attachments

(1 file)

Right now we can't use the profiler out of the box on b2g because SIGUSR2 is also used by B2G. The reason we use SIGUSR2 is because SIGUSR1 is used by the dalvik VM. Since we have the flexibility on B2G can we change that signal number?
Yes, we can grab one of the SIGRT's.
Actually, we could use SIGUSR1, since we don't have dalvik.
Attached patch Use SIGUSR1Splinter Review
Attachment #645877 - Flags: review?(jones.chris.g)
Comment on attachment 645877 [details] [diff] [review] Use SIGUSR1 hg blame says that I pulled SIGUSR2 out of my butt in bug 749551 comment 53, so I don't see anything wrong a priori with switching to SIGUSR1 if it's indeed dalvik using that and not bionic.
Attachment #645877 - Flags: review?(jones.chris.g) → review+
Assignee: nobody → jmuizelaar
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla17
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: