From owner-svn-src-all@freebsd.org Sat Jun 9 17:56:23 2018 Return-Path: Delivered-To: svn-src-all@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 59DD210186DA; Sat, 9 Jun 2018 17:56:23 +0000 (UTC) (envelope-from bdrewery@FreeBSD.org) Received: from freefall.freebsd.org (freefall.freebsd.org [96.47.72.132]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client CN "freefall.freebsd.org", Issuer "Let's Encrypt Authority X3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id EAECD69129; Sat, 9 Jun 2018 17:56:22 +0000 (UTC) (envelope-from bdrewery@FreeBSD.org) Received: from mail.xzibition.com (unknown [127.0.1.132]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by freefall.freebsd.org (Postfix) with ESMTPS id B858AF1F3; Sat, 9 Jun 2018 17:56:22 +0000 (UTC) (envelope-from bdrewery@FreeBSD.org) Received: from mail.xzibition.com (localhost [172.31.3.2]) by mail.xzibition.com (Postfix) with ESMTP id 05DB89EC; Sat, 9 Jun 2018 17:56:22 +0000 (UTC) X-Virus-Scanned: amavisd-new at mail.xzibition.com Received: from mail.xzibition.com ([172.31.3.2]) by mail.xzibition.com (mail.xzibition.com [172.31.3.2]) (amavisd-new, port 10026) with LMTP id o-Qjp_yXYcjC; Sat, 9 Jun 2018 17:56:18 +0000 (UTC) Subject: Re: svn commit: r334883 - head/cddl/contrib/opensolaris/tools/ctf/cvt DKIM-Filter: OpenDKIM Filter v2.10.3 mail.xzibition.com D378B9E7 To: Mark Johnston , "O. Hartmann" Cc: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org References: <201806091510.w59FAnIW036815@repo.freebsd.org> <20180609193527.46a0f9a6@thor.intern.walstatt.dynvpn.de> <20180609173951.GB1394@pesky> From: Bryan Drewery Openpgp: preference=signencrypt Autocrypt: addr=bdrewery@FreeBSD.org; prefer-encrypt=mutual; keydata= xsBNBFJphmsBCADiFgmS4bIzwZijrS31SjEMzg+n5zNellgM+HkShwehpqCiyhXdWrvH6dTZ a6u50pbUIX7doTR7W7PQHCjCTqtpwvcj0eulZva+iHFp+XrbgSFHn+VVXgkYP2MFySyZRFab D2qqzJBEJofhpv4HvY6uQI5K99pMqKr1Z/lHqsijYYu4RH2OfwB5PinId7xeldzWEonVoCr+ rfxzO/UrgA6v/3layGZcKNHFjmc3NqoN1DXtdaEHqtjIozzbndVkH6lkFvIpIrI6i5ox8pwp VxsxLCr/4Musd5CWgHiet5kSw2SzNeA8FbxdLYCpXNVu+uBACEbCUP+CSNy3NVfEUxsBABEB AAHNJEJyeWFuIERyZXdlcnkgPGJkcmV3ZXJ5QEZyZWVCU0Qub3JnPsLAgAQTAQoAKgIbAwUL CQgHAwUVCgkICwUWAwIBAAIeAQIXgAIZAQUCWujOIgUJCmB7NwAKCRA113G7bkaXz/xpB/9b /UWIPbieY1IeIuHF2pyYPE7Hytkh3HVsxMA0F5Ma2AYQsXZZeKNKWrF7RPyDyDwUklLHJkhm k3EfClBbHxf08kMIm1vWCJRtgxic9knY/bzYGiWMpHjg3cSd1XfrYH1autYqTZAjDwIkgOjU dR//Tbn4V36sY7y2jz+kdMVWvK53U32aZqiwBbCn4DPe1wSZcUs17mV/0uZdIoGdj74B1orN A/0py5vHYo6HcbBNoaR8pKRLf5VZNRsxqGIMhTucx4SJWcHpuRBWYyvJSFzwvxdK4ZD4Yqoc kFGPVtOXktVMai9exrLvP3G77fKMu8DI6j4QRU4wCesnHuIfRPFuzsBNBFJphmsBCACiVFPf kNfaFtUSuY0395ueo/rMyHPGPQ2iwvERFCpeFGSQSgagpenNHLpFQKTg/dl6FOoST5tqyxMq fyHGHDzzU51bvA/IfaGoNi/BIhTe/toZNMRvpcI3PLjiGcnJnuwCCbAVOAGdb+t5cZtpNdOI cKYmrYG3u9RiBpe6dTF+qLrD/8Bs1wjhduQ8fcNNgnkXu8xDH4ZxY0lIc3QgvYWp9vimlQe6 iKjUd2/DX28ETZcD5h6pYV331KMPTrEI0p0yvFijUZce8c1XHFyL1j9sBAha5qpszJl6Uq5i LolhKRcGfcdmtD72vHQjUYglUyudSJUVyo2gMYjdbiFKzJulABEBAAHCwGUEGAEKAA8FAlJp hmsCGwwFCQlmAYAACgkQNddxu25Gl89UPggA2mGQp28yCUKsJ6KHFVy/lpHfoQrKF+s7HfKT U2ObVeVNX4I8ZdW1UO48mRqxEOwY8r5YSH6X06OmiqCX2aSMXg3N06/l+ztlB0+UGGlkXBjv l9/nii+bC6b8XWuu0X7Qpb9oYBK9YtoaoyuVplAmjdj/cPou65meKIaS1yDTjHh450DrW8Qg he6l0bFX4BHKTSm99U90ML7EY19B6iI2BZSqWutVsyD71oAREY6NGgDpCOIO6FS41+WeYCDR j8vsa/BiaoX2d2SBDsCwsEwe9fg5PYMi2uVIhvL6OrxnwOdB+TkgvOy5zZSNO29UG/JilZKo Ndz2wpEaUzChGGqLvcLAZQQYAQoADwIbDAUCWujOKAUJCmB7PQAKCRA113G7bkaXz6bkB/9H dUR3E0wBwMh6z0AOFDKh+PbRI9Xd4IncdhE55tNK410650a3gADIDwqz3i72GIinkgaxzpEO xP1bs7a+BeF3p5Xd6Jjk6J/nEshisgNW7VjUbJHFGs8Sf9A6oM3q4VkI/ArVo5qkZxgKs72U HSAy5NV+AdqdTrWuAL20xfQ6gA7JF35Xf8zyUM2GMl0X8ik7dJ1jMp+TB27LipqbDgamFzH9 F9hC9gur94OQ/x3nQ+mFZ1uipYHA1EdrKuhb/Ts4bN/Ezl8nmYGxc9Bw7ZBxGOTId/rEIzoe LWpAvg6dcw0T9lNfSWc6PX+kf3dOXNIdkw9NqKID8wEPe8axcGYG Organization: FreeBSD Message-ID: <80b8176b-61c5-c3b2-49b1-36ba9be77d1b@FreeBSD.org> Date: Sat, 9 Jun 2018 13:56:18 -0400 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.12; rv:52.0) Gecko/20100101 Thunderbird/52.8.0 MIME-Version: 1.0 In-Reply-To: <20180609173951.GB1394@pesky> Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="xAno8dCexq5pH0BiwSog1ZnXvV9NY1ODs" X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.26 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 09 Jun 2018 17:56:23 -0000 This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --xAno8dCexq5pH0BiwSog1ZnXvV9NY1ODs Content-Type: multipart/mixed; boundary="N11eYmmkGd9QdcWYJEoq1ie8ISq3Cx3ZB"; protected-headers="v1" From: Bryan Drewery To: Mark Johnston , "O. Hartmann" Cc: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Message-ID: <80b8176b-61c5-c3b2-49b1-36ba9be77d1b@FreeBSD.org> Subject: Re: svn commit: r334883 - head/cddl/contrib/opensolaris/tools/ctf/cvt References: <201806091510.w59FAnIW036815@repo.freebsd.org> <20180609193527.46a0f9a6@thor.intern.walstatt.dynvpn.de> <20180609173951.GB1394@pesky> In-Reply-To: <20180609173951.GB1394@pesky> --N11eYmmkGd9QdcWYJEoq1ie8ISq3Cx3ZB Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: quoted-printable On 6/9/18 1:39 PM, Mark Johnston wrote: > On Sat, Jun 09, 2018 at 07:35:00PM +0200, O. Hartmann wrote: >> -----BEGIN PGP SIGNED MESSAGE----- >> Hash: SHA512 >> >> Am Sat, 9 Jun 2018 15:10:49 +0000 (UTC) >> Mark Johnston schrieb: >> >>> Author: markj >>> Date: Sat Jun 9 15:10:49 2018 >>> New Revision: 334883 >>> URL: https://svnweb.freebsd.org/changeset/base/334883 >>> >>> Log: >>> Don't process DWARF generated from non-C/C++ code. >>> =20 >>> ctfconvert(1) is not designed to handle DWARF generated from such c= ode, >>> and will generally fail in non-obvious ways. Use an explicit check= to >>> help catch such potential failures. >>> =20 >>> Reported by: Johannes Lundberg >>> MFC after: 2 weeks >>> >>> Modified: >>> head/cddl/contrib/opensolaris/tools/ctf/cvt/dwarf.c >>> >>> Modified: head/cddl/contrib/opensolaris/tools/ctf/cvt/dwarf.c >>> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D >>> --- head/cddl/contrib/opensolaris/tools/ctf/cvt/dwarf.c Sat Jun 9 15= :10:39 >>> 2018 (r334882) +++ head/cddl/contrib/opensolaris/tools/ctf/cvt/dwarf.= c >>> Sat Jun 9 15:10:49 2018 (r334883) @@ -1901,7 +1901,7 @@ should_have_= dwarf(Elf >>> *elf) int >>> dw_read(tdata_t *td, Elf *elf, char *filename __unused) >>> { >>> - Dwarf_Unsigned abboff, hdrlen, nxthdr; >>> + Dwarf_Unsigned abboff, hdrlen, lang, nxthdr; >>> Dwarf_Half vers, addrsz, offsz; >>> Dwarf_Die cu =3D 0; >>> Dwarf_Die child =3D 0; >>> @@ -1941,8 +1941,8 @@ dw_read(tdata_t *td, Elf *elf, char *filename _= _unused >>> } >>> =20 >>> if ((rc =3D dwarf_next_cu_header_b(dw.dw_dw, &hdrlen, &vers, &abbof= f, >>> - &addrsz, &offsz, NULL, &nxthdr, &dw.dw_err)) !=3D DW_DLV_OK) { >>> - if (dw.dw_err.err_error =3D=3D DW_DLE_NO_ENTRY) >>> + &addrsz, &offsz, NULL, &nxthdr, &dw.dw_err)) !=3D DW_DLV_OK) { >>> + if (dw.dw_err.err_error =3D=3D DW_DLE_NO_ENTRY) >>> exit(0); >>> else >>> terminate("rc =3D %d %s\n", rc, dwarf_errmsg(dw.dw_err)); >>> @@ -1971,6 +1971,25 @@ dw_read(tdata_t *td, Elf *elf, char *filename = __unused >>> debug(1, "DWARF emitter: %s\n", prod); >>> free(prod); >>> } >>> + >>> + if (dwarf_attrval_unsigned(cu, DW_AT_language, &lang, &dw.dw_err) =3D= =3D 0) >>> + switch (lang) { >>> + case DW_LANG_C: >>> + case DW_LANG_C89: >>> + case DW_LANG_C99: >>> + case DW_LANG_C11: >>> + case DW_LANG_C_plus_plus: >>> + case DW_LANG_C_plus_plus_03: >>> + case DW_LANG_C_plus_plus_11: >>> + case DW_LANG_C_plus_plus_14: >>> + break; >>> + default: >>> + terminate("file contains DWARF for unsupported " >>> + "language %d", lang); >>> + } >>> + else >>> + warning("die %llu: failed to get language attribute: %s\n", >>> + die_off(&dw, cu), dwarf_errmsg(dw.dw_err)); >>> =20 >>> if ((dw.dw_cuname =3D die_name(&dw, cu)) !=3D NULL) { >>> char *base =3D xstrdup(basename(dw.dw_cuname)); >>> _______________________________________________ >>> svn-src-head@freebsd.org mailing list >>> https://lists.freebsd.org/mailman/listinfo/svn-src-head >>> To unsubscribe, send any mail to "svn-src-head-unsubscribe@freebsd.or= g" >> This commit makes buildworld (WITH_META_MODE) failing: >=20 > I added these identifiers in r334881, and the CI build is passing. Migh= t > this be a problem with META_MODE? >=20 >> [...] >> =3D=3D=3D> cddl/usr.bin/ctfconvert (obj,all,install) >> Building /usr/obj/usr/src/amd64.amd64/tmp/obj-tools/cddl/usr.bin/ctfco= nvert/dwarf.o obj-tools means it is trying to build a HOST NATIVE version of this. Where are these defined? In this case they need to be defined in /usr/include already unless the build has -I.CURDIR/somewhere/to/headers instead. >> - --- dwarf.o --- >> /usr/src/cddl/contrib/opensolaris/tools/ctf/cvt/dwarf.c:1980:8: error:= use of undeclared >> identifier 'DW_LANG_C11' case DW_LANG_C11: >> ^ >> /usr/src/cddl/contrib/opensolaris/tools/ctf/cvt/dwarf.c:1982:8: error:= use of undeclared >> identifier 'DW_LANG_C_plus_plus_03' case DW_LANG_C_plus_plus_03: >> ^ >> /usr/src/cddl/contrib/opensolaris/tools/ctf/cvt/dwarf.c:1983:8: error:= use of undeclared >> identifier 'DW_LANG_C_plus_plus_11' case DW_LANG_C_plus_plus_11: >> ^ >> /usr/src/cddl/contrib/opensolaris/tools/ctf/cvt/dwarf.c:1984:8: error:= use of undeclared >> identifier 'DW_LANG_C_plus_plus_14' case DW_LANG_C_plus_plus_14: >> ^ >> 4 errors generated. >> *** [dwarf.o] Error code 1 --=20 Regards, Bryan Drewery --N11eYmmkGd9QdcWYJEoq1ie8ISq3Cx3ZB-- --xAno8dCexq5pH0BiwSog1ZnXvV9NY1ODs Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- iQEzBAEBCAAdFiEE+Rc8ssOq6npcih8JNddxu25Gl88FAlscFMIACgkQNddxu25G l88pIAgApDMCc6Fp4TsWioe0uuL9ro98tgHa2rtemKiyB/W20I7UhBniUOLSv8oQ X8UFUM10NEeBT0qpYpH1T2x1JfN4f0fu5K0mDqL67j6hgJItTZqSr+O7Q7y8t6v9 ki1LFDBQpurwXPTQ54UpL2Oi+dfsqCZB//7ja/AsSUlvqXyt9ybnWA7AtBbriF8l j6jARKc4jFhhsgg+xxbUGnak8B/5CMYU3/uQK5m9TqaDJdb5DRzB+d43DG0rNM41 EXmS7dyfipBZguLpAkU+J1rFLhB5ul1sP6V2YcvsCL/qcTwIhQ++5qjG89SC6lg2 TIFSU9A0Wrt7Rsk84LlGK8CsCCp/Rw== =MbHG -----END PGP SIGNATURE----- --xAno8dCexq5pH0BiwSog1ZnXvV9NY1ODs--