svn commit: r298482 - in head/sys: amd64/amd64 amd64/linux amd64/linux32 compat/linux dev/cxgbe/tom i386/linux vm

[Pedro F. Giffuni]

Author: pfg
Date: Fri Apr 22 16:57:42 2016
New Revision: 298482
URL: https://svnweb.freebsd.org/changeset/base/298482

Log:
  Cleanup redundant parenthesis from existing howmany()/roundup() macro uses.

Modified:
  head/sys/amd64/amd64/pmap.c
  head/sys/amd64/linux/linux_sysvec.c
  head/sys/amd64/linux32/linux32_sysvec.c
  head/sys/compat/linux/linux_file.c
  head/sys/dev/cxgbe/tom/t4_tom.c
  head/sys/i386/linux/linux_sysvec.c
  head/sys/vm/vm_radix.c

Modified: head/sys/amd64/amd64/pmap.c
==============================================================================
--- head/sys/amd64/amd64/pmap.c	Fri Apr 22 16:28:41 2016	(r298481)
+++ head/sys/amd64/amd64/pmap.c	Fri Apr 22 16:57:42 2016	(r298482)
@@ -676,7 +676,7 @@ allocpages(vm_paddr_t *firstaddr, int n)
 CTASSERT(powerof2(NDMPML4E));
 
 /* number of kernel PDP slots */
-#define	NKPDPE(ptpgs)		howmany((ptpgs), NPDEPG)
+#define	NKPDPE(ptpgs)		howmany(ptpgs, NPDEPG)
 
 static void
 nkpt_init(vm_paddr_t addr)

Modified: head/sys/amd64/linux/linux_sysvec.c
==============================================================================
--- head/sys/amd64/linux/linux_sysvec.c	Fri Apr 22 16:28:41 2016	(r298481)
+++ head/sys/amd64/linux/linux_sysvec.c	Fri Apr 22 16:57:42 2016	(r298482)
@@ -346,7 +346,7 @@ linux_copyout_strings(struct image_param
 	destp =	(caddr_t)arginfo - SPARE_USRSPACE -
 	    roundup(sizeof(canary), sizeof(char *)) -
 	    roundup(execpath_len, sizeof(char *)) -
-	    roundup((ARG_MAX - imgp->args->stringspace), sizeof(char *));
+	    roundup(ARG_MAX - imgp->args->stringspace, sizeof(char *));
 
 	if (execpath_len != 0) {
 		imgp->execpathp = (uintptr_t)arginfo - execpath_len;

Modified: head/sys/amd64/linux32/linux32_sysvec.c
==============================================================================
--- head/sys/amd64/linux32/linux32_sysvec.c	Fri Apr 22 16:28:41 2016	(r298481)
+++ head/sys/amd64/linux32/linux32_sysvec.c	Fri Apr 22 16:57:42 2016	(r298482)
@@ -862,7 +862,7 @@ linux_copyout_strings(struct image_param
 	destp =	(caddr_t)arginfo - SPARE_USRSPACE -
 	    roundup(sizeof(canary), sizeof(char *)) -
 	    roundup(execpath_len, sizeof(char *)) -
-	    roundup((ARG_MAX - imgp->args->stringspace), sizeof(char *));
+	    roundup(ARG_MAX - imgp->args->stringspace, sizeof(char *));
 
 	if (execpath_len != 0) {
 		imgp->execpathp = (uintptr_t)arginfo - execpath_len;

Modified: head/sys/compat/linux/linux_file.c
==============================================================================
--- head/sys/compat/linux/linux_file.c	Fri Apr 22 16:28:41 2016	(r298481)
+++ head/sys/compat/linux/linux_file.c	Fri Apr 22 16:57:42 2016	(r298482)
@@ -305,11 +305,10 @@ struct l_dirent64 {
  * at least glibc-2.7 requires it. That is why l_dirent is padded with 2 bytes.
  */
 #define LINUX_RECLEN(namlen)						\
-    roundup((offsetof(struct l_dirent, d_name) + (namlen) + 2),		\
-    sizeof(l_ulong))
+    roundup(offsetof(struct l_dirent, d_name) + (namlen) + 2, sizeof(l_ulong))
 
 #define LINUX_RECLEN64(namlen)						\
-    roundup((offsetof(struct l_dirent64, d_name) + (namlen) + 1),	\
+    roundup(offsetof(struct l_dirent64, d_name) + (namlen) + 1,		\
     sizeof(uint64_t))
 
 #define LINUX_MAXRECLEN		max(LINUX_RECLEN(LINUX_NAME_MAX),	\

Modified: head/sys/dev/cxgbe/tom/t4_tom.c
==============================================================================
--- head/sys/dev/cxgbe/tom/t4_tom.c	Fri Apr 22 16:28:41 2016	(r298481)
+++ head/sys/dev/cxgbe/tom/t4_tom.c	Fri Apr 22 16:57:42 2016	(r298482)
@@ -129,7 +129,7 @@ alloc_toepcb(struct vi_info *vi, int txq
 	 * units of 16 byte.  Calculate the maximum work requests possible.
 	 */
 	txsd_total = tx_credits /
-	    howmany((sizeof(struct fw_ofld_tx_data_wr) + 1), 16);
+	    howmany(sizeof(struct fw_ofld_tx_data_wr) + 1, 16);
 
 	if (txqid < 0)
 		txqid = (arc4random() % vi->nofldtxq) + vi->first_ofld_txq;

Modified: head/sys/i386/linux/linux_sysvec.c
==============================================================================
--- head/sys/i386/linux/linux_sysvec.c	Fri Apr 22 16:28:41 2016	(r298481)
+++ head/sys/i386/linux/linux_sysvec.c	Fri Apr 22 16:57:42 2016	(r298482)
@@ -320,7 +320,7 @@ linux_copyout_strings(struct image_param
 	destp = (caddr_t)arginfo - SPARE_USRSPACE - linux_szplatform -
 	    roundup(sizeof(canary), sizeof(char *)) -
 	    roundup(execpath_len, sizeof(char *)) -
-	    roundup((ARG_MAX - imgp->args->stringspace), sizeof(char *));
+	    roundup(ARG_MAX - imgp->args->stringspace, sizeof(char *));
 
 	/*
 	 * install LINUX_PLATFORM

Modified: head/sys/vm/vm_radix.c
==============================================================================
--- head/sys/vm/vm_radix.c	Fri Apr 22 16:28:41 2016	(r298481)
+++ head/sys/vm/vm_radix.c	Fri Apr 22 16:57:42 2016	(r298482)
@@ -82,7 +82,7 @@ __FBSDID("$FreeBSD$");
 #define	VM_RADIX_COUNT	(1 << VM_RADIX_WIDTH)
 #define	VM_RADIX_MASK	(VM_RADIX_COUNT - 1)
 #define	VM_RADIX_LIMIT							\
-	(howmany((sizeof(vm_pindex_t) * NBBY), VM_RADIX_WIDTH) - 1)
+	(howmany(sizeof(vm_pindex_t) * NBBY, VM_RADIX_WIDTH) - 1)
 
 /* Flag bits stored in node pointers. */
 #define	VM_RADIX_ISLEAF	0x1