Date: Sat, 18 May 2002 20:23:00 -0400 (EDT) From: John Baldwin <jhb@FreeBSD.org> To: Robert Watson <rwatson@FreeBSD.org> Cc: cvs-all@FreeBSD.org, cvs-committers@FreeBSD.org Subject: Re: cvs commit: src/sys/compat/linprocfs linprocfs.c src/sys/fs/ Message-ID: <XFMail.20020518202300.jhb@FreeBSD.org> In-Reply-To: <Pine.NEB.3.96L.1020518201742.79313C-100000@fledge.watson.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On 19-May-2002 Robert Watson wrote:
> Great! Now I get to suffer as I propagate the locking changes into the
> MAC framework :-). BTW, what was the rationale for not renaming the
> functions?
1) It was more work.
2) They are still process-related functions as we are checking to see
if we can do certain things to target processes, so being in a
process namespace still makes sense.
> On Sat, 18 May 2002, John Baldwin wrote:
>
>> jhb 2002/05/18 17:14:50 PDT
>>
>> Modified files:
>> sys/compat/linprocfs linprocfs.c
>> sys/fs/procfs procfs.c procfs_ctl.c procfs_dbregs.c
>> procfs_fpregs.c procfs_mem.c
>> procfs_regs.c procfs_status.c
>> sys/fs/pseudofs pseudofs_vnops.c
>> sys/kern kern_event.c kern_ktrace.c kern_proc.c
>> kern_prot.c kern_resource.c kern_sig.c
>> sys_process.c
>> sys/posix4 p1003_1b.c
>> sys/security/lomac kernel_lkm.c
>> sys/sys proc.h
>> Log:
>> Change p_can{debug,see,sched,signal}()'s first argument to be a thread
>> pointer instead of a proc pointer and require the process pointed to
>> by the second argument to be locked. We now use the thread ucred
>> reference
>> for the credential checks in p_can*() as a result. p_canfoo() should now
>> no longer need Giant.
>>
>> Revision Changes Path
>> 1.52 +1 -1 src/sys/compat/linprocfs/linprocfs.c
>> 1.5 +3 -1 src/sys/fs/procfs/procfs.c
>> 1.40 +2 -2 src/sys/fs/procfs/procfs_ctl.c
>> 1.20 +1 -1 src/sys/fs/procfs/procfs_dbregs.c
>> 1.26 +1 -1 src/sys/fs/procfs/procfs_fpregs.c
>> 1.58 +1 -1 src/sys/fs/procfs/procfs_mem.c
>> 1.25 +1 -1 src/sys/fs/procfs/procfs_regs.c
>> 1.44 +1 -1 src/sys/fs/procfs/procfs_status.c
>> 1.27 +1 -1 src/sys/fs/pseudofs/pseudofs_vnops.c
>> 1.39 +1 -1 src/sys/kern/kern_event.c
>> 1.66 +1 -1 src/sys/kern/kern_ktrace.c
>> 1.135 +3 -3 src/sys/kern/kern_proc.c
>> 1.156 +69 -60 src/sys/kern/kern_prot.c
>> 1.105 +9 -9 src/sys/kern/kern_resource.c
>> 1.165 +3 -4 src/sys/kern/kern_sig.c
>> 1.93 +2 -2 src/sys/kern/sys_process.c
>> 1.19 +5 -5 src/sys/posix4/p1003_1b.c
>> 1.3 +2 -4 src/sys/security/lomac/kernel_lkm.c
>> 1.221 +4 -4 src/sys/sys/proc.h
>>
>
--
John Baldwin <jhb@FreeBSD.org> <>< http://www.FreeBSD.org/~jhb/
"Power Users Use the Power to Serve!" - http://www.FreeBSD.org/
To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?XFMail.20020518202300.jhb>