Re: [PATCH v3 4/5] remoteproc: qcom: Introduce sysmon

From: Chris Lew
Date: Fri Nov 17 2017 - 20:27:36 EST




On 11/16/2017 9:58 PM, Bjorn Andersson wrote:
On Thu 16 Nov 12:05 PST 2017, Chris Lew wrote:
+ req.event = SSCTL_SSR_EVENT_BEFORE_SHUTDOWN;

Are there plans to add the other SSR events to sysmon notifiers? I think the
SSCTL service expects to receive events about remote procs starting as well.


We could easily add support here to send out the AFTER_BOOTUP
notification, beyond that we would need to extend the subdev notifiers
as well.

But the downstream code paths does confuse me, can you confirm that
these messages are actually sent to the remote ssctl services?

Regards,
Bjorn


Yea the other three events are definitely sent to the remote. The remote side posts the events for other modules to query. I'm not sure if all of the events are used by other other modules though.

Thanks,
Chris
--
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project