Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 8 Jan 2012 14:58:56 -0800
From:      Adrian Chadd <adrian@freebsd.org>
To:        Pawel Jakub Dawidek <pjd@freebsd.org>
Cc:        svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org
Subject:   Re: svn commit: r229800 - head/sys/conf
Message-ID:  <CAJ-VmonrxxhMha9e-cgdySZ9kt%2BwGpJp36Z%2BOfEJ3S2rVP8Vpg@mail.gmail.com>
In-Reply-To: <20120108104330.GC1674@garage.freebsd.pl>
References:  <201201080055.q080tMlJ063808@svn.freebsd.org> <20120108104330.GC1674@garage.freebsd.pl>

next in thread | previous in thread | raw e-mail | index | archive | help
On 8 January 2012 02:43, Pawel Jakub Dawidek <pjd@freebsd.org> wrote:
> If someone is actually using GENERIC kernel. This change will break all
> my system next time I upgrade. Adding UFS_ACL option to the kernel
> config to make ZFS kernel module to work doesn't sound very intuitive.
>
> I understand what you are trying to accomplish, but we really need to
> find better way to do this. Until then, could you back it out?

I'd rather find a cleaner solution - it's pulling in code which just
isn't being used if you aren't using UFS_ACL or ZFS.

How about wrapping those two up in a module which zfs can register a
dependency on?


Adrian



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAJ-VmonrxxhMha9e-cgdySZ9kt%2BwGpJp36Z%2BOfEJ3S2rVP8Vpg>