From owner-freebsd-x11@freebsd.org Fri Jan 15 08:19:47 2016 Return-Path: Delivered-To: freebsd-x11@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 77547A836A9 for ; Fri, 15 Jan 2016 08:19:47 +0000 (UTC) (envelope-from jean-sebastien.pedron@dumbbell.fr) Received: from mail.made4.biz (mail.made4.biz [IPv6:2001:41d0:2:c018::1:3]) (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 447B71D93 for ; Fri, 15 Jan 2016 08:19:47 +0000 (UTC) (envelope-from jean-sebastien.pedron@dumbbell.fr) Received: from 141.7.19.93.rev.sfr.net ([93.19.7.141] helo=magellan.dumbbell.fr) by mail.made4.biz with esmtpsa (TLSv1.2:ECDHE-RSA-AES128-GCM-SHA256:128) (Exim 4.86 (FreeBSD)) (envelope-from ) id 1aJzbl-00022U-Cw; Fri, 15 Jan 2016 09:19:45 +0100 Subject: Re: build-able drm-i915-update-38 To: Nikola Pajkovsky References: <87oacpdguk.fsf@gooddata.com> <5696A19A.2030009@dumbbell.fr> <864mehduk6.fsf@freebsd.lan> Cc: freebsd-x11@freebsd.org From: =?UTF-8?Q?Jean-S=c3=a9bastien_P=c3=a9dron?= X-Enigmail-Draft-Status: N1110 Message-ID: <5698AB9C.9070005@dumbbell.fr> Date: Fri, 15 Jan 2016 09:19:40 +0100 User-Agent: Mozilla/5.0 (X11; FreeBSD amd64; rv:38.0) Gecko/20100101 Thunderbird/38.5.0 MIME-Version: 1.0 In-Reply-To: <864mehduk6.fsf@freebsd.lan> Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="HQXEtHvxqUXVI8r9barKrQoWDPulK6URu" X-BeenThere: freebsd-x11@freebsd.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: X11 on FreeBSD -- maintaining and support List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 15 Jan 2016 08:19:47 -0000 This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --HQXEtHvxqUXVI8r9barKrQoWDPulK6URu Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable On 13/01/2016 22:43, Nikola Pajkovsky wrote: >> I rebuild a full kernel from scratch and it does build fine here. Ther= e >> is no "unused variable" warning for me. Do you build a non-debug kerne= l >> perhaps? I don't plan to build-test such a kernel now, I'll do it when= >> the patch is finished. >=20 > Oh, everytime? I have found out, that I use "make buildkernel -DKERNFAST" so only the modifed files are recompiled. I just rebuild the kernel from scratch this time to double-check if I missed a warning. > $ cd sys/modules/drm2 > $ make -j9 > $ make DESTDIR=3D/boot/ KMODDIR=3Dkernel.i915 install >=20 > is engough (only after I have fully built and booted kernel) Yes, this works fine, but this doesn't use many defaults set by GENERIC, such as debugging options or 32-bit support on 64-bit archs. The problem you hit is likely because I build a debug kernel (the variable is used) and not you (only the declaration remains). > gen6_ggtt_bind_object() > const int max_entries =3D dev_priv->mm.gtt->gtt_total_entries - fir= st_entry; > BUG_ON(i > max_entries); >=20 > Seems like BUG_ON is noop, which should not be. BUG_ON() is a KASSERT() which is defined to nothing on a non-debug kernel. The variable should be declared in: #if defined(INVARIANTS) =2E.. #endif If you are still willing to send a pull request, you're welcome to do so! I know the change is small, so if you don't feel like cloning an entire repository just for that, I understand and I will fix it. Unlike what I said in my previous email, the pull request should be against the drm-i915-update-38 branch. --=20 Jean-S=C3=A9bastien P=C3=A9dron --HQXEtHvxqUXVI8r9barKrQoWDPulK6URu 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 iQJ8BAEBCgBmBQJWmKucXxSAAAAAAC4AKGlzc3Vlci1mcHJAbm90YXRpb25zLm9w ZW5wZ3AuZmlmdGhob3JzZW1hbi5uZXQ2NzA4N0ZEMUFFQUUwRTEyREJDNkE2RjAz OUU5OTc2MUE1RkQ5NENDAAoJEDnpl2Gl/ZTM8R0QAIPknRRbg0kQRPZxI8NT4bo3 erbkiQyx0JVsjw987IENeHQETaRXFt70g//n2hO8u8uCWCLyn5SKi2/+M4yJrjXi WcHMLDwn94oGlyyF6RyVCVAbY48sdepQ7JTLTuu/u99Qrs8rMZIAz+8PfeaPB0Sh r/wmUFN99qELKIaYS1iQu5A0nIdB0kVsov8qNL6re3GJymKObwzAs7FzeB5ufTZW lopNddkm6++iDTU+NFbEQsdHjEk7/5AcF9whaaNAXANvoKQ5OfVLetjDJ4xqOmV9 9a3iR9S34YW6pRENIXE9g4bgrJANUwU91pC14bE883xQFmE9RMHS7DDRULMY+pmw 21QXqtoEjY7xmBZ2WMYh6nVA7JcPXJnHaL1NtZXjP4vc1g/YOhT3kLyPWHIDBOei It/bD4AlozvYMUEBEKkjxHSlYQDGTiep7l4qdKYNT/F5SSpNoD3Kd4venUS8cHNX fa4jPGd7VjdIdVzPey4lkLWGH5CKDCcvUrqy/2W8IwkmI2GH7Fwp1xQnaexBx9IU QjTMYjg90JpXw5ySd3yGOvkDxAZ71m2rn/+H3el2Mm+Ov+SFmNUGcZcJmhWGKjsh 9UtVhIWIz9r8jPpCigFHEdaKs5uyoUzddVzJwbf7h89OClJiItTFSjnZBF8aRNnx 8zpsSHjeKARNdhDHaat/ =ZC4I -----END PGP SIGNATURE----- --HQXEtHvxqUXVI8r9barKrQoWDPulK6URu--