linux-kernel - Re: [PATCHv3 10/13] scripts/gdb: Add cpu iterators

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:	Sun, 13 Mar 2016 18:39:27 +0000
From:	Kieran Bingham <kieran.bingham@...aro.org>
To:	Jan Kiszka <jan.kiszka@...mens.com>, linux-kernel@...r.kernel.org
Cc:	lee.jones@...aro.org, peter.griffin@...aro.org,
	maxime.coquelin@...com
Subject: Re: [PATCHv3 10/13] scripts/gdb: Add cpu iterators

On 13/03/16 16:33, Jan Kiszka wrote:
> On 2016-03-03 12:41, Kieran Bingham wrote:
>> The linux kernel provides macro's for iterating against values from the
>> cpu_list masks. By providing some commonly used masks, we can mirror the
>> kernels helper macros with easy to use generators.
>>
>> Signed-off-by: Kieran Bingham <kieran.bingham@...aro.org>
>> ---
>>  scripts/gdb/linux/cpus.py | 21 +++++++++++++++++++++
>>  1 file changed, 21 insertions(+)
>>
>> diff --git a/scripts/gdb/linux/cpus.py b/scripts/gdb/linux/cpus.py
>> index 4297b83fedef..59e24472c507 100644
>> --- a/scripts/gdb/linux/cpus.py
>> +++ b/scripts/gdb/linux/cpus.py
>> @@ -100,6 +100,27 @@ def cpu_list(mask_name):
>>          yield cpu
>>  
>>  
>> +def each_online_cpu():
>> +    for cpu in cpu_list("cpu_online_mask"):
>> +        yield cpu
>> +
>> +
>> +def each_present_cpu():
>> +    for cpu in cpu_list("cpu_present_mask"):
>> +        yield cpu
>> +
>> +
>> +def each_possible_cpu():
>> +    for cpu in cpu_list("cpu_possible_mask"):
>> +        yield cpu
>> +
> 
> This patch requires rebasing over master (these masks became macros).

Ah, yes Ok - I see.

Changed between 4b804c85dc37d...5aec01b834fd6

I'll fixup, and rebase on next.

> In general, please base on master or even next before posting.

Ack. Of course,

And of course this highlights your concerns about the need for continual
testing as well!

> 
>> +
>> +def print_cpus():
>> +    gdb.write("Possible CPUS : {}\n".format(list(each_possible_cpu())))
>> +    gdb.write("Present CPUS  : {}\n".format(list(each_present_cpu())))
>> +    gdb.write("Online CPUS   : {}\n".format(list(each_online_cpu())))
> 
> I think this can be useful. Let's make it a real, discoverable command.
> lx-cpus?

Ack.

Looking through the above breakage, I see there is also a
cpu_active_mask, so I'll add that for completeness.

--
Kieran

> 
>> +
>> +
>>  class PerCpu(gdb.Function):
>>      """Return per-cpu variable.
>>  
>>
> 
> Jan
>