linux-kernel - Re: [PATCH UPDATED 08/32] drm/nouveau: use system_wq instead of dev

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <1296784411.7970.1.camel@nisroch>
Date:	Fri, 04 Feb 2011 11:53:25 +1000
From:	Ben Skeggs <skeggsb@...il.com>
To:	Tejun Heo <tj@...nel.org>
Cc:	linux-kernel@...r.kernel.org, dri-devel@...ts.freedesktop.org
Subject: Re: [PATCH UPDATED 08/32] drm/nouveau: use system_wq instead of
 dev_priv->wq

On Tue, 2011-02-01 at 11:41 +0100, Tejun Heo wrote:
> On Wed, Jan 26, 2011 at 05:49:18PM +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.  Each work item is sync
> > flushed when the containing structure is unregistered/destroyed.
> > 
> > Note that this change also makes sure that nv50_gpio_handler is not
> > freed while the contained work item is still running.
> > 
> > Signed-off-by: Tejun Heo <tj@...nel.org>
> > Cc: David Airlie <airlied@...ux.ie>
> > Cc: dri-devel@...ts.freedesktop.org
> 
> Ping.  Can you please put this through the drm tree?
Hey Tejun,

Thanks for this again.  I've just made some changes to the nv50 display
bottom half handling that these will conflict with, they'll hit the
nouveau tree next week.

I'll push the still-relevant bits of your nouveau patch through the
nouveau tree for the 2.6.39-rc1 merge window.  That ok?

Thanks,
Ben.

> 
> Thank you.
> 


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/