From owner-freebsd-current@FreeBSD.ORG Tue Mar 5 15:27:13 2013 Return-Path: Delivered-To: freebsd-current@freebsd.org Received: from mx1.freebsd.org (mx1.FreeBSD.org [8.8.178.115]) by hub.freebsd.org (Postfix) with ESMTP id E00E02B5 for ; Tue, 5 Mar 2013 15:27:13 +0000 (UTC) (envelope-from jean-sebastien.pedron@dumbbell.fr) Received: from mail.made4.biz (unknown [IPv6:2001:41d0:1:7018::1:3]) by mx1.freebsd.org (Postfix) with ESMTP id 9667686E for ; Tue, 5 Mar 2013 15:27:13 +0000 (UTC) Received: from [2001:1b48:10b:cafe:225:64ff:febe:589f] (helo=viking.yzserv.com) by mail.made4.biz with esmtpsa (TLSv1:DHE-RSA-CAMELLIA256-SHA:256) (Exim 4.80.1 (FreeBSD)) (envelope-from ) id 1UCtlo-0001NI-BC for freebsd-current@freebsd.org; Tue, 05 Mar 2013 16:27:12 +0100 Message-ID: <51360ECB.4000407@dumbbell.fr> Date: Tue, 05 Mar 2013 16:27:07 +0100 From: =?UTF-8?B?SmVhbi1Tw6liYXN0aWVuIFDDqWRyb24=?= User-Agent: Mozilla/5.0 (X11; FreeBSD amd64; rv:17.0) Gecko/20130220 Thunderbird/17.0.3 MIME-Version: 1.0 To: freebsd-current@freebsd.org Subject: Re: r247835: drm2 code breaks buildkernel References: <5135C70B.50906@zedat.fu-berlin.de> <5135CD0E.8040801@dumbbell.fr> <5135DE36.9010303@zedat.fu-berlin.de> <20130305123016.GE1483@glenbarber.us> <5135FD78.1050608@FreeBSD.org> <20130305095417.3780d487@shibato> In-Reply-To: <20130305095417.3780d487@shibato> X-Enigmail-Version: 1.5.1 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="----enig2QHTXUAOPWHLISLOMTTGH" X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.14 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, 05 Mar 2013 15:27:13 -0000 This is an OpenPGP/MIME signed message (RFC 4880 and 3156) ------enig2QHTXUAOPWHLISLOMTTGH Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable On 05.03.2013 15:54, J.R. Oldroyd wrote: > A fix for these is in my big "get it to compile" patch that I emailed > you both the other day. Sorry, I didn't take the time to read it yet :-/ >>> dev/drm2/ttm/ttm_page_alloc.c:122: warning: declaration does not decl= are anything >>> dev/drm2/ttm/ttm_page_alloc.c:123: warning: declaration does not decl= are anything > > Experimentation shows that this warning is triggered because we use > -std=3Diso9899:1999. It can be turned off again by adding --ms-extensi= ons > too. >=20 > Alternatively, my big patch replaces all these anon unions with > named ones. There are lots of these in this code, though. Doing > this adds lots of patch bloat. Yes, the flag is preferable. I didn't have the time to test it earlier either, until now. I confirm that it works with both clang and gcc. > There is a further problem not mentioned here. Three of the files > make use of a pointer to a volatile int but later cast this to a > (void *). Because we also have -Wcast-qual, this cast triggers > "cast discards qualifier on pointer target type" warnings and because > of -Werror, this then aborts. What's the best way to fix that? Those warnings are in the radeon driver, not ttm, aren't they? At least, the build finishes properly on my computer with gcc and clang with just -fms-extensions. --=20 Jean-S=C3=A9bastien P=C3=A9dron ------enig2QHTXUAOPWHLISLOMTTGH Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.19 (FreeBSD) Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/ iEYEARECAAYFAlE2DtAACgkQa+xGJsFYOlPvpgCglJEZ5Zb45iB5sBJUWR98QriB 1ykAn1Qj/ZYXjUT5hBGfLIJgzQZMhH+G =jWus -----END PGP SIGNATURE----- ------enig2QHTXUAOPWHLISLOMTTGH--