[PATCH 1/2] drm: Shortcircuit vblank queries

2015-04-15 Thread Michel Dänzer
On 14.04.2015 23:21, Chris Wilson wrote: > On Tue, Apr 14, 2015 at 06:42:03PM +0900, Michel Dänzer wrote: >> Also, the two patches should have different and more specific shortlogs. > > Second patch: > > drm: Query vblank counters directly for known accurate state > > ? Sounds good to me.

[PATCH 1/2] drm: Shortcircuit vblank queries

2015-04-14 Thread Mario Kleiner
On 04/14/2015 08:36 PM, Chris Wilson wrote: > On Tue, Apr 14, 2015 at 08:22:20PM +0200, Mario Kleiner wrote: >> On 04/05/2015 05:40 PM, Chris Wilson wrote: >>> Avoid adding to the waitqueue and reprobing the current vblank if the >>> caller is only querying the current vblank sequence and

[PATCH 1/2] drm: Shortcircuit vblank queries

2015-04-14 Thread Mario Kleiner
On 04/05/2015 05:40 PM, Chris Wilson wrote: > Avoid adding to the waitqueue and reprobing the current vblank if the > caller is only querying the current vblank sequence and timestamp and > so we would return immediately. > > Signed-off-by: Chris Wilson > Cc: Ville Syrjälä > Cc: Daniel Vetter

[PATCH 1/2] drm: Shortcircuit vblank queries

2015-04-14 Thread Mario Kleiner
On 04/14/2015 04:21 PM, Chris Wilson wrote: > On Tue, Apr 14, 2015 at 06:42:03PM +0900, Michel Dänzer wrote: >> Also, the two patches should have different and more specific shortlogs. > > Second patch: > > drm: Query vblank counters directly for known accurate state > > ? > When i applied your

[PATCH 1/2] drm: Shortcircuit vblank queries

2015-04-14 Thread Chris Wilson
On Tue, Apr 14, 2015 at 08:22:20PM +0200, Mario Kleiner wrote: > On 04/05/2015 05:40 PM, Chris Wilson wrote: > >Avoid adding to the waitqueue and reprobing the current vblank if the > >caller is only querying the current vblank sequence and timestamp and > >so we would return immediately. > > >

[PATCH 1/2] drm: Shortcircuit vblank queries

2015-04-14 Thread Chris Wilson
On Tue, Apr 14, 2015 at 08:17:21PM +0200, Mario Kleiner wrote: > On 04/14/2015 04:21 PM, Chris Wilson wrote: > >On Tue, Apr 14, 2015 at 06:42:03PM +0900, Michel Dänzer wrote: > >>Also, the two patches should have different and more specific shortlogs. > > > >Second patch: > > > >drm: Query vblank

[PATCH 1/2] drm: Shortcircuit vblank queries

2015-04-14 Thread Michel Dänzer
On 06.04.2015 00:40, Chris Wilson wrote: > Avoid adding to the waitqueue and reprobing the current vblank if the > caller is only querying the current vblank sequence and timestamp and > so we would return immediately. > > Signed-off-by: Chris Wilson > Cc: Ville Syrjälä > Cc: Daniel Vetter >

[PATCH 1/2] drm: Shortcircuit vblank queries

2015-04-14 Thread Chris Wilson
On Tue, Apr 14, 2015 at 06:42:03PM +0900, Michel Dänzer wrote: > Also, the two patches should have different and more specific shortlogs. Second patch: drm: Query vblank counters directly for known accurate state ? -- Chris Wilson, Intel Open Source Technology Centre

[PATCH 1/2] drm: Shortcircuit vblank queries

2015-04-05 Thread Chris Wilson
Avoid adding to the waitqueue and reprobing the current vblank if the caller is only querying the current vblank sequence and timestamp and so we would return immediately. Signed-off-by: Chris Wilson Cc: Ville Syrjälä Cc: Daniel Vetter Cc: Michel Dänzer Cc: Laurent Pinchart Cc: Dave