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 1e787f8..8ae2a6a 100644 > --- a/drivers/gpu/drm/drm_irq.c > +++ b/drivers/gpu/drm/drm_irq.c > @@ -531,8 +542,11 @@ int drm_wait_vblank(struct drm_device *dev, void *data, > > ret = drm_vblank_get(dev, crtc); > if (ret) { > - DRM_ERROR("failed to acquire vblank counter, %d\n", ret); > - return ret; > + DRM_ERROR("failed to acquire vblank counter, %d, assuming to > other pipe\n", ret);
I don't know what "assuming to other pipe" means. Maybe "trying other pipe"? > + crtc = !crtc; > + ret = drm_vblank_get(dev, crtc); > + if (ret) > + return ret; Shouldn't there also be an error message here? > } > seq = drm_vblank_count(dev, crtc);
pgpb6D6hgjUrq.pgp
Description: PGP signature
------------------------------------------------------------------------------ Check out the new SourceForge.net Marketplace. It is the best place to buy or sell services for just about anything Open Source. http://p.sf.net/sfu/Xq1LFB
-- _______________________________________________ Dri-devel mailing list Dri-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/dri-devel