public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* 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