On 12/1/18 6:30 AM, Peter Maydell wrote: > Patchset structure: > * patch 1 does the "make sure we have the iothread lock for > calls into QEMU" (which is effectively a no-op initially > since we'll already be holding that lock when our refresh > etc callbacks are called) > * patch 2 makes switchSurface directly take the pixman image > (which is refcounted) rather than the DisplaySurface (which > is not), so we can make the calls to it asynchronous later > * patches 3 and 4 are just trivial code motion > * patch 5 does the bulk of the work (and can't really be split > further without the UI being broken at the intermediate point)
FWIW, this makes sense and is relatively easy to follow. That said, I've never touched OSX at all, so can't even test. r~
