[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [Qemu-devel] [PATCHv2 05/12] cuda: rename frequency prope
Re: [Qemu-ppc] [Qemu-devel] [PATCHv2 05/12] cuda: rename frequency property to tb_frequency
Sun, 11 Feb 2018 22:22:12 +1100
On Sun, Feb 11, 2018 at 10:59:05AM +0000, Mark Cave-Ayland wrote:
> On 10/02/18 23:11, David Gibson wrote:
> > On Sun, Feb 11, 2018 at 09:32:14AM +1100, David Gibson wrote:
> > > On Fri, Feb 09, 2018 at 06:51:35PM +0000, Mark Cave-Ayland wrote:
> > > > This allows us to more easily differentiate between the timebase
> > > > frequency used
> > > > to calibrate the MacOS timers and the actual frequency of the hardware
> > > > clock as
> > > > indicated by CUDA_TIMER_FREQ.
> > > >
> > > > Signed-off-by: Mark Cave-Ayland <address@hidden>
> > > > Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
> > >
> > > Applied, thanks.
> > Actually, this patch doesn't compile, because you've changed
> > ti->frequency in a couple of places where you should only be changing
> > s->frequency. I've fixed it up in my tree.
> Ah apologies there - this was a mistake I made quite early on in the
> patchset which I must have accidentally brought back in during a subsequent
> rebase :(
> I've just done a check on patch 7 ("cuda: set timer 1 frequency property to
> CUDA_TIMER_FREQ") and confirmed that the cuda.c version in your ppc-for-2.12
> branch matches that in my local branch, so your fixups are good.
> Any thoughts on the last few patches? I know Philippe had a question for
> Stefan re: the trace-events patch, however it would be useful for patches
> 10/11 to be applied as they complete the transition from CUDA over to using
> the mos6522 device (also providing an initial example as to how it can be
Haven't had a chance to look at them yet.
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
Description: PGP signature
[Qemu-ppc] [PATCHv2 01/12] cuda: do not use old_mmio accesses, Mark Cave-Ayland, 2018/02/09
[Qemu-ppc] [PATCHv2 04/12] cuda: introduce CUDAState parameter to get_counter(), Mark Cave-Ayland, 2018/02/09
[Qemu-ppc] [PATCHv2 09/12] misc: introduce new mos6522 VIA device and enable it for ppc builds, Mark Cave-Ayland, 2018/02/09
[Qemu-ppc] [PATCHv2 11/12] ppc: move CUDAState and other CUDA-related definitions into separate cuda.h file, Mark Cave-Ayland, 2018/02/09
- [Qemu-ppc] [PATCHv2 07/12] cuda: set timer 1 frequency property to CUDA_TIMER_FREQ, (continued)