Re: [PATCH v2] MIPS - fix cycle counter timing calculations

2021-12-14 Thread Simon Burge
=?UTF-8?Q?Philippe_Mathieu-Daud=c3=a9?= wrote: > Minor comment, it is better to post patch iterations as new thread, > and not as reply to older patch, because in thread view your new > patch might ended hidden / lost. Ah, my bad. I misread the part about using in-reply-to in the patch submissio

Re: [PATCH] MIPS - fix cycle counter timing calculations

2021-12-13 Thread Simon Burge
Hi Phil, =?UTF-8?Q?Philippe_Mathieu-Daud=c3=a9?= wrote: > Oops, missing your Signed-off-by tag, see: > https://www.qemu.org/docs/master/devel/submitting-a-patch.html#patch-emails-must-includ e-a-signed-off-by-line > > Do you mind re-sending with your S-o-b? Meanwhile, patch dropped. Hopefully I'

[PATCH v2] MIPS - fix cycle counter timing calculations

2021-12-13 Thread Simon Burge
of 1. Signed-off-by: Simon Burge --- target/mips/cpu.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/target/mips/cpu.c b/target/mips/cpu.c index 4aae23934b..0766e25693 100644 --- a/target/mips/cpu.c +++ b/target/mips/cpu.c @@ -440,8 +440,9 @@ static void

[PATCH] MIPS - fix cycle counter timing calculations

2021-11-16 Thread Simon Burge
The cp0_count_ns value is calculated from the CP0_COUNT_RATE_DEFAULT constant in target/mips/cpu.c. The cycle counter resolution is defined per-CPU in target/mips/cpu-defs.c.inc; use this value for calculating cp0_count_ns. Fixings timing problems on guest OSs for the 20Kc CPU which has a CCRes o