svn commit: r323087 - in head/sys: amd64/conf arm64/conf i386/conf powerpc/conf riscv/conf sparc64/conf

Nathan Whitehorn nwhitehorn at freebsd.org
Fri Sep 1 21:46:12 UTC 2017



On 09/01/17 09:54, Ian Lepore wrote:
> On Fri, 2017-09-01 at 09:06 -0700, Conrad Meyer wrote:
>> Wait, why?  The PR doesn't explain it any better than this commit
>> message.
>>
>> On Fri, Sep 1, 2017 at 8:54 AM, Josh Paetzel <jpaetzel at freebsd.org>
>> wrote:
>>> Author: jpaetzel
>>> Date: Fri Sep  1 15:54:53 2017
>>> New Revision: 323087
>>> URL: https://svnweb.freebsd.org/changeset/base/323087
>>>
>>> Log:
>>>    Take options IPSEC out of GENERIC
>>>
>>>    PR:   220170
>>>    Submitted by: delphij
>>>    Reviewed by:  ae, glebius
>>>    MFC after:    2 weeks
>>>    Differential Revision:        D11806
> The differential revision (which should be cited using the complete url
> in the commit) seems to indicate that it's because it can be kldloaded.
>
> This change misses the armv6 platform, where the option appears in
> std.armv6, which gets included in GENERIC (and all other armv6
> configs).
>
> This change seems to warrant an entry in UPDATING, because it's going
> to catch a lot of people by surprise.  Even people with custom kernels,
> if they follow the advice of the handbook and "include GENERIC" then
> override things.
>
> -- Ian
>

I realize this is immediately moot because the commit has been reverted, 
but please remember about powerpc/conf/GENERIC64 for future such 
changes. This is the "GENERIC" kernel for the powerpc64 architecture, 
which does not have its own arch directory.
-Nathan


More information about the svn-src-all mailing list