[PATCH xserver 1/9] Create/Destroy/Trigger/Reset Fence Sync objects

2010-09-21 Thread James Jones
. However, it is also desireable to associate a screen with fence sync objects at creation time so that the associated screen's driver can allocate any HW- specific resources needed by the fence object up front. Signed-off-by: James Jones jajo...@nvidia.com Reviewed-by: Aaron Plattner aplatt

[PATCH xserver 2/9] Add XSyncQueryFence()

2010-09-21 Thread James Jones
Allows callers to query whether or not a given fence sync object is currently triggered. Signed-off-by: James Jones jajo...@nvidia.com Reviewed-by: Aaron Plattner aplatt...@nvidia.com Reviewed-by: Robert Morell rmor...@nvidia.com --- Xext/sync.c | 49

[PATCH xserver 3/9] Create SyncObject base type.

2010-09-21 Thread James Jones
SyncObject is now the base type for SyncCounter and SyncFence. Data to be used by both types is stored in the base object. Both SyncCounter and SyncFence can be safely cast to SyncObject, and a SyncObject can be cast to the correct type based on SyncObject::type. Signed-off-by: James Jones jajo

[PATCH xserver 4/9] Make Await SyncTrigger functions generic

2010-09-21 Thread James Jones
Update all the functions dealing with Await sync triggers handle generic sync objects instead of just counters. This will facilitate code sharing between the counter sync waits and the fence sync waits. Signed-off-by: James Jones jajo...@nvidia.com Reviewed-by: Aaron Plattner aplatt

[PATCH xserver 5/9] Generalize comment above Sync CheckTriggered funcs

2010-09-21 Thread James Jones
The comment referred only to counter sync objects and did not include the new fence sync CheckTriggered function. Generalize the language so it applies to all the CheckTriggered functions. Signed-off-by: James Jones jajo...@nvidia.com Reviewed-by: Aaron Plattner aplatt...@nvidia.com Reviewed

[PATCH xserver 6/9] Add XSyncAwaitFence() handler

2010-09-21 Thread James Jones
in the previous refactoring changes. Signed-off-by: James Jones jajo...@nvidia.com Reviewed-by: Aaron Plattner aplatt...@nvidia.com Reviewed-by: Robert Morell rmor...@nvidia.com --- Xext/sync.c | 304 +++ 1 files changed, 223 insertions(+), 81

[PATCH xserver 7/9] Add XDamageSubtractAndTrigger operation

2010-09-21 Thread James Jones
XDamageSubtractAndTrigger behaves exactly like XDamageSubtract except it receives an optional fence sync object. If the value of this object is not None, it is triggered by X once all the rendering associated with the damage regions being subtracted has completed. Signed-off-by: James Jones jajo

[PATCH xserver 8/9] Add fence sync driver interface

2010-09-21 Thread James Jones
to notify driver when adding/removing triggers to/ from the sync object. Signed-off-by: James Jones jajo...@nvidia.com Reviewed-by: Aaron Plattner aplatt...@nvidia.com Reviewed-by: Robert Morell rmor...@nvidia.com --- Xext/sync.c | 38 +++ dix/privates.c |1

[PATCH xserver 9/9] Export SyncVerifyFence() in new SDK header

2010-09-21 Thread James Jones
Add syncsdk.h, a new xorg SDK header. It contains SyncVerifyFence() and the helper functions that use it to look up fence sync objects. Exporting this functionality in an SDK header allows 3rd party extensions to look up fence objects in their interop APIs. Signed-off-by: James Jones jajo

Re: [PATCH xextproto 1/4] Document changes in XSync version 3.1

2010-08-17 Thread James Jones
On Tuesday 17 August 2010 15:01:53 Aaron Plattner wrote: On Tue, Aug 17, 2010 at 02:49:41PM -0700, Adam Jackson wrote: Apologies for the slow review here... No worries. Thanks for taking the time. On Thu, 2010-08-12 at 16:40 -0700, James Jones wrote: para -The extension adds

[PATCH xextproto 0/4] XSync Fence Objects (rev. 2)

2010-08-12 Thread James Jones
below whatever the lib code picks up from compile-time constants in xextproto. James Jones (4): Document changes in XSync version 3.1 Initial Fence Sync support Add XSyncQueryFence() Add protocol for XSyncAwaitFence() specs/sync.xml | 219

[PATCH xextproto 1/4] Document changes in XSync version 3.1

2010-08-12 Thread James Jones
Signed-off-by: James Jones jajo...@nvidia.com Reviewed-by: Pierre-Loup Griffais pgriff...@nvidia.com Reviewed-by: Aaron Plattner aplatt...@nvidia.com Reviewed-by: Robert Morell rmor...@nvidia.com --- specs/sync.xml | 219 +-- 1 files changed

[PATCH xextproto 2/4] Initial Fence Sync support

2010-08-12 Thread James Jones
Defines the protocol for creation and basic management of binary state sync objects. The following operations are defined: -Creation -Destruction -Trigger -Reset Signed-off-by: James Jones jajo...@nvidia.com Reviewed-by: Aaron Plattner aplatt...@nvidia.com --- syncconst.h |6

[PATCH xextproto 3/4] Add XSyncQueryFence()

2010-08-12 Thread James Jones
Allows callers to query whether a given fence sync object is currently triggered or not. Signed-off-by: James Jones jajo...@nvidia.com Reviewed-by: Aaron Plattner aplatt...@nvidia.com --- syncproto.h | 28 1 files changed, 28 insertions(+), 0 deletions(-) diff

[PATCH xextproto 4/4] Add protocol for XSyncAwaitFence()

2010-08-12 Thread James Jones
Add the fence sync object equivalent of XSyncAwait() Signed-off-by: James Jones jajo...@nvidia.com Reviewed-by: Aaron Plattner aplatt...@nvidia.com --- syncproto.h | 12 1 files changed, 12 insertions(+), 0 deletions(-) diff --git a/syncproto.h b/syncproto.h index 7e0a568

[PATCH damageproto 0/2] Add XDamageSubtractAndTrigger

2010-08-12 Thread James Jones
Adds the XDamageSubtractAndTrigger() request and bumps the damage protocol to version 1.2. Now with protocol spec updates including overview, justification, and intended usage of the new reqeust. James Jones (2): Document changes in damage proto version 1.2 Add XDamageSubtractAndTrigger

[PATCH damageproto 1/2] Document changes in damage proto version 1.2

2010-08-12 Thread James Jones
Signed-off-by: James Jones jajo...@nvidia.com Reviewed-by: Pierre-Loup Griffais pgriff...@nvidia.com Reviewed-by: Aaron Plattner aplatt...@nvidia.com Reviewed-by: Robert Morell rmor...@nvidia.com --- damageproto.txt | 57 +- 1 files changed

[PATCH damageproto 2/2] Add XDamageSubtractAndTrigger operation

2010-08-12 Thread James Jones
XDamageSubtractAndTrigger behaves exactly like XDamageSubtract except it receives an optional fence sync object. If the value of this object is not None, it is triggered by X once all the rendering associated with the damage regions being subtracted has completed. Signed-off-by: James Jones jajo

[PATCH xextproto 0/4] XSync Fence Objects

2010-08-09 Thread James Jones
cross-API synchronization with X rendering operations. James Jones (4): Initial Fence Sync support Add XSyncQueryFence() Add protocol for XSyncAwaitFence() Add a Drawable argument to XSyncCreateFence() syncconst.h |6 ++- syncproto.h | 96

[PATCH xextproto 1/4] Initial Fence Sync support

2010-08-09 Thread James Jones
Defines the protocol for creation and basic management of binary state sync objects. The following operations are defined: -Creation -Destruction -Trigger -Reset Signed-off-by: James Jones jajo...@nvidia.com Reviewed-by: Aaron Plattner aplatt...@nvidia.com --- syncconst.h |6

[PATCH xextproto 4/4] Add a Drawable argument to XSyncCreateFence()

2010-08-09 Thread James Jones
-by: James Jones jajo...@nvidia.com Reviewed-by: Aaron Plattner aplatt...@nvidia.com --- syncproto.h |5 - 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/syncproto.h b/syncproto.h index 6e851e4..1453e44 100644 --- a/syncproto.h +++ b/syncproto.h @@ -81,6 +81,7 @@ PERFORMANCE

Re: [PATCH xextproto 0/4] XSync Fence Objects

2010-08-09 Thread James Jones
On Monday 09 August 2010 12:52:30 pm Alan Coopersmith wrote: James Jones wrote: Adds support for binary sync objects. Objects are set to triggered using X commands that are executed relative to X rendering commands. Clients can wait for fence sync objcts to reach the triggered state

Re: [PATCH xextproto 0/4] XSync Fence Objects

2010-08-09 Thread James Jones
On Monday 09 August 2010 1:56:34 pm Keith Packard wrote: * PGP Signed by an unknown key On Mon, 9 Aug 2010 13:41:54 -0700, James Jones jajo...@nvidia.com wrote: If anyone thinks docs would make reviewing the code changes easier, I can write them up now rather than waiting for more feedback

Review Request: X Synchronization Fences

2010-06-28 Thread James Jones
to get the basic changes reviewed first. Thanks, -James Jones --- This email message is for the sole use of the intended recipient(s) and may contain confidential information. Any unauthorized review, use, disclosure

[PATCH] Cast void* to pointer* to appease some compilers.

2010-06-18 Thread James Jones
When this privates.h is included in C++ builds, the compiler complains about implicitly casting void* to void**. This small patch fixes that up. Signed-off-by: James Jones jajo...@nvidia.com --- include/privates.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/include

<    1   2