Current-Users archive

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

Re: amd64 -current crashs at boot



Andrew Doran wrote:
> On Mon, Dec 22, 2008 at 12:52:22PM +0100, Christoph Egger wrote:
> 
>> Attached patch sprinkles some KASSERTs which explicitely check
>> for assumptions not check otherwise in the code path.
>>
>> The last hunk in the diff to ffs_vnode.c fixes the crash for me
>> and makes my machine boot again.
>>
>> Patch ok to commit ?
> 
> No because it was a hack like this that caused the problem in the first
> place. Don't commit without being able to explain the failure and at least
> convince yourself that you understand what the code is doing.
> 
> It should be fixed now, I'm about to check.

No, but ffs_vnops.c, rev. 1.106 fixed a panic on shutdown/panic
*with* my diff.
The panic at boot *w/o* my diff is still there.

Christoph


Home | Main Index | Thread Index | Old Index