All of lore.kernel.org
 help / color / mirror / Atom feed
From: Pavel Machek <pavel@ucw.cz>
To: Schmidt Michal <xschmi00@stud.feec.vutbr.cz>
Cc: Ondrej Zary <linux@rainbow-software.org>,
	Robert Hancock <hancockr@shaw.ca>,
	Arjan van de Ven <arjan@linux.intel.com>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: [announce] Intel announces the PowerTOP utility for Linux
Date: Wed, 16 May 2007 14:20:48 +0000	[thread overview]
Message-ID: <20070516142048.GA4577@ucw.cz> (raw)
In-Reply-To: <464AB84F.8060609@stud.feec.vutbr.cz>

Hi!

> >On Tuesday 15 May 2007 02:13:15 Robert Hancock wrote:
> >>Looks like the radeon driver has the same problem as 
> >>the i915 driver
> >>mentioned on the known problems page - I get 60 
> >>wakeups/sec from it on
> >>my Compaq X1000 laptop (Radeon 9000 graphics) while in 
> >>X, which
> >>essentially prevents entry into C3.
> >
> >And Rage 128 too.
> >
> 
> I have a Radeon 7500 in my laptop. The attached hack to 
> disable the VBLANK interrupt works for me. I don't know 
> if it breaks 3D, I've only tried 2D.

Hmm, you should probably find where it is used, and only enable irq
when neccessary. Disabling it always will break something.

> diff --git a/drivers/char/drm/radeon_irq.c b/drivers/char/drm/radeon_irq.c
> index 3ff0baa..a990bb6 100644
> --- a/drivers/char/drm/radeon_irq.c
> +++ b/drivers/char/drm/radeon_irq.c
> @@ -227,9 +227,8 @@ void radeon_driver_irq_postinstall(drm_device_t * dev)
>  	atomic_set(&dev_priv->swi_emitted, 0);
>  	DRM_INIT_WAITQUEUE(&dev_priv->swi_queue);
>  
> -	/* Turn on SW and VBL ints */
> -	RADEON_WRITE(RADEON_GEN_INT_CNTL,
> -		     RADEON_CRTC_VBLANK_MASK | RADEON_SW_INT_ENABLE);
> +	/* Turn on the SW interrupt */
> +	RADEON_WRITE(RADEON_GEN_INT_CNTL, RADEON_SW_INT_ENABLE);
>  }
>  
>  void radeon_driver_irq_uninstall(drm_device_t * dev)


-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

  reply	other threads:[~2007-05-16 19:19 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <fa.FTEkheuP0nHOUv0cCp0k8pBuoMU@ifi.uio.no>
2007-05-15  0:13 ` [announce] Intel announces the PowerTOP utility for Linux Robert Hancock
2007-05-15 16:30   ` Ondrej Zary
2007-05-16  7:52     ` Schmidt Michal
2007-05-16 14:20       ` Pavel Machek [this message]
2007-05-11 23:07 Arjan van de Ven
2007-05-12  0:17 ` Matt Mackall
2007-05-12 11:00 ` Jan Engelhardt
2007-05-12 17:44   ` Arjan van de Ven
2007-05-12 21:12 ` Alistair John Strachan
2007-05-14 20:22   ` Jan Engelhardt
2007-05-14 20:42     ` Alistair John Strachan

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20070516142048.GA4577@ucw.cz \
    --to=pavel@ucw.cz \
    --cc=arjan@linux.intel.com \
    --cc=hancockr@shaw.ca \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux@rainbow-software.org \
    --cc=xschmi00@stud.feec.vutbr.cz \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.