From owner-svn-src-stable-10@freebsd.org Wed Jan 13 16:56:42 2016 Return-Path: Delivered-To: svn-src-stable-10@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 31C80A81B00; Wed, 13 Jan 2016 16:56:42 +0000 (UTC) (envelope-from slw@zxy.spb.ru) Received: from zxy.spb.ru (zxy.spb.ru [195.70.199.98]) (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 DE8F8163D; Wed, 13 Jan 2016 16:56:41 +0000 (UTC) (envelope-from slw@zxy.spb.ru) Received: from slw by zxy.spb.ru with local (Exim 4.86 (FreeBSD)) (envelope-from ) id 1aJOis-00093t-ID; Wed, 13 Jan 2016 19:56:38 +0300 Date: Wed, 13 Jan 2016 19:56:38 +0300 From: Slawa Olhovchenkov To: Matthew Ahrens Cc: Warner Losh , Allan Jude , src-committers , svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-10@freebsd.org Subject: Re: svn commit: r293776 - stable/10/usr.sbin/fstyp Message-ID: <20160113165638.GP4535@zxy.spb.ru> References: <201601121638.u0CGc9cJ076217@repo.freebsd.org> <20160113131858.GN70867@zxy.spb.ru> <56967AA1.3050201@freebsd.org> <8694B9BA-0F48-4D1E-9974-2C095FF519A8@bsdimp.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/1.5.24 (2015-08-30) X-SA-Exim-Connect-IP: X-SA-Exim-Mail-From: slw@zxy.spb.ru X-SA-Exim-Scanned: No (on zxy.spb.ru); SAEximRunCond expanded to false X-BeenThere: svn-src-stable-10@freebsd.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: SVN commit messages for only the 10-stable src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 13 Jan 2016 16:56:42 -0000 On Wed, Jan 13, 2016 at 08:35:14AM -0800, Matthew Ahrens wrote: > On Wed, Jan 13, 2016 at 8:32 AM, Warner Losh wrote: > > > > > > On Jan 13, 2016, at 9:26 AM, Allan Jude wrote: > > > > > > On 2016-01-13 08:18, Slawa Olhovchenkov wrote: > > >> On Tue, Jan 12, 2016 at 04:38:09PM +0000, Allan Jude wrote: > > >> > > >>> Author: allanjude > > >>> Date: Tue Jan 12 16:38:09 2016 > > >>> New Revision: 293776 > > >>> URL: https://svnweb.freebsd.org/changeset/base/293776 > > >>> > > >>> Log: > > >>> MFC: r284589 > > >>> Add the ability to detect ZFS and GELI encrypted file systems to > > fstyp(8) > > >>> > > >>> MFC: r284644 > > >>> Fix GCC Warnings > > >>> > > >>> MFC: r284728 > > >>> Only build ZFS support in absense of WITHOUT_ZFS > > >> > > >> What purpose for switch off ZFS support in case WITHOUT_ZFS? > > >> > > > > > > If the user builds FreeBSD WITHOUT_CDDL or WITHOUT_ZFS, then they are > > > requesting that we not build the ZFS code, so building it anyway would > > > be against their wishes. The most likely reason they would request this > > > is for license reasons. And what main reason in this case: license or memory consumption? > > For x86, that may be true. For arm and mips platforms, ZFS is too much of a > > memory pig-dog to run on much of the kit we run on in that space. Turning > > it > > off is part of removing the type 2 muda from the system. How many additional memory footpint need for support ZFS detecting? > > In any event, the double negative is true. Don’t build ZFS when its > > absence has > > been requested with WITHOUT_ZFS is correct. > > > > Some of the confusion may be because this is not building ZFS - > just support for detecting ZFS pools in fstyp. Which would be reasonable > to do even if you are not building support for ZFS. Yes, for diagnostic purpose (interactive and in srcipts). Also, ZFS may be in FUSE or may be exported by iSCSI (and ZFS detecting need for selecting raw disk for exporting). > However, the fstyp ZFS code relies on libzfs and libnvpair which are CDDL > licensed. So if you did not build ZFS (or CDDL), the fstyp code could not > work. May be better relies on WITHOUT_CDDL? If main purpose is license reasons?