On 11/10/25 12:03 PM, Mathieu Poirier wrote:
On Thu, Oct 30, 2025 at 12:21:24PM +0800, Peng Fan wrote:
Hi Tanmay,

On Wed, Oct 29, 2025 at 06:51:51PM -0500, Tanmay Shah wrote:
...


Hi Peng,

I don't understand why it should fail. The patch simply implements
rproc_detach() -> rproc_attach() sequence.


Hi Peng,

Thanks for testing the patch. I appreciate your quick response. I think
rproc_boot() should be used instead of rproc_attach(). That should probably
solve the issue you are facing. I will send v2 with this change for you to
try.

Thanks,
Tanmay

In your case, when you do detach -> attach via sysfs that sequence works?
If that works, then crash recovery should work as well.

sysfs does not have attach option, only start/stop/detach are there.


Could you give steps how do you generate the crash?

I have not look into the details on why it fails at my side for the 2nd time.

On my board, the M4 core use watchdog to reset itself and notify Linux, then
linux side imx_rproc driver will do
"rproc_report_crash(priv->rproc, RPROC_WATCHDOG);"

I will give a debug on the failures in a few days.


So what is happening here - Peng, do you plan on providing more debugging
information? Tanmay - are you planning on sending a second revision?

Mathieu,

I will be providing the v2, that will replace rproc_attach with rproc_boot. I am testing it, so far have not seen any issues.

I hope that will resolve Peng's problem. V2 will be posted this week sometime.


Thanks,
Tanmay

Thanks,
Peng


Thanks,
Tanmay

Thanks,
Peng




Reply via email to