On Sun, 21 Feb 2021 at 20:07, Philippe Mathieu-Daudé <f4...@amsat.org> wrote: > > On 2/11/21 12:43 AM, Philippe Mathieu-Daudé wrote: > > Use the new clock_ns_to_ticks() function in cp0_timer where > > appropriate. This allows us to remove CPUMIPSState::cp0_count_ns > > and mips_cp0_period_set(). > > > > Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> > > --- > > Based-on: <20210209132040.5091-1-peter.mayd...@linaro.org> > > > > RFC because this is just a starter patch to test Peter's series > > providing a handy function which "tells me how many times this > > clock would tick in this length of time". > > > > Also because we could rethink the MIPS CP0 timer logic to avoid > > too frequent divu128 calls (painful on 32-bit hosts). > > > > The style should be updated, using more variables for easier > > review. env_archcpu() could eventually be dropped (by passing > > MIPSCPU* instead of CPUMIPSState*). > > I guess it is better to wait for Peter patches to get merged > before posting the updated patches (not much, just one previous > patch MIPSCPU* instead of CPUMIPSState*).
This patch is still on my to-review queue, fwiw (though I am taking a week off work next week, so I probably won't get to it til March.) -- PMM
