tech-kern archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: New sysctl entry: proc.PID.realpath



On Mon, Sep 07, 2015 at 09:23:21AM +0200, Martin Husemann wrote:
> On Mon, Sep 07, 2015 at 03:50:21AM +0200, Kamil Rytarowski wrote:
> > +	error = vnode_to_path(path, MAXPATHLEN,vp, l, p);
> 
> Two nits:
> 
>  1) vnode_to_path(9) is undocumented
>  2) it only works if you are lucky (IIUC) - which you mostly are
> 
> The former is easy to fix, the latter IMHO is a killer before we expose
> this interface prominently and make debuggers depend on it. We then should
> also make $ORIGIN work in ld.elf_so ;-}

My suggestion was to just provide the filesystem id and inode number as
fallback. I still believe we should just turn on the code that remembers
the realpath on exec in first place, if you want to debug
something_with_a_very_very_very_very_..._very_long_name, you can always
override the (missing) default.

Joerg


Home | Main Index | Thread Index | Old Index