On 6/29/20 8:03 PM, David Miller wrote: > Never BUG_ON() unless you absolutely cannot continue executing kernel > without corrupting memory or similar.
Yes, that's basically why I don't use it. But the reason I was considering it conditional on a config option is that Qualcomm has a crash analysis tool that expects a BUG() call to stop the system so its instant state can be captured. I don't use this tool, and I might be mistaken about what's required. What I would *really* like to do is have a way to gracefully shut down just the IPA driver when an unexpected condition occurs, so I can stop everything without crashing the system. But doing that in a way that works in all cases is Hard. Do you have any suggestions? -Alex