Re: [RFC PATCH v3 33/37] kvx: Add support for cpuinfo

[Krzysztof Kozlowski]

On 22/07/2024 11:41, ysionneau@xxxxxxxxxxxxx wrote:
> From: Yann Sionneau <ysionneau@xxxxxxxxxxxxx>
> 
> Add support for cpuinfo on kvx arch.
> 
> Co-developed-by: Clement Leger <clement@xxxxxxxxxxxxxxxx>
> Signed-off-by: Clement Leger <clement@xxxxxxxxxxxxxxxx>
> Co-developed-by: Guillaume Thouvenin <thouveng@xxxxxxxxx>
> Signed-off-by: Guillaume Thouvenin <thouveng@xxxxxxxxx>
> Co-developed-by: Julian Vetter <jvetter@xxxxxxxxxxxxx>
> Signed-off-by: Julian Vetter <jvetter@xxxxxxxxxxxxx>
> Signed-off-by: Jules Maselbas <jmaselbas@xxxxxxxx>
> Signed-off-by: Yann Sionneau <ysionneau@xxxxxxxxxxxxx>
> ---
> 
> Notes:
> V1 -> V2: No changes
> V2 -> V3:
> - add missing function declaration for setup_cpuinfo()
> - replace printk(KERN_WARNING... with pr_warn(...
> ---
>  arch/kvx/include/asm/cpuinfo.h |  7 +++
>  arch/kvx/kernel/cpuinfo.c      | 95 ++++++++++++++++++++++++++++++++++
>  2 files changed, 102 insertions(+)
>  create mode 100644 arch/kvx/include/asm/cpuinfo.h
>  create mode 100644 arch/kvx/kernel/cpuinfo.c
> 
> diff --git a/arch/kvx/include/asm/cpuinfo.h b/arch/kvx/include/asm/cpuinfo.h
> new file mode 100644
> index 0000000000000..ace9b85fbafaf
> --- /dev/null
> +++ b/arch/kvx/include/asm/cpuinfo.h
> @@ -0,0 +1,7 @@
> +/* SPDX-License-Identifier: GPL-2.0-or-later */
> +#ifndef __ASM_KVX_CPUINFO_H
> +#define __ASM_KVX_CPUINFO_H
> +
> +extern void setup_cpuinfo(void);
> +
> +#endif /* __ASM_KVX_CPUINFO_H */
> diff --git a/arch/kvx/kernel/cpuinfo.c b/arch/kvx/kernel/cpuinfo.c
> new file mode 100644
> index 0000000000000..6d46fd2a1bd93
> --- /dev/null
> +++ b/arch/kvx/kernel/cpuinfo.c
> @@ -0,0 +1,95 @@
> +// SPDX-License-Identifier: GPL-2.0-only
> +/*
> + * Copyright (C) 2017-2023 Kalray Inc.
> + * Author(s): Clement Leger
> + *            Guillaume Thouvenin
> + */
> +
> +#include <linux/seq_file.h>
> +#include <linux/delay.h>
> +#include <linux/clk.h>
> +#include <linux/cpu.h>
> +#include <linux/of.h>
> +
> +unsigned long elf_hwcap __read_mostly;
> +
> +static int show_cpuinfo(struct seq_file *m, void *v)
> +{
> +	int cpu_num = *(unsigned int *)v;
> +	struct cpuinfo_kvx *n = per_cpu_ptr(&cpu_info, cpu_num);
> +
> +	seq_printf(m, "processor\t: %d\nvendor_id\t: Kalray\n", cpu_num);
> +
> +	seq_printf(m,
> +		   "copro enabled\t: %s\n"
> +		   "arch revision\t: %d\n"
> +		   "uarch revision\t: %d\n",
> +		   n->copro_enable ? "yes" : "no",
> +		   n->arch_rev,
> +		   n->uarch_rev);
> +
> +	seq_printf(m,
> +		   "bogomips\t: %lu.%02lu\n"
> +		   "cpu MHz\t\t: %llu.%03llu\n\n",
> +		   (loops_per_jiffy * HZ) / 500000,
> +		   ((loops_per_jiffy * HZ) / 5000) % 100,
> +		   n->freq / 1000000, (n->freq / 10000) % 100);
> +
> +	return 0;
> +}
> +
> +static void *c_start(struct seq_file *m, loff_t *pos)
> +{
> +	if (*pos == 0)
> +		*pos = cpumask_first(cpu_online_mask);
> +	if (*pos >= num_online_cpus())
> +		return NULL;
> +
> +	return pos;
> +}
> +
> +static void *c_next(struct seq_file *m, void *v, loff_t *pos)
> +{
> +	*pos = cpumask_next(*pos, cpu_online_mask);
> +
> +	return c_start(m, pos);
> +}
> +
> +static void c_stop(struct seq_file *m, void *v)
> +{
> +}
> +
> +const struct seq_operations cpuinfo_op = {

Where is this used?

> +	.start = c_start,
> +	.next = c_next,
> +	.stop = c_stop,
> +	.show = show_cpuinfo,
> +};
> +

Best regards,
Krzysztof