Date: Sun, 11 May 2014 06:53:46 -0600 From: Ian Lepore <ian@FreeBSD.org> To: Konstantin Belousov <kostikbel@gmail.com> Cc: svn-src-head@FreeBSD.org, svn-src-all@FreeBSD.org, src-committers@FreeBSD.org Subject: Re: svn commit: r265861 - in head/sys: arm/include modules Message-ID: <1399812826.22079.427.camel@revolution.hippie.lan> In-Reply-To: <20140511071002.GV74331@kib.kiev.ua> References: <201405110043.s4B0h6B9040965@svn.freebsd.org> <20140511071002.GV74331@kib.kiev.ua>
next in thread | previous in thread | raw e-mail | index | archive | help
On Sun, 2014-05-11 at 10:10 +0300, Konstantin Belousov wrote: > On Sun, May 11, 2014 at 12:43:06AM +0000, Ian Lepore wrote: > > Author: ian > > Date: Sun May 11 00:43:06 2014 > > New Revision: 265861 > > URL: http://svnweb.freebsd.org/changeset/base/265861 > > > > Log: > > Make the hardware memory and instruction barrier functions work on armv4 > > and armv5 as well. > > > > Modified: > > head/sys/arm/include/atomic.h > > head/sys/modules/Makefile > > > > Modified: head/sys/arm/include/atomic.h > > ============================================================================== > > --- head/sys/arm/include/atomic.h Sat May 10 22:27:01 2014 (r265860) > > +++ head/sys/arm/include/atomic.h Sun May 11 00:43:06 2014 (r265861) > > @@ -58,9 +58,9 @@ > > #define dsb() __asm __volatile("mcr p15, 0, %0, c7, c10, 4" : : "r" (0) : "memory") > > #define dmb() __asm __volatile("mcr p15, 0, %0, c7, c10, 5" : : "r" (0) : "memory") > > #else > > -#define isb() > > -#define dsb() > > -#define dmb() > > +#define isb() __asm __volatile("mcr p15, 0, %0, c7, c5, 4" : : "r" (0) : "memory") > > +#define dsb() __asm __volatile("mcr p15, 0, %0, c7, c10, 4" : : "r" (0) : "memory") > > +#define dmb() dsb() > > #endif > > > > #define mb() dmb() > > > > Modified: head/sys/modules/Makefile > > ============================================================================== > > --- head/sys/modules/Makefile Sat May 10 22:27:01 2014 (r265860) > > +++ head/sys/modules/Makefile Sun May 11 00:43:06 2014 (r265861) > > @@ -2,6 +2,8 @@ > > > > .include <src.opts.mk> > > > > +SUBDIR_PARALLEL= > > + > > # Modules that include binary-only blobs of microcode should be selectable by > > # MK_SOURCELESS_UCODE option (see below). > > > This part is unrelated and should be reverted, I suppose ? Ooops, indeed, thanks. Although... it's a good change in terms of speeding up the build, I just didn't intend to commit it until it got tested with -j levels higher than I can test with my little 6-core machine. -- Ian
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?1399812826.22079.427.camel>