Re: [PATCH] try other pipe in wait_vblank if drm_vblank_get fails

2009-01-09 Thread Jesse Barnes
On Friday, January 9, 2009 12:28 am Michel Dänzer wrote: > On Thu, 2009-01-08 at 10:45 -0800, Jesse Barnes wrote: > > Dunno if this is a good idea or not, but it would have helped with some > > of the recent bugs. > > Sure about that? The sequence numbers may be completely different > between CRTCs

Re: [PATCH] try other pipe in wait_vblank if drm_vblank_get fails

2009-01-09 Thread Michel Dänzer
On Thu, 2009-01-08 at 10:45 -0800, Jesse Barnes wrote: > Dunno if this is a good idea or not, but it would have helped with some of the > recent bugs. Sure about that? The sequence numbers may be completely different between CRTCs. -- Earthling Michel Dänzer |http://ww

Re: [PATCH] try other pipe in wait_vblank if drm_vblank_get fails

2009-01-08 Thread Keith Packard
On Thu, 2009-01-08 at 17:30 -0800, Jesse Barnes wrote: > "assuming you meant the other pipe" but less wordy. "Trying other pipe" is > probably better. We shouldn't spam the syslog with messages here; if it happens, it's going to happen often. Perhaps just complaining once? -- keith.pack...@in

Re: [PATCH] try other pipe in wait_vblank if drm_vblank_get fails

2009-01-08 Thread Jesse Barnes
On Thursday, January 8, 2009 5:25 pm Ian Romanick wrote: > On Thu, Jan 08, 2009 at 10:45:16AM -0800, Jesse Barnes wrote: > > Dunno if this is a good idea or not, but it would have helped with some > > of the recent bugs. > > > > -- > > Jesse Barnes, Intel Open Source Technology Center > > > > diff

Re: [PATCH] try other pipe in wait_vblank if drm_vblank_get fails

2009-01-08 Thread Ian Romanick
On Thu, Jan 08, 2009 at 10:45:16AM -0800, Jesse Barnes wrote: > Dunno if this is a good idea or not, but it would have helped with some of the > recent bugs. > > -- > Jesse Barnes, Intel Open Source Technology Center > > diff --git a/drivers/gpu/drm/drm_irq.c b/drivers/gpu/drm/drm_irq.c > index

[PATCH] try other pipe in wait_vblank if drm_vblank_get fails

2009-01-08 Thread Jesse Barnes
Dunno if this is a good idea or not, but it would have helped with some of the recent bugs. -- Jesse Barnes, Intel Open Source Technology Center diff --git a/drivers/gpu/drm/drm_irq.c b/drivers/gpu/drm/drm_irq.c index 1e787f8..8ae2a6a 100644 --- a/drivers/gpu/drm/drm_irq.c +++ b/drivers/gpu/drm/