All of lore.kernel.org
 help / color / mirror / Atom feed
From: Shailabh Nagar <nagar@watson.ibm.com>
To: Jay Lan <jlan@sgi.com>
Cc: Balbir Singh <balbir@in.ibm.com>, Andrew Morton <akpm@osdl.org>,
	lkml <linux-kernel@vger.kernel.org>,
	csturtiv@sgi.com, jamal <hadi@cyberus.ca>
Subject: Re: taskstats interface for accounting
Date: Tue, 06 Jun 2006 22:55:44 -0400	[thread overview]
Message-ID: <44864030.6010906@watson.ibm.com> (raw)
In-Reply-To: <44863376.5020701@sgi.com>

Jay Lan wrote:

> Hi Balbir and Shailabh,
>
> I finally have time to think about implementation details of CSA over
> taskstats interface. I took another look at the taskstats interface
> proposal and was a little bit nervous.
>
> Do you remember i suggested to use #ifdef to cut down traffic and i
> was told a generic netlink header would serve the purpose?
> What i see now at Documentation/accounting/taskstats.txt saying
> NETLINK_GENERIC family is used for unicast query/reply mode. The
> NETLINK_GENERIC family provides great flexsibility on what to receive. 
> However, CSA only uses the multicast mode to receive from kernel
> whenever tasks are existing. I guess i would need to read the netlink
> documentation more carefully to see whether my understanding is
> correct.

I don't think there's a problem here. The netlink socket opened for 
listening to
taskstat data sent on task exit can be done in multicast mode....the 
example code in the
Documentation does that.

>
> Another thing i overlooked when i did the review was that taskstats
> interface is designed to provide _BOTH_ per task _AND_ per thread
> accounting data EVERY TIME a task exists. A thread is an aggregate
> of (per-pid) tasks. Since this type of aggregation is not used in
> CSA, half of data traffic would be useless. Can we add a way to
> configure to not send per-thread data to the socket?

I don't see why not. We could extend the command set to set tgid sending 
on/off.

Regards,
Shailabh

> Regards,
> jay



  reply	other threads:[~2006-06-07  2:55 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-06-07  2:01 taskstats interface for accounting Jay Lan
2006-06-07  2:55 ` Shailabh Nagar [this message]
2006-06-07 23:45   ` Jay Lan
2006-06-08 15:45     ` Shailabh Nagar

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=44864030.6010906@watson.ibm.com \
    --to=nagar@watson.ibm.com \
    --cc=akpm@osdl.org \
    --cc=balbir@in.ibm.com \
    --cc=csturtiv@sgi.com \
    --cc=hadi@cyberus.ca \
    --cc=jlan@sgi.com \
    --cc=linux-kernel@vger.kernel.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.