linux-kernel - Re: [PATCH RFC/RFT v3 4/9] s390: move cacheinfo sysfs to generic cacheinfo infrastructure

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]

Message-ID: <53061311.9000800@arm.com>
Date:	Thu, 20 Feb 2014 14:37:05 +0000
From:	Sudeep Holla <Sudeep.Holla@....com>
To:	Heiko Carstens <heiko.carstens@...ibm.com>
CC:	Sudeep.Holla@....com,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	Martin Schwidefsky <schwidefsky@...ibm.com>,
	"linux390@...ibm.com" <linux390@...ibm.com>,
	"linux-s390@...r.kernel.org" <linux-s390@...r.kernel.org>
Subject: Re: [PATCH RFC/RFT v3 4/9] s390: move cacheinfo sysfs to generic
 cacheinfo infrastructure

On 20/02/14 14:07, Heiko Carstens wrote:
> On Thu, Feb 20, 2014 at 01:33:56PM +0000, Sudeep Holla wrote:
>> Thanks, this info was helpful and looks like it's stupid mistake I did. I
>> deleted a line unknowingly while trying to minimise the diff for show_cacheinfo.
>> The below fix-up must work IIUC the issue. I will squash this in my next update
>> if it works.
>>
>> Regards,
>> Sudeep
>>
>> --->8
>>
>> From 77690bc34b2b4b492377998019371f6e2f8f90b7 Mon Sep 17 00:00:00 2001
>> From: Sudeep Holla <sudeep.holla@....com>
>> Date: Thu, 20 Feb 2014 13:14:09 +0000
>> Subject: [PATCH] fixup! s390: move cacheinfo sysfs to generic cacheinfo
>>  infrastructure
>>
>> ---
>>  arch/s390/kernel/cache.c | 5 +++--
>>  1 file changed, 3 insertions(+), 2 deletions(-)
>>
>> diff --git a/arch/s390/kernel/cache.c b/arch/s390/kernel/cache.c
>> index e064f95..aeedb7f 100644
>> --- a/arch/s390/kernel/cache.c
>> +++ b/arch/s390/kernel/cache.c
>> @@ -68,14 +68,15 @@ void show_cacheinfo(struct seq_file *m)
>>  {
>>  	int cpu = smp_processor_id(), idx;
>>  	struct cpu_cacheinfo *this_cpu_ci = get_cpu_cacheinfo(cpu);
>> -	struct cache_info *cache = this_cpu_ci->info_list;
>> +	struct cache_info *cache;
>>
>>  	for (idx = 0; idx < this_cpu_ci->num_leaves; idx++) {
>> +		cache = this_cpu_ci->info_list + idx;
>>  		seq_printf(m, "cache%-11d: ", idx);
>>  		seq_printf(m, "level=%d ", cache->level);
>>  		seq_printf(m, "type=%s ", cache_type_string[cache->type]);
>>  		seq_printf(m, "scope=%s ",
>> -			   cache->disable_sysfs ? "Private" : "Shared");
>> +			   cache->disable_sysfs ? "Shared" : "Private");
>>  		seq_printf(m, "size=%dK ", cache->size >> 10);
>>  		seq_printf(m, "line_size=%u ", cache->coherency_line_size);
>>  		seq_printf(m, "associativity=%d", cache->ways_of_associativity);
> 
> With this patch applied the output looks ok again.
> 

Thanks a lot for retesting and confirming results with this fix-up, will squash
it in my next update.

Regards,
Sudeep

--
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/