On Thursday 09 April 2015 19:13:55 Rafael J. Wysocki wrote: > On Thursday, April 09, 2015 08:36:57 AM Pali Rohár wrote: > > --nextPart2566388.gOmNIJrIqI > > Content-Type: Text/Plain; > > > > charset="utf-8" > > > > Content-Transfer-Encoding: quoted-printable > > > > On Thursday 09 April 2015 02:28:41 Rafael J. Wysocki wrote: > > > On Sunday, April 05, 2015 07:20:17 PM Pali Roh=C3=A1r > > > wrote: > > > > To prevent race conditions on userspace processes with > > > > I/O some taks must be called after processes are > > > > freezed. This patch adds new events which are delivered > > > > by > > > > pm_notifier_call_chain() after freezing processes when > > > > doing suspend or hibernate action. > > > > > > > >=20 > > > > > > > > Signed-off-by: Pali Roh=C3=A1r <pali.rohar@xxxxxxxxx> > > > > > >=20 > > > > > > Please don't add more notifiers. Just call whatever you > > > need directly from where you need to call that. > > > > > >=20 > > > > > > If that is device-related, try to use device PM > > > suspend/hibernate callbacks instead. > > > > > >=20 > > > > Hi! It is not possible to use any exiting pm notifiers! This > > is=20 reason why I added new ones. As I wrote wiping dm > > crypt keys must=20 be done *after* userspace processes are > > freezed to prevent race=20 conditions... > > I'm not talking about using the existing notifiers. I'm > talking about calling the function you need to call directly > from a suitable place in the system suspend code. I need to wipe crypto keys from dm-crypt module. That module can be compiled as external .ko file and so kernel cannot call directly needed function. This is reason why I'm adding new notifier event. -- Pali Rohár pali.rohar@xxxxxxxxx
Attachment:
signature.asc
Description: This is a digitally signed message part.
-- dm-devel mailing list dm-devel@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/dm-devel
