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

[Heiko Carstens]

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@xxxxxxx>
> 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,
Heiko

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