[PATCH v2 3/5] riscv: sifive: Add SiFive alternative ports
Vincent Chen
vincent.chen at sifive.com
Mon Mar 22 07:50:46 GMT 2021
On Mon, Mar 22, 2021 at 2:08 PM Anup Patel <anup at brainfault.org> wrote:
>
> On Mon, Mar 22, 2021 at 11:10 AM Vincent Chen <vincent.chen at sifive.com> wrote:
> >
> > On Mon, Mar 22, 2021 at 8:26 AM Vincent Chen <vincent.chen at sifive.com> wrote:
> > >
> > > On Wed, Mar 17, 2021 at 7:43 PM Anup Patel <anup at brainfault.org> wrote:
> > > >
> > > > On Wed, Mar 17, 2021 at 3:04 PM Vincent Chen <vincent.chen at sifive.com> wrote:
> > > > >
> > > > > Add required ports of the Alternative scheme for SiFive.
> > > > >
> > > > > Signed-off-by: Vincent Chen <vincent.chen at sifive.com>
> > > > > ---
> > > > > arch/riscv/Kconfig.erratas | 11 ++++++
> > > > > arch/riscv/Kconfig.socs | 1 +
> > > > > arch/riscv/errata/Makefile | 1 +
> > > > > arch/riscv/errata/alternative.c | 5 +++
> > > > > arch/riscv/errata/sifive/Makefile | 1 +
> > > > > arch/riscv/errata/sifive/errata.c | 68 ++++++++++++++++++++++++++++++++++++
> > > > > arch/riscv/include/asm/alternative.h | 3 ++
> > > > > 7 files changed, 90 insertions(+)
> > > > > create mode 100644 arch/riscv/errata/sifive/Makefile
> > > > > create mode 100644 arch/riscv/errata/sifive/errata.c
> > > > >
> > > > > diff --git a/arch/riscv/Kconfig.erratas b/arch/riscv/Kconfig.erratas
> > > > > index 4d0bafc536df..e50ab67f4830 100644
> > > > > --- a/arch/riscv/Kconfig.erratas
> > > > > +++ b/arch/riscv/Kconfig.erratas
> > > > > @@ -9,4 +9,15 @@ config RISCV_ERRATA_ALTERNATIVE
> > > > > code patching is performed once in the boot stages. It means
> > > > > that the overhead from this mechanism is just taken once.
> > > > >
> > > > > +config ERRATA_SIFIVE
> > > > > + bool "SiFive errata"
> > > > > + depends on RISCV_ERRATA_ALTERNATIVE
> > > > > + default y
> > > >
> > > > Remove the "default y" here.
> > > >
> > > > Let the ERRATA Kconfig be explicitly selected from riscv/Kconfig.socs
> > > >
> > > This is a good suggestion. I will modify it in my next version patch.
> > > Thank you,
> > >
> > Sorry, I found the "default y" may not be removed here. It is because
> > SiFive's customers possibly only license our Core IP. In this case, it
> > does not enable CONFIG_SOC_SIFIVE. Therefore, I prefer to keep the
> > "default y" feature. (In this patch, I have made the CONFIG_SOC_SIFIVE
> > select the CONFIG_SOC_SIFIVE )
>
> If you are using SiFive IP for your <xyz> SOC then your
> CONFIG_SOC_<xyz> in Kconfig option should select
> ERRATA_SIFIVE
>
> Basically, SOC Kconfig options should explicitly select required
> ERRATA configs so that if someone wants to disable ERRATs from
> menuconfig then it can be easily done without knowledge of which
> SOCs need which ERRATAs.
>
> Regards,
> Anup
>
I got it. I will remove it in my next version patch.
Thank you for the explanation.
Best regards,
Vincent
> >
> > > > > + help
> > > > > + All SiFive errata Kconfig depend on this Kconfig. Disabling
> > > > > + this Kconfig will disable all SiFive errata. Please say "Y"
> > > > > + here if your platform uses SiFive CPU cores.
> > > > > +
> > > > > + Otherwise, please say "N" here to avoid unnecessary overhead.
> > > > > +
> > > > > endmenu
> > > > > diff --git a/arch/riscv/Kconfig.socs b/arch/riscv/Kconfig.socs
> > > > > index 7efcece8896c..b9eda857fc87 100644
> > > > > --- a/arch/riscv/Kconfig.socs
> > > > > +++ b/arch/riscv/Kconfig.socs
> > > > > @@ -7,6 +7,7 @@ config SOC_SIFIVE
> > > > > select CLK_SIFIVE
> > > > > select CLK_SIFIVE_PRCI
> > > > > select SIFIVE_PLIC
> > > > > + select ERRATA_SIFIVE
> > > > > help
> > > > > This enables support for SiFive SoC platform hardware.
> > > > >
> > > > > diff --git a/arch/riscv/errata/Makefile b/arch/riscv/errata/Makefile
> > > > > index 43e6d5424367..b8f8740a3e44 100644
> > > > > --- a/arch/riscv/errata/Makefile
> > > > > +++ b/arch/riscv/errata/Makefile
> > > > > @@ -1 +1,2 @@
> > > > > obj-y += alternative.o
> > > > > +obj-$(CONFIG_ERRATA_SIFIVE) += sifive/
> > > > > diff --git a/arch/riscv/errata/alternative.c b/arch/riscv/errata/alternative.c
> > > > > index 8efa60ad69b7..3b15885db70b 100644
> > > > > --- a/arch/riscv/errata/alternative.c
> > > > > +++ b/arch/riscv/errata/alternative.c
> > > > > @@ -42,6 +42,11 @@ static void __init init_alternative(void)
> > > > > riscv_fill_cpu_mfr_info();
> > > > >
> > > > > switch (cpu_mfr_info.vendor_id) {
> > > > > +#ifdef CONFIG_ERRATA_SIFIVE
> > > > > + case SIFIVE_VENDOR_ID:
> > > > > + vendor_patch_func = sifive_errata_patch_func;
> > > > > + break;
> > > > > +#endif
> > > > > default:
> > > > > vendor_patch_func = NULL;
> > > > > }
> > > > > diff --git a/arch/riscv/errata/sifive/Makefile b/arch/riscv/errata/sifive/Makefile
> > > > > new file mode 100644
> > > > > index 000000000000..2d644e19caef
> > > > > --- /dev/null
> > > > > +++ b/arch/riscv/errata/sifive/Makefile
> > > > > @@ -0,0 +1 @@
> > > > > +obj-y += errata.o
> > > > > diff --git a/arch/riscv/errata/sifive/errata.c b/arch/riscv/errata/sifive/errata.c
> > > > > new file mode 100644
> > > > > index 000000000000..826cd391fc55
> > > > > --- /dev/null
> > > > > +++ b/arch/riscv/errata/sifive/errata.c
> > > > > @@ -0,0 +1,68 @@
> > > > > +// SPDX-License-Identifier: GPL-2.0-only
> > > > > +/*
> > > > > + * Copyright (C) 2021 Sifive.
> > > > > + */
> > > > > +
> > > > > +#include <linux/kernel.h>
> > > > > +#include <linux/string.h>
> > > > > +#include <linux/bug.h>
> > > > > +#include <asm/patch.h>
> > > > > +#include <asm/alternative.h>
> > > > > +#include <asm/vendorid_list.h>
> > > > > +#include <asm/errata_list.h>
> > > > > +
> > > > > +struct errata_info_t {
> > > > > + char name[ERRATA_STRING_LENGTH_MAX];
> > > > > + bool (*check_func)(unsigned long arch_id, unsigned long impid);
> > > > > +};
> > > > > +
> > > > > +static u32 __init sifive_errata_probe(unsigned long archid, unsigned long impid)
> > > > > +{
> > > > > + int idx;
> > > > > + u32 cpu_req_errata = 0;
> > > > > +
> > > > > + for (idx = 0; idx < ERRATA_SIFIVE_NUMBER; idx++)
> > > > > + if (errata_list[idx].check_func(archid, impid))
> > > > > + cpu_req_errata |= (1U << idx);
> > > > > +
> > > > > + return cpu_req_errata;
> > > > > +}
> > > > > +
> > > > > +static void __init warn_miss_errata(u32 miss_errata)
> > > > > +{
> > > > > + int i;
> > > > > +
> > > > > + pr_warn("----------------------------------------------------------------\n");
> > > > > + pr_warn("WARNING: Missing the following errata may cause potential issues\n");
> > > > > + for (i = 0; i < ERRATA_SIFIVE_NUMBER; i++)
> > > > > + if (miss_errata & 0x1 << i)
> > > > > + pr_warn("\tSiFive Errata[%d]:%s\n", i, errata_list[i].name);
> > > > > + pr_warn("Please enable the corresponding Kconfig to apply them\n");
> > > > > + pr_warn("----------------------------------------------------------------\n");
> > > > > +}
> > > > > +
> > > > > +void __init sifive_errata_patch_func(struct alt_entry *begin, struct alt_entry *end,
> > > > > + unsigned long archid, unsigned long impid)
> > > > > +{
> > > > > + struct alt_entry *alt;
> > > > > + u32 cpu_req_errata = sifive_errata_probe(archid, impid);
> > > > > + u32 cpu_apply_errata = 0;
> > > > > + u32 tmp;
> > > > > +
> > > > > + for (alt = begin; alt < end; alt++) {
> > > > > + if (alt->vendor_id != SIFIVE_VENDOR_ID)
> > > > > + continue;
> > > > > + if (alt->errata_id >= ERRATA_SIFIVE_NUMBER) {
> > > > > + WARN(1, "This errata id:%d is not in kernel errata list", alt->errata_id);
> > > > > + continue;
> > > > > + }
> > > > > +
> > > > > + tmp = (1U << alt->errata_id);
> > > > > + if (cpu_req_errata & tmp) {
> > > > > + patch_text_nosync(alt->old_ptr, alt->alt_ptr, alt->alt_len);
> > > > > + cpu_apply_errata |= tmp;
> > > > > + }
> > > > > + }
> > > > > + if (cpu_apply_errata != cpu_req_errata)
> > > > > + warn_miss_errata(cpu_req_errata - cpu_apply_errata);
> > > > > +}
> > > > > diff --git a/arch/riscv/include/asm/alternative.h b/arch/riscv/include/asm/alternative.h
> > > > > index 430bc4fea133..e625d3cafbed 100644
> > > > > --- a/arch/riscv/include/asm/alternative.h
> > > > > +++ b/arch/riscv/include/asm/alternative.h
> > > > > @@ -32,5 +32,8 @@ struct errata_checkfunc_id {
> > > > > bool (*func)(struct alt_entry *alt);
> > > > > };
> > > > >
> > > > > +void sifive_errata_patch_func(struct alt_entry *begin, struct alt_entry *end,
> > > > > + unsigned long archid, unsigned long impid);
> > > > > +
> > > > > #endif
> > > > > #endif
> > > > > --
> > > > > 2.7.4
> > > > >
> > > > >
> > > > > _______________________________________________
> > > > > linux-riscv mailing list
> > > > > linux-riscv at lists.infradead.org
> > > > > http://lists.infradead.org/mailman/listinfo/linux-riscv
> > > >
> > > > Apart from the minor comment above, this patch looks good to me.
> > > >
> > > > Reviewed-by: Anup Patel <anup at brainfault.org>
> > > >
> > > > Regards,
> > > > Anup
More information about the linux-riscv
mailing list