linux-kernel - Re: [PATCH] video: don't use flush_scheduled_work() in fb

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]

Date:	Thu, 16 Jun 2011 15:56:18 +0900
From:	Paul Mundt <lethal@...ux-sh.org>
To:	Tejun Heo <tj@...nel.org>
Cc:	Jaya Kumar <jayalk@...works.biz>,
	Andrew Morton <akpm@...ux-foundation.org>,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH] video: don't use flush_scheduled_work() in fb_defio

On Wed, Jun 15, 2011 at 04:57:21PM +0200, Tejun Heo wrote:
> flush_scheduled_work() in on the way out.  fb_deferred_io_cleanup()
> can simply sync-cancel info->deferred_work instead of cancel +
> flush_scheduled_work().  Drop flush_scheduled_work() usage.
> 
> Signed-off-by: Tejun Heo <tj@...nel.org>
> Cc: Jaya Kumar <jayalk@...works.biz>
> ---
> Compile tested.  I'm planning on marking flush_scheduled_work()
> deprecated in linux-next so it would be great if this can show up
> there soonish.  Thank you.
> 
>  drivers/video/fb_defio.c |    3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
I've applied it to my fbdev tree, so it should show up in -next as of
tomorrow.

In the future, please also Cc linux-fbdev@...r.kernel.org on these
things, as they're likely to be overlooked otherwise.
--
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/