From owner-freebsd-current@freebsd.org Tue Oct 31 21:52:38 2017 Return-Path: Delivered-To: freebsd-current@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 117D2E63F3A for ; Tue, 31 Oct 2017 21:52:38 +0000 (UTC) (envelope-from andreast-list@fgznet.ch) Received: from smtp.fgznet.ch (smtp.fgznet.ch [157.161.14.52]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id AE06A65973; Tue, 31 Oct 2017 21:52:36 +0000 (UTC) (envelope-from andreast-list@fgznet.ch) Received: from [192.168.225.14] (dhclient-91-190-10-49.flashcable.ch [91.190.10.49]) (using TLSv1 with cipher ECDHE-RSA-AES128-SHA (128/128 bits)) (No client certificate requested) by fgznet.ch (Postfix) with ESMTPSA id DA550C3BBA; Tue, 31 Oct 2017 22:52:33 +0100 (CET) Subject: Re: Segfault in _Unwind_* code called from pthread_exit To: Gerald Pfeifer Cc: Konstantin Belousov , Tijl Coosemans , freebsd-current@FreeBSD.org References: <20170824180830.199885b0@kalimero.tijl.coosemans.org> <20170825173851.09116ddc@kalimero.tijl.coosemans.org> <20170825234442.GO1700@kib.kiev.ua> <20170826202813.1240a1ef@kalimero.tijl.coosemans.org> <20170826184034.GR1700@kib.kiev.ua> <20171029182351.502f53cf@kalimero.tijl.coosemans.org> <20171029191358.GU2566@kib.kiev.ua> <9a724da4-70f1-4330-9a77-619739008a14@fgznet.ch> <20171030153207.15a42a1e@kalimero.tijl.coosemans.org> <9468430e-fda4-10f4-b6a0-aa40d7d64f5b@fgznet.ch> <20171031092803.GA2566@kib.kiev.ua> <9619a1f8-bb42-abc9-a4ee-3b93192f365e@fgznet.ch> From: Andreas Tobler Message-ID: Date: Tue, 31 Oct 2017 22:52:33 +0100 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.11; rv:52.0) Gecko/20100101 Thunderbird/52.3.0 MIME-Version: 1.0 In-Reply-To: Content-Type: multipart/mixed; boundary="------------40C450CC1D50A0A8104883DE" Content-Language: de-CH X-Scanned-By: Idefix Submit on 127.0.1.1 X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 31 Oct 2017 21:52:38 -0000 This is a multi-part message in MIME format. --------------40C450CC1D50A0A8104883DE Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit On 31.10.17 22:36, Gerald Pfeifer wrote: > On Tue, 31 Oct 2017, Andreas Tobler wrote: >> Do we, FreeBSD'ers, want to have gcc unwind support on older than >> FreeBSD 9.3 releases? I think the gcc folks do not care, but we are the >> ones who might have an need for such a support? >> @Gerald, do you have an opinion? > > Yes. No. :-) > > Those possibly still stuck on obsolete versions of FreeBSD don't > need/want fancy new compilers and GCC 4.9 is still available for > use and does not exhibit this issue, correct? (If it does, nobody > reported any problems.) It is limited to gcc >=5, gcc-4.9 does not have the MD_FALLBACK_FRAME_STATE_FOR defined. >> I can 'ifdef' the new code and in the 'else' case we fall back to >> the already existing path. > > If it's "cheap", that might be nice. Attached, the test is running on gcc trunk and gcc-6. gcc-6 is the last one with java support and there we have quite extensive test cases which really test for this MD_FALLBACK_FRAME_STATE_FOR macro. These test cases, Throw_2 and co do pass. So I think the new bits should be fine. Also some coming asan test cases do pass with this addition too. > Thanks to the three of you - Tijl, Konstantin, and Andreas! Gruss, Andreas --------------40C450CC1D50A0A8104883DE Content-Type: text/plain; charset=UTF-8; x-mac-type="0"; x-mac-creator="0"; name="libgcc-fbsd-unwind.diff" Content-Transfer-Encoding: base64 Content-Disposition: attachment; filename="libgcc-fbsd-unwind.diff" SW5kZXg6IGxpYmdjYy9jb25maWcvaTM4Ni9mcmVlYnNkLXVud2luZC5oCj09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT0KLS0tIGxpYmdjYy9jb25maWcvaTM4Ni9mcmVlYnNkLXVud2luZC5oCShyZXZpc2lvbiAy NTQyMDUpCisrKyBsaWJnY2MvY29uZmlnL2kzODYvZnJlZWJzZC11bndpbmQuaAkod29ya2lu ZyBjb3B5KQpAQCAtMjgsNyArMjgsMTAgQEAKIAogI2luY2x1ZGUgPHN5cy90eXBlcy5oPgog I2luY2x1ZGUgPHNpZ25hbC5oPgorI2luY2x1ZGUgPHVuaXN0ZC5oPgorI2luY2x1ZGUgPHN5 cy9zeXNjdGwuaD4KICNpbmNsdWRlIDxzeXMvdWNvbnRleHQuaD4KKyNpbmNsdWRlIDxzeXMv dXNlci5oPgogI2luY2x1ZGUgPG1hY2hpbmUvc2lnZnJhbWUuaD4KIAogI2RlZmluZSBSRUdf TkFNRShyZWcpCXNmX3VjLnVjX21jb250ZXh0Lm1jXyMjIHJlZwpAQCAtMzYsNiArMzksMzkg QEAKICNpZmRlZiBfX3g4Nl82NF9fCiAjZGVmaW5lIE1EX0ZBTExCQUNLX0ZSQU1FX1NUQVRF X0ZPUiB4ODZfNjRfZnJlZWJzZF9mYWxsYmFja19mcmFtZV9zdGF0ZQogCisjaWZkZWYgS0VS Tl9QUk9DX1NJR1RSQU1QCisgLyogTmV3ZXIgdmVyc2lvbnMgb2YgRnJlZUJTRCwgPiBGcmVl QlNEIDkuMywgcHJvdmlkZSBhCisgICAga2Vybi5wcm9jLnNpZ3RyYW1wLjxwaWQ+IHN5c2N0 bCB0aGF0IHJldHVybnMgdGhlIGxvY2F0aW9uIG9mIHRoZQorICAgIHNpZ25hbCB0cmFtcG9s aW5lLiBXZSB1c2UgdGhpcyBpbmZvcm1hdGlvbiB0byBmaW5kIG91dCBpZiB3ZSdyZSBpbgor ICAgIGEgdHJhbXBvbGluZSBvciBub3QuCisgKi8KK3N0YXRpYyBpbnQKK3g4Nl82NF9vdXRz aWRlX3NpZ3RyYW1wX3JhbmdlICh1bnNpZ25lZCBjaGFyICpwYykKK3sKKyAgc3RhdGljIGlu dCBzaWd0cmFtcF9yYW5nZV9kZXRlcm1pbmVkID0gMDsKKyAgc3RhdGljIHVuc2lnbmVkIGNo YXIgKnNpZ3RyYW1wX3N0YXJ0LCAqc2lndHJhbXBfZW5kOworCisgIGlmIChzaWd0cmFtcF9y YW5nZV9kZXRlcm1pbmVkID09IDApCisgICAgeworICAgICAgc3RydWN0IGtpbmZvX3NpZ3Ry YW1wIGtzdCA9IHswfTsKKyAgICAgIHNpemVfdCBsZW4gPSBzaXplb2YgKGtzdCk7CisgICAg ICBpbnQgbWliWzRdID0geyBDVExfS0VSTiwgS0VSTl9QUk9DLCBLRVJOX1BST0NfU0lHVFJB TVAsIGdldHBpZCgpIH07CisKKyAgICAgIHNpZ3RyYW1wX3JhbmdlX2RldGVybWluZWQgPSAx OworICAgICAgaWYgKHN5c2N0bCAobWliLCA0LCAma3N0LCAmbGVuLCBOVUxMLCAwKSA9PSAw KQorICAgICAgeworICAgICAgICBzaWd0cmFtcF9yYW5nZV9kZXRlcm1pbmVkID0gMjsKKyAg ICAgICAgc2lndHJhbXBfc3RhcnQgPSBrc3Qua3NpZ3RyYW1wX3N0YXJ0OworICAgICAgICBz aWd0cmFtcF9lbmQgICA9IGtzdC5rc2lndHJhbXBfZW5kOworICAgICAgfQorICAgIH0KKyAg aWYgKHNpZ3RyYW1wX3JhbmdlX2RldGVybWluZWQgPCAyKSAgLyogc3lzY3RsIGZhaWxlZCBp ZiA8IDIgKi8KKyAgICByZXR1cm4gMTsKKworICByZXR1cm4gKHBjIDwgc2lndHJhbXBfc3Rh cnQgfHwgcGMgPj0gc2lndHJhbXBfZW5kKTsKK30KKyNlbmRpZgorCiBzdGF0aWMgX1Vud2lu ZF9SZWFzb25fQ29kZQogeDg2XzY0X2ZyZWVic2RfZmFsbGJhY2tfZnJhbWVfc3RhdGUKIChz dHJ1Y3QgX1Vud2luZF9Db250ZXh0ICpjb250ZXh0LCBfVW53aW5kX0ZyYW1lU3RhdGUgKmZz KQpAQCAtNDMsNiArNzksNyBAQAogICBzdHJ1Y3Qgc2lnZnJhbWUgKnNmOwogICBsb25nIG5l d19jZmE7CiAKKyNpZm5kZWYgS0VSTl9QUk9DX1NJR1RSQU1QCiAgIC8qIFByaW9yIHRvIEZy ZWVCU0QgOSwgdGhlIHNpZ25hbCB0cmFtcG9saW5lIHdhcyBsb2NhdGVkIGltbWVkaWF0ZWx5 CiAgICAgIGJlZm9yZSB0aGUgcHNfc3RyaW5ncy4gIFRvIHN1cHBvcnQgbm9uLWV4ZWN1dGFi bGUgc3RhY2tzIG9uIEFNRDY0LAogICAgICB0aGUgc2lndHJhbXAgd2FzIG1vdmVkIHRvIGEg c2hhcmVkIHBhZ2UgZm9yIEZyZWVCU0QgOS4gIFVuZm9ydHVuYXRlbHkKQEAgLTYyLDEyICs5 OSwxNSBAQAogICAgICAgICAmJiAqKHVuc2lnbmVkIGludCAqKShjb250ZXh0LT5yYSArICA4 KSA9PSAweDAxYTFjMGM3CiAgICAgICAgICYmICoodW5zaWduZWQgaW50ICopKGNvbnRleHQt PnJhICsgMTIpID09IDB4MDUwZjAwMDAgKSkKICAgICByZXR1cm4gX1VSQ19FTkRfT0ZfU1RB Q0s7CisjZWxzZQorICBpZiAoeDg2XzY0X291dHNpZGVfc2lndHJhbXBfcmFuZ2UoY29udGV4 dC0+cmEpKQorICAgIHJldHVybiBfVVJDX0VORF9PRl9TVEFDSzsKKyNlbmRpZgogCiAgIHNm ID0gKHN0cnVjdCBzaWdmcmFtZSAqKSBjb250ZXh0LT5jZmE7CiAgIG5ld19jZmEgPSBzZi0+ UkVHX05BTUUocnNwKTsKICAgZnMtPnJlZ3MuY2ZhX2hvdyA9IENGQV9SRUdfT0ZGU0VUOwot ICAvKiBSZWdpc3RlciA3IGlzIHJzcCAgKi8KLSAgZnMtPnJlZ3MuY2ZhX3JlZyA9IDc7Cisg IGZzLT5yZWdzLmNmYV9yZWcgPSAgX19MSUJHQ0NfU1RBQ0tfUE9JTlRFUl9SRUdOVU1fXzsK ICAgZnMtPnJlZ3MuY2ZhX29mZnNldCA9IG5ld19jZmEgLSAobG9uZykgY29udGV4dC0+Y2Zh OwogCiAgIC8qIFRoZSBTVlI0IHJlZ2lzdGVyIG51bWJlcmluZyBtYWNyb3MgYXJlbid0IHVz YWJsZSBpbiBsaWJnY2MuICAqLwo= --------------40C450CC1D50A0A8104883DE--