From: Joe Perches <joe@perches.com> To: Evan Green <evan@rivosinc.com>, Palmer Dabbelt <palmer@rivosinc.com> Cc: heiko@sntech.de, Conor Dooley <conor@kernel.org>, slewis@rivosinc.com, vineetg@rivosinc.com, Albert Ou <aou@eecs.berkeley.edu>, Andrew Bresticker <abrestic@rivosinc.com>, Andrew Jones <ajones@ventanamicro.com>, Anup Patel <apatel@ventanamicro.com>, Atish Patra <atishp@rivosinc.com>, Celeste Liu <coelacanthus@outlook.com>, Conor Dooley <conor.dooley@microchip.com>, Guo Ren <guoren@kernel.org>, Jisheng Zhang <jszhang@kernel.org>, Jonathan Corbet <corbet@lwn.net>, Palmer Dabbelt <palmer@dabbelt.com>, Paul Walmsley <paul.walmsley@sifive.com>, Ruizhe Pan <c141028@gmail.com>, Sunil V L <sunilvl@ventanamicro.com>, Tsukasa OI <research_trasio@irq.a4lg.com>, Xianting Tian <xianting.tian@linux.alibaba.com>, linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, linux-riscv@lists.infradead.org Subject: Re: [PATCH v3 5/7] RISC-V: hwprobe: Support probing of misaligned access performance Date: Wed, 22 Feb 2023 01:39:55 -0800 [thread overview] Message-ID: <2cf5852092ac5940b8bc6664f54bd6066fe03be2.camel@perches.com> (raw) In-Reply-To: <20230221190858.3159617-6-evan@rivosinc.com> On Tue, 2023-02-21 at 11:08 -0800, Evan Green wrote: > This allows userspace to select various routines to use based on the > performance of misaligned access on the target hardware. [] > diff --git a/arch/riscv/kernel/cpufeature.c b/arch/riscv/kernel/cpufeature.c [] > @@ -89,11 +94,11 @@ static bool riscv_isa_extension_check(int id) > void __init riscv_fill_hwcap(void) > { > struct device_node *node; > - const char *isa; > + const char *isa, *misaligned; > char print_str[NUM_ALPHA_EXTS + 1]; > int i, j, rc; > unsigned long isa2hwcap[26] = {0}; > - unsigned long hartid; > + unsigned long hartid, cpu; > > isa2hwcap['i' - 'a'] = COMPAT_HWCAP_ISA_I; > isa2hwcap['m' - 'a'] = COMPAT_HWCAP_ISA_M; > @@ -246,6 +251,28 @@ void __init riscv_fill_hwcap(void) > bitmap_copy(riscv_isa, this_isa, RISCV_ISA_EXT_MAX); > else > bitmap_and(riscv_isa, riscv_isa, this_isa, RISCV_ISA_EXT_MAX); > + > + /* > + * Check for the performance of misaligned accesses. > + */ > + cpu = hartid_to_cpuid_map(hartid); > + if (cpu < 0) > + continue; unsigned long can't be less than 0 Likely cpu should be long not unsigned long It seems cpu is rather randomly either int or long. Perhaps standardizing on int would be better.
WARNING: multiple messages have this Message-ID (diff)
From: Joe Perches <joe@perches.com> To: Evan Green <evan@rivosinc.com>, Palmer Dabbelt <palmer@rivosinc.com> Cc: heiko@sntech.de, linux-doc@vger.kernel.org, Andrew Bresticker <abrestic@rivosinc.com>, Atish Patra <atishp@rivosinc.com>, Conor Dooley <conor.dooley@microchip.com>, Celeste Liu <coelacanthus@outlook.com>, Jisheng Zhang <jszhang@kernel.org>, linux-riscv@lists.infradead.org, Jonathan Corbet <corbet@lwn.net>, Xianting Tian <xianting.tian@linux.alibaba.com>, Tsukasa OI <research_trasio@irq.a4lg.com>, Andrew Jones <ajones@ventanamicro.com>, Albert Ou <aou@eecs.berkeley.edu>, vineetg@rivosinc.com, Paul Walmsley <paul.walmsley@sifive.com>, Ruizhe Pan <c141028@gmail.com>, Anup Patel <apatel@ventanamicro.com>, slewis@rivosinc.com, linux-kernel@vger.kernel.org, Conor Dooley <conor@kernel.org>, Palmer Dabbelt <palmer@dabbelt.com>, Guo Ren <guoren@kernel.org> Subject: Re: [PATCH v3 5/7] RISC-V: hwprobe: Support probing of misaligned access performance Date: Wed, 22 Feb 2023 01:39:55 -0800 [thread overview] Message-ID: <2cf5852092ac5940b8bc6664f54bd6066fe03be2.camel@perches.com> (raw) In-Reply-To: <20230221190858.3159617-6-evan@rivosinc.com> On Tue, 2023-02-21 at 11:08 -0800, Evan Green wrote: > This allows userspace to select various routines to use based on the > performance of misaligned access on the target hardware. [] > diff --git a/arch/riscv/kernel/cpufeature.c b/arch/riscv/kernel/cpufeature.c [] > @@ -89,11 +94,11 @@ static bool riscv_isa_extension_check(int id) > void __init riscv_fill_hwcap(void) > { > struct device_node *node; > - const char *isa; > + const char *isa, *misaligned; > char print_str[NUM_ALPHA_EXTS + 1]; > int i, j, rc; > unsigned long isa2hwcap[26] = {0}; > - unsigned long hartid; > + unsigned long hartid, cpu; > > isa2hwcap['i' - 'a'] = COMPAT_HWCAP_ISA_I; > isa2hwcap['m' - 'a'] = COMPAT_HWCAP_ISA_M; > @@ -246,6 +251,28 @@ void __init riscv_fill_hwcap(void) > bitmap_copy(riscv_isa, this_isa, RISCV_ISA_EXT_MAX); > else > bitmap_and(riscv_isa, riscv_isa, this_isa, RISCV_ISA_EXT_MAX); > + > + /* > + * Check for the performance of misaligned accesses. > + */ > + cpu = hartid_to_cpuid_map(hartid); > + if (cpu < 0) > + continue; unsigned long can't be less than 0 Likely cpu should be long not unsigned long It seems cpu is rather randomly either int or long. Perhaps standardizing on int would be better. _______________________________________________ linux-riscv mailing list linux-riscv@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-riscv
next prev parent reply other threads:[~2023-02-22 9:58 UTC|newest] Thread overview: 48+ messages / expand[flat|nested] mbox.gz Atom feed top 2023-02-21 19:08 [PATCH v3 0/7] RISC-V Hardware Probing User Interface Evan Green 2023-02-21 19:08 ` Evan Green 2023-02-21 19:08 ` [PATCH v3 1/7] RISC-V: Move struct riscv_cpuinfo to new header Evan Green 2023-02-21 19:08 ` Evan Green 2023-02-21 19:08 ` [PATCH v3 2/7] RISC-V: Add a syscall for HW probing Evan Green 2023-02-21 19:08 ` Evan Green 2023-02-23 10:06 ` Arnd Bergmann 2023-02-23 10:06 ` Arnd Bergmann 2023-03-30 18:30 ` Evan Green 2023-03-30 18:30 ` Evan Green 2023-03-30 20:20 ` Heiko Stübner 2023-03-30 20:20 ` Heiko Stübner 2023-03-30 21:24 ` Evan Green 2023-03-30 21:24 ` Evan Green 2023-03-31 13:21 ` Arnd Bergmann 2023-03-31 13:21 ` Arnd Bergmann 2023-03-31 17:51 ` Evan Green 2023-03-31 17:51 ` Evan Green 2023-02-27 22:19 ` Conor Dooley 2023-02-27 22:19 ` Conor Dooley 2023-02-21 19:08 ` [PATCH v3 3/7] RISC-V: hwprobe: Add support for RISCV_HWPROBE_BASE_BEHAVIOR_IMA Evan Green 2023-02-21 19:08 ` Evan Green 2023-02-27 22:47 ` Conor Dooley 2023-02-27 22:47 ` Conor Dooley 2023-03-03 0:56 ` Evan Green 2023-03-03 0:56 ` Evan Green 2023-02-21 19:08 ` [PATCH v3 4/7] dt-bindings: Add RISC-V misaligned access performance Evan Green 2023-02-21 19:08 ` Evan Green 2023-02-27 22:57 ` Conor Dooley 2023-02-27 22:57 ` Conor Dooley 2023-02-28 14:57 ` Rob Herring 2023-02-28 14:57 ` Rob Herring 2023-02-21 19:08 ` [PATCH v3 5/7] RISC-V: hwprobe: Support probing of " Evan Green 2023-02-21 19:08 ` Evan Green 2023-02-22 9:39 ` Joe Perches [this message] 2023-02-22 9:39 ` Joe Perches 2023-02-27 23:14 ` Conor Dooley 2023-02-27 23:14 ` Conor Dooley 2023-02-21 19:08 ` [PATCH v3 6/7] selftests: Test the new RISC-V hwprobe interface Evan Green 2023-02-21 19:08 ` Evan Green 2023-02-21 22:45 ` Mark Brown 2023-02-21 22:45 ` Mark Brown 2023-02-21 19:08 ` [PATCH v3 7/7] RISC-V: Add hwprobe vDSO function and data Evan Green 2023-02-21 19:08 ` Evan Green 2023-02-21 21:17 ` kernel test robot 2023-02-21 21:17 ` kernel test robot 2023-02-22 6:55 ` kernel test robot 2023-02-22 6:55 ` kernel test robot
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=2cf5852092ac5940b8bc6664f54bd6066fe03be2.camel@perches.com \ --to=joe@perches.com \ --cc=abrestic@rivosinc.com \ --cc=ajones@ventanamicro.com \ --cc=aou@eecs.berkeley.edu \ --cc=apatel@ventanamicro.com \ --cc=atishp@rivosinc.com \ --cc=c141028@gmail.com \ --cc=coelacanthus@outlook.com \ --cc=conor.dooley@microchip.com \ --cc=conor@kernel.org \ --cc=corbet@lwn.net \ --cc=evan@rivosinc.com \ --cc=guoren@kernel.org \ --cc=heiko@sntech.de \ --cc=jszhang@kernel.org \ --cc=linux-doc@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-riscv@lists.infradead.org \ --cc=palmer@dabbelt.com \ --cc=palmer@rivosinc.com \ --cc=paul.walmsley@sifive.com \ --cc=research_trasio@irq.a4lg.com \ --cc=slewis@rivosinc.com \ --cc=sunilvl@ventanamicro.com \ --cc=vineetg@rivosinc.com \ --cc=xianting.tian@linux.alibaba.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.