Hi Marcel, > I am still not buying this tail call optimization. However we might need to thing about how we can make sure we do not need this. It feels bad to me every time I look at it. However this is not something that holds up merging this patchset. It is just something that I prefer to get somehow worked out. Especially since I am afraid we have then similar issues in the whole codebase if this holds true. Maybe we need a magic gcc option. > To be honest, this doesn't make sense to me either. Any decent compiler should be able to detect that a pointer to a local variable is being passed to the tail call and determine not to pop the stack. I don't really know why I've been getting these weird crashes; then again, they stopped happening after I moved things around. It is a mystery to me why the tail call actually made a difference, but for now I'm removing the comment and the weird work around from the patch. We can investigate later if this is actually an issue or has something to do with the compiler configuration I'm building with. -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
