linux-kernel - Re: Linux-2.6.31-rc4 - shut the annoying "failed to acquire vblank..."

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20090806143128.2c5b76fe@jbarnes-g45>
Date:	Thu, 6 Aug 2009 14:31:28 -0700
From:	Jesse Barnes <jbarnes@...tuousgeek.org>
To:	Frans Pop <elendil@...net.nl>
Cc:	Paul Rolland <rol@...917.net>, torvalds@...ux-foundation.org,
	linux-kernel@...r.kernel.org, rol@...be.net
Subject: Re: Linux-2.6.31-rc4 - shut the annoying "failed to acquire
 vblank..."

On Thu, 30 Jul 2009 13:06:50 +0200
Frans Pop <elendil@...net.nl> wrote:

> Paul Rolland wrote:
> > Some applications/hardware combinations are triggering the message
> > "failed to acquire vblank counter" to be issued up to 20 times a
> > second, which makes it both useless and dangerous, as this may hide
> > other important messages. This changes makes it only appear when
> > people are debugging. 
> > 
> > Signed-off-by: Paul Rolland <rol@...917.net>
> > Reviewed-by: Jesse Barnes <jbarnes@...tuousgeek.org>
> > 
> > --- linux-2.6.31-rc4/drivers/gpu/drm/drm_irq.c.orig     2009-07-26
> > 21:36:20.000000000 +0200 +++
> > linux-2.6.31-rc4/drivers/gpu/drm/drm_irq.c  2009-07-26
> > 21:36:34.000000000 +0200 @@ -566,7 +566,7 @@ int
> > drm_wait_vblank(struct drm_device *d
> > 
> > ret = drm_vblank_get(dev, crtc);
> > 	if (ret) {
> > -		DRM_ERROR("failed to acquire vblank counter,
> > %d\n", ret);
> > +		DRM_DEBUG("failed to acquire vblank counter,
> > %d\n", ret); return ret;
> > 	}
> > 	seq = drm_vblank_count(dev, crtc);
> 
> Actually, I just see that I get a different, but related error on
> suspend and when opening/closing a second X session:
>    *ERROR* trying to get vblank count for disabled pipe 0/1
> 
> Would it make sense to add the following to this patch:

Yes, that would be a good second patch (I also sent this one a few
times but it keeps getting lost).  If you want to send it to Linus more
formally, you can add my reviewed-by.

Thanks,
-- 
Jesse Barnes, Intel Open Source Technology Center
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/