[0/1] Fix issue with module unload and CTI cleanup

Message

Mike Leach July 29, 2020, 7:52 p.m. UTC

In the proposed Coresight module set v5 [1] from Tingwei, unloading the
ETM module before the CTI module will crash on unload of the CTI module
due to the cleanup callback from Coresight to the CTI module not working
correctly in clearing sysfs callbacks.

Patch fixes this issue. Applies on [1].

Tingwei - could you consider adding this to your set for v6?

[1] https://lists.linaro.org/pipermail/coresight/2020-July/004349.html

Mike Leach (1):
  coresight: cti: Fix bug clearing sysfs links on callback

 drivers/hwtracing/coresight/coresight-core.c     | 4 ++--
 drivers/hwtracing/coresight/coresight-cti-core.c | 3 +--
 2 files changed, 3 insertions(+), 4 deletions(-)

Comments

Tingwei Zhang July 30, 2020, 2:41 a.m. UTC | #1

On Thu, Jul 30, 2020 at 03:52:32AM +0800, Mike Leach wrote:
> In the proposed Coresight module set v5 [1] from Tingwei, unloading the
> ETM module before the CTI module will crash on unload of the CTI module
> due to the cleanup callback from Coresight to the CTI module not working
> correctly in clearing sysfs callbacks.
> 
> Patch fixes this issue. Applies on [1].
> 
> Tingwei - could you consider adding this to your set for v6?

Sure, Mike.  I'll pick this change into v6.

Thanks, Tingwei
> 
> [1] https://lists.linaro.org/pipermail/coresight/2020-July/004349.html
> 
> Mike Leach (1):
>   coresight: cti: Fix bug clearing sysfs links on callback
> 
>  drivers/hwtracing/coresight/coresight-core.c     | 4 ++--
>  drivers/hwtracing/coresight/coresight-cti-core.c | 3 +--
>  2 files changed, 3 insertions(+), 4 deletions(-)
> 
> -- 
> 2.17.1
>