From owner-freebsd-current@FreeBSD.ORG Tue Nov 17 16:07:20 2009 Return-Path: Delivered-To: current@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 72398106568D for ; Tue, 17 Nov 2009 16:07:20 +0000 (UTC) (envelope-from ndenev@gmail.com) Received: from mail-fx0-f227.google.com (mail-fx0-f227.google.com [209.85.220.227]) by mx1.freebsd.org (Postfix) with ESMTP id 00C358FC20 for ; Tue, 17 Nov 2009 16:07:19 +0000 (UTC) Received: by fxm27 with SMTP id 27so145307fxm.3 for ; Tue, 17 Nov 2009 08:07:19 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:received:received:subject:mime-version :content-type:from:in-reply-to:date:cc:content-transfer-encoding :message-id:references:to:x-mailer; bh=aSLi0M0/KEJ4hbivMFcCZSmE0ZYUqRawNFebS09plaY=; b=CLsmQqYuWrOkPbOzB09ZXXaYdEkTpD7O/VPncLWoUfPCcBv3ro9BcK2x/VDqqQuLws UYa0FbweB2XrHit6lUUxIq4Ez6AGYRShORlptXOG3YykXFENgNLkm4i+aqHwD9VXd8ZM BeICI62WxJEf3Q4jbxl7nnAScNkcR2MAdvHGs= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=subject:mime-version:content-type:from:in-reply-to:date:cc :content-transfer-encoding:message-id:references:to:x-mailer; b=g0Cr+o3veHSHGDcYA1jzJpOQtWng5V9h7n9kvrbiZ8uTD/wyesZ8OPm5fRJ/MqHc3t gPBROdRbarCOxTHeYTYV7SYbJ49Wi0g9YY1QRA0ib/JSpYTbYWv9QVDkTBUOdw6h+ErX WLM73tV+UEBJ0972QtbLnTATrfIEcef1Y4AHA= Received: by 10.86.158.7 with SMTP id g7mr129891fge.53.1258472622661; Tue, 17 Nov 2009 07:43:42 -0800 (PST) Received: from ndenev.cmotd.com (blah.sun-fish.com [217.18.249.150]) by mx.google.com with ESMTPS id e11sm1444639fga.12.2009.11.17.07.43.39 (version=TLSv1/SSLv3 cipher=RC4-MD5); Tue, 17 Nov 2009 07:43:40 -0800 (PST) Mime-Version: 1.0 (Apple Message framework v1077) Content-Type: text/plain; charset=us-ascii From: Nikolay Denev In-Reply-To: Date: Tue, 17 Nov 2009 17:43:34 +0200 Content-Transfer-Encoding: quoted-printable Message-Id: References: <200911171017.58140.jhb@freebsd.org> To: Thomas Backman X-Mailer: Apple Mail (2.1077) Cc: current@FreeBSD.org Subject: Re: [PATCH] Build a separate ZFS-enabled loader.zfs X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.5 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, 17 Nov 2009 16:07:20 -0000 On Nov 17, 2009, at 5:26 PM, Thomas Backman wrote: > On Nov 17, 2009, at 4:17 PM, John Baldwin wrote: >=20 >> This patch is a workaround to enabling ZFS support by default in the = boot=20 >> loader. It enables building a loader.zfs which is a ZFS-enabled = loader and=20 >> changing zfsboot and gptzfsboot to use /boot/loader.zfs instead=20 >> of /boot/loader. I have only tested that things built ok, I have not=20= >> boot-tested it as I don't have ZFS setup anywhere. The patch is = available at=20 >> http://www.FreeBSD.org/~jhb/loader.zfs/. You will also need to copy=20= >> the 'loader.zfs/Makefile' file from that URL into a new=20 >> sys/boot/i386/loader.zfs directory after applying the patch. > If I may ask (and sorry for my ignorance, but): what problem does this = workaround solve? > Isn't the whole problem with ZFS loaders the license, and because of = the licence, that a ZFS-capable loader isn't built by default? > In other words: Why not use LOADER_ZFS_SUPPORT as long as you have to = choose between setting an option or using a patch? Neither works out of = the box, so you might as well pick the solution that already exists, no = patching involved. > I can only assume I'm missing something vital. :) >=20 > (Please note: Makefiles aren't my strongest suite; I tried to figure = the answer out by reading the patch, but couldn't. I wrote my first own = Makefile, ~15 lines, this weekend.) >=20 > Regards, > Thomas_______________________________________________ > freebsd-current@freebsd.org mailing list > http://lists.freebsd.org/mailman/listinfo/freebsd-current > To unsubscribe, send any mail to = "freebsd-current-unsubscribe@freebsd.org" I think that you are missing the fact that "built by default" and = "loaded by default" are two different things. Regards, Niki Denev