On 26/04/16 23:35, Laurent Pinchart wrote: > The omapdrm driver uses a custom API to synchronize with the SGX GPU. > This is unusable as such in the mainline kernel as the API is only > partially implemented and requires additional out-of-tree patches. > Furthermore, as no SGX driver is available in the mainline kernel, the > API can't be considered as a stable mainline API. > > Remove buffer synchronization support to prepare for its replacement by > an implementation based on standard fences and reservation objects. I thought one thing the OMAP_GEM_CPU_PREP/FINI ioctls were supposed to do it cache flushing, if the buffer is OMAP_BO_CACHED. I don't think that works, though, which might be considered as an omapdrm bug. Also, if an app is using the prep/fini ioctls at the moment, even if it doesn't exactly require flushing (or any kind of synchronization), after this patch the app might fail. So even if we remove all the code behind, perhaps we should leave no-op ioctls. Tomi
Attachment:
signature.asc
Description: OpenPGP digital signature
_______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
