Re: kern/52598 (/proc/self/exe misbehaves if it's on a null mount)

[christos%zoulas.com@localhost (Christos Zoulas)]

The following reply was made to PR kern/52598; it has been noted by GNATS.

From: christos%zoulas.com@localhost (Christos Zoulas)
To: =?UTF-8?Q?Fr=C3=A9d=C3=A9ric_Fauberteau?= <triaxx%NetBSD.org@localhost>, 
	gnats-bugs%netbsd.org@localhost
Cc: kern-bug-people%netbsd.org@localhost, netbsd-bugs%netbsd.org@localhost, coypu%sdf.org@localhost
Subject: Re: kern/52598 (/proc/self/exe misbehaves if it's on a null mount)
Date: Tue, 14 Jan 2020 17:31:52 -0500

 On Jan 14, 10:10pm, triaxx%NetBSD.org@localhost (=?UTF-8?Q?Fr=C3=A9d=C3=A9ric_Fauberteau?=) wrote:
 -- Subject: Re: kern/52598 (/proc/self/exe misbehaves if it's on a null mount
 
 | > http://cvsweb.netbsd.org/bsdweb.cgi/src/sys/sys/proc.h?rev=1.343&content-type=text/x-cvsweb-markup&only_with_tag=MAIN
 | > 
 | >  So I continue my investigations and tests following this way.
 | 
 | Ok, by applying these changes, I have a NetBSD 8.0 that can mount a 
 | working /proc in a chroot environment:
 | cvs rdiff -u -r1.90.4.1  -r1.93  src/sys/kern/exec_elf.c
 | cvs rdiff -u -r1.442.4.3 -r1.450 src/sys/kern/kern_exec.c
 | cvs rdiff -u -r1.268.8.1 -r1.270 src/sys/kern/kern_exit.c
 | cvs rdiff -u -r1.202     -r1.203 src/sys/kern/kern_fork.c
 | cvs rdiff -u -r1.206.6.4 -r1.208 src/sys/kern/kern_proc.c
 | cvs rdiff -u -r1.63      -r1.64  src/sys/kern/subr_kmem.c
 | cvs rdiff -u -r1.197.2.2 -r1.200 src/sys/miscfs/procfs/procfs_vnops.c
 | cvs rdiff -u -r1.151     -r1.152 src/sys/sys/exec.h
 | cvs rdiff -u -r1.9       -r1.10  src/sys/sys/kmem.h
 | cvs rdiff -u -r1.542.2.4 -r1.550 src/sys/sys/param.h
 | cvs rdiff -u -r1.340.6.1 -r1.343 src/sys/sys/proc.h
 | 
 | I should be able to build lang/rust with pbulk in a chrooted sandbox. I 
 | am testing.
 | 
 | But I fear it is not possible to submit a pullup request since a bump 
 | revision seems to be done for these changes (8.99.6 if I am correct). Do 
 | you confirm?
 
 Well, you need a kernel bump because sizeof(struct proc) changes...
 This would probably be put on 8.2...
 
 christos