Re: [v6 PATCH] RISC-V: Remove unsupported isa string info print

2019-10-07 Thread h...@infradead.org
On Wed, Oct 02, 2019 at 06:28:59AM +, Atish Patra wrote: > On Wed, 2019-10-02 at 09:53 +0800, Alan Kao wrote: > > On Tue, Oct 01, 2019 at 03:10:16AM -0700, h...@infradead.org wrote: > > > On Tue, Oct 01, 2019 at 08:22:37AM +, Atish Patra wrote: > > > > riscv_of_processor_hartid() or seems

Re: [v6 PATCH] RISC-V: Remove unsupported isa string info print

2019-10-02 Thread Atish Patra
On Wed, 2019-10-02 at 09:53 +0800, Alan Kao wrote: > On Tue, Oct 01, 2019 at 03:10:16AM -0700, h...@infradead.org wrote: > > On Tue, Oct 01, 2019 at 08:22:37AM +, Atish Patra wrote: > > > riscv_of_processor_hartid() or seems to be a better candidate. We > > > already check if "rv" is present

Re: [v6 PATCH] RISC-V: Remove unsupported isa string info print

2019-10-01 Thread Alan Kao
On Tue, Oct 01, 2019 at 03:10:16AM -0700, h...@infradead.org wrote: > On Tue, Oct 01, 2019 at 08:22:37AM +, Atish Patra wrote: > > riscv_of_processor_hartid() or seems to be a better candidate. We > > already check if "rv" is present in isa string or not. I will extend > > that to check for

Re: [v6 PATCH] RISC-V: Remove unsupported isa string info print

2019-10-01 Thread h...@infradead.org
On Tue, Oct 01, 2019 at 08:22:37AM +, Atish Patra wrote: > riscv_of_processor_hartid() or seems to be a better candidate. We > already check if "rv" is present in isa string or not. I will extend > that to check for rv64i or rv32i. Is that okay ? I'd rather lift the checks out of that into a

Re: [v6 PATCH] RISC-V: Remove unsupported isa string info print

2019-10-01 Thread Atish Patra
On Tue, 2019-10-01 at 00:02 -0700, Christoph Hellwig wrote: > On Mon, Sep 30, 2019 at 05:23:18PM -0700, Atish Patra wrote: > > /proc/cpuinfo should just print all the isa string as an > > information > > instead of determining what is supported or not. ELF hwcap can be > > used by the userspace to

Re: [v6 PATCH] RISC-V: Remove unsupported isa string info print

2019-10-01 Thread Christoph Hellwig
On Mon, Sep 30, 2019 at 05:23:18PM -0700, Atish Patra wrote: > /proc/cpuinfo should just print all the isa string as an information > instead of determining what is supported or not. ELF hwcap can be > used by the userspace to figure out that. > > Simplify the isa string printing by removing the

[v6 PATCH] RISC-V: Remove unsupported isa string info print

2019-09-30 Thread Atish Patra
/proc/cpuinfo should just print all the isa string as an information instead of determining what is supported or not. ELF hwcap can be used by the userspace to figure out that. Simplify the isa string printing by removing the unsupported isa string print and all related code. The relevant