From: Mark Rutland <mark.rutland@arm.com> To: Jisheng Zhang <jszhang3@mail.ustc.edu.cn>, Thomas Gleixner <tglx@linutronix.de> Cc: "Paul Walmsley" <paul.walmsley@sifive.com>, "Palmer Dabbelt" <palmer@dabbelt.com>, "Albert Ou" <aou@eecs.berkeley.edu>, "Björn Töpel" <bjorn@kernel.org>, "Alexei Starovoitov" <ast@kernel.org>, "Daniel Borkmann" <daniel@iogearbox.net>, "Andrii Nakryiko" <andrii@kernel.org>, "John Fastabend" <john.fastabend@gmail.com>, "KP Singh" <kpsingh@kernel.org>, "Masahiro Yamada" <masahiroy@kernel.org>, "Michal Marek" <michal.lkml@markovi.net>, "Nick Desaulniers" <ndesaulniers@google.com>, "Kefeng Wang" <wangkefeng.wang@huawei.com>, "Tong Tiangen" <tongtiangen@huawei.com>, linux-riscv@lists.infradead.org, linux-kernel@vger.kernel.org, netdev@vger.kernel.org, bpf@vger.kernel.org, linux-kbuild@vger.kernel.org Subject: Re: [PATCH 09/12] riscv: extable: add `type` and `data` fields Date: Thu, 18 Nov 2021 15:21:55 +0000 [thread overview] Message-ID: <20211118152155.GB9977@lakrids.cambridge.arm.com> (raw) In-Reply-To: <20211118193332.79799a9c@xhacker> On Thu, Nov 18, 2021 at 07:42:49PM +0800, Jisheng Zhang wrote: > On Thu, 18 Nov 2021 19:26:05 +0800 Jisheng Zhang wrote: > > > From: Jisheng Zhang <jszhang@kernel.org> > > > > This is a riscv port of commit d6e2cc564775("arm64: extable: add `type` > > and `data` fields"). > > > > We will add specialized handlers for fixups, the `type` field is for > > fixup handler type, the `data` field is used to pass specific data to > > each handler, for example register numbers. > > > > Signed-off-by: Jisheng Zhang <jszhang@kernel.org> > > diff --git a/scripts/sorttable.c b/scripts/sorttable.c > > index 0c031e47a419..5b5472b543f5 100644 > > --- a/scripts/sorttable.c > > +++ b/scripts/sorttable.c > > @@ -376,9 +376,11 @@ static int do_file(char const *const fname, void *addr) > > case EM_PARISC: > > case EM_PPC: > > case EM_PPC64: > > - case EM_RISCV: > > custom_sort = sort_relative_table; > > break; > > + case EM_RISCV: > > + custom_sort = arm64_sort_relative_table; > > Hi Mark, Thomas, > > x86 and arm64 version of sort_relative_table routine are the same, I want to > unify them, and then use the common function for riscv, but I'm not sure > which name is better. Could you please suggest? I sent a patch last week which unifies them as sort_relative_table_with_data(): https://lore.kernel.org/linux-arm-kernel/20211108114220.32796-1-mark.rutland@arm.com/ Thomas, are you happy with that patch? With your ack it could go via the riscv tree for v5.17 as a preparatory cleanup in this series. Maybe we could get it in as a cleanup for v5.16-rc{2,3} ? Thanks, Mark.
WARNING: multiple messages have this Message-ID (diff)
From: Mark Rutland <mark.rutland@arm.com> To: Jisheng Zhang <jszhang3@mail.ustc.edu.cn>, Thomas Gleixner <tglx@linutronix.de> Cc: "Paul Walmsley" <paul.walmsley@sifive.com>, "Palmer Dabbelt" <palmer@dabbelt.com>, "Albert Ou" <aou@eecs.berkeley.edu>, "Björn Töpel" <bjorn@kernel.org>, "Alexei Starovoitov" <ast@kernel.org>, "Daniel Borkmann" <daniel@iogearbox.net>, "Andrii Nakryiko" <andrii@kernel.org>, "John Fastabend" <john.fastabend@gmail.com>, "KP Singh" <kpsingh@kernel.org>, "Masahiro Yamada" <masahiroy@kernel.org>, "Michal Marek" <michal.lkml@markovi.net>, "Nick Desaulniers" <ndesaulniers@google.com>, "Kefeng Wang" <wangkefeng.wang@huawei.com>, "Tong Tiangen" <tongtiangen@huawei.com>, linux-riscv@lists.infradead.org, linux-kernel@vger.kernel.org, netdev@vger.kernel.org, bpf@vger.kernel.org, linux-kbuild@vger.kernel.org Subject: Re: [PATCH 09/12] riscv: extable: add `type` and `data` fields Date: Thu, 18 Nov 2021 15:21:55 +0000 [thread overview] Message-ID: <20211118152155.GB9977@lakrids.cambridge.arm.com> (raw) In-Reply-To: <20211118193332.79799a9c@xhacker> On Thu, Nov 18, 2021 at 07:42:49PM +0800, Jisheng Zhang wrote: > On Thu, 18 Nov 2021 19:26:05 +0800 Jisheng Zhang wrote: > > > From: Jisheng Zhang <jszhang@kernel.org> > > > > This is a riscv port of commit d6e2cc564775("arm64: extable: add `type` > > and `data` fields"). > > > > We will add specialized handlers for fixups, the `type` field is for > > fixup handler type, the `data` field is used to pass specific data to > > each handler, for example register numbers. > > > > Signed-off-by: Jisheng Zhang <jszhang@kernel.org> > > diff --git a/scripts/sorttable.c b/scripts/sorttable.c > > index 0c031e47a419..5b5472b543f5 100644 > > --- a/scripts/sorttable.c > > +++ b/scripts/sorttable.c > > @@ -376,9 +376,11 @@ static int do_file(char const *const fname, void *addr) > > case EM_PARISC: > > case EM_PPC: > > case EM_PPC64: > > - case EM_RISCV: > > custom_sort = sort_relative_table; > > break; > > + case EM_RISCV: > > + custom_sort = arm64_sort_relative_table; > > Hi Mark, Thomas, > > x86 and arm64 version of sort_relative_table routine are the same, I want to > unify them, and then use the common function for riscv, but I'm not sure > which name is better. Could you please suggest? I sent a patch last week which unifies them as sort_relative_table_with_data(): https://lore.kernel.org/linux-arm-kernel/20211108114220.32796-1-mark.rutland@arm.com/ Thomas, are you happy with that patch? With your ack it could go via the riscv tree for v5.17 as a preparatory cleanup in this series. Maybe we could get it in as a cleanup for v5.16-rc{2,3} ? Thanks, Mark. _______________________________________________ linux-riscv mailing list linux-riscv@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-riscv
next prev parent reply other threads:[~2021-11-18 15:22 UTC|newest] Thread overview: 47+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-11-18 11:21 [PATCH v4 0/12] riscv: switch to relative extable and other improvements Jisheng Zhang 2021-11-18 11:21 ` Jisheng Zhang 2021-11-18 11:22 ` [PATCH 01/12] riscv: remove unused __cmpxchg_user() macro Jisheng Zhang 2021-11-18 11:22 ` Jisheng Zhang 2021-11-18 11:22 ` [PATCH 02/12] riscv: consolidate __ex_table construction Jisheng Zhang 2021-11-18 11:22 ` Jisheng Zhang 2021-11-18 11:22 ` [PATCH 03/12] riscv: switch to relative exception tables Jisheng Zhang 2021-11-18 11:22 ` Jisheng Zhang 2021-11-19 2:09 ` tongtiangen 2021-11-19 2:09 ` tongtiangen 2021-11-18 11:23 ` [PATCH 04/12] riscv: bpf: move rv_bpf_fixup_exception signature to extable.h Jisheng Zhang 2021-11-18 11:23 ` Jisheng Zhang 2021-11-18 11:24 ` [PATCH 05/12] riscv: extable: make fixup_exception() return bool Jisheng Zhang 2021-11-18 11:24 ` Jisheng Zhang 2021-11-18 11:24 ` [PATCH 06/12] riscv: extable: use `ex` for `exception_table_entry` Jisheng Zhang 2021-11-18 11:24 ` Jisheng Zhang 2021-11-18 11:25 ` [PATCH 07/12] riscv: lib: uaccess: fold fixups into body Jisheng Zhang 2021-11-18 11:25 ` Jisheng Zhang 2021-11-18 11:25 ` [PATCH 08/12] riscv: extable: consolidate definitions Jisheng Zhang 2021-11-18 11:25 ` Jisheng Zhang 2021-11-18 11:26 ` [PATCH 09/12] riscv: extable: add `type` and `data` fields Jisheng Zhang 2021-11-18 11:26 ` Jisheng Zhang 2021-11-18 11:42 ` Jisheng Zhang 2021-11-18 11:42 ` Jisheng Zhang 2021-11-18 15:21 ` Mark Rutland [this message] 2021-11-18 15:21 ` Mark Rutland 2022-01-06 3:21 ` Palmer Dabbelt 2022-01-06 3:21 ` Palmer Dabbelt 2022-01-06 10:23 ` Mark Rutland 2022-01-06 10:23 ` Mark Rutland 2021-11-19 2:35 ` tongtiangen 2021-11-19 2:35 ` tongtiangen 2021-11-18 11:26 ` [PATCH 10/12] riscv: add gpr-num.h Jisheng Zhang 2021-11-18 11:26 ` Jisheng Zhang 2021-11-18 11:26 ` [PATCH 11/12] riscv: extable: add a dedicated uaccess handler Jisheng Zhang 2021-11-18 11:26 ` Jisheng Zhang 2021-11-18 17:31 ` kernel test robot 2022-01-20 18:15 ` Mayuresh Chitale 2022-01-20 18:15 ` Mayuresh Chitale 2022-01-21 12:16 ` Jisheng Zhang 2022-01-21 12:16 ` Jisheng Zhang 2022-01-23 9:12 ` Jisheng Zhang 2022-01-23 9:12 ` Jisheng Zhang 2022-01-24 17:02 ` Mayuresh Chitale 2022-01-24 17:02 ` Mayuresh Chitale 2021-11-18 11:27 ` [PATCH 12/12] riscv: vmlinux.lds.S|vmlinux-xip.lds.S: remove `.fixup` section Jisheng Zhang 2021-11-18 11:27 ` Jisheng Zhang
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20211118152155.GB9977@lakrids.cambridge.arm.com \ --to=mark.rutland@arm.com \ --cc=andrii@kernel.org \ --cc=aou@eecs.berkeley.edu \ --cc=ast@kernel.org \ --cc=bjorn@kernel.org \ --cc=bpf@vger.kernel.org \ --cc=daniel@iogearbox.net \ --cc=john.fastabend@gmail.com \ --cc=jszhang3@mail.ustc.edu.cn \ --cc=kpsingh@kernel.org \ --cc=linux-kbuild@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-riscv@lists.infradead.org \ --cc=masahiroy@kernel.org \ --cc=michal.lkml@markovi.net \ --cc=ndesaulniers@google.com \ --cc=netdev@vger.kernel.org \ --cc=palmer@dabbelt.com \ --cc=paul.walmsley@sifive.com \ --cc=tglx@linutronix.de \ --cc=tongtiangen@huawei.com \ --cc=wangkefeng.wang@huawei.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.