On Sat, 20 Jan 2018 04:19:53 -0800 Tejun Heo <tj@xxxxxxxxxx> wrote: > I'm a bit worried tho because this essentially seems like "detect > recursion, ignore messages" approach. netcons can have a very large > surface for bugs. Suppressing those messages would make them > difficult to debug. For example, all our machines have both serial > console (thus the slowness) and netconsole hooked up and netcons code > has had its fair share of issues. This would likely make tracking > down those problems more challenging. Well, it's not totally ignoring them. There's a variable that tells printk how many to print before it starts ignoring them. I picked 3, but that could very well be 5 or 10. Probably 10 is the best, because then it would give us enough idea why printk is recursing on itself without overloading the buffer. And I made it a variable to easily make it a knob for userspace to tweak if need be. > > Can we discuss pros and cons of this approach against offloading > before committing to this? I'm open. I was just thinking about the scenario that you mentioned and how what the best way to solve it would be. We need to define the exact problem(s) we are dealing with before we offer a solution. The one thing I don't want is a solution looking for a problem. I want a full understanding of what the problem exactly is and then we can discuss various solutions, and how they solve the problem(s). Otherwise we are just doing (to quote Linus) code masturbation. -- Steve -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>
