137282 2014-09-30 18:04:52 -0700 [SOUP] Consider reducing max simultaneous connections 2016-01-07 07:00:40 -0800 1 1 1 Unclassified WebKit Platform 528+ (Nightly build) PC Linux RESOLVED FIXED P5 Enhancement --- 1 mcatanzaro mcatanzaro berto cgarcia commit-queue danw gustavo mcatanzaro mrobinson svillar oldest_to_newest 1038426 0 mcatanzaro 2014-09-30 18:04:52 -0700 I noticed this code in Source/WebCore/platform/network/soup/SoupNetworkSession.cpp: // Values taken from http://www.browserscope.org/ following // the rule "Do What Every Other Modern Browser Is Doing". They seem // to significantly improve page loading time compared to soup's // default values. static const int maxConnections = 35; static const int maxConnectionsPerHost = 6; I checked Browserscope [1], and Firefox, Internet Explorer, and Safari all allow 17 connections nowadays. Chrome only allows 10. I have no clue if reducing max connections will improve performance, but I like the sound of "Do What Every Other Modern Browser Is Doing".... [1] http://www.browserscope.org/?category=network&v=top 1038495 1 svillar 2014-10-01 00:15:44 -0700 (In reply to comment #0) > I noticed this code in Source/WebCore/platform/network/soup/SoupNetworkSession.cpp: > > // Values taken from http://www.browserscope.org/ following > // the rule "Do What Every Other Modern Browser Is Doing". They seem > // to significantly improve page loading time compared to soup's > // default values. > static const int maxConnections = 35; > static const int maxConnectionsPerHost = 6; > > I checked Browserscope [1], and Firefox, Internet Explorer, and Safari all allow 17 connections nowadays. Chrome only allows 10. I have no clue if reducing max connections will improve performance, but I like the sound of "Do What Every Other Modern Browser Is Doing".... > > [1] http://www.browserscope.org/?category=network&v=top In general a higher amount of max_connections allows more parallelism, and thus faster downloads. Said that, the only reason to limit that number (apart from not overflowing your router's NAT) is to be "nice" with the servers and the other clients. There should be a reason why they lowered that number, perhaps server admins are actually reducing that number. In any case I'm pretty sure that if FF, IE and Safari have a lower number right now it's because they have deeply studied all the consequences and concluded that having half the connections is better. 1097110 2 mcatanzaro 2015-05-23 11:41:20 -0700 (In reply to comment #1) > In general a higher amount of max_connections allows more parallelism, and > thus faster downloads. Said that, the only reason to limit that number > (apart from not overflowing your router's NAT) is to be "nice" with the > servers and the other clients. I presume that major browsers use a lower number because having more connections open could slow them down, and they prefer to fully load a few resources and then fully load a few more rather than have many loading at the same time. It probably makes the browser feel "faster." 1153077 3 268374 mcatanzaro 2016-01-06 10:26:49 -0800 Created attachment 268374 Patch 1153404 4 268374 commit-queue 2016-01-07 07:00:36 -0800 Comment on attachment 268374 Patch Clearing flags on attachment: 268374 Committed r194697: <http://trac.webkit.org/changeset/194697> 1153405 5 commit-queue 2016-01-07 07:00:40 -0800 All reviewed patches have been landed. Closing bug. 268374 2016-01-06 10:26:49 -0800 2016-01-07 07:00:36 -0800 Patch bug-137282-20160106122616.patch text/plain 1795 mcatanzaro U3VidmVyc2lvbiBSZXZpc2lvbjogMTk0NjI0CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggODRlMjRmYTI0OGM3ZGIx NmQ2YjE5MGNmZjkxMWI2MjllNDkwNTE1Yi4uMTY0NjQyNzg2OTdlZTY1ZTM1NzA0NTU1YTcwZTNi YmQ0MDAzYzEyNyAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE3IEBACisyMDE2LTAxLTA2ICBNaWNo YWVsIENhdGFuemFybyAgPG1jYXRhbnphcm9AaWdhbGlhLmNvbT4KKworICAgICAgICBbU09VUF0g Q29uc2lkZXIgcmVkdWNpbmcgbWF4IHNpbXVsdGFuZW91cyBjb25uZWN0aW9ucworICAgICAgICBo dHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTM3MjgyCisKKyAgICAgICAg UmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgUmVkdWNlIG1heCBzaW11bHRh bmVvdXMgY29ubmVjdGlvbnMgZnJvbSAzNSB0byAxNywgYmVjYXVzZSAoYSkgYWxsIG1ham9yIGJy b3dzZXJzIGV4Y2VwdAorICAgICAgICBDaHJvbWUgdXNlIDE3LCBhbmQgKGIpIENocm9tZSB1c2Vz IDEwLiBUaGUgb25seSByZWFzb24gd2UgcHJldmlvdXNseSB1c2VkIDM1IHdhcyB0aGF0IG90aGVy CisgICAgICAgIGJyb3dzZXJzIHdlcmUgdXNpbmcgMzUgYXQgdGhlIHRpbWUuCisKKyAgICAgICAg KiBwbGF0Zm9ybS9uZXR3b3JrL3NvdXAvU291cE5ldHdvcmtTZXNzaW9uLmNwcDoKKyAgICAgICAg KFdlYkNvcmU6OlNvdXBOZXR3b3JrU2Vzc2lvbjo6U291cE5ldHdvcmtTZXNzaW9uKToKKwogMjAx Ni0wMS0wNSAgUHJhbmphbCBKdW1kZSAgPHBqdW1kZUBhcHBsZS5jb20+CiAKICAgICAgICAgSW1h Z2Ugc2hvdWxkIG5vdCBiZSByZS1yZWdpc3RlcmVkIGlmIG1fZm9ybSBhbHJlYWR5IGV4aXN0cy4g VGhpcyBsZWFkcyB0byBhbiBhc3NlcnRpb24gZmFpbHVyZS4KZGlmZiAtLWdpdCBhL1NvdXJjZS9X ZWJDb3JlL3BsYXRmb3JtL25ldHdvcmsvc291cC9Tb3VwTmV0d29ya1Nlc3Npb24uY3BwIGIvU291 cmNlL1dlYkNvcmUvcGxhdGZvcm0vbmV0d29yay9zb3VwL1NvdXBOZXR3b3JrU2Vzc2lvbi5jcHAK aW5kZXggOGJlZWJmYWUyNjJjMjU4NTZjZDgyN2MyZjNkNDhlZjRiY2UxMGM1Mi4uOGM2ODRmYTE4 ZmViODZhYjk2NzI2ZTU4N2FhZDRmMzg1ODhiNDllYiAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNv cmUvcGxhdGZvcm0vbmV0d29yay9zb3VwL1NvdXBOZXR3b3JrU2Vzc2lvbi5jcHAKKysrIGIvU291 cmNlL1dlYkNvcmUvcGxhdGZvcm0vbmV0d29yay9zb3VwL1NvdXBOZXR3b3JrU2Vzc2lvbi5jcHAK QEAgLTk3LDcgKzk3LDcgQEAgU291cE5ldHdvcmtTZXNzaW9uOjpTb3VwTmV0d29ya1Nlc3Npb24o U291cENvb2tpZUphciogY29va2llSmFyKQogICAgIC8vIHRoZSBydWxlICJEbyBXaGF0IEV2ZXJ5 IE90aGVyIE1vZGVybiBCcm93c2VyIElzIERvaW5nIi4gVGhleSBzZWVtCiAgICAgLy8gdG8gc2ln bmlmaWNhbnRseSBpbXByb3ZlIHBhZ2UgbG9hZGluZyB0aW1lIGNvbXBhcmVkIHRvIHNvdXAncwog ICAgIC8vIGRlZmF1bHQgdmFsdWVzLgotICAgIHN0YXRpYyBjb25zdCBpbnQgbWF4Q29ubmVjdGlv bnMgPSAzNTsKKyAgICBzdGF0aWMgY29uc3QgaW50IG1heENvbm5lY3Rpb25zID0gMTc7CiAgICAg c3RhdGljIGNvbnN0IGludCBtYXhDb25uZWN0aW9uc1Blckhvc3QgPSA2OwogCiAgICAgZ19vYmpl Y3Rfc2V0KG1fc291cFNlc3Npb24uZ2V0KCksCg==