From patchwork Mon Jul 3 12:16:01 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Szabolcs Nagy X-Patchwork-Id: 21390 Received: (qmail 80631 invoked by alias); 3 Jul 2017 12:16:25 -0000 Mailing-List: contact libc-alpha-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: libc-alpha-owner@sourceware.org Delivered-To: mailing list libc-alpha@sourceware.org Received: (qmail 80493 invoked by uid 89); 3 Jul 2017 12:16:23 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-24.5 required=5.0 tests=AWL, BAYES_00, GIT_PATCH_0, GIT_PATCH_1, GIT_PATCH_2, GIT_PATCH_3, RCVD_IN_DNSWL_NONE, SPF_HELO_PASS, SPF_PASS autolearn=ham version=3.3.2 spammy=deadlock, unlock, modifications X-HELO: EUR03-VE1-obe.outbound.protection.outlook.com Authentication-Results: redhat.com; dkim=none (message not signed) header.d=none; redhat.com; dmarc=none action=none header.from=arm.com; Message-ID: <595A3581.7000007@arm.com> Date: Mon, 03 Jul 2017 13:16:01 +0100 From: Szabolcs Nagy User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.8.0 MIME-Version: 1.0 To: Carlos O'Donell , Siddhesh Poyarekar , Joseph Myers CC: nd@arm.com, GNU C Library , "triegel@redhat.com" Subject: [PATCH v4] Single threaded stdio optimization References: <594AA0A4.7010600@arm.com> <594B92ED.6060809@arm.com> <2dc4477c-c349-86b1-f8a5-95d69c397f24@gotplt.org> <98711966-b1c2-1234-ebee-301e5102aa46@gotplt.org> <8e7e9ce1-4f50-217e-b99f-e640fcca5490@redhat.com> <595640FB.8030904@arm.com> <648bb45e-9573-3328-b575-ff68970ddc5e@redhat.com> <595662AC.7080607@arm.com> In-Reply-To: X-ClientProxiedBy: DB6PR07CA0085.eurprd07.prod.outlook.com (2603:10a6:6:2b::23) To HE1PR0802MB2490.eurprd08.prod.outlook.com (2603:10a6:3:d9::22) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 2a4aa4ac-cda2-4a6e-2ab1-08d4c20d47da X-MS-Office365-Filtering-HT: Tenant X-Microsoft-Antispam: UriScan:; BCL:0; PCL:0; RULEID:(300000500095)(300135000095)(300000501095)(300135300095)(22001)(300000502095)(300135100095)(48565401081)(300000503095)(300135400095)(49563074)(201703131423075)(201703031133081)(300000504095)(300135200095)(300000505095)(300135600095)(300000506095)(300135500095); SRVR:HE1PR0802MB2490; X-Microsoft-Exchange-Diagnostics: 1; HE1PR0802MB2490; 3:i2jUhAQH1/fbHyE3c46vWlIip5I5TcNJGr68HXPFapd40AZ/AdbrF3lQzjL4DYCF6q6KRUFdmQ5+GI3lc92WEznsiaU1QrDFe119upqiu2GZ1G1gISHWBh+IfcuyINNL1xXlulXohrvwANT1tpmpAtra7TSNvR7FS7/wkRlcvAYwG8grvG/uibaJYL2xKCWe/ON7FNUVuHHYjQEeGXkJHMImtQt2FMxIW4kYqmnGd6Beyr6SiSCte3jg59aBUwxHzaMNxCz/PAUl+PmZoAm90nJ2qwaIkZBhy21qeb2X0N1sDddkBDb4kU8jPeFA7idtbjEse39caHmkEJKYzuo9jfjN0fXij1pxzeqBGwl6FLFzuzHxx86ohQlrxkRrKH5L9ypmSDKcjB1oDSkWqPrSk/GNeIK2wJrh2BT4IElgHMKTA6HGVakC3bllo8sDGnd6AlvRpHsFtRiOejr5q3F6i4qYbm1fO5mU18JFnqF8w968RSfFNLLPDPxe+jZsxRhYmh5bxhw2KlZ5WsiNtVdcUJeuuk91pVM6Y5F8hzvzcPHqPXh9FEl1CZR1jvcmPbf5SZAG2oChtwEL3QaJyBmRxf92oV5KJxIBeO+Unc5YXxYBby5J58iTjncfGMVZtNVDbSaUD6SqbIUJB6nv4d3SUbB6qQ0+rv6E931Ioc1TSq6uQCtJ2JYZhCbf4gSCCqI3MkBc2nDW8JUch55tP38Wj20EGjph7X9tc/Kz2aZiDObDLn1e2VKlTmts3MX6rGSN7rM+/fBLNRVS6vCHKRjHzinRVfMPh2Vb+GyfP6nKnsU= X-MS-TrafficTypeDiagnostic: HE1PR0802MB2490: X-Microsoft-Exchange-Diagnostics: 1; HE1PR0802MB2490; 25:kCMCfp0RO0PJ8mgseLcmlJHkfEQ7ba/CgY0CLSX7kkoNy4st6qRtJg7M71j39kFi4e7p7ozrFAGjt8uiFWyERrVNb6D7EdWFtUoS8koAYKEszpV8xnEDPZK4Q1wVl7Vath5dUTSycv6WX+I4nMmWwtVHjvoEMHqHMHv51yhhMHQOWXihMCkm5Q7K8mIys7dM7cw8wC3r8KTasQeFsSbrHjCdUyR4SF6THEOUAXdnL5g2VBFioqH3VPPBuQYDXiPc3HZQMLxGJHuIUTKPiUm76TEhUUwre9e2gft9uQxSNTZuZ1DJOG1ZymCRlJLGPxZDSfkJevPb7oMaS6MMNOSYKi05xL5Nnrl153qczgAVNSmcKim/srFTq35Zcx0bNy/O0zxzfm4asmiPDqBCZhjRAWPG95xF8ydqc9y/6zbYCtcctApbnd08pPVhgXwFJWdNZWA5aRV6/1znGLEXl6O1G0ypFdR/YNyvA/rJcDzCD44ptuVedIXYp9k8iy21C+VNFIKkcCZNtdVYp0wyyAelK9N3CLGnJYMqpvcTBNgHAi1pgxr00nF3SLVSnSgf1wksT/4TgUo6yzoLP2h96pwsydiIs6y8ukTcTSlIoQavOyAHHLBKBkxZOgUYinKlHJpEm7kTEeWk3c/GadR2xftn3UV7zSwMfLBSkUWWr3oNQC0fIsqPN+MGxRw+GbTFS6pDgGuQ6eDCsroCACTKTYXaNZwB1jX29cVoGM08Tnad2VBGpfNJ2UNNv0A6LhITl6K9uyQtDy6qYr2YASptaC8w3ltcTE2h+MdP6q44P3Ygq0qSndey5OeIMbEpDZfMH7XCs0c0z5d6JbEre9G+fjWT6PtfspnaLkttLehPf0z5DAvjdDtmVJFapcxeKaVZQCaiL7CBgL6zDm9FV/0g/wlaBDCUW+yMzck/uoFrRlcou2AhCkmWJq8c7PD6fKXC4uJK X-Microsoft-Exchange-Diagnostics: 1; HE1PR0802MB2490; 31:ZHNE1lr45pw3yo3Etg5SwfMOVV5/CnQFwCk+BBqQswtJg4fAqINe32UDyAxKhOSTLYZXQJvcDvToXlVBKT54KlOw3F0pdDli4rX74y00a18IyY4dI+dMSYS+oxbHnHs8yygVgfM5kGDVogSCsr37q8yUASQy5SyBJbrhoDVl40B9yJ7xJGOKq2xaPARLG30Wx+AUx5erI+KJ2Uw84W3ctfpxMfeFNAN8b0Ggsl20tk6hOBDsKz/K6B8p9O+BzsoHRKDzXgW6mh2ZI6c+eKmHfTYy+ZzieCfPqcJ/hfmMjrXkqXFt3ETSDOKeT26AzE5XfBf2a+emwABJkCJVGDjQRSIeen6HP1Y9fabazwxaifFUW5sNH0ABGFM5LT6Ye5bpCitWzHkd+cXLfTy41oQZ/DS/6BOaW4PpGBvEkOySJys2N9lJAkDkGOZpLKdrBVpo1eHDBSNs2tm37PRVxCFu6gLEjrTpO93BCLW5yh8zRXruNoG5YcdNWoApM3JlfKtsiNfmV1OI4aJYP+xpr44SMHMoi7zJgy1TItptR1d3FDnvngciWEQ9W699TgC8kDO7GT4L5eJuBY6UmlTOBIMbtaJN5xtrhsGYfVe7MzQv90SdaDX25MWxQQ7RP0zsZ9SPpRDCoTYTGUqta8tYTHLjZ8+vU8XxrGJR5v0RMkZGVW0tFJVXZvdWeLGaYagwUEHzCSdRBs019rZ0I4g/mL8gvQ==; 20:79iAOpbSaQJ0VA8pKVxO7E6ePUde1tm4y7HIK01wjJLeecuaTlElY53QyVyUhIteHuFOsB/N8RbQSN/Cub8RD6Fm5OWi00JWDjpskiMa3A68yzABVZk9Rc6y5cxz/wRAvWgbwIz50huOHZ8qOoaEEW+vXeSxr0eZ+a9kw9uumJE= NoDisclaimer: True X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:(180628864354917)(236129657087228)(148574349560750); X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(100000700101)(100105000095)(100000701101)(100105300095)(100000702101)(100105100095)(102415395)(6040450)(601004)(2401047)(8121501046)(5005006)(93006095)(93001095)(100000703101)(100105400095)(10201501046)(3002001)(6055026)(6041248)(20161123555025)(20161123562025)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(20161123560025)(20161123558100)(20161123564025)(6072148)(100000704101)(100105200095)(100000705101)(100105500095); SRVR:HE1PR0802MB2490; BCL:0; PCL:0; RULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(100000804101)(100110200095)(100000805101)(100110500095); SRVR:HE1PR0802MB2490; X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; HE1PR0802MB2490; 4:Igkm+oftdpEn/fuWe1YevFqNXYHG1aZ4jLgCLD30?= =?us-ascii?Q?T9cdD+TGm1qTQjKnPf+ifIk1e5noREGIyyk2D3w1dgivBgCYkyXZRRVyEWDB?= =?us-ascii?Q?XbZQBnljdudRFdMLbN4sAVkjfEHgt6kJbzkgkpnvItVaeFWBkYYY1yzBYnmr?= =?us-ascii?Q?NNzqanR1s5wYzE9tohRUzaWFmoy2+XTH1w9TPmEgmC0cQz5jMEtmmuntfbyz?= =?us-ascii?Q?T5IMCeUG/lgOFDqBka4SzF8d7SNV79FD5QDJqKxKJdnCqhEWrkajfiWqzkkv?= =?us-ascii?Q?sDPe+2UaL/ZGQy34d2VZgidBi1ieXGZnx3JDVIFUrvvrw2Mdb3S0ucai5Oc2?= =?us-ascii?Q?BXMnOE7dsKb1/XPXju6wgb0Vp3dYnnEVuCLDaNsXtbQn4FJlqfHzxqmFTo3Q?= =?us-ascii?Q?HrX/7CEPZ44yFawGLP/tn9wLzUWiFOB316A6zMbTNdWRAm+SfzH1CKfPb6E8?= =?us-ascii?Q?qfxQNP9jTIneqm8rQc/LN+vkWtpKK0MvAEFfBB2Fst2zsjMu+6AH3CHUggn/?= =?us-ascii?Q?qama/4xXhHqlAZbwbiYyncwrAFrOCCTfGJNAe1pFikUMVfGcOs6neCvtnGGW?= =?us-ascii?Q?N0/lQCNgRBU7q4mT/eDl9gG+Sh1TSc1Y7/Eg+l7Ff0Ylx+cw1uwxWsoVz7Fk?= =?us-ascii?Q?DNX4AhAxKvei8IjSwJC1ypHmTgLsfFBa8ZGSGng/XPpwOlWp8u3N9uUajmuQ?= =?us-ascii?Q?p4PJp7YOPS8cvxtPhxerK6N+OK0z6hc3AP3Ko+/eqWmu+ZYSRMLTkHjA4OO0?= =?us-ascii?Q?I1jwX6In11LXg/WZfSHv/JpL/I3ggry8QsTv7bZmWzvE/AV4cfYAC+Hi2U+K?= =?us-ascii?Q?rfnsBqoDPEY/6+tuQhbd+WSTbJot9RJTlx4vKbl4KdRw+zMqsgVVvFumDgpr?= =?us-ascii?Q?9S6NLbF06sfWiDPEYpnIxG9bVZu9C3Gj58raMqdJEsiNeDN+l+N0E5RN4Fp4?= =?us-ascii?Q?SM/mhCpSd6yDrrVJuuHE0pKNQfTnukWNM8v8aA4UaEtuc9DvPvdVwQxnW9nx?= =?us-ascii?Q?34YM8hglNnFA8zZ0Yi0OwAudv/8lwbCVByvrFp4/OjuOGb7p3k5Wer6RiFvl?= =?us-ascii?Q?uVdonKVH5yZBdWas69DMm9I/cU4vz5Z+MRcY5gajc6aEQEEmg2ruwiJBYWSH?= =?us-ascii?Q?r2KM2dTPjOQj5gkiwVzhlcLidbjtffr3XvVeh1wDxy2fiu4gQmwWIXpWIi3B?= =?us-ascii?Q?A7kjg5f5JPJonPhNRhH4L2FDPf22izgpPrZ2r1BpPT+u9rlfTqqVaMqwulK7?= =?us-ascii?Q?8+e26UsFKd77RY2xrZFuJeRtxMdviuRJB/Nc0KwgWVp7AFdgar0Gkoh40qdC?= =?us-ascii?Q?FA=3D=3D?= X-Forefront-PRVS: 035748864E X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(4630300001)(979002)(6009001)(6049001)(39840400002)(39850400002)(39410400002)(39860400002)(39450400003)(39400400002)(24454002)(377424004)(65806001)(72206003)(568964002)(2950100002)(4001350100001)(270700001)(2476003)(33656002)(54906002)(6306002)(3846002)(6116002)(65956001)(53936002)(6666003)(54356999)(87266999)(76176999)(50986999)(65816999)(86362001)(21490400002)(966005)(42186005)(64126003)(36756003)(2906002)(4610100001)(25786009)(77096006)(6486002)(478600001)(59896002)(80316001)(5890100001)(66066001)(83506001)(7736002)(8676002)(93886004)(305945005)(189998001)(81166006)(5660300001)(38730400002)(84326002)(53546010)(4326008)(969003)(989001)(999001)(1009001)(1019001); DIR:OUT; SFP:1101; SCL:1; SRVR:HE1PR0802MB2490; H:[10.2.206.69]; FPR:; SPF:None; MLV:ovrnspm; PTR:InfoNoRecords; LANG:en; X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; HE1PR0802MB2490; 23:ONj5mOLwyG9K1X++rfvw771V0m9voZ26paa5WA3?= =?us-ascii?Q?34jY6vroep/ub5oJrGGxQFIp1Z7N7Hammrys23qT/LaG/C3e3XmyFToZGQmG?= =?us-ascii?Q?hnkubpH+x3U3XPBZLMD//5r4gRSjtXwhSLohsuC9u+XL9iklHXfmwD132TNT?= =?us-ascii?Q?J9M7PTWxdnT9Ix0oTJPg6VMCHY/VUP+Yd4afDReN+oW6ABVb2QS6Xqw9Lajp?= =?us-ascii?Q?uwWet1cqgj8A/QuOkyHsKONs4jXAGxwci+YTmHeBdS9ler2zNmfKyJaZNYPG?= =?us-ascii?Q?qtYJQZirmC0PmJGkZQJK3a5WzHCFN2rg7TY5WdtcnjBbEOMrQnGP5CZDCM9/?= =?us-ascii?Q?aWSgDzdN3OqGB33Iqo/FXGp9dBvORZCJQ3GtnHsvfVJgk54fFPzk8fhj2nM9?= =?us-ascii?Q?49i8h49Ow2c5240HMHshbexnT3AFy10rp6vrzx+vcpGiNp7UJRXrXu1c7wwU?= =?us-ascii?Q?Oil1D3vC74PytTswihM6wvbuoccf3cUcsVVg8KsWJzRJV5iAwcCVf8DG0Ypz?= =?us-ascii?Q?i14VGPcLGuLeNQ/vlykXhQHZbHO/OkAkdC94DpW6rNvoAIusVoAWTv2vgpf8?= =?us-ascii?Q?wpLj8YmYnZRhRJJJMi31g7IPlDHn46jZITvg6UKCO1yICw5jtZULU30bZDB2?= =?us-ascii?Q?4r9I6Zg3VftWJygOyblTEzcUC4F91xxMPLjeLiCLdo8Chvjcph6I9HxHix77?= =?us-ascii?Q?VHWF2Sdk2hA1YZhDrFfuEHeW0E8cUOTejwrxGZdXaSVAISs3KZVNTfV077V3?= =?us-ascii?Q?ASIdM3vUr0OrQpnhzabaC1c8rFTUE0tM+Y4nsURrlt0P7yLt6yySdekEJZ6x?= =?us-ascii?Q?OKKPBjkRmwyYT0VIt3Y4m8l86PExiNZZh0GnswYZ0/RnidaXjP2d3PaUKwIZ?= =?us-ascii?Q?5RD9llZWqhdGVZbGA3cY/FQ9iA5xtmtSM3o1NCTkgGmjs9k1jmDnEfZzWDYG?= =?us-ascii?Q?nfSy34DUdiuMDcSYVOuDFnNw28B47Lzk4r+Sga6DoLKILcvFAYe09/60WJ8W?= =?us-ascii?Q?lN5rfnPYFQNjBVh2kOaT5PoApzWBrl9kpp4jqn1rZBAqT/hbTGxY0tXTVf/y?= =?us-ascii?Q?JqgWgVzy1AP1bUsPfF9EA2QXkq6Qv9D418HFxG2cSqy1v/11kks4jCiRvuPo?= =?us-ascii?Q?0kQ1Sn3aQfPzSVwBECzcaVz9EHLvOjBAugMB9rFeibo8JCpGkl5Fn4zo7XRe?= =?us-ascii?Q?Ix6/PGHT0kTq+ceVmOk6G1wSQk2iNgWylgyybcNIa7im/DLHff/GaxqVYHnL?= =?us-ascii?Q?4Wuhn7ar3YLSTWWsx1iY5TF3PxJEUx/L9oRB2/QuA0qKWqsvNuHcpN9tTy5R?= =?us-ascii?Q?EsIt6NXPGkfh6Ic3yKTemwyMVG1vTcJ+wIDtAjZCiYSiyE/xxmf824sh+65j?= =?us-ascii?Q?l1K44V/Pa/3USLDsZooBwJiA6/TxgTr+/IJroxYxhMLwcx9z9t7YYI1mXu83?= =?us-ascii?Q?jGHkM3hb30DF+++1ySo78jYYl+iodjKFduX07FEyVnzMTQlX4ZZo+nkDB4AZ?= =?us-ascii?Q?KZyfm2HpuZOBev2blY0VXN3cggHO0J3dY6HSbtuUqmaWI0Q5ZS0JIhwA1K5q?= =?us-ascii?Q?bf7mwhXFKYGFTtUk7vA=3D=3D?= X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; HE1PR0802MB2490; 6:Yq2YEO37OzdCPlmx0ype02xci3pfACP5vyT+S4n0?= =?us-ascii?Q?Lss4H04W5+/+AMS9t5wjqag6mFLNKaLzX9QcWC/LYjlP1ROvRpKJnwOApyHU?= =?us-ascii?Q?+/mLjr7AKQkFrs6gCsJIvQ4K8r0Rp5sYmosAnO+aauNtvz+6/og8xubhJlDw?= =?us-ascii?Q?DbQKv0WER5H/ktLOmi7kS50ZxwxcoSfPwiYez7yMhW3PV6o11V8RGqUymzXC?= =?us-ascii?Q?E5W6qqnVnYzPzKTOu1qDNrmRI7oQtSwCrredsAO9r8QKjM6ddyD4PI7Rq4MP?= =?us-ascii?Q?bcHzhWdfQneg+YEtMA/hb/+/BiCtxy3wUp9rTZsGtg2jiTGVr8v+3uo6hArg?= =?us-ascii?Q?1pFcufuOJopbYlK0fPOnL/Wyc+6zpmH+jiBh+Ztn5S/6BsI6wVfeTsirDdjK?= =?us-ascii?Q?Yknt+qQVmJO6L4UIV6idjnNQnqT/saIHhW//RE9bXP5IRoavCWp8NkmVAsYp?= =?us-ascii?Q?J5rNRvjNRZeSE+WDXcGxBctF2zTWaeTzsYvv1pJu59MDrUQa+FRgvCDZ+f4b?= =?us-ascii?Q?JLaMeJw+NAGl8OgI/M9NgHoi2lMVJjHiPuVXUmjhljypbXGXoeECzXBYIxiD?= =?us-ascii?Q?/Mn5sIsiMgzp1sXt9zAAAFd/Iy8yL/CaP1uEHPPHUkv0gak98J1kmgaWST5Q?= =?us-ascii?Q?T9oHNyPljRf15DhhCaUYHnwLDxgnXaTs6Qxtr2MPDXInzHw8KR24Vl0gQ6JL?= =?us-ascii?Q?iMMr0mLOVvA1jmzA+org8DS+dGHbxaBVfrASPCLEj9BpTe2AM6zQDqPhk1LG?= =?us-ascii?Q?9sMqvxGzdLucmLMLX/rUMUPIriaDEfIgIePz+siEZr9M49PyyQ6IGfITdteD?= =?us-ascii?Q?wew24VZV970bJFZZ5U07DnyM8sDdJHKiupFEusDtiyPm4dxDbLazjiRvItXw?= =?us-ascii?Q?NPWS6i2bxTSWA52b6AASm6lQ66IBufAqkXT1wL5WcAkuTynRF5/z01IcbvO2?= =?us-ascii?Q?hVqiXek3fxFahc6/aWu/0tQ7hKtoReS5K2ITqaPhylsvwpx6EuYJgQQhy3Fp?= =?us-ascii?Q?dVQvFzyVqoJ80wuNyIWqiLwB?= X-Microsoft-Exchange-Diagnostics: 1; HE1PR0802MB2490; 5:JbPN838Z66rf1UOnux2oHRK279woio2XfgsxQV0RgPZYpBEtsPuDhHmDH28hZjEKaE4jZ8EZm63ArugrFwU78+uhOZxrU9+bmayt48R/cxvDIeP6oHJ9JLVEsdR8HZ+sP/eSvno3BgA2pz5Z23S6mj5R0VVBoGF309OjACRwCtQj4ZhKJRqjn/zRRHp4Devf5GTNT+6/bbcR+EB0Dt+V0S5WaWxfDwrzw6DG2i9Gg5PBFp4sDk684/Yx2a86ToL29JngEfyEJexliiyteY5HKONe1HYgXrQU+oBFXx6QofEG7TNK5WUbkWeIjoKbju2EBKcSS+S/hHHw//8ZkFwG8DfS06u6JGa3j4JrvXan8gdN+tlzUw1Jw3Wjx0bGS7BLx4OBZskQuK+XoLrxXi1O7gbquP53pDUNnk4EVX+NeqvZqpAkBPusk+8tYYyg6Suq7FBzrO2mjGbe7WebhqSVyxYU1IjOIJjQ19r6Hs9XVOWcgqVYnfPnJvpaphEcC6cC; 24:uT6plFs/IJmXS1Qx+S2j15+JsvSzD3q8JL+NwXtHuzSF+QIwj+a7a568eBzHxuY6cM6r4KMRY5hZRoBlL9XboiJ81DNssYjzr6ye8ahpf9M= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1; HE1PR0802MB2490; 7:FsZ81c0JPV+jcVZjbg5HZX0Kbv9i1VL7Rb2aqQYcIqOP8DzERC/J91X8f/mCyS4DDCj6mq6di2wivJMi+i8/pbSq+HDoWec2cv6P5RPH1f1KqNEOIDuLrpOufreI6cGVZH7wOWTScEIuBwk5kGRwIuQRePWGZkDCVNAHusGzipNxFnS7zrMNGlVj/R45saSw87f0qBUT6IrN+9xBfB0W0VUN554jEYfx+BGSpe3E7utB9PJSlOngjOmFejJJlVt5GLCowdx+JlMpBjngh/tzB02LjnA/5rd+o5P9FMORd84Ym/avdJdlCAYhMrWv48qgbPmoLGOReAvtg74D1hFdg4aXCB8NX186HXLnjw2+e2ZPQfA8oBjYqkWgTx6/J55Fqs381MYH46hbJyClw1DXBgkrAxyk3+rU0iZ9QP7snQCWy+IYB7LM8AO0VnxBC+xaHkshighSZ0LpZewDQv1dtDGuxcKXjq+kOTXRDB22NJJfqoIx/wC6vFzCYfKK1vU673DlDNYE8ugdwLjn+cEiXdlp6CnUj/2KawxJJoXk8p+qWj/rSNFMecLAGKrWPj6fwzwDEsBN3I/DaEEdg5+AZnpJzYFTgp13fFXV6vCU4vRUxMHmriaJd1W4YPFiUEgZPXobICymOhjneFK/hjGtGpjn3PVsEeYz3jKEh1OAgn6qymDfr9snTjwx5lkqpJhkTMSTdsE1DOiDVtm6mwJUAOmR1ylvxVCdxd6gWgCeiwjEDaqzj+FKNvsJH052jhplYlSuL/qaOPGXZD40Ob6PXUgYfKX55LRQbmhLW9Bmg18= X-OriginatorOrg: arm.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 03 Jul 2017 12:16:05.6231 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: HE1PR0802MB2490 On 30/06/17 17:07, Carlos O'Donell wrote: > > I'm OK with the patch for 2.26. > v4: i did two modifications: i dropped the hunk #if _G_HAVE_MMAP -# ifdef _LIBC +# if IS_IN (libc) /* When using this code in the GNU libc we must not pollute the name space. */ # define mmap __mmap # define munmap __munmap because https://sourceware.org/ml/libc-alpha/2017-06/msg00854.html got committed which made __mmap work in pthread_create. and added comments to the _IO_enable_locks definition. if there are no complaints about these changes i'll commit the patch tomorrow as: Locking overhead can be significant in some stdio operations that are common in single threaded applications. This patch adds the _IO_FLAGS2_NEED_LOCK flag to indicate if an _IO_FILE object needs to be locked and some of the stdio functions just jump to their _unlocked variant when not. The flag is set on all _IO_FILE objects when the first thread is created. A new GLIBC_PRIVATE libc symbol, _IO_enable_locks, was added to do this from libpthread. The optimization can be applied to more stdio functions, currently it is only applied to single flag check or single non-wide-char standard operations. The flag should probably be never set for files with _IO_USER_LOCK, but that's just a further optimization, not a correctness requirement. The optimization is valid in a single thread because stdio operations are non-as-safe (so lock state is not observable from a signal handler) and stdio locks are recursive (so lock state is not observable via deadlock). The optimization is not valid if a thread may be created while an stdio lock is taken and thus it should be disabled if any user code may run during an stdio operation (interposed malloc, printf hooks, etc). This makes the optimization more complicated for some stdio operations (e.g. printf), but those are bigger and thus less important to optimize so this patch does not try to do that. 2017-07-03 Szabolcs Nagy * libio/libio.h (_IO_FLAGS2_NEED_LOCK, _IO_need_lock): Define. * libio/libioP.h (_IO_enable_locks): Declare. * libio/Versions (_IO_enable_locks): New symbol. * libio/genops.c (_IO_enable_locks): Define. (_IO_old_init): Initialize flags2. * libio/feof.c.c (_IO_feof): Avoid locking when not needed. * libio/ferror.c (_IO_ferror): Likewise. * libio/fputc.c (fputc): Likewise. * libio/putc.c (_IO_putc): Likewise. * libio/getc.c (_IO_getc): Likewise. * libio/getchar.c (getchar): Likewise. * libio/ioungetc.c (_IO_ungetc): Likewise. * nptl/pthread_create.c (__pthread_create_2_1): Enable stdio locks. * libio/iofopncook.c (_IO_fopencookie): Enable locking for the file. * sysdeps/pthread/flockfile.c (__flockfile): Likewise. diff --git a/libio/Versions b/libio/Versions index 2a1d6e6c85..77123347e3 100644 --- a/libio/Versions +++ b/libio/Versions @@ -155,5 +155,8 @@ libc { GLIBC_PRIVATE { # Used by NPTL and librt __libc_fatal; + + # Used by NPTL + _IO_enable_locks; } } diff --git a/libio/feof.c b/libio/feof.c index 9712a81e78..8890a5f51f 100644 --- a/libio/feof.c +++ b/libio/feof.c @@ -32,6 +32,8 @@ _IO_feof (_IO_FILE *fp) { int result; CHECK_FILE (fp, EOF); + if (!_IO_need_lock (fp)) + return _IO_feof_unlocked (fp); _IO_flockfile (fp); result = _IO_feof_unlocked (fp); _IO_funlockfile (fp); diff --git a/libio/ferror.c b/libio/ferror.c index 01e3bd8e2b..d10fcd9fff 100644 --- a/libio/ferror.c +++ b/libio/ferror.c @@ -32,6 +32,8 @@ _IO_ferror (_IO_FILE *fp) { int result; CHECK_FILE (fp, EOF); + if (!_IO_need_lock (fp)) + return _IO_ferror_unlocked (fp); _IO_flockfile (fp); result = _IO_ferror_unlocked (fp); _IO_funlockfile (fp); diff --git a/libio/fputc.c b/libio/fputc.c index a7cd682fe2..b72305c06f 100644 --- a/libio/fputc.c +++ b/libio/fputc.c @@ -32,6 +32,8 @@ fputc (int c, _IO_FILE *fp) { int result; CHECK_FILE (fp, EOF); + if (!_IO_need_lock (fp)) + return _IO_putc_unlocked (c, fp); _IO_acquire_lock (fp); result = _IO_putc_unlocked (c, fp); _IO_release_lock (fp); diff --git a/libio/genops.c b/libio/genops.c index a466cfa337..6ad7346cae 100644 --- a/libio/genops.c +++ b/libio/genops.c @@ -570,11 +570,39 @@ _IO_init (_IO_FILE *fp, int flags) _IO_init_internal (fp, flags); } +static int stdio_needs_locking; + +/* In a single-threaded process most stdio locks can be omitted. After + _IO_enable_locks is called, locks are not optimized away any more. + It must be first called while the process is still single-threaded. + + This lock optimization can be disabled on a per-file basis by setting + _IO_FLAGS2_NEED_LOCK, because a file can have user-defined callbacks + or can be locked with flockfile and then a thread may be created + between a lock and unlock, so omitting the lock is not valid. + + Here we have to make sure that the flag is set on all existing files + and files created later. */ +void +_IO_enable_locks (void) +{ + _IO_ITER i; + + if (stdio_needs_locking) + return; + stdio_needs_locking = 1; + for (i = _IO_iter_begin (); i != _IO_iter_end (); i = _IO_iter_next (i)) + _IO_iter_file (i)->_flags2 |= _IO_FLAGS2_NEED_LOCK; +} +libc_hidden_def (_IO_enable_locks) + void _IO_old_init (_IO_FILE *fp, int flags) { fp->_flags = _IO_MAGIC|flags; fp->_flags2 = 0; + if (stdio_needs_locking) + fp->_flags2 |= _IO_FLAGS2_NEED_LOCK; fp->_IO_buf_base = NULL; fp->_IO_buf_end = NULL; fp->_IO_read_base = NULL; diff --git a/libio/getc.c b/libio/getc.c index b58fd62308..fd66ef93cf 100644 --- a/libio/getc.c +++ b/libio/getc.c @@ -34,6 +34,8 @@ _IO_getc (FILE *fp) { int result; CHECK_FILE (fp, EOF); + if (!_IO_need_lock (fp)) + return _IO_getc_unlocked (fp); _IO_acquire_lock (fp); result = _IO_getc_unlocked (fp); _IO_release_lock (fp); diff --git a/libio/getchar.c b/libio/getchar.c index 5b41595d17..d79932114e 100644 --- a/libio/getchar.c +++ b/libio/getchar.c @@ -33,6 +33,8 @@ int getchar (void) { int result; + if (!_IO_need_lock (_IO_stdin)) + return _IO_getc_unlocked (_IO_stdin); _IO_acquire_lock (_IO_stdin); result = _IO_getc_unlocked (_IO_stdin); _IO_release_lock (_IO_stdin); diff --git a/libio/iofopncook.c b/libio/iofopncook.c index a08dfdaa42..982f464a68 100644 --- a/libio/iofopncook.c +++ b/libio/iofopncook.c @@ -172,6 +172,8 @@ _IO_cookie_init (struct _IO_cookie_file *cfile, int read_write, _IO_mask_flags (&cfile->__fp.file, read_write, _IO_NO_READS+_IO_NO_WRITES+_IO_IS_APPENDING); + cfile->__fp.file._flags2 |= _IO_FLAGS2_NEED_LOCK; + /* We use a negative number different from -1 for _fileno to mark that this special stream is not associated with a real file, but still has to be treated as such. */ diff --git a/libio/ioungetc.c b/libio/ioungetc.c index 951064fa12..917cad8abb 100644 --- a/libio/ioungetc.c +++ b/libio/ioungetc.c @@ -33,6 +33,8 @@ _IO_ungetc (int c, _IO_FILE *fp) CHECK_FILE (fp, EOF); if (c == EOF) return EOF; + if (!_IO_need_lock (fp)) + return _IO_sputbackc (fp, (unsigned char) c); _IO_acquire_lock (fp); result = _IO_sputbackc (fp, (unsigned char) c); _IO_release_lock (fp); diff --git a/libio/libio.h b/libio/libio.h index 518ffd8e44..14bcb92332 100644 --- a/libio/libio.h +++ b/libio/libio.h @@ -119,6 +119,7 @@ # define _IO_FLAGS2_SCANF_STD 16 # define _IO_FLAGS2_NOCLOSE 32 # define _IO_FLAGS2_CLOEXEC 64 +# define _IO_FLAGS2_NEED_LOCK 128 #endif /* These are "formatting flags" matching the iostream fmtflags enum values. */ @@ -451,6 +452,9 @@ extern int _IO_ftrylockfile (_IO_FILE *) __THROW; #define _IO_cleanup_region_end(_Doit) /**/ #endif +#define _IO_need_lock(_fp) \ + (((_fp)->_flags2 & _IO_FLAGS2_NEED_LOCK) != 0) + extern int _IO_vfscanf (_IO_FILE * __restrict, const char * __restrict, _IO_va_list, int *__restrict); extern int _IO_vfprintf (_IO_FILE *__restrict, const char *__restrict, diff --git a/libio/libioP.h b/libio/libioP.h index eb93418c8d..1832b44cc7 100644 --- a/libio/libioP.h +++ b/libio/libioP.h @@ -444,6 +444,8 @@ extern void _IO_list_unlock (void) __THROW; libc_hidden_proto (_IO_list_unlock) extern void _IO_list_resetlock (void) __THROW; libc_hidden_proto (_IO_list_resetlock) +extern void _IO_enable_locks (void) __THROW; +libc_hidden_proto (_IO_enable_locks) /* Default jumptable functions. */ diff --git a/libio/putc.c b/libio/putc.c index b591c5538b..6e1fdeef3a 100644 --- a/libio/putc.c +++ b/libio/putc.c @@ -25,6 +25,8 @@ _IO_putc (int c, _IO_FILE *fp) { int result; CHECK_FILE (fp, EOF); + if (!_IO_need_lock (fp)) + return _IO_putc_unlocked (c, fp); _IO_acquire_lock (fp); result = _IO_putc_unlocked (c, fp); _IO_release_lock (fp); diff --git a/nptl/pthread_create.c b/nptl/pthread_create.c index 7a970ffc5b..2f8ada34d6 100644 --- a/nptl/pthread_create.c +++ b/nptl/pthread_create.c @@ -32,6 +32,7 @@ #include #include #include +#include "libioP.h" #include @@ -756,6 +757,9 @@ __pthread_create_2_1 (pthread_t *newthread, const pthread_attr_t *attr, collect_default_sched (pd); } + if (__glibc_unlikely (__nptl_nthreads == 1)) + _IO_enable_locks (); + /* Pass the descriptor to the caller. */ *newthread = (pthread_t) pd; diff --git a/sysdeps/pthread/flockfile.c b/sysdeps/pthread/flockfile.c index 7fe8e99161..a8e6c28ed9 100644 --- a/sysdeps/pthread/flockfile.c +++ b/sysdeps/pthread/flockfile.c @@ -25,6 +25,7 @@ void __flockfile (FILE *stream) { + stream->_flags2 |= _IO_FLAGS2_NEED_LOCK; _IO_lock_lock (*stream->_lock); } strong_alias (__flockfile, _IO_flockfile)