svn commit: r299930 - in head/sys/compat/linuxkpi/common: include/linux src

Conrad Meyer cem at FreeBSD.org
Mon May 16 20:07:05 UTC 2016


On Mon, May 16, 2016 at 2:16 AM, Hans Petter Selasky
<hselasky at freebsd.org> wrote:
> Author: hselasky
> Date: Mon May 16 09:16:15 2016
> New Revision: 299930
> URL: https://svnweb.freebsd.org/changeset/base/299930
>
> Log:
>   Properly implement "cpu_has_clflush" macro.
> ...
> @@ -51,6 +51,10 @@ __FBSDID("$FreeBSD$");
>
>  #include <machine/stdarg.h>
>
> +#if defined(__i386__) || defined(__amd64__)
> +#include <machine/md_var.h>
> +#endif
> +
>  #include <linux/kobject.h>
>  #include <linux/device.h>
>  #include <linux/slab.h>
> @@ -67,6 +71,7 @@ __FBSDID("$FreeBSD$");
>  #include <linux/rcupdate.h>
>  #include <linux/interrupt.h>
>  #include <linux/uaccess.h>
> +#include <linux/kernel.h>

Hi Hans,

In Linux code, usually linux/kernel.h is included first.  In FreeBSD,
we mostly sort alphabetically.  Pick a system, but this is the wrong
place for this include :-).

Best,
Conrad


More information about the svn-src-head mailing list