cvs commit: src/sys/kern kern_thread.c src/sys/security/audit audit.c audit.h src/sys/sys proc.h

[John Baldwin]

On Wednesday 01 February 2006 19:37, Robert Watson wrote:
> rwatson     2006-02-02 00:37:06 UTC
>
>   FreeBSD src repository
>
>   Modified files:
>     sys/kern             kern_thread.c
>     sys/security/audit   audit.c audit.h
>     sys/sys              proc.h
>   Log:
>   Add new fields to process-related data structures:
>
>   - td_ar to struct thread, which holds the in-progress audit record during
>     a system call.
>
>   - p_au to struct proc, which holds per-process audit state, such as the
>     audit identifier, audit terminal, and process audit masks.
>
>   In the earlier implementation, td_ar was added to the zero'd section of
>   struct thread.  In order to facilitate merging to RELENG_6, it has been
>   moved to the end of the data structure, requiring explicit
>   initalization in the thread constructor.
>
>   Much help from: wsalamon
>   Obtained from:  TrustedBSD Project

Have you considered putting td_ar in the logical place in HEAD and only doing 
the ABI-friendly "hack" for RELENG_6?  Maybe you could do the ABI thing in 
HEAD to make the MFC easier and then update HEAD after the MFC to be more 
intuitive?

-- 
John Baldwin <jhb at FreeBSD.org>  <><  http://www.FreeBSD.org/~jhb/
"Power Users Use the Power to Serve"  =  http://www.FreeBSD.org