diff options
| author | yvt <i@yvt.jp> | 2021-07-19 00:50:04 +0900 |
|---|---|---|
| committer | yvt <i@yvt.jp> | 2021-07-19 00:57:48 +0900 |
| commit | 41b372627161c8473222d0cd83e5d904ba8c1799 (patch) | |
| tree | 9d447d591a4f8f311cef1664aa7df9ee46862e06 /components/canvas/canvas_paint_thread.rs | |
| parent | e8cb9f56e3fd0273d8984bc89639723c4f1d5482 (diff) | |
| download | servo-41b372627161c8473222d0cd83e5d904ba8c1799.tar.gz servo-41b372627161c8473222d0cd83e5d904ba8c1799.zip | |
feat: shorten thread names
The Linux kernel imposes a 15-byte limit on thread names[1]. This means
information that does not fit in this limit, e.g., the pipeline ID of
layout and script threads, is lost in a debugger and profiler (see the
first column of the table below).
This commit shortens the thread names used in Servo to maximize the
amount of information conveyed. It also rectifies some inconsistencies
in the names.
| Before | After |
|-------------------|-------------------|
| `BluetoothThread` | `Bluetooth` |
| `CanvasThread` | `Canvas` |
| `display alert d` | `AlertDialog` |
| `FontCacheThread` | `FontCache` |
| `GLPlayerThread` | `GLPlayer` |
| `HTML Parser` | `Parse:www.examp` |
| `LayoutThread Pi` | `Layout(1,1)` |
| `Memory profiler` | `MemoryProfiler` |
| `Memory profiler` | `MemoryProfTimer` |
| `OfflineAudioCon` | `OfflineACResolv` |
| `PullTimelineMar` | `PullTimelineDat` |
| `ScriptThread Pi` | `Script(1,1)` |
| `WebWorker for h` | `WW:www.example.` |
| `ServiceWorker f` | `SW:www.example.` |
| `ServiceWorkerMa` | `SvcWorkerManage` |
| `Time profiler t` | `TimeProfTimer` |
| `Time profiler` | `TimeProfiler` |
| `WebGL thread` | `WebGL` |
| `Choose a device` | `DevicePicker` |
| `Pick a file` | `FilePicker` |
| `Pick files` | `FilePicker` |
[1]: https://stackoverflow.com/questions/5026531/thread-name-longer-than-15-chars
Diffstat (limited to 'components/canvas/canvas_paint_thread.rs')
| -rw-r--r-- | components/canvas/canvas_paint_thread.rs | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/components/canvas/canvas_paint_thread.rs b/components/canvas/canvas_paint_thread.rs index b426d398a72..f7c1d38b76f 100644 --- a/components/canvas/canvas_paint_thread.rs +++ b/components/canvas/canvas_paint_thread.rs @@ -62,7 +62,7 @@ impl<'a> CanvasPaintThread<'a> { let msg_receiver = ROUTER.route_ipc_receiver_to_new_crossbeam_receiver(ipc_receiver); let (create_sender, create_receiver) = unbounded(); thread::Builder::new() - .name("CanvasThread".to_owned()) + .name("Canvas".to_owned()) .spawn(move || { let mut canvas_paint_thread = CanvasPaintThread::new(webrender_api, font_cache_thread); loop { |