public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Jay Lan <jlan@sgi.com>
To: balbir@in.ibm.com
Cc: Shailabh Nagar <nagar@watson.ibm.com>,
	LKML <linux-kernel@vger.kernel.org>,
	Andrew Morton <akpm@osdl.org>,
	Chris Sturtivant <csturtiv@sgi.com>
Subject: Re: ON/OFF control of taskstats accounting data at do_exit
Date: Thu, 15 Jun 2006 11:57:05 -0700	[thread overview]
Message-ID: <4491AD81.4040700@sgi.com> (raw)
In-Reply-To: <4491A398.4050204@in.ibm.com>

Balbir Singh wrote:
> 
>> For that reason, i think exposing the switch at sysfs is not a good
>> idea. Instead, /etc/init.d/taskstats script would be right for
>> this purpose. At kernel side, we would need to make this possible.
> 
> 
> When you say we need to make this possible at the kernel side, do you
> want the kernel not to accumulate taskstats in the do_exit() path
> (avoid calling fill_xxxx routines) if there are no listeners
> on TASKSTATS_LISTEN_GROUP? As Shailabh, mentioned sending out does not
> happen if there are no listeners.

I did not understand that when i made the request because i did not
see from taskstats code how you can control that. Shailabh later said
that it was done at netlink layer.

I do not know how deep down in netlink layer to exit the proces,
but it may turn out that the need of this feature not that importan...

Yet, we still allocate memory, fill taskstats struct, invoke netlink,
and free memory. Ideally when taskstats is turned off, we do not need
to do any of these. So, if we can fail memory allocation when taskstats
is turned off, the taskstats_exit_send() will return right away.

To answer Shailabh's discussion in a separate email, yes, i meant
the second case that all taskstats applications will be stopped or
suspended. The different between a /etc/init.d/taskstats and a
sysfs/procfs control of taskstats is that the system will not
be activated at init time. However, i admit customers can always turn
off taskstats via sysfs before starts whatever they like to do.

I do not know what is community's preference: a /etc/init.d mechnism
or a sysfs mechnism. I would be happy either way. Thanks!

Regards,
  jay


> 
>>
>> What do you think?
>>
>> Regards,
>>  - jay
>>


  reply	other threads:[~2006-06-15 18:57 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-06-14 22:55 ON/OFF control of taskstats accounting data at do_exit Jay Lan
2006-06-15  3:01 ` Balbir Singh
2006-06-15  3:02 ` Shailabh Nagar
2006-06-15  3:33   ` Jay Lan
2006-06-15 14:52     ` Shailabh Nagar
2006-06-15 15:55     ` Shailabh Nagar
2006-06-15 17:24       ` Jay Lan
2006-06-15 18:14         ` Balbir Singh
2006-06-15 18:57           ` Jay Lan [this message]
2006-06-15 18:28         ` Shailabh Nagar
2006-06-15 23:41       ` Peter Williams

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4491AD81.4040700@sgi.com \
    --to=jlan@sgi.com \
    --cc=akpm@osdl.org \
    --cc=balbir@in.ibm.com \
    --cc=csturtiv@sgi.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=nagar@watson.ibm.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox