[systemd-devel] [PATCH] journald: collect process data from socket data

Piotr Wilczek p.wilczek at samsung.com
Thu Jul 3 04:37:26 PDT 2014



> -----Original Message-----
> From: Lennart Poettering [mailto:lennart at poettering.net]
> Sent: Thursday, July 03, 2014 1:13 PM
> To: Piotr Wilczek
> Cc: systemd-devel at lists.freedesktop.org; Kyungmin Park; Juho Son;
> Bartlomiej Zolnierkiewicz
> Subject: Re: [systemd-devel] [PATCH] journald: collect process data
> from socket data
> 
> On Mon, 02.06.14 12:56, Piotr Wilczek (p.wilczek at samsung.com) wrote:
> 
> >
> > On 06/01/2014 07:52 AM, Lennart Poettering wrote:
> > >On Fri, 30.05.14 12:45, Piotr Wilczek (p.wilczek at samsung.com) wrote:
> > >
> > >>This patch adds posibility for journal to get process data from
> > >>socket data (if available) instead of from procfs.
> > >>
> > >>Additionally a new procinfo structure is added to store all process
> > >>data in single place. This data include creditentials and status.
> > >>
> > >>To enable this functionality, SO_PASSPROC socket option must be
> set.
> > >>This option is introduced in:
> > >>http://lwn.net/Articles/600564/
> > >
> > >I really *love* this work! It would be fantastic if we could get
> this done.
> > >
> > >Hmm, I am not seeting that SO_PASSPROC is actually turned on in this
> > >patch? Am I missing something? This really needs to be an option
> that
> > >can be turned on from the sending side, otherwise it would be only
> > >half as useful...
> > >
> > I turn this option in a separate patch, that I will post when the
> > kernel patch is accepted. For now I wanted to know opinions about
> this
> > patch.
> 
> In case this wasn't clear: the patch for systemd looks pretty OK. I'd
> love to merge something like this, but of course we need the kernel
> support first. If you can make this happen, I'd be delighted.

Thanks, I'm working on that and will let you know when ready.

Piotr

> 
> Lennart
> 
> --
> Lennart Poettering, Red Hat




More information about the systemd-devel mailing list