* Re: task_diag: add a new interface to get information about processes
@ 2016-05-04 20:22 Stephen Hemminger
2016-05-04 22:34 ` Andrey Vagin
0 siblings, 1 reply; 2+ messages in thread
From: Stephen Hemminger @ 2016-05-04 20:22 UTC (permalink / raw)
To: Andrey Vagin; +Cc: linux-kernel, David Miller
I understand how reading /proc or /sys can be a bottleneck, but this
proposed method using a system call is the wrong way to do this.
Why not use netlink like other systems do which allows a message
based response which allows for future changes (no fixed datastructures),
and is message based.
Generic netlink has already been used by several other subsystems.
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: task_diag: add a new interface to get information about processes
2016-05-04 20:22 task_diag: add a new interface to get information about processes Stephen Hemminger
@ 2016-05-04 22:34 ` Andrey Vagin
0 siblings, 0 replies; 2+ messages in thread
From: Andrey Vagin @ 2016-05-04 22:34 UTC (permalink / raw)
To: Stephen Hemminger, Andy Lutomirski; +Cc: LKML, David Miller
Hi Stephen,
On Wed, May 4, 2016 at 1:22 PM, Stephen Hemminger
<stephen@networkplumber.org> wrote:
> I understand how reading /proc or /sys can be a bottleneck, but this
> proposed method using a system call is the wrong way to do this.
>
> Why not use netlink like other systems do which allows a message
> based response which allows for future changes (no fixed datastructures),
> and is message based.
>
> Generic netlink has already been used by several other subsystems.
I used netlink in two first versions of task_diag, but then Andy
convinced me that netlink interfaces are not ideal for this case. I
added him into Cс.
Here is a thread with our discussion about using netlink for
task_diag: https://lkml.org/lkml/2015/12/15/520
Can I ask you to read it and give your comments? It would be nice to
find a way how to use netlink sockets instead of creating a new
interface.
Thanks,
Andrew
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2016-05-04 22:34 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-05-04 20:22 task_diag: add a new interface to get information about processes Stephen Hemminger
2016-05-04 22:34 ` Andrey Vagin
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox