1239961 - Do less PR_Poll during shutdown

Status: RESOLVED FIXED

(Core :: Networking, defect)

Description

[Dragana Damjanovic [:dragana]]

We can also do less PR_Poll during shutdown.

We do not need to do PR_Connect, PR_ConnectContinue and also PR_Poll during shutdown (at least as soon as nsIOService knows that we are in shutdown)

Also nsHttpConnection::Close does PR_Read to take all data from the socket sockets so that we do not cause TCP RST. We could also skip that.

Only thing left should be PR_Close and less calls that can hang :)

Comment 1

[Dragana Damjanovic [:dragana]]

Looking at the code, nsHttpConnectionMgr posts nsHttpConnectionMgr::OnMsgShutdown event on socketThread. This needs to be done before socketThread shuts down, but right after that we can start shutting down thread, I think.

nsHttpConnectionMgr::OnMsgShutdown closes all nsHttpConnection so only connections left are non http connection (e.g. udp).
So DoPollIteration coming after that function will detach all socket (we will do that also during socketThread shutdown) and poll for non http connection (I have not look at them jet).
To optimize this I will only prevent PR_Poll call (and let socketThread call detachSocket while main thread is still busy with some other stuff and not do thate later while main thread is waiting for it).

Comment 2

[Dragana Damjanovic [:dragana]]

Attachment: bug_1239961_v1.patch

Comment 3

[Dragana Damjanovic [:dragana]]

Attachment: bug_1239961_v1.patch

Comment 4

[Dragana Damjanovic [:dragana]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=574c0eb67f3e

Comment 5

[Patrick McManus [:mcmanus]]

Comment on attachment 8708438 [details] [diff] [review]
bug_1239961_v1.patch

Review of attachment 8708438 [details] [diff] [review]:
-----------------------------------------------------------------

I think all of these changes will really help in aggregate. thanks!

Comment 6

[Dragana Damjanovic [:dragana]]

The next step would be to limit number of PR_Close calls.
Something like: if we are in shutdown already for more than 3-4min do not call PR_Close let them leak.
Probably this can be done in: http://mxr.mozilla.org/mozilla-central/source/netwerk/base/nsSocketTransport2.cpp#1734

Comment 7

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/c12b2d098ae0

Comment 8

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/c12b2d098ae0