On Mon, May 16, 2016 at 2:16 AM, Hans Petter Selasky <hsela...@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 _______________________________________________ svn-src-head@freebsd.org mailing list https://lists.freebsd.org/mailman/listinfo/svn-src-head To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"