[PATCH 08/32] drm/nouveau: use system_wq instead of dev_priv->wq

2011-01-03 Thread Tejun Heo
With cmwq, there's no reason for nouveau to use a dedicated workqueue. Drop dev_priv->wq and use system_wq instead. Because nouveau_irq_uninstall() may be called from unsleepable context, the work items can't be flushed from there. Instead, init and flush from nouveau_load/unload(). Signed-off-b

[PATCH 08/32] drm/nouveau: use system_wq instead of dev_priv->wq

2011-01-03 Thread Tejun Heo
With cmwq, there's no reason for nouveau to use a dedicated workqueue. Drop dev_priv->wq and use system_wq instead. Because nouveau_irq_uninstall() may be called from unsleepable context, the work items can't be flushed from there. Instead, init and flush from nouveau_load/unload(). Signed-off-b

[PATCH 08/32] drm/nouveau: use system_wq instead of dev_priv->wq

2011-01-05 Thread Ben Skeggs
On Mon, 2011-01-03 at 14:49 +0100, Tejun Heo wrote: > With cmwq, there's no reason for nouveau to use a dedicated workqueue. > Drop dev_priv->wq and use system_wq instead. > > Because nouveau_irq_uninstall() may be called from unsleepable > context, the work items can't be flushed from there. Ins

[PATCH 08/32] drm/nouveau: use system_wq instead of dev_priv->wq

2011-01-05 Thread Ben Skeggs
On Wed, 2011-01-05 at 11:07 +1000, Ben Skeggs wrote: > On Mon, 2011-01-03 at 14:49 +0100, Tejun Heo wrote: > > With cmwq, there's no reason for nouveau to use a dedicated workqueue. > > Drop dev_priv->wq and use system_wq instead. > > > > Because nouveau_irq_uninstall() may be called from unsleepa

[PATCH 08/32] drm/nouveau: use system_wq instead of dev_priv->wq

2011-01-06 Thread Tejun Heo
Hello, On Wed, Jan 05, 2011 at 11:16:05AM +1000, Ben Skeggs wrote: > On Wed, 2011-01-05 at 11:07 +1000, Ben Skeggs wrote: > > On Mon, 2011-01-03 at 14:49 +0100, Tejun Heo wrote: > > > With cmwq, there's no reason for nouveau to use a dedicated workqueue. > > > Drop dev_priv->wq and use system_wq i

[PATCH 08/32] drm/nouveau: use system_wq instead of dev_priv->wq

2011-01-03 Thread Tejun Heo
With cmwq, there's no reason for nouveau to use a dedicated workqueue. Drop dev_priv->wq and use system_wq instead. Because nouveau_irq_uninstall() may be called from unsleepable context, the work items can't be flushed from there. Instead, init and flush from nouveau_load/unload(). Signed-off-b

[PATCH 08/32] drm/nouveau: use system_wq instead of dev_priv->wq

2011-01-03 Thread Tejun Heo
With cmwq, there's no reason for nouveau to use a dedicated workqueue. Drop dev_priv->wq and use system_wq instead. Because nouveau_irq_uninstall() may be called from unsleepable context, the work items can't be flushed from there. Instead, init and flush from nouveau_load/unload(). Signed-off-b

[PATCH 08/32] drm/nouveau: use system_wq instead of dev_priv->wq

2011-01-05 Thread Ben Skeggs
On Mon, 2011-01-03 at 14:49 +0100, Tejun Heo wrote: > With cmwq, there's no reason for nouveau to use a dedicated workqueue. > Drop dev_priv->wq and use system_wq instead. > > Because nouveau_irq_uninstall() may be called from unsleepable > context, the work items can't be flushed from there. Ins

[PATCH 08/32] drm/nouveau: use system_wq instead of dev_priv->wq

2011-01-05 Thread Ben Skeggs
On Wed, 2011-01-05 at 11:07 +1000, Ben Skeggs wrote: > On Mon, 2011-01-03 at 14:49 +0100, Tejun Heo wrote: > > With cmwq, there's no reason for nouveau to use a dedicated workqueue. > > Drop dev_priv->wq and use system_wq instead. > > > > Because nouveau_irq_uninstall() may be called from unsleepa

[PATCH 08/32] drm/nouveau: use system_wq instead of dev_priv->wq

2011-01-06 Thread Tejun Heo
Hello, On Wed, Jan 05, 2011 at 11:16:05AM +1000, Ben Skeggs wrote: > On Wed, 2011-01-05 at 11:07 +1000, Ben Skeggs wrote: > > On Mon, 2011-01-03 at 14:49 +0100, Tejun Heo wrote: > > > With cmwq, there's no reason for nouveau to use a dedicated workqueue. > > > Drop dev_priv->wq and use system_wq i

[PATCH 08/32] drm/nouveau: use system_wq instead of dev_priv->wq

2011-01-03 Thread Tejun Heo
With cmwq, there's no reason for nouveau to use a dedicated workqueue. Drop dev_priv->wq and use system_wq instead. Because nouveau_irq_uninstall() may be called from unsleepable context, the work items can't be flushed from there. Instead, init and flush from nouveau_load/unload(). Signed-off-b

[PATCH 08/32] drm/nouveau: use system_wq instead of dev_priv->wq

2011-01-03 Thread Tejun Heo
With cmwq, there's no reason for nouveau to use a dedicated workqueue. Drop dev_priv->wq and use system_wq instead. Because nouveau_irq_uninstall() may be called from unsleepable context, the work items can't be flushed from there. Instead, init and flush from nouveau_load/unload(). Signed-off-b

[PATCH 08/32] drm/nouveau: use system_wq instead of dev_priv->wq

2011-01-03 Thread Tejun Heo
With cmwq, there's no reason for nouveau to use a dedicated workqueue. Drop dev_priv->wq and use system_wq instead. Because nouveau_irq_uninstall() may be called from unsleepable context, the work items can't be flushed from there. Instead, init and flush from nouveau_load/unload(). Signed-off-b

[PATCH 08/32] drm/nouveau: use system_wq instead of dev_priv->wq

2011-01-05 Thread Ben Skeggs
On Mon, 2011-01-03 at 14:49 +0100, Tejun Heo wrote: > With cmwq, there's no reason for nouveau to use a dedicated workqueue. > Drop dev_priv->wq and use system_wq instead. > > Because nouveau_irq_uninstall() may be called from unsleepable > context, the work items can't be flushed from there. Ins

[PATCH 08/32] drm/nouveau: use system_wq instead of dev_priv->wq

2011-01-05 Thread Ben Skeggs
On Wed, 2011-01-05 at 11:07 +1000, Ben Skeggs wrote: > On Mon, 2011-01-03 at 14:49 +0100, Tejun Heo wrote: > > With cmwq, there's no reason for nouveau to use a dedicated workqueue. > > Drop dev_priv->wq and use system_wq instead. > > > > Because nouveau_irq_uninstall() may be called from unsleepa

[PATCH 08/32] drm/nouveau: use system_wq instead of dev_priv->wq

2011-01-06 Thread Tejun Heo
Hello, On Wed, Jan 05, 2011 at 11:16:05AM +1000, Ben Skeggs wrote: > On Wed, 2011-01-05 at 11:07 +1000, Ben Skeggs wrote: > > On Mon, 2011-01-03 at 14:49 +0100, Tejun Heo wrote: > > > With cmwq, there's no reason for nouveau to use a dedicated workqueue. > > > Drop dev_priv->wq and use system_wq i

[PATCH 08/32] drm/nouveau: use system_wq instead of dev_priv->wq

2011-01-03 Thread Tejun Heo
With cmwq, there's no reason for nouveau to use a dedicated workqueue. Drop dev_priv->wq and use system_wq instead. Because nouveau_irq_uninstall() may be called from unsleepable context, the work items can't be flushed from there. Instead, init and flush from nouveau_load/unload(). Signed-off-b

[PATCH 08/32] drm/nouveau: use system_wq instead of dev_priv->wq

2011-01-03 Thread Tejun Heo
With cmwq, there's no reason for nouveau to use a dedicated workqueue. Drop dev_priv->wq and use system_wq instead. Because nouveau_irq_uninstall() may be called from unsleepable context, the work items can't be flushed from there. Instead, init and flush from nouveau_load/unload(). Signed-off-b

[PATCH 08/32] drm/nouveau: use system_wq instead of dev_priv->wq

2011-01-05 Thread Ben Skeggs
On Mon, 2011-01-03 at 14:49 +0100, Tejun Heo wrote: > With cmwq, there's no reason for nouveau to use a dedicated workqueue. > Drop dev_priv->wq and use system_wq instead. > > Because nouveau_irq_uninstall() may be called from unsleepable > context, the work items can't be flushed from there. Ins

[PATCH 08/32] drm/nouveau: use system_wq instead of dev_priv->wq

2011-01-05 Thread Ben Skeggs
On Wed, 2011-01-05 at 11:07 +1000, Ben Skeggs wrote: > On Mon, 2011-01-03 at 14:49 +0100, Tejun Heo wrote: > > With cmwq, there's no reason for nouveau to use a dedicated workqueue. > > Drop dev_priv->wq and use system_wq instead. > > > > Because nouveau_irq_uninstall() may be called from unsleepa

[PATCH 08/32] drm/nouveau: use system_wq instead of dev_priv->wq

2011-01-06 Thread Tejun Heo
Hello, On Wed, Jan 05, 2011 at 11:16:05AM +1000, Ben Skeggs wrote: > On Wed, 2011-01-05 at 11:07 +1000, Ben Skeggs wrote: > > On Mon, 2011-01-03 at 14:49 +0100, Tejun Heo wrote: > > > With cmwq, there's no reason for nouveau to use a dedicated workqueue. > > > Drop dev_priv->wq and use system_wq i

[PATCH 08/32] drm/nouveau: use system_wq instead of dev_priv->wq

2011-01-03 Thread Tejun Heo
With cmwq, there's no reason for nouveau to use a dedicated workqueue. Drop dev_priv->wq and use system_wq instead. Because nouveau_irq_uninstall() may be called from unsleepable context, the work items can't be flushed from there. Instead, init and flush from nouveau_load/unload(). Signed-off-b

[PATCH 08/32] drm/nouveau: use system_wq instead of dev_priv->wq

2011-01-05 Thread Ben Skeggs
On Mon, 2011-01-03 at 14:49 +0100, Tejun Heo wrote: > With cmwq, there's no reason for nouveau to use a dedicated workqueue. > Drop dev_priv->wq and use system_wq instead. > > Because nouveau_irq_uninstall() may be called from unsleepable > context, the work items can't be flushed from there. Ins

[PATCH 08/32] drm/nouveau: use system_wq instead of dev_priv->wq

2011-01-05 Thread Ben Skeggs
On Wed, 2011-01-05 at 11:07 +1000, Ben Skeggs wrote: > On Mon, 2011-01-03 at 14:49 +0100, Tejun Heo wrote: > > With cmwq, there's no reason for nouveau to use a dedicated workqueue. > > Drop dev_priv->wq and use system_wq instead. > > > > Because nouveau_irq_uninstall() may be called from unsleepa

[PATCH 08/32] drm/nouveau: use system_wq instead of dev_priv->wq

2011-01-06 Thread Tejun Heo
Hello, On Wed, Jan 05, 2011 at 11:16:05AM +1000, Ben Skeggs wrote: > On Wed, 2011-01-05 at 11:07 +1000, Ben Skeggs wrote: > > On Mon, 2011-01-03 at 14:49 +0100, Tejun Heo wrote: > > > With cmwq, there's no reason for nouveau to use a dedicated workqueue. > > > Drop dev_priv->wq and use system_wq i

[PATCH 08/32] drm/nouveau: use system_wq instead of dev_priv->wq

2011-01-03 Thread Tejun Heo
With cmwq, there's no reason for nouveau to use a dedicated workqueue. Drop dev_priv->wq and use system_wq instead. Because nouveau_irq_uninstall() may be called from unsleepable context, the work items can't be flushed from there. Instead, init and flush from nouveau_load/unload(). Signed-off-b

[PATCH 08/32] drm/nouveau: use system_wq instead of dev_priv->wq

2011-01-03 Thread Tejun Heo
With cmwq, there's no reason for nouveau to use a dedicated workqueue. Drop dev_priv->wq and use system_wq instead. Because nouveau_irq_uninstall() may be called from unsleepable context, the work items can't be flushed from there. Instead, init and flush from nouveau_load/unload(). Signed-off-b

[PATCH 08/32] drm/nouveau: use system_wq instead of dev_priv->wq

2011-01-05 Thread Ben Skeggs
On Mon, 2011-01-03 at 14:49 +0100, Tejun Heo wrote: > With cmwq, there's no reason for nouveau to use a dedicated workqueue. > Drop dev_priv->wq and use system_wq instead. > > Because nouveau_irq_uninstall() may be called from unsleepable > context, the work items can't be flushed from there. Ins

[PATCH 08/32] drm/nouveau: use system_wq instead of dev_priv->wq

2011-01-05 Thread Ben Skeggs
On Wed, 2011-01-05 at 11:07 +1000, Ben Skeggs wrote: > On Mon, 2011-01-03 at 14:49 +0100, Tejun Heo wrote: > > With cmwq, there's no reason for nouveau to use a dedicated workqueue. > > Drop dev_priv->wq and use system_wq instead. > > > > Because nouveau_irq_uninstall() may be called from unsleepa

[PATCH 08/32] drm/nouveau: use system_wq instead of dev_priv->wq

2011-01-06 Thread Tejun Heo
Hello, On Wed, Jan 05, 2011 at 11:16:05AM +1000, Ben Skeggs wrote: > On Wed, 2011-01-05 at 11:07 +1000, Ben Skeggs wrote: > > On Mon, 2011-01-03 at 14:49 +0100, Tejun Heo wrote: > > > With cmwq, there's no reason for nouveau to use a dedicated workqueue. > > > Drop dev_priv->wq and use system_wq i

Re: [PATCH 08/32] drm/nouveau: use system_wq instead of dev_priv->wq

2011-01-04 Thread Ben Skeggs
On Mon, 2011-01-03 at 14:49 +0100, Tejun Heo wrote: > With cmwq, there's no reason for nouveau to use a dedicated workqueue. > Drop dev_priv->wq and use system_wq instead. > > Because nouveau_irq_uninstall() may be called from unsleepable > context, the work items can't be flushed from there. Ins

Re: [PATCH 08/32] drm/nouveau: use system_wq instead of dev_priv->wq

2011-01-04 Thread Ben Skeggs
On Wed, 2011-01-05 at 11:07 +1000, Ben Skeggs wrote: > On Mon, 2011-01-03 at 14:49 +0100, Tejun Heo wrote: > > With cmwq, there's no reason for nouveau to use a dedicated workqueue. > > Drop dev_priv->wq and use system_wq instead. > > > > Because nouveau_irq_uninstall() may be called from unsleepa

Re: [PATCH 08/32] drm/nouveau: use system_wq instead of dev_priv->wq

2011-01-06 Thread Tejun Heo
Hello, On Wed, Jan 05, 2011 at 11:16:05AM +1000, Ben Skeggs wrote: > On Wed, 2011-01-05 at 11:07 +1000, Ben Skeggs wrote: > > On Mon, 2011-01-03 at 14:49 +0100, Tejun Heo wrote: > > > With cmwq, there's no reason for nouveau to use a dedicated workqueue. > > > Drop dev_priv->wq and use system_wq i

Re: [PATCH 08/32] drm/nouveau: use system_wq instead of dev_priv->wq

2011-01-04 Thread Ben Skeggs
On Mon, 2011-01-03 at 14:49 +0100, Tejun Heo wrote: > With cmwq, there's no reason for nouveau to use a dedicated workqueue. > Drop dev_priv->wq and use system_wq instead. > > Because nouveau_irq_uninstall() may be called from unsleepable > context, the work items can't be flushed from there. Ins

Re: [PATCH 08/32] drm/nouveau: use system_wq instead of dev_priv->wq

2011-01-04 Thread Ben Skeggs
On Wed, 2011-01-05 at 11:07 +1000, Ben Skeggs wrote: > On Mon, 2011-01-03 at 14:49 +0100, Tejun Heo wrote: > > With cmwq, there's no reason for nouveau to use a dedicated workqueue. > > Drop dev_priv->wq and use system_wq instead. > > > > Because nouveau_irq_uninstall() may be called from unsleepa

Re: [PATCH 08/32] drm/nouveau: use system_wq instead of dev_priv->wq

2011-01-06 Thread Tejun Heo
Hello, On Wed, Jan 05, 2011 at 11:16:05AM +1000, Ben Skeggs wrote: > On Wed, 2011-01-05 at 11:07 +1000, Ben Skeggs wrote: > > On Mon, 2011-01-03 at 14:49 +0100, Tejun Heo wrote: > > > With cmwq, there's no reason for nouveau to use a dedicated workqueue. > > > Drop dev_priv->wq and use system_wq i

Re: [PATCH 08/32] drm/nouveau: use system_wq instead of dev_priv->wq

2011-01-04 Thread Ben Skeggs
On Mon, 2011-01-03 at 14:49 +0100, Tejun Heo wrote: > With cmwq, there's no reason for nouveau to use a dedicated workqueue. > Drop dev_priv->wq and use system_wq instead. > > Because nouveau_irq_uninstall() may be called from unsleepable > context, the work items can't be flushed from there. Ins

Re: [PATCH 08/32] drm/nouveau: use system_wq instead of dev_priv->wq

2011-01-04 Thread Ben Skeggs
On Wed, 2011-01-05 at 11:07 +1000, Ben Skeggs wrote: > On Mon, 2011-01-03 at 14:49 +0100, Tejun Heo wrote: > > With cmwq, there's no reason for nouveau to use a dedicated workqueue. > > Drop dev_priv->wq and use system_wq instead. > > > > Because nouveau_irq_uninstall() may be called from unsleepa

Re: [PATCH 08/32] drm/nouveau: use system_wq instead of dev_priv->wq

2011-01-06 Thread Tejun Heo
Hello, On Wed, Jan 05, 2011 at 11:16:05AM +1000, Ben Skeggs wrote: > On Wed, 2011-01-05 at 11:07 +1000, Ben Skeggs wrote: > > On Mon, 2011-01-03 at 14:49 +0100, Tejun Heo wrote: > > > With cmwq, there's no reason for nouveau to use a dedicated workqueue. > > > Drop dev_priv->wq and use system_wq i

Re: [PATCH 08/32] drm/nouveau: use system_wq instead of dev_priv->wq

2011-01-04 Thread Ben Skeggs
On Mon, 2011-01-03 at 14:49 +0100, Tejun Heo wrote: > With cmwq, there's no reason for nouveau to use a dedicated workqueue. > Drop dev_priv->wq and use system_wq instead. > > Because nouveau_irq_uninstall() may be called from unsleepable > context, the work items can't be flushed from there. Ins

Re: [PATCH 08/32] drm/nouveau: use system_wq instead of dev_priv->wq

2011-01-04 Thread Ben Skeggs
On Wed, 2011-01-05 at 11:07 +1000, Ben Skeggs wrote: > On Mon, 2011-01-03 at 14:49 +0100, Tejun Heo wrote: > > With cmwq, there's no reason for nouveau to use a dedicated workqueue. > > Drop dev_priv->wq and use system_wq instead. > > > > Because nouveau_irq_uninstall() may be called from unsleepa

Re: [PATCH 08/32] drm/nouveau: use system_wq instead of dev_priv->wq

2011-01-06 Thread Tejun Heo
Hello, On Wed, Jan 05, 2011 at 11:16:05AM +1000, Ben Skeggs wrote: > On Wed, 2011-01-05 at 11:07 +1000, Ben Skeggs wrote: > > On Mon, 2011-01-03 at 14:49 +0100, Tejun Heo wrote: > > > With cmwq, there's no reason for nouveau to use a dedicated workqueue. > > > Drop dev_priv->wq and use system_wq i

Re: [PATCH 08/32] drm/nouveau: use system_wq instead of dev_priv->wq

2011-01-04 Thread Ben Skeggs
On Mon, 2011-01-03 at 14:49 +0100, Tejun Heo wrote: > With cmwq, there's no reason for nouveau to use a dedicated workqueue. > Drop dev_priv->wq and use system_wq instead. > > Because nouveau_irq_uninstall() may be called from unsleepable > context, the work items can't be flushed from there. Ins

Re: [PATCH 08/32] drm/nouveau: use system_wq instead of dev_priv->wq

2011-01-04 Thread Ben Skeggs
On Wed, 2011-01-05 at 11:07 +1000, Ben Skeggs wrote: > On Mon, 2011-01-03 at 14:49 +0100, Tejun Heo wrote: > > With cmwq, there's no reason for nouveau to use a dedicated workqueue. > > Drop dev_priv->wq and use system_wq instead. > > > > Because nouveau_irq_uninstall() may be called from unsleepa

Re: [PATCH 08/32] drm/nouveau: use system_wq instead of dev_priv->wq

2011-01-06 Thread Tejun Heo
Hello, On Wed, Jan 05, 2011 at 11:16:05AM +1000, Ben Skeggs wrote: > On Wed, 2011-01-05 at 11:07 +1000, Ben Skeggs wrote: > > On Mon, 2011-01-03 at 14:49 +0100, Tejun Heo wrote: > > > With cmwq, there's no reason for nouveau to use a dedicated workqueue. > > > Drop dev_priv->wq and use system_wq i

Re: [PATCH 08/32] drm/nouveau: use system_wq instead of dev_priv->wq

2011-01-04 Thread Ben Skeggs
On Mon, 2011-01-03 at 14:49 +0100, Tejun Heo wrote: > With cmwq, there's no reason for nouveau to use a dedicated workqueue. > Drop dev_priv->wq and use system_wq instead. > > Because nouveau_irq_uninstall() may be called from unsleepable > context, the work items can't be flushed from there. Ins

Re: [PATCH 08/32] drm/nouveau: use system_wq instead of dev_priv->wq

2011-01-04 Thread Ben Skeggs
On Wed, 2011-01-05 at 11:07 +1000, Ben Skeggs wrote: > On Mon, 2011-01-03 at 14:49 +0100, Tejun Heo wrote: > > With cmwq, there's no reason for nouveau to use a dedicated workqueue. > > Drop dev_priv->wq and use system_wq instead. > > > > Because nouveau_irq_uninstall() may be called from unsleepa

Re: [PATCH 08/32] drm/nouveau: use system_wq instead of dev_priv->wq

2011-01-06 Thread Tejun Heo
Hello, On Wed, Jan 05, 2011 at 11:16:05AM +1000, Ben Skeggs wrote: > On Wed, 2011-01-05 at 11:07 +1000, Ben Skeggs wrote: > > On Mon, 2011-01-03 at 14:49 +0100, Tejun Heo wrote: > > > With cmwq, there's no reason for nouveau to use a dedicated workqueue. > > > Drop dev_priv->wq and use system_wq i