linux-kernel - Re: [PATCH 2/3] iio: ina2xx: give the capture kthread a more useful name string.

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Sat, 12 Dec 2015 15:59:20 +0000
From:	Jonathan Cameron <jic23@...nel.org>
To:	Marc Titinger <mtitinger@...libre.com>, knaack.h@....de,
	lars@...afoo.de, pmeerw@...erw.net
Cc:	linux-iio@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 2/3] iio: ina2xx: give the capture kthread a more useful
 name string.

On 11/12/15 16:49, Marc Titinger wrote:
>   PID  PPID USER     STAT   VSZ %VSZ %CPU COMMAND
>   144     2 root     DW       0   0%  33% [ina226:1-8800us]
>   141     2 root     DW       0   0%  25% [ina226:0-8800us]
>    40     2 root     SW       0   0%  15% [irq/156-4802a00]
>   147     2 root     DW       0   0%   7% [ina226:2-8800us]
>   145     1 root     S     1236   0%   6% dd if /dev/iio:device1 of /dev/null
>   148     1 root     S     1236   0%   4% dd if /dev/iio:device2 of /dev/null
>   149   137 root     R     1244   0%   3% top -d 1
>   142     1 root     S     1236   0%   2% dd if /dev/iio:device0 of /dev/null
> 
> Signed-off-by: Marc Titinger <mtitinger@...libre.com>
Could have slightly improved your patch, but saying what it was before in the
description but otherwise seems reasonable so applied to the
togreg branch of iio.git - initially pushed out as testing for the
autobuilders to play with it.

Thanks,

Jonathan
> ---
>  drivers/iio/adc/ina2xx-adc.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/iio/adc/ina2xx-adc.c b/drivers/iio/adc/ina2xx-adc.c
> index fe42872..99afa6e 100644
> --- a/drivers/iio/adc/ina2xx-adc.c
> +++ b/drivers/iio/adc/ina2xx-adc.c
> @@ -542,7 +542,8 @@ int ina2xx_buffer_enable(struct iio_dev *indio_dev)
>  	chip->prev_ns = iio_get_time_ns();
>  
>  	chip->task = kthread_run(ina2xx_capture_thread, (void *)indio_dev,
> -				 "ina2xx-%uus", sampling_us);
> +				 "%s:%d-%uus", indio_dev->name, indio_dev->id,
> +				 sampling_us);
>  
>  	return PTR_ERR_OR_ZERO(chip->task);
>  }
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/