Re: cvs commit: src/bin/ps extern.h keyword.c print.c ps.c ps.h src/contrib/ipfilter/ipsend sock.c src/lib/libkcore kcore_file.c src/lib/libkvm Makefile kvm.c kvm.h kvm_getprocs.3 kvm_private.h kvm_proc.c src/sys/conf files src/sys/kern imgact_aout.c ...

[YONETANI Tomokazu]

On Thu, Feb 01, 2007 at 02:33:26AM -0800, Simon Schubert wrote:
>   Log:
>   Change kinfo_proc interface between kernel and userland.
>   
> http://www.dragonflybsd.org/cvsweb/src/bin/ps/print.c.diff?r1=1.25&r2=1.26&f=u

I just noticed after this change and "1:1 Userland threading stage 2.11/4",
that jail processes are no longer marked 'J' in ps output.  bin/ps/print.c
has been changed this way:

@@ -166,16 +166,14 @@ logname(const KINFO *k, const struct var
 void
 state(const KINFO *k, const struct varent *vent)
 {
-       struct proc *p;
        int flag;
        char *cp;
        char buf[16];

-       p = KI_PROC(k);
-       flag = p->p_flag;
+       flag = KI_LWP(k, flags);
        cp = buf;


However, since fork1() sets P_JAILED to p2->p_flag and not lp2->lwp_flag,
KI_LWP(k, flags) never contains P_JAILED (or any other P_* flags set to
p_flag).  Is this problem going to be resolved in the later stage?

Cheers.