Hi Neil,

On Wed, Jul 22, 2015 at 12:08 AM, NeilBrown [off-list ref] wrote:

On Mon, 20 Jul 2015 23:46:32 +0200 "Rafael J. Wysocki"
[off-list ref] wrote:

quoted

On Monday, July 20, 2015 09:32:26 AM Pali Rohár wrote:

quoted

On Saturday 18 July 2015 01:27:15 Rafael J. Wysocki wrote:

quoted

On Thursday, July 16, 2015 09:33:02 AM Pali Rohár wrote:

quoted

On Thursday 16 July 2015 03:02:03 Rafael J. Wysocki wrote:

quoted

Also, if you're adding AFTER_FREEZE, it would be good to add BEFORE_THAW too
for symmetry.

But there is no use case for BEFORE_THAW. At least it is not needed for now.

For your use case, a single function pointer would be sufficient too.

What do you mean by single function pointer? kernel/power is part of
kernel image and dm-crypt is external kernel module.

Well, if there is a function pointer in the core suspend code initially set to
NULL and exported to modules such that the dm-crypt code can set it to
something else, that should be sufficient, shouldn't it?

As long as the dm-crypt module is never unloaded.

OK, there is a race related to that.

And as long as no other module could very possible want functionality like this. Ever.

The point was that there were no other users currently, so dm-crypt is
going to be the only user for the time being.

If a module wants to be notified - the providing a notifier chain
really seems like the right thing to do...

Well, so please see my last response in this thread. :-)

Thanks,
Rafael