* [PATCH] omapdrm: use alloc_ordered_workqueue() instead of UNBOUND w/ max_active = 1
@ 2012-08-22 23:49 Tejun Heo
2012-08-23 20:08 ` Clark, Rob
0 siblings, 1 reply; 3+ messages in thread
From: Tejun Heo @ 2012-08-22 23:49 UTC (permalink / raw)
To: Rob Clark, Tony Lindgren; +Cc: linux-omap, linux-kernel
This is an equivalent conversion and will ease scheduled removal of
WQ_NON_REENTRANT.
Only compile tested.
Signed-off-by: Tejun Heo <tj@kernel.org>
---
drivers/staging/omapdrm/omap_drv.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/staging/omapdrm/omap_drv.c b/drivers/staging/omapdrm/omap_drv.c
index 4beab94..672cb3a 100644
--- a/drivers/staging/omapdrm/omap_drv.c
+++ b/drivers/staging/omapdrm/omap_drv.c
@@ -571,8 +571,7 @@ static int dev_load(struct drm_device *dev, unsigned long flags)
dev->dev_private = priv;
- priv->wq = alloc_workqueue("omapdrm",
- WQ_UNBOUND | WQ_NON_REENTRANT, 1);
+ priv->wq = alloc_ordered_workqueue("omapdrm", 0);
INIT_LIST_HEAD(&priv->obj_list);
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] omapdrm: use alloc_ordered_workqueue() instead of UNBOUND w/ max_active = 1
2012-08-22 23:49 [PATCH] omapdrm: use alloc_ordered_workqueue() instead of UNBOUND w/ max_active = 1 Tejun Heo
@ 2012-08-23 20:08 ` Clark, Rob
2012-08-23 20:16 ` Tejun Heo
0 siblings, 1 reply; 3+ messages in thread
From: Clark, Rob @ 2012-08-23 20:08 UTC (permalink / raw)
To: Tejun Heo; +Cc: Tony Lindgren, linux-omap, linux-kernel, gregkh
On Wed, Aug 22, 2012 at 6:49 PM, Tejun Heo <tj@kernel.org> wrote:
> This is an equivalent conversion and will ease scheduled removal of
> WQ_NON_REENTRANT.
>
> Only compile tested.
>
> Signed-off-by: Tejun Heo <tj@kernel.org>
I've tested it
Signed-off-by: Rob Clark <rob@ti.com>
> ---
> drivers/staging/omapdrm/omap_drv.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/drivers/staging/omapdrm/omap_drv.c b/drivers/staging/omapdrm/omap_drv.c
> index 4beab94..672cb3a 100644
> --- a/drivers/staging/omapdrm/omap_drv.c
> +++ b/drivers/staging/omapdrm/omap_drv.c
> @@ -571,8 +571,7 @@ static int dev_load(struct drm_device *dev, unsigned long flags)
>
> dev->dev_private = priv;
>
> - priv->wq = alloc_workqueue("omapdrm",
> - WQ_UNBOUND | WQ_NON_REENTRANT, 1);
> + priv->wq = alloc_ordered_workqueue("omapdrm", 0);
>
> INIT_LIST_HEAD(&priv->obj_list);
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] omapdrm: use alloc_ordered_workqueue() instead of UNBOUND w/ max_active = 1
2012-08-23 20:08 ` Clark, Rob
@ 2012-08-23 20:16 ` Tejun Heo
0 siblings, 0 replies; 3+ messages in thread
From: Tejun Heo @ 2012-08-23 20:16 UTC (permalink / raw)
To: Clark, Rob; +Cc: Tony Lindgren, linux-omap, linux-kernel, gregkh
On Thu, Aug 23, 2012 at 03:08:59PM -0500, Clark, Rob wrote:
> On Wed, Aug 22, 2012 at 6:49 PM, Tejun Heo <tj@kernel.org> wrote:
> > This is an equivalent conversion and will ease scheduled removal of
> > WQ_NON_REENTRANT.
> >
> > Only compile tested.
> >
> > Signed-off-by: Tejun Heo <tj@kernel.org>
>
> I've tested it
>
> Signed-off-by: Rob Clark <rob@ti.com>
Can you please route this with other omap patches?
Thanks!
--
tejun
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2012-08-23 20:16 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-08-22 23:49 [PATCH] omapdrm: use alloc_ordered_workqueue() instead of UNBOUND w/ max_active = 1 Tejun Heo
2012-08-23 20:08 ` Clark, Rob
2012-08-23 20:16 ` Tejun Heo
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).