On Wed, Jun 29, 2011 at 03:09:36PM +0200, Gerd Hoffmann wrote: > Hi, > > >+ case QXL_IO_DESTROY_ALL_SURFACES_ASYNC: > >+ d->mode = QXL_MODE_UNDEFINED; > > Should go to the async thread.
doesn't it make more sense to do all state changes from the vcpu thread? async thread can run much later, if you have a QXL_IO_DESTROY_ALL_SURFACES_ASYNC followed by a QXL_IO_CREATE_PRIMARY_ASYNC where the driver did not wait for the completion of the ASYNC first, I would still like to support that, but it won't work if I move this to the async thread. > > cheers, > Gerd
