Source-Changes archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: src/sys/arch



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




Home | Main Index | Thread Index | Old Index