RE: [PATCH v3 2/2] drivers: devfreq: add tracing for scheduling work

2019-02-20 Thread MyungJoo Ham
>This patch add basic tracing of the devfreq workqueue and delayed work.
>It aims to capture changes of the polling intervals and device state.
>
>Signed-off-by: Lukasz Luba 
>---
> drivers/devfreq/devfreq.c | 5 +
> 1 file changed, 5 insertions(+)
>
>diff --git a/drivers/devfreq/devfreq.c b/drivers/devfreq/devfreq.c
>index 0ae3de7..660365a 100644
>--- a/drivers/devfreq/devfreq.c
>+++ b/drivers/devfreq/devfreq.c

Acked-by: MyungJoo Ham 





Re: [PATCH v3 2/2] drivers: devfreq: add tracing for scheduling work

2019-02-18 Thread Chanwoo Choi
Hi Lukasz,

On 19. 2. 19. 오전 3:21, Lukasz Luba wrote:
> This patch add basic tracing of the devfreq workqueue and delayed work.
> It aims to capture changes of the polling intervals and device state.
> 
> Signed-off-by: Lukasz Luba 
> ---
>  drivers/devfreq/devfreq.c | 5 +
>  1 file changed, 5 insertions(+)
> 
> diff --git a/drivers/devfreq/devfreq.c b/drivers/devfreq/devfreq.c
> index 0ae3de7..660365a 100644
> --- a/drivers/devfreq/devfreq.c
> +++ b/drivers/devfreq/devfreq.c
> @@ -29,6 +29,9 @@
>  #include 
>  #include "governor.h"
>  
> +#define CREATE_TRACE_POINTS
> +#include 
> +
>  static struct class *devfreq_class;
>  
>  /*
> @@ -394,6 +397,8 @@ static void devfreq_monitor(struct work_struct *work)
>   queue_delayed_work(devfreq_wq, >work,
>   msecs_to_jiffies(devfreq->profile->polling_ms));
>   mutex_unlock(>lock);
> +
> + trace_devfreq_monitor(devfreq);
>  }
>  
>  /**
> 

Reviewed-by: Chanwoo Choi 

-- 
Best Regards,
Chanwoo Choi
Samsung Electronics