more efficient current_is_keventd macro? [was Re: [lhcs-devel] Re: Kthread_create() never returns when called from worker_thread]

[Srivatsa Vaddagiri]

Hi Ingo,
	current_is_keventd() macro checks "current" with the per-cpu
thread of _all_ possible cpus attached to keventd_wq. Can't it just check 
against the per-cpu thread of _current_ cpu alone (since the per-cpu workqueue
threads are anyway bound only to their cpus)?  

int current_is_keventd(void)
{
        struct cpu_workqueue_struct *cwq;
-       int cpu;
+       int cpu = smp_processor_id();

        BUG_ON(!keventd_wq);

-       for_each_cpu(cpu) {
-               cwq = keventd_wq->cpu_wq + cpu;
-               if (current == cwq->thread)
-                       return 1;
-       }

-       return 0;

+	cwq = keventd_wq->cpu_wq + cpu;
+	if (current == cwq->thread)
+		return 1;
+	else
+		return 0;
}


----- Forwarded message from Rusty Russell <rusty@xxxxxxxxxxxxxxx> -----

Subject: Re: [lhcs-devel] Re: Kthread_create() never returns when called
	from worker_thread
From: Rusty Russell <rusty@xxxxxxxxxxxxxxx>
To: vatsa@xxxxxxxxxx
Cc: rusty@xxxxxxxxxxx,
	"Keshavamurthy, Anil S" <anil.s.keshavamurthy@xxxxxxxxx>,
	lhcs-devel@xxxxxxxxxxxxxxxxxxxxx, "Raj, Ashok" <ashok.raj@xxxxxxxxx>,
	"Shah, Rajesh" <rajesh.shah@xxxxxxxxx>
X-Mailer: Ximian Evolution 1.4.5 
Date: Mon, 08 Mar 2004 11:29:27 +1100

On Wed, 2004-03-03 at 20:08, Srivatsa Vaddagiri wrote: 
> On Tue, Mar 02, 2004 at 12:10:08PM +1100, Rusty Russell wrote:
> >  	/* If we're being called to start the first workqueue, we
> >  	 * can't use keventd. */
> > -	if (!keventd_up())
> > +	if (!keventd_up() || current_is_keventd())
> >  		work.func(work.data);
> >  	else {
> >  		schedule_work(&work);
> 
> I noticed that current_is_keventd() check actually goes and compares
> "current" with all the per-cpu threads attached to keventd() workqueue ..
> Is that really necessary? Can't it compare with the per-cpu thread
> attached to _current_ cpu only? 

You'd think so, but it's not my code.  I believe you are correct though.

Cheers,
Rusty.
-- 
Anyone who quotes me in their signature is an idiot -- Rusty Russell



-------------------------------------------------------
This SF.Net email is sponsored by: IBM Linux Tutorials
Free Linux tutorial presented by Daniel Robbins, President and CEO of
GenToo technologies. Learn everything from fundamentals to system
administration.http://ads.osdn.com/?ad_id=1470&alloc_id=3638&op=click
_______________________________________________
lhcs-devel mailing list
lhcs-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.sourceforge.net/lists/listinfo/lhcs-devel

----- End forwarded message -----

-- 


Thanks and Regards,
Srivatsa Vaddagiri,
Linux Technology Center,
IBM Software Labs,
Bangalore, INDIA - 560017
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/