public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* Fix missing refrigerator invocation in jffs2.
@ 2005-07-18  4:14 Nigel Cunningham
  2005-07-18 18:18 ` Jörn Engel
  0 siblings, 1 reply; 3+ messages in thread
From: Nigel Cunningham @ 2005-07-18  4:14 UTC (permalink / raw)
  To: Andrew Morton, jffs-dev, Linux Kernel Mailing List

Hi.

Here's a patch to fix a missing refrigerator call in jffs2.

Regards,

Nigel

Signed-off by: Nigel Cunningham <nigel@suspend2.net>

 intrep.c |    3 +++
 1 files changed, 3 insertions(+)
diff -ruNp 235-jffs-intrep.patch-old/fs/jffs/intrep.c 235-jffs-intrep.patch-new/fs/jffs/intrep.c
--- 235-jffs-intrep.patch-old/fs/jffs/intrep.c	2005-07-18 06:36:59.000000000 +1000
+++ 235-jffs-intrep.patch-new/fs/jffs/intrep.c	2005-07-18 14:02:27.000000000 +1000
@@ -3397,6 +3397,9 @@ jffs_garbage_collect_thread(void *ptr)
 			siginfo_t info;
 			unsigned long signr = 0;
 
+			if (try_to_freeze())
+				continue;
+
 			spin_lock_irq(&current->sighand->siglock);
 			signr = dequeue_signal(current, &current->blocked, &info);
 			spin_unlock_irq(&current->sighand->siglock);

-- 
Evolution.
Enumerate the requirements.
Consider the interdependencies.
Calculate the probabilities.
Be amazed that people believe it happened. 


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Fix missing refrigerator invocation in jffs2.
  2005-07-18  4:14 Fix missing refrigerator invocation in jffs2 Nigel Cunningham
@ 2005-07-18 18:18 ` Jörn Engel
  2005-07-19 12:13   ` Nigel Cunningham
  0 siblings, 1 reply; 3+ messages in thread
From: Jörn Engel @ 2005-07-18 18:18 UTC (permalink / raw)
  To: Nigel Cunningham; +Cc: Andrew Morton, jffs-dev, Linux Kernel Mailing List

On Mon, 18 July 2005 14:14:53 +1000, Nigel Cunningham wrote:
> 
> Here's a patch to fix a missing refrigerator call in jffs2.
                                                           ^
You should shorten the description by one letter, roughly. ;)

> 
> Signed-off by: Nigel Cunningham <nigel@suspend2.net>
> 
>  intrep.c |    3 +++
>  1 files changed, 3 insertions(+)
> diff -ruNp 235-jffs-intrep.patch-old/fs/jffs/intrep.c 235-jffs-intrep.patch-new/fs/jffs/intrep.c
> --- 235-jffs-intrep.patch-old/fs/jffs/intrep.c	2005-07-18 06:36:59.000000000 +1000
> +++ 235-jffs-intrep.patch-new/fs/jffs/intrep.c	2005-07-18 14:02:27.000000000 +1000
> @@ -3397,6 +3397,9 @@ jffs_garbage_collect_thread(void *ptr)
>  			siginfo_t info;
>  			unsigned long signr = 0;
>  
> +			if (try_to_freeze())
> +				continue;
> +
>  			spin_lock_irq(&current->sighand->siglock);
>  			signr = dequeue_signal(current, &current->blocked, &info);
>  			spin_unlock_irq(&current->sighand->siglock);
> 
> -- 
> Evolution.
> Enumerate the requirements.
> Consider the interdependencies.
> Calculate the probabilities.
> Be amazed that people believe it happened. 
> 
> -
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/

Jörn

-- 
A quarrel is quickly settled when deserted by one party; there is
no battle unless there be two.
-- Seneca

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Fix missing refrigerator invocation in jffs2.
  2005-07-18 18:18 ` Jörn Engel
@ 2005-07-19 12:13   ` Nigel Cunningham
  0 siblings, 0 replies; 3+ messages in thread
From: Nigel Cunningham @ 2005-07-19 12:13 UTC (permalink / raw)
  To: Jörn Engel; +Cc: Andrew Morton, jffs-dev, Linux Kernel Mailing List

Hi.

On Tue, 2005-07-19 at 04:18, Jörn Engel wrote:
> On Mon, 18 July 2005 14:14:53 +1000, Nigel Cunningham wrote:
> > 
> > Here's a patch to fix a missing refrigerator call in jffs2.
>                                                            ^
> You should shorten the description by one letter, roughly. ;)

Oh oops! Sorry!

Nigel

> > 
> > Signed-off by: Nigel Cunningham <nigel@suspend2.net>
> > 
> >  intrep.c |    3 +++
> >  1 files changed, 3 insertions(+)
> > diff -ruNp 235-jffs-intrep.patch-old/fs/jffs/intrep.c 235-jffs-intrep.patch-new/fs/jffs/intrep.c
> > --- 235-jffs-intrep.patch-old/fs/jffs/intrep.c	2005-07-18 06:36:59.000000000 +1000
> > +++ 235-jffs-intrep.patch-new/fs/jffs/intrep.c	2005-07-18 14:02:27.000000000 +1000
> > @@ -3397,6 +3397,9 @@ jffs_garbage_collect_thread(void *ptr)
> >  			siginfo_t info;
> >  			unsigned long signr = 0;
> >  
> > +			if (try_to_freeze())
> > +				continue;
> > +
> >  			spin_lock_irq(&current->sighand->siglock);
> >  			signr = dequeue_signal(current, &current->blocked, &info);
> >  			spin_unlock_irq(&current->sighand->siglock);
> > 
> > -- 
> > Evolution.
> > Enumerate the requirements.
> > Consider the interdependencies.
> > Calculate the probabilities.
> > Be amazed that people believe it happened. 
> > 
> > -
> > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> > the body of a message to majordomo@vger.kernel.org
> > More majordomo info at  http://vger.kernel.org/majordomo-info.html
> > Please read the FAQ at  http://www.tux.org/lkml/
> 
> Jörn
-- 
Evolution.
Enumerate the requirements.
Consider the interdependencies.
Calculate the probabilities.


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2005-07-19 12:11 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2005-07-18  4:14 Fix missing refrigerator invocation in jffs2 Nigel Cunningham
2005-07-18 18:18 ` Jörn Engel
2005-07-19 12:13   ` Nigel Cunningham

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox