From owner-svn-src-all@FreeBSD.ORG Tue Jul 30 20:40:54 2013 Return-Path: Delivered-To: svn-src-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTP id A6376338; Tue, 30 Jul 2013 20:40:54 +0000 (UTC) (envelope-from etnapierala@gmail.com) Received: from mail-bk0-x231.google.com (mail-bk0-x231.google.com [IPv6:2a00:1450:4008:c01::231]) (using TLSv1 with cipher ECDHE-RSA-RC4-SHA (128/128 bits)) (No client certificate requested) by mx1.freebsd.org (Postfix) with ESMTPS id A908C2A01; Tue, 30 Jul 2013 20:40:53 +0000 (UTC) Received: by mail-bk0-f49.google.com with SMTP id r7so2239546bkg.8 for ; Tue, 30 Jul 2013 13:40:51 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=sender:content-type:mime-version:subject:from:in-reply-to:date:cc :content-transfer-encoding:message-id:references:to:x-mailer; bh=VWSDJS5u0t3OGtAUa6UTteb3J3OJApVMOA8Kzezpw9A=; b=Vn/sgyE3euFYXPHyEgS2ejQeXLhn+gAEzrzGAfRRVfl070Kc9qlHKiAFs52wewAfxr lZAmpSvMGGCbCD3qG23eCcTsfMEGK7nWY+LkxHkO8zGwPKP2cITyvZPxAwD+T8CZqmVz 82PBi5bI9cuJjILD77eVOv1VbSt3LzHTH97kpxrS/0pJtLcxCZOhWyRbE5Wdh8f4rFWX xWLFz4y21VooBBEHHfx9tu1uNzvqO9JpQAsZx+bAF2unRINoVI9XwuAmNlI3zkL93u1F VeNuJNiEq59DmI3wC5Rv5ixfv7Y6aMtT7AQTW+/xo3Sh8yCvSsauB2gi+6Mt93OZMxC4 3LXA== X-Received: by 10.205.21.129 with SMTP id qs1mr9771397bkb.13.1375216851644; Tue, 30 Jul 2013 13:40:51 -0700 (PDT) Received: from [192.168.1.4] (adea192.neoplus.adsl.tpnet.pl. [79.184.78.192]) by mx.google.com with ESMTPSA id ok9sm17910731bkb.8.2013.07.30.13.40.50 for (version=TLSv1 cipher=ECDHE-RSA-RC4-SHA bits=128/128); Tue, 30 Jul 2013 13:40:51 -0700 (PDT) Sender: =?UTF-8?Q?Edward_Tomasz_Napiera=C5=82a?= Content-Type: text/plain; charset=iso-8859-2 Mime-Version: 1.0 (Mac OS X Mail 6.5 \(1508\)) Subject: Re: svn commit: r249410 - in head/sys: amd64/conf arm/conf cam/ctl conf i386/conf ia64/conf modules/ctl sparc64/conf From: =?iso-8859-2?Q?Edward_Tomasz_Napiera=B3a?= In-Reply-To: <20130730202820.GM56034@alchemy.franken.de> Date: Tue, 30 Jul 2013 22:40:49 +0200 Content-Transfer-Encoding: quoted-printable Message-Id: References: <201304121625.r3CGP3Dv068400@svn.freebsd.org> <20130729203848.GG948@alchemy.franken.de> <517A021B-6C7B-48C3-9AC2-CBA937BBAC80@FreeBSD.org> <20130730202820.GM56034@alchemy.franken.de> To: Marius Strobl X-Mailer: Apple Mail (2.1508) Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org, ken@freebsd.org X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.14 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: Tue, 30 Jul 2013 20:40:54 -0000 Wiadomo=B6=E6 napisana przez Marius Strobl w = dniu 30 lip 2013, o godz. 22:28: > On Tue, Jul 30, 2013 at 09:57:21PM +0200, Edward Tomasz Napiera?a = wrote: >> Wiadomo?? napisana przez Marius Strobl w = dniu 29 lip 2013, o godz. 22:38: >>> On Fri, Apr 12, 2013 at 04:25:03PM +0000, Edward Tomasz Napierala = wrote: >>>> Author: trasz >>>> Date: Fri Apr 12 16:25:03 2013 >>>> New Revision: 249410 >>>> URL: http://svnweb.freebsd.org/changeset/base/249410 >>>>=20 >>>> Log: >>>> Remove ctl(4) from GENERIC. Also remove 'options CTL_DISABLE' >>>> and kern.cam.ctl.disable tunable; those were introduced as a = workaround >>>> to make it possible to boot GENERIC on low memory machines. >>>>=20 >>>> With ctl(4) being built as a module and automatically loaded by = ctladm(8), >>>> this makes CTL work out of the box. >>>>=20 >>>=20 >>> Uhm, shouldn't r249328 and the above change be MFCed to stable/9 in >>> order to reduce the default memory footprint there? >>=20 >> Yup, my bad. I'm waiting for reply from re@ whether it's a good = thing >> to do at this point of the release process. >>=20 >=20 > Yeah, I saw that request and was looking into what it would actually > take to remove ctl(4) from GENERICs in stable/9. Unfortunately, it's > not exactly as straight forward as MFCing the two above revisions 1:1. > For one, IMO kern.cam.ctl.disable shouldn't be removed there in case > someone uses an old kernel configuration file having "device ctl" but > relies on that tunable to work. That's no big deal, though, and makes > the merge even less intrusive. However, the real problem is that = ctl.ko > currently is globally disabled in stable/9 as it doesn't build with = PAE/ > XEN. Actually, the compiler is right to complain as a 32-bit = void-pointer > is casted to a 64-bit integer in that case. What I don't get so far is > why on earth this doesn't break compilation in head ... Damn. Now that you mention it... yeah, it was me who disconnected it, in r249530. I had no idea what's going on either.