RESOLVED FIXED 166663
Use prctl to name thread on Linux 
https://bugs.webkit.org/show_bug.cgi?id=166663
Summary Use prctl to name thread on Linux
Yusuke Suzuki
Reported 2017-01-03 14:22:40 PST
Use prctl to name thread on Linux
Attachments
Patch (1.43 KB, patch)
2017-01-03 14:24 PST, Yusuke Suzuki 		mcatanzaro: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Yusuke Suzuki
Comment 1 2017-01-03 14:24:00 PST
Created attachment 297953 [details] Patch
Michael Catanzaro
Comment 2 2017-01-03 15:26:16 PST
Comment on attachment 297953 [details] Patch OK, I don't see why not. Did you do this to help Carlos keep track of which thread is the main thread? :D What do our thread names look like? I see from prctl(2) that they are truncated to 16 characters; I wonder if our names are often longer than that?
Yusuke Suzuki
Comment 3 2017-01-03 23:37:58 PST
(In reply to comment #2) > Comment on attachment 297953 [details] > Patch > > OK, I don't see why not. Did you do this to help Carlos keep track of which > thread is the main thread? :D > > What do our thread names look like? I see from prctl(2) that they are > truncated to 16 characters; I wonder if our names are often longer than that? Yeah, characters are truncated. The typical form is like, jsc.watchdog.queue org.webkit.ThreadedCompositorWorkQueue So I think we should retrieve trailing 16 characters.
Yusuke Suzuki
Comment 4 2017-01-04 00:27:04 PST
(In reply to comment #3) > (In reply to comment #2) > > Comment on attachment 297953 [details] > > Patch > > > > OK, I don't see why not. Did you do this to help Carlos keep track of which > > thread is the main thread? :D > > > > What do our thread names look like? I see from prctl(2) that they are > > truncated to 16 characters; I wonder if our names are often longer than that? > > Yeah, characters are truncated. The typical form is like, > > jsc.watchdog.queue > org.webkit.ThreadedCompositorWorkQueue > > So I think we should retrieve trailing 16 characters. I'll just port WorkQueue::platformInitialize's threadName handling.
Yusuke Suzuki
Comment 5 2017-01-04 00:27:57 PST
(In reply to comment #4) > (In reply to comment #3) > > (In reply to comment #2) > > > Comment on attachment 297953 [details] > > > Patch > > > > > > OK, I don't see why not. Did you do this to help Carlos keep track of which > > > thread is the main thread? :D > > > > > > What do our thread names look like? I see from prctl(2) that they are > > > truncated to 16 characters; I wonder if our names are often longer than that? > > > > Yeah, characters are truncated. The typical form is like, > > > > jsc.watchdog.queue > > org.webkit.ThreadedCompositorWorkQueue > > > > So I think we should retrieve trailing 16 characters. > > I'll just port WorkQueue::platformInitialize's threadName handling. I'll do that in different patch :)
Yusuke Suzuki
Comment 6 2017-01-04 00:30:52 PST
Committed r210272: <http://trac.webkit.org/changeset/210272>
Yusuke Suzuki
Comment 7 2017-01-04 01:44:32 PST
(In reply to comment #2) > Comment on attachment 297953 [details] > Patch > > OK, I don't see why not. Did you do this to help Carlos keep track of which > thread is the main thread? :D > > What do our thread names look like? I see from prctl(2) that they are > truncated to 16 characters; I wonder if our names are often longer than that? I found it when debugging WorkQueue thing. I'm super happy if it would help Carlos :)
Note You need to log in before you can comment on or make changes to this bug.