![]() | |||||||||||
Messages in this thread |
jamal wrote: >On Mon, 2006-13-03 at 18:33 -0800, Matt Helsley wrote: > > >>On Mon, 2006-03-13 at 19:56 -0500, Shailabh Nagar wrote: >> >> > > > >>Jamal, was your Mon, 13 Mar 2006 21:29:09 -0500 reply: >> >> >>>Note, you are still not following the standard scheme of doing things. >>>Example: using command = GET and the message carrying the TGID to note >>>which TGID is of interest. Instead you have command = TGID. >>> >>> >>> > > > >>meant to suggest that TASKSTATS_CMD_(P|TG)ID should be renamed to >>TASKSTATS_CMD_GET_(P|TG)ID ? Is that sufficient? Or am I >>misunderstanding? >> >> >> > >I had a long description in an earlier email feedback; but the summary >of it is the GET command is generic like TASKSTATS_CMD_GET; the message >itself carries TLVs of what needs to be gotten which are >either PID and/or TGID etc. Anyways, theres a long spill of what i am >saying in that earlier email. Perhaps the current patch is a transition >towards that? > > Yes, the comments you'd made in the previous mail have not been incorporated and this is still the older version of the patch. We'd been avoiding TLV usage so far :-) >cheers, >jamal > > > - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/ | ||||||||||
| Last update: 2006-03-14 05:20 [from the cache] ©2003-2008 | |||||||||||