The current MSFBUFSIZE on armv7 is a bit small and I'd like to
increase its default size to somethin on-par with i386 or amd64.
While trying to do so, I ran into the issue that we (conditionally)
define MSGBUFSIZE in both <arm/param.h> and <machine/param.h>.
However, because <machine/param.h> includes <arm/param.h> before
attempting to define MSGBUFSIZE, change the definition in
<machine/param.h> has no effect.

There are of course two ways to make things a bit more sensible.
Either removing the definition from <machine/param.h>, or removing it
from <arm/param.h>.  In our tree, mips64 takes the first approach and
powerpc the second.  My preference is to follow powerpc here.  That
allows us to change the default for armv7 while keeping a smaller
value on zaurus.

ok?


Index: arch/arm/include/param.h
===================================================================
RCS file: /cvs/src/sys/arch/arm/include/param.h,v
retrieving revision 1.21
diff -u -p -r1.21 param.h
--- arch/arm/include/param.h    13 Jun 2013 05:13:12 -0000      1.21
+++ arch/arm/include/param.h    20 Aug 2016 17:38:13 -0000
@@ -57,10 +57,6 @@
 
 #define        NMBCLUSTERS     4096                    /* map size, max 
cluster allocation */
 
-#ifndef        MSGBUFSIZE
-#define        MSGBUFSIZE      (1 * PAGE_SIZE)         /* default message 
buffer size */
-#endif
-
 /*
  * Maximum size of the kernel malloc arena in PAGE_SIZE-sized
  * logical pages.

Reply via email to