Re: [PATCH net-next 1/5] net: ipa: head-of-line block registers are RX only

From: Alex Elder
Date: Mon Jun 29 2020 - 21:10:04 EST


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