[Kernel-janitors] Re: [PATCH] md: replace schedule_timeout() with

From: Nishanth Aravamudan <nacc@us.ibm.com>
To: kernel-janitors@vger.kernel.org
Subject: [Kernel-janitors] Re: [PATCH] md: replace schedule_timeout() with
Date: Mon, 26 Jul 2004 17:31:06 +0000	[thread overview]
Message-ID: <20040726173106.GC1897@us.ibm.com> (raw)
In-Reply-To: <Pine.LNX.4.58.0407240731440.16034@devserv.devel.redhat.com>

[-- Attachment #1: Type: text/plain, Size: 971 bytes --]

On Sat, Jul 24, 2004 at 07:33:31AM -0400, Ingo Molnar wrote:
> 
> On Fri, 23 Jul 2004, Nishanth Aravamudan wrote:
> 
> > --- linux-vanilla/drivers/md/md.c	2004-06-16 05:19:03.000000000 +0000
> > +++ linux-dev/drivers/md/md.c	2004-07-12 18:47:53.000000000 +0000
> > @@ -3438,8 +3438,7 @@ static void md_do_sync(mddev_t *mddev)
> >  		if (currspeed > sysctl_speed_limit_min) {
> >  			if ((currspeed > sysctl_speed_limit_max) ||
> >  					!is_mddev_idle(mddev)) {
> > -				current->state = TASK_INTERRUPTIBLE;
> > -				schedule_timeout(HZ/4);
> > +				msleep(250);
> 
> well, it's not 100% equivalent because msleep() is uninterruptible so
> stoppage of the md thread (upon shutdown) will occur with only a 250 msec
> delay. Someone should add a msleep_interruptible() function to
> kernel/timer.c.

You are exactly right. That's what I tried to indicated in my posts.
There isn't any msleep_interruptible() unfortunately, so the code can just
be left as is.

Thanks,
Nish

[-- Attachment #2: Type: text/plain, Size: 167 bytes --]

_______________________________________________
Kernel-janitors mailing list
Kernel-janitors@lists.osdl.org
http://lists.osdl.org/mailman/listinfo/kernel-janitors