Re: [Intel-gfx] [PATCH] [drm] Move drm vblank initialization/cleanup to driver load/unload

2008-11-24 Thread Jesse Barnes
On Tuesday, November 18, 2008 5:27 pm Keith Packard wrote: On Tue, 2008-11-18 at 13:00 -0800, Jesse Barnes wrote: Just remembered that other drivers will need updates here too, otherwise they may leak memory due to multiple drm_vblank_init calls on every IRQ postinstall. Here's an

Re: [Intel-gfx] [PATCH] [drm] Move drm vblank initialization/cleanup to driver load/unload

2008-11-21 Thread Stefano Avallone
Hi I applied this patch on top of kernel 2.6.28-rc5 and I found that VT switch works under KDE4 with composite enabled. However, suspend to disk does not work. After resuming, I can only see a black screen and the cursor, which I can move. At that point, I can only ssh into the machine and

Re: [Intel-gfx] [PATCH] [drm] Move drm vblank initialization/cleanup to driver load/unload

2008-11-18 Thread Jesse Barnes
On Tuesday, November 18, 2008 12:17 pm Jesse Barnes wrote: On Tuesday, November 18, 2008 9:48 am Keith Packard wrote: This should fix the compiz issues across vt switch (works for me). From 087e40e095de737d948f26bb1c28bba34a274fdb Mon Sep 17 00:00:00 2001 From: Keith Packard [EMAIL

Re: [Intel-gfx] [PATCH] [drm] Move drm vblank initialization/cleanup to driver load/unload

2008-11-18 Thread Keith Packard
On Tue, 2008-11-18 at 13:00 -0800, Jesse Barnes wrote: Just remembered that other drivers will need updates here too, otherwise they may leak memory due to multiple drm_vblank_init calls on every IRQ postinstall. Here's an untested patch that fixes the other drivers in the same way: From