Re: [rfc][patch] Avoid taking global tasklist_lock for single threaded process at getrusage()

On Wed, Dec 21, 2005 at 01:22:24PM -0800, Christoph Lameter wrote:
> On Wed, 21 Dec 2005, Ravikiran G Thirumalai wrote:
> 
> > We did look at that. Cases RUSAGE_CHILDREN and RUSAGE_SELF are always called by the 
> > current task, so we can avoid tasklist locking there.
> > getrusage for non-current tasks are always called with RUSAGE_BOTH.
> > We ensure we  always take the siglock for RUSAGE_BOTH case, so that the
> > p->signal* fields are protected and take the tasklist_lock only if we have 
> > to traverse the tasklist hashlist. Isn't this safe?
> 
> Sounds okay. But its complex in the way its is coded now and its easy to 
> assume that one can call getrusage with any parameter from inside the 
> kernel. Maybe we can have a couple of separate functions 
> 
> rusage_children()
> rusage_self()
> rusage_both()
> 
> ?
> 
> Only rusage_both would take a task_struct * parameter. The others would 
> only operate on current. Change all the locations that call getrusage with 
> RUSAGE_BOTH to call rusage_both().

Yes.  This would indeed be better. I will do that change.

Thanks,
Kiran
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

---

• References:
  • [rfc][patch] Avoid taking global tasklist_lock for single threaded process at getrusage()
    • From: Ravikiran G Thirumalai <[email protected]>
  • Re: [rfc][patch] Avoid taking global tasklist_lock for single threaded process at getrusage()
    • From: Christoph Lameter <[email protected]>
  • Re: [rfc][patch] Avoid taking global tasklist_lock for single threaded process at getrusage()
    • From: Ravikiran G Thirumalai <[email protected]>
  • Re: [rfc][patch] Avoid taking global tasklist_lock for single threaded process at getrusage()
    • From: Christoph Lameter <[email protected]>

• Prev by Date: Re: 2.6.15-rc5 and later: USB mouse IRQ post kills the computer post resume.
• Next by Date: Re: About 4k kernel stack size....
• Previous by thread: Re: [rfc][patch] Avoid taking global tasklist_lock for single threaded process at getrusage()
• Next by thread: Re: [rfc][patch] Avoid taking global tasklist_lock for single threaded process at getrusage()
• Index(es):
  • Date
  • Thread

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]