[PATCH] [MMC] Add try_to_freeze to kauditd

[PATCH] [MMC] Add try_to_freeze to kauditd

[Pierre Ossman]

kauditd was causing suspends to fail because it refused to freeze.
Adding a try_to_freeze() to its sleep loop solves the issue.

Signed-off-by: Pierre Ossman <drzeus@drzeus.cx>
Acked-by: Pavel Machek <pavel@suse.cz>
---

 kernel/audit.c |    4 +++-
 1 files changed, 3 insertions(+), 1 deletions(-)

diff --git a/kernel/audit.c b/kernel/audit.c
--- a/kernel/audit.c
+++ b/kernel/audit.c
@@ -291,8 +291,10 @@ int kauditd_thread(void *dummy)
 			set_current_state(TASK_INTERRUPTIBLE);
 			add_wait_queue(&kauditd_wait, &wait);
 
-			if (!skb_queue_len(&audit_skb_queue))
+			if (!skb_queue_len(&audit_skb_queue)) {
+				try_to_freeze();
 				schedule();
+			}
 
 			__set_current_state(TASK_RUNNING);
 			remove_wait_queue(&kauditd_wait, &wait);

Re: [PATCH] [MMC] Add try_to_freeze to kauditd

[Pierre Ossman]

Pierre Ossman wrote:
> kauditd was causing suspends to fail because it refused to freeze.
> Adding a try_to_freeze() to its sleep loop solves the issue.
>
> Signed-off-by: Pierre Ossman <drzeus@drzeus.cx>
> Acked-by: Pavel Machek <pavel@suse.cz>
> ---
>   


This has of course nothing to do with [MMC]. Andrew, could you strip 
this of the patch before you add it to your set?

Rgds
Pierre

Re: [PATCH] [MMC] Add try_to_freeze to kauditd

[Pierre Ossman]

Pierre Ossman wrote:
> kauditd was causing suspends to fail because it refused to freeze.
> Adding a try_to_freeze() to its sleep loop solves the issue.
>
> Signed-off-by: Pierre Ossman <drzeus@drzeus.cx>
> Acked-by: Pavel Machek <pavel@suse.cz>
> ---
>
>  kernel/audit.c |    4 +++-
>  1 files changed, 3 insertions(+), 1 deletions(-)
>
>   

Did anyone actually pick this up? Its not in -mm or Linus' tree.

Rgds
Pierre

Re: [PATCH] [MMC] Add try_to_freeze to kauditd

[Pavel Machek]

Hi!

> >kauditd was causing suspends to fail because it refused to freeze.
> >Adding a try_to_freeze() to its sleep loop solves the issue.
> >
> >Signed-off-by: Pierre Ossman <drzeus@drzeus.cx>
> >Acked-by: Pavel Machek <pavel@suse.cz>
> >---
> >
> > kernel/audit.c |    4 +++-
> > 1 files changed, 3 insertions(+), 1 deletions(-)
> >
> >  
> 
> Did anyone actually pick this up? Its not in -mm or Linus' tree.

Not me... I do not think it needs to go through my trees, should be
simple enough to go to -mm directly.
								Pavel
-- 
Thanks, Sharp!