linux-kernel - Re: [PATCH 04/16] freezer: implement and use kthread_freezable_should

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:	Fri, 19 Aug 2011 17:07:12 -0300
From:	Henrique de Moraes Holschuh <hmh@....eng.br>
To:	Tejun Heo <tj@...nel.org>
Cc:	rjw@...k.pl, menage@...gle.com, linux-kernel@...r.kernel.org,
	arnd@...db.de, oleg@...hat.com, Jens Axboe <axboe@...nel.dk>,
	Henrique de Moraes Holschuh <ibm-acpi@....eng.br>
Subject: Re: [PATCH 04/16] freezer: implement and use
 kthread_freezable_should_stop()

On Fri, 19 Aug 2011, Tejun Heo wrote:
> Writeback and thinkpad_acpi have been using thaw_process() to prevent
> deadlock between the freezer and kthread_stop(); unfortunately, this
> is inherently racy - nothing prevents freezing from happening between
> thaw_process() and kthread_stop().
> 
> This patch implements kthread_freezable_should_stop() which enters
> refrigerator if necessary but is guaranteed to return if
> kthread_stop() is invoked.  Both thaw_process() users are converted to
> use the new function.
> 
> Note that this deadlock condition exists for many of freezable
> kthreads.  They need to be converted to use the new should_stop or
> freezable workqueue.
> 
> Tested with synthetic test case.
> 
> Signed-off-by: Tejun Heo <tj@...nel.org>
> Cc: Jens Axboe <axboe@...nel.dk>
> Cc: Henrique de Moraes Holschuh <ibm-acpi@....eng.br>

For the thinkpad-acpi bits:
Acked-by: Henrique de Moraes Holschuh <hmh@....eng.br>

-- 
  "One disk to rule them all, One disk to find them. One disk to bring
  them all and in the darkness grind them. In the Land of Redmond
  where the shadows lie." -- The Silicon Valley Tarot
  Henrique Holschuh
--
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/