[Intel-gfx] [PATCH] drm/vblank: Fixup and document timestamp update/read barriers

2015-05-07 Thread Mario Kleiner
On 05/07/2015 01:56 PM, Peter Hurley wrote: > On 05/06/2015 04:56 AM, Daniel Vetter wrote: >> On Tue, May 05, 2015 at 11:57:42AM -0400, Peter Hurley wrote: >>> On 05/05/2015 11:42 AM, Daniel Vetter wrote: On Tue, May 05, 2015 at 10:36:24AM -0400, Peter Hurley wrote: > On 05/04/2015 12:52 A

[Intel-gfx] [PATCH] drm/vblank: Fixup and document timestamp update/read barriers

2015-05-07 Thread Peter Hurley
On 05/06/2015 04:56 AM, Daniel Vetter wrote: > On Tue, May 05, 2015 at 11:57:42AM -0400, Peter Hurley wrote: >> On 05/05/2015 11:42 AM, Daniel Vetter wrote: >>> On Tue, May 05, 2015 at 10:36:24AM -0400, Peter Hurley wrote: On 05/04/2015 12:52 AM, Mario Kleiner wrote: > On 04/16/2015 03:03

[Intel-gfx] [PATCH] drm/vblank: Fixup and document timestamp update/read barriers

2015-05-06 Thread Daniel Vetter
On Tue, May 05, 2015 at 11:57:42AM -0400, Peter Hurley wrote: > On 05/05/2015 11:42 AM, Daniel Vetter wrote: > > On Tue, May 05, 2015 at 10:36:24AM -0400, Peter Hurley wrote: > >> On 05/04/2015 12:52 AM, Mario Kleiner wrote: > >>> On 04/16/2015 03:03 PM, Daniel Vetter wrote: > On Thu, Apr 16,

[Intel-gfx] [PATCH] drm/vblank: Fixup and document timestamp update/read barriers

2015-05-05 Thread Daniel Vetter
On Tue, May 05, 2015 at 10:36:24AM -0400, Peter Hurley wrote: > On 05/04/2015 12:52 AM, Mario Kleiner wrote: > > On 04/16/2015 03:03 PM, Daniel Vetter wrote: > >> On Thu, Apr 16, 2015 at 08:30:55AM -0400, Peter Hurley wrote: > >>> On 04/15/2015 01:31 PM, Daniel Vetter wrote: > On Wed, Apr 15,

[Intel-gfx] [PATCH] drm/vblank: Fixup and document timestamp update/read barriers

2015-05-05 Thread Peter Hurley
On 05/05/2015 11:57 AM, Peter Hurley wrote: > On 05/05/2015 11:42 AM, Daniel Vetter wrote: >> I'm also somewhat confused about how you to a line across both cpus for >> barriers because barriers only have cpu-local effects (which is why we >> always need a barrier on both ends of a transaction). I

[Intel-gfx] [PATCH] drm/vblank: Fixup and document timestamp update/read barriers

2015-05-05 Thread Peter Hurley
On 05/05/2015 11:42 AM, Daniel Vetter wrote: > On Tue, May 05, 2015 at 10:36:24AM -0400, Peter Hurley wrote: >> On 05/04/2015 12:52 AM, Mario Kleiner wrote: >>> On 04/16/2015 03:03 PM, Daniel Vetter wrote: On Thu, Apr 16, 2015 at 08:30:55AM -0400, Peter Hurley wrote: > On 04/15/2015 01:31

[Intel-gfx] [PATCH] drm/vblank: Fixup and document timestamp update/read barriers

2015-05-05 Thread Peter Hurley
On 05/04/2015 12:52 AM, Mario Kleiner wrote: > On 04/16/2015 03:03 PM, Daniel Vetter wrote: >> On Thu, Apr 16, 2015 at 08:30:55AM -0400, Peter Hurley wrote: >>> On 04/15/2015 01:31 PM, Daniel Vetter wrote: On Wed, Apr 15, 2015 at 09:00:04AM -0400, Peter Hurley wrote: > Hi Daniel, > >>>

[Intel-gfx] [PATCH] drm/vblank: Fixup and document timestamp update/read barriers

2015-05-04 Thread Mario Kleiner
On 04/16/2015 03:03 PM, Daniel Vetter wrote: > On Thu, Apr 16, 2015 at 08:30:55AM -0400, Peter Hurley wrote: >> On 04/15/2015 01:31 PM, Daniel Vetter wrote: >>> On Wed, Apr 15, 2015 at 09:00:04AM -0400, Peter Hurley wrote: Hi Daniel, On 04/15/2015 03:17 AM, Daniel Vetter wrote: >

[Intel-gfx] [PATCH] drm/vblank: Fixup and document timestamp update/read barriers

2015-04-16 Thread Daniel Vetter
On Thu, Apr 16, 2015 at 08:30:55AM -0400, Peter Hurley wrote: > On 04/15/2015 01:31 PM, Daniel Vetter wrote: > > On Wed, Apr 15, 2015 at 09:00:04AM -0400, Peter Hurley wrote: > >> Hi Daniel, > >> > >> On 04/15/2015 03:17 AM, Daniel Vetter wrote: > >>> This was a bit too much cargo-culted, so lets m

[Intel-gfx] [PATCH] drm/vblank: Fixup and document timestamp update/read barriers

2015-04-16 Thread Peter Hurley
On 04/15/2015 01:31 PM, Daniel Vetter wrote: > On Wed, Apr 15, 2015 at 09:00:04AM -0400, Peter Hurley wrote: >> Hi Daniel, >> >> On 04/15/2015 03:17 AM, Daniel Vetter wrote: >>> This was a bit too much cargo-culted, so lets make it solid: >>> - vblank->count doesn't need to be an atomic, writes are

[Intel-gfx] [PATCH] drm/vblank: Fixup and document timestamp update/read barriers

2015-04-15 Thread Daniel Vetter
On Wed, Apr 15, 2015 at 09:00:04AM -0400, Peter Hurley wrote: > Hi Daniel, > > On 04/15/2015 03:17 AM, Daniel Vetter wrote: > > This was a bit too much cargo-culted, so lets make it solid: > > - vblank->count doesn't need to be an atomic, writes are always done > > under the protection of dev->v

[Intel-gfx] [PATCH] drm/vblank: Fixup and document timestamp update/read barriers

2015-04-15 Thread Daniel Vetter
On Wed, Apr 15, 2015 at 09:17:03AM +0100, Chris Wilson wrote: > On Wed, Apr 15, 2015 at 09:17:02AM +0200, Daniel Vetter wrote: > > diff --git a/drivers/gpu/drm/drm_irq.c b/drivers/gpu/drm/drm_irq.c > > index c8a34476570a..23bfbc61a494 100644 > > --- a/drivers/gpu/drm/drm_irq.c > > +++ b/drivers/gpu

[Intel-gfx] [PATCH] drm/vblank: Fixup and document timestamp update/read barriers

2015-04-15 Thread Chris Wilson
On Wed, Apr 15, 2015 at 11:25:00AM +0200, Daniel Vetter wrote: > On Wed, Apr 15, 2015 at 09:17:03AM +0100, Chris Wilson wrote: > > On Wed, Apr 15, 2015 at 09:17:02AM +0200, Daniel Vetter wrote: > > > diff --git a/drivers/gpu/drm/drm_irq.c b/drivers/gpu/drm/drm_irq.c > > > index c8a34476570a..23bfbc