155260 2016-03-09 14:10:46 -0800 use ulimit command to check process limit in webkitpy 2016-03-09 15:07:42 -0800 1 1 1 Unclassified WebKit Tools / Tests WebKit Local Build Unspecified Unspecified RESOLVED FIXED P2 Normal --- 1 aakash_jain aakash_jain aakash_jain ap commit-queue glenn lforschler oldest_to_newest 1172754 0 aakash_jain 2016-03-09 14:10:46 -0800 We currently use "launchctl limit maxproc" command to check the process limit, this is sometimes different from the limit reported by "ulimit -u" command. Output by ulimit command seems to be more reliable (the one on which we actually start hitting process limit and start getting fork errors). We should use ulimit command output. 1172758 1 273474 aakash_jain 2016-03-09 14:14:55 -0800 Created attachment 273474 Proposed patch 1172807 2 273474 commit-queue 2016-03-09 15:07:38 -0800 Comment on attachment 273474 Proposed patch Clearing flags on attachment: 273474 Committed r197891: <http://trac.webkit.org/changeset/197891> 1172808 3 commit-queue 2016-03-09 15:07:42 -0800 All reviewed patches have been landed. Closing bug. 273474 2016-03-09 14:14:55 -0800 2016-03-09 15:07:38 -0800 Proposed patch patch_ulimit text/plain 1724 aakash_jain SW5kZXg6IFRvb2xzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBUb29scy9DaGFuZ2VMb2cJKHJl dmlzaW9uIDE5Nzg4NikKKysrIFRvb2xzL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwz ICsxLDE0IEBACisyMDE2LTAzLTA5ICBBYWthc2ggSmFpbiAgPGFha2FzaF9qYWluQGFwcGxlLmNv bT4KKworICAgICAgICB1c2UgdWxpbWl0IGNvbW1hbmQgdG8gY2hlY2sgcHJvY2VzcyBsaW1pdCBp biB3ZWJraXRweQorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/ aWQ9MTU1MjYwCisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAg ICAgKiBTY3JpcHRzL3dlYmtpdHB5L3BvcnQvaW9zLnB5OgorICAgICAgICAoSU9TU2ltdWxhdG9y UG9ydC5kZWZhdWx0X2NoaWxkX3Byb2Nlc3Nlcyk6IFVzZSB1bGltaXQgY29tbWFuZCBvdXRwdXQg aW5zdGVhZCBvZiAKKyAgICAgICAgbGF1bmNoY3RsIGxpbWl0IG1heHByb2MgY29tbWFuZC4KKwog MjAxNi0wMy0wOSAgQWxleGV5IFByb3NrdXJ5YWtvdiAgPGFwQGFwcGxlLmNvbT4KIAogICAgICAg ICBTdG9wIGJ1aWxkaW5nIGFybXY3IG9uIGlPUyBkZXZpY2UgYnVpbGRlcnMKSW5kZXg6IFRvb2xz L1NjcmlwdHMvd2Via2l0cHkvcG9ydC9pb3MucHkKPT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gVG9vbHMvU2NyaXB0 cy93ZWJraXRweS9wb3J0L2lvcy5weQkocmV2aXNpb24gMTk3ODg2KQorKysgVG9vbHMvU2NyaXB0 cy93ZWJraXRweS9wb3J0L2lvcy5weQkod29ya2luZyBjb3B5KQpAQCAtMTM2LDcgKzEzNiw3IEBA IGNsYXNzIElPU1NpbXVsYXRvclBvcnQoUG9ydCk6CiAgICAgZGVmIGRlZmF1bHRfY2hpbGRfcHJv Y2Vzc2VzKHNlbGYpOgogICAgICAgICAiIiJSZXR1cm4gdGhlIG51bWJlciBvZiBTaW11bGF0b3Jz IGluc3RhbmNlcyB0byB1c2UgZm9yIHRoaXMgcG9ydC4iIiIKICAgICAgICAgYmVzdF9jaGlsZF9w cm9jZXNzX2NvdW50X2Zvcl9jcHUgPSBzZWxmLl9leGVjdXRpdmUuY3B1X2NvdW50KCkgLyAyCi0g ICAgICAgIHN5c3RlbV9wcm9jZXNzX2NvdW50X2xpbWl0ID0gaW50KHN1YnByb2Nlc3MuY2hlY2tf b3V0cHV0KFsibGF1bmNoY3RsIiwgImxpbWl0IiwgIm1heHByb2MiXSkuc3RyaXAoKS5zcGxpdCgp WzFdKQorICAgICAgICBzeXN0ZW1fcHJvY2Vzc19jb3VudF9saW1pdCA9IGludChzdWJwcm9jZXNz LmNoZWNrX291dHB1dChbInVsaW1pdCIsICItdSJdKS5zdHJpcCgpKQogICAgICAgICBjdXJyZW50 X3Byb2Nlc3NfY291bnQgPSBsZW4oc3VicHJvY2Vzcy5jaGVja19vdXRwdXQoWyJwcyIsICJhdXgi XSkuc3RyaXAoKS5zcGxpdCgnXG4nKSkKICAgICAgICAgX2xvZy5pbmZvKCdQcm9jZXNzIGxpbWl0 OiAlZCwgY3VycmVudCAjcHJvY2Vzc2VzOiAlZCcgJSAoc3lzdGVtX3Byb2Nlc3NfY291bnRfbGlt aXQsIGN1cnJlbnRfcHJvY2Vzc19jb3VudCkpCiAgICAgICAgIG1heGltdW1fc2ltdWxhdG9yX2Nv dW50X29uX3RoaXNfc3lzdGVtID0gKHN5c3RlbV9wcm9jZXNzX2NvdW50X2xpbWl0IC0gY3VycmVu dF9wcm9jZXNzX2NvdW50KSAvLyBzZWxmLlBST0NFU1NfQ09VTlRfRVNUSU1BVEVfUEVSX1NJTVVM QVRPUl9JTlNUQU5DRQo=