lkml.org 
[lkml]   [2006]   [Oct]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: BUG-lockdep and freeze (was: Arrr! Linux 2.6.18)
    >>> Linus Torvalds <torvalds@osdl.org> 30.09.06 23:11 >>>
    >It could - and _should_ dammit! - do some basic sanity tests like "is the
    >thing even in the same stack page"? But nooo... It seems _designed_ to be
    >fragile and broken.

    Your opinion. Mine's different, as the unwind directives are specifically
    meant to allow changing to a different stack page. Also, there shouldn't
    be any potential for corrupt unwind data (once all annotations are
    correct, the lack or incorrectness of which I'd rather attribute as bugs
    of the old code or, in one case, the compiler) as long as it's being
    properly write protected; corruption of the stack should be affecting
    the quality of old and new style back traces in similar ways.

    >Here's a simple test: if the next stack-slot isn't on the same page, the
    >unwind information is bogus unless you had the IRQ stack-switch signature
    >there. Does the code do that? No. It just assumes that unwind information
    >is complete and perfect.
    >
    >That's not the kind of code we write in the kernel. In the kernel, we
    >write code that _works_, regardless of the kind of horrible stuff people
    >feed it. That's _doubly_ true for something like a stack frame debugger,
    >which is invoced when there is trouble, and for all we know the stack
    >itself MIGHT BE CORRUPT.
    >
    >In short, I think the stack unwinder is just _broken_. It has made all the
    >wrong policy decisions - it only works when everything is perfect, yet
    >it's actually meant to be _used_ when somethign bad happened. Doesn't that
    >strike anybody else as a totally flawed design?

    Again, a corrupt stack will not allow you getting reliable data out of the
    old unwinder either. Even worse when you consider a stack overflow and
    your request for range checks (or pointers into the stack) - you might not
    get a stack trace then at all.

    Jan
    -
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

    \
     
     \ /
      Last update: 2006-10-04 11:23    [W:2.672 / U:0.068 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site