Networking: fold websocket and http tokio runtime into one. about servo HOT 9 CLOSED

But it seems it #31517 (comment) even more threads.

Yes that's correct, it uses a threadpool which will see threads created upfront, instead of spawning one thread for each image that is decoded.

I'm wondering if we could make whole net crate to share the same tokio runtime without manually spawning threads and thread pool.

We can use a single runtime for all async task, and a single core resource threadpool for the other non-async bacground work.

from servo.

@gterzian Could you add some good-first-issue or E-candidate-for-mentoring label?
This issue seems like a good chance for anyone who want to contribute.

from servo.

I would like to take this up.

from servo.

I saw your comment in zulip mentioned there's another PR dealing with thread spawning too. But it seems it created even more threads. I'm wondering if we could make whole net crate to share the same tokio runtime without manually spawning threads and thread pool.

from servo.

Hey @tannal what is your status with the issue. Can I join

from servo.

@MunishMummadi Yes you can.
The quick fix for this issue is done.

We can use a single runtime for all async task, and a single core resource threadpool for the other non-async bacground work.

You can open another pr to do this part if you are interested in this issue.

from servo.

a single core resource threadpool for the other non-async bacground work.

Yes indeed, this part would be good for a new PR @MunishMummadi

The idea is to use a single CoreResourceThreadPool in components/net, see for guidance this issue, which added a second one. You could make changes to have a single pool used where now two are used.

from servo.

Yes I would love to work on it @gterzian @tannal . Can I ask the bot to "assign me"!!!

from servo.

Actually, because the image cache runs in the script thread, not in the core resource thread, this is quite complicated and perhaps not a good idea. I withdraw my earlier conclusion.

from servo.