Re: [Qemu-devel] [PATCH 0/2] target/riscv: Bugfixes found in decodetree conversion

2018-11-08 Thread Richard Henderson
On 11/8/18 8:12 PM, Palmer Dabbelt wrote: > * We don't take advantage of the ordering bits on fences, which could allow us >  to emit less conservative fences.  This would presumably increase   > performance. Yes. > * We treat fences as NOPs under "#ifndef CONFIG_USER_ONLY", which is a bug. Ah y

Re: [Qemu-devel] [PATCH 0/2] target/riscv: Bugfixes found in decodetree conversion

2018-11-08 Thread Palmer Dabbelt
On Thu, 08 Nov 2018 09:29:26 PST (-0800), Bastian Koppelmann wrote: On 11/8/18 4:53 PM, Richard Henderson wrote: On 11/8/18 1:06 PM, Bastian Koppelmann wrote: while going through the reviews of the riscv-decodetree patches, two bugs came up that I fix here. There is one more problem [1] mentio

Re: [Qemu-devel] [PATCH 0/2] target/riscv: Bugfixes found in decodetree conversion

2018-11-08 Thread Bastian Koppelmann
On 11/8/18 4:53 PM, Richard Henderson wrote: On 11/8/18 1:06 PM, Bastian Koppelmann wrote: while going through the reviews of the riscv-decodetree patches, two bugs came up that I fix here. There is one more problem [1] mentioned by Richard but I don't have the time to investigate it further.

Re: [Qemu-devel] [PATCH 0/2] target/riscv: Bugfixes found in decodetree conversion

2018-11-08 Thread Richard Henderson
On 11/8/18 1:06 PM, Bastian Koppelmann wrote: > while going through the reviews of the riscv-decodetree patches, two bugs came > up that I fix here. There is one more problem [1] mentioned by Richard but > I don't have the time to investigate it further. > > [1] https://patchwork.kernel.org/patch

[Qemu-devel] [PATCH 0/2] target/riscv: Bugfixes found in decodetree conversion

2018-11-08 Thread Bastian Koppelmann
Hi, while going through the reviews of the riscv-decodetree patches, two bugs came up that I fix here. There is one more problem [1] mentioned by Richard but I don't have the time to investigate it further. [1] https://patchwork.kernel.org/patch/10650293/ Bastian Koppelmann (2): target/riscv: