re: CVS commit: src/sys/miscfs/procfs

[matthew green <mrg%eterna.com.au@localhost>]

Nicolas Joly writes:
> On Sat, Jul 12, 2014 at 08:11:46PM +1000, matthew green wrote:
> > 
> > "Nicolas Joly" writes:
> > > Module Name:      src
> > > Committed By:     njoly
> > > Date:             Sat Jul 12 09:58:39 UTC 2014
> > > 
> > > Modified Files:
> > >   src/sys/miscfs/procfs: procfs_linux.c
> > > 
> > > Log Message:
> > > Use kproc2 to provide sensible informations for /proc/<pid>/stat.
> > 
> > hmm, kernel code should really avoid using the userland compat layer.
> > you're processing something from one known layout to another known
> > layout.. userland needs it because it only knows one of them, which
> > has a known forwards/backwards compat layout.
> 
> I was trying to avoid duplicating all the needed computation ...
> /proc/<pid>/stat is another exact version (kernel) of the userland
> processes infos.
> 
> Do we need another intermediate layer to fill a kernel only structure
> which can then be easily translated to kproc2 ?

hmmm, OK.  your idea is probably a good one, but i see why
you'd chosen what you have.

thanks.


.mrg.