Re: Keep vblank disabled after VT switch

2007-05-10 Thread Keith Packard
On Thu, 2007-05-10 at 20:08 +0200, Michel Dänzer wrote:

> Looks good.

pushed.

-- 
[EMAIL PROTECTED]


signature.asc
Description: This is a digitally signed message part
-
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: Change linux kernel install to use kernel/drivers/char/drm

2007-05-10 Thread Dawid Gajownik
Dnia 05/10/2007 08:06 PM, Użytkownik Keith Packard napisał:
> The kernel version of the DRM drivers installs everything to
> kernel/drivers/char/drm, but the out-of-tree version sticks them in
> extras/, which isn't very nice.

According to Linux documentation (kbuild/modules.txt) this seems to be 
correct:

--8<

=== 6. Module installation

Modules which are included in the kernel are installed in the directory:

 /lib/modules/$(KERNELRELEASE)/kernel

External modules are installed in the directory:

 /lib/modules/$(KERNELRELEASE)/extra

--8<

What is more, I find it very useful. This feature allows me to update 
modules without overwriting ones from the original kernel RPM package.

Regards,
Dawid

-- 

   ^_*


-
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: Keep vblank disabled after VT switch

2007-05-10 Thread Michel Dänzer
On Thu, 2007-05-10 at 11:02 -0700, Keith Packard wrote:
> Here's a cleaner patch that keeps interrupts disabled when VT switch
> occurs (which calls i915_driver_irq_postinstall). It's quite simple --
> it sets the default vblank_pipe at init time to pipe A, then doesn't
> check for 0 in the postinstall function.

Looks good.


-- 
Earthling Michel Dänzer   |  http://tungstengraphics.com
Libre software enthusiast |  Debian, X and DRI developer


-
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Change linux kernel install to use kernel/drivers/char/drm

2007-05-10 Thread Keith Packard
The kernel version of the DRM drivers installs everything to
kernel/drivers/char/drm, but the out-of-tree version sticks them in
extras/, which isn't very nice.

diff --git a/linux-core/Makefile b/linux-core/Makefile
index af29bd6..d478bb1 100644
--- a/linux-core/Makefile
+++ b/linux-core/Makefile
@@ -33,6 +33,8 @@
 
 SHELL=/bin/sh
 
+INSTALL_MOD_DIR = kernel/drivers/char/drm
+
 .SUFFIXES:
 
 ifndef LINUXDIR
@@ -237,8 +239,8 @@ dristat: dristat.c
 drmstat: drmstat.c
$(CC) $(PRGCFLAGS) $< -o $@ $(DRMSTATLIBS)
 
-install:
-   make -C $(LINUXDIR) $(GETCONFIG) SUBDIRS=`pwd` DRMSRCDIR=`pwd` 
modules_install
+install modules_install:
+   make -C $(LINUXDIR) $(GETCONFIG) INSTALL_MOD_DIR=$(INSTALL_MOD_DIR) 
SUBDIRS=`pwd` DRMSRCDIR=`pwd` modules_install
 
 else
 


-- 
[EMAIL PROTECTED]


signature.asc
Description: This is a digitally signed message part
-
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Keep vblank disabled after VT switch

2007-05-10 Thread Keith Packard
Here's a cleaner patch that keeps interrupts disabled when VT switch
occurs (which calls i915_driver_irq_postinstall). It's quite simple --
it sets the default vblank_pipe at init time to pipe A, then doesn't
check for 0 in the postinstall function.

index cb5dcea..8991a3c 100644
--- a/shared-core/i915_dma.c
+++ b/shared-core/i915_dma.c
@@ -177,6 +177,10 @@ static int i915_initialize(drm_device_t * dev,
 */
dev_priv->allow_batchbuffer = 1;
 
+   /* Enable vblank on pipe A for older X servers
+*/
+   dev_priv->vblank_pipe = DRM_I915_VBLANK_PIPE_A;
+
/* Program Hardware Status Page */
dev_priv->status_page_dmah = drm_pci_alloc(dev, PAGE_SIZE, PAGE_SIZE, 
0x);
diff --git a/shared-core/i915_irq.c b/shared-core/i915_irq.c
index 8748b64..e333545 100644
--- a/shared-core/i915_irq.c
+++ b/shared-core/i915_irq.c
@@ -726,9 +726,6 @@ void i915_driver_irq_postinstall(drm_device_t * dev)
INIT_LIST_HEAD(&dev_priv->vbl_swaps.head);
dev_priv->swaps_pending = 0;
 
-   if (!dev_priv->vblank_pipe)
-   dev_priv->vblank_pipe = DRM_I915_VBLANK_PIPE_A;
-
dev_priv->swaps_lock = SPIN_LOCK_UNLOCKED;
INIT_LIST_HEAD(&dev_priv->vbl_swaps.head);
dev_priv->swaps_pending = 0;
@@ -736,8 +733,6 @@ void i915_driver_irq_postinstall(drm_device_t * dev)
dev_priv->user_irq_lock = SPIN_LOCK_UNLOCKED;
dev_priv->user_irq_refcount = 0;
 
-   if (!dev_priv->vblank_pipe)
-   dev_priv->vblank_pipe = DRM_I915_VBLANK_PIPE_A;
i915_enable_interrupt(dev);
DRM_INIT_WAITQUEUE(&dev_priv->irq_queue);
 


-- 
[EMAIL PROTECTED]


signature.asc
Description: This is a digitally signed message part
-
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: [PATCH] make radeons fire fewer vblank interrupts

2007-05-10 Thread Jesse Barnes
On Wednesday, May 9, 2007 8:56 am Eric Anholt wrote:
> > I suspect doing it like this might break userspace expectations
> > about the behaviour of the vblank counter. It would be better to do
> > it similarly to how Eric Anholt did it for i915, i.e. by toggling
> > the vblank interrupt in the 2D driver TransitionTo2/3D hooks. Or
> > possibly even better (as this would e.g. still leave it enabled all
> > the time when using a GLX compositing manager), the 3D driver could
> > tell the DRM whether it needs the vblank interrupt or not.
>
> Yeah, I wasn't sure if the 3D driver would be able to know easily
> when it might need to wait on an absolute vblank.  Arjan suggested
> that we not turn off the vblank when 3d is running until nobody's
> waited one one for (for example) a second, and then if someone waits
> on one after that, the kernel can re-enable the interrupt and
> extrapolate the vblank counter using the system clock.  Hiding vblank
> enable/disable knowledge in the kernel sounds nicer than what I did
> on i915.

Yeah, most drivers will probably have to grow power management state 
machines like this if we're really going to be aggressive in turning 
them off.  I'll try to come up with a patch that does that (then on to 
ipw2200...).

Thanks,
Jesse

-
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 10855] on Intel 945G, (beryl or compiz) + glxgears = DRM_I830_CMDBUFFER: -22

2007-05-10 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=10855


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||DUPLICATE




--- Comment #1 from [EMAIL PROTECTED]  2007-05-10 01:03 PST ---


*** This bug has been marked as a duplicate of bug 10702 ***


-- 
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel