cvs commit: src/sys/sys pcpu.h

John Baldwin jhb at freebsd.org
Fri Mar 7 14:19:06 UTC 2008


On Thursday 06 March 2008 12:24:28 pm Marcel Moolenaar wrote:
> marcel      2008-03-06 17:24:28 UTC
>
>   FreeBSD src repository
>
>   Modified files:
>     sys/sys              pcpu.h
>   Log:
>   Move the PCPU_MD_FIELDS last in struct pcpu. While this header is
>   private to the kernel, some ports define _KERNEL and include this
>   header. While arguably this is wrong, it's also reality. By having
>   the MD fields last, architectures that have CPU-specific variations
>   of PCPU_MD_FIELDS will at least have the MI fields at a constant
>   offset.  Of course, having all MI fields first helps kernel debugging
>   as well, so this is not a change without some benefits to us.
>
>   This change does not result in an ABI breakage, because this header
>   is not part of the ABI. Recompilation of lsof is required though :-)

It actually started out being the last "field" in the structure but some new 
fields were incorrectly added after it rather than before it.  It might 
warrant a comment to prevent similar regressions in the future.

-- 
John Baldwin


More information about the cvs-src mailing list