Re: CVS commit: src/sys/arch

[christos%tac.gw.com@localhost (Christos Zoulas)]

In article <20050211173930.GR2715%snowdrop.l8s.co.uk@localhost>,
David Laight  <david%l8s.co.uk@localhost> wrote:
>On Fri, Feb 11, 2005 at 08:29:48AM -0800, Jason Thorpe wrote:
>> 
>> On Feb 11, 2005, at 12:25 AM, David Laight wrote:
>> 
>> >Add 'option FFS_SNAPSHOT' to most of the config files.
>> >Commented out for kernels that appear to hace space constraints.
>> 
>> Again, you have it backwards.  Please change the sense of this option.
>
>Why? It should never have been a default in the first place.

The author meant it to be the default.

>You also need to disable it on the other small kernels - I didn't
>even add entries to some of the 68k ones.
>
>I'm also not sure that snapshots will survive many forms of system
>reboot at all anyway.
>
>The effect of fsck on a snapshot could be amazing!

This is orthogonal to if it should be a default or not. If you
disagree, have core@ arbitrate. So far we have phone and thorpej
wanting the polarity changed, and I tend to agree with them.

christos