Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 15 Oct 2020 20:21:16 +0000 (UTC)
From:      Mitchell Horne <mhorne@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r366737 - in head/sys: amd64/amd64 arm/arm arm64/arm64 riscv/riscv
Message-ID:  <202010152021.09FKLGsb016515@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: mhorne
Date: Thu Oct 15 20:21:15 2020
New Revision: 366737
URL: https://svnweb.freebsd.org/changeset/base/366737

Log:
  Simplify preload_dump() condition
  
  Hiding this feature behind RB_VERBOSE is gratuitous. The tunable is enough
  to limit its use to only those who explicitly request it.
  
  Suggested by:	kevans

Modified:
  head/sys/amd64/amd64/machdep.c
  head/sys/arm/arm/machdep.c
  head/sys/arm64/arm64/machdep.c
  head/sys/riscv/riscv/machdep.c

Modified: head/sys/amd64/amd64/machdep.c
==============================================================================
--- head/sys/amd64/amd64/machdep.c	Thu Oct 15 18:03:14 2020	(r366736)
+++ head/sys/amd64/amd64/machdep.c	Thu Oct 15 20:21:15 2020	(r366737)
@@ -1859,8 +1859,7 @@ hammer_time(u_int64_t modulep, u_int64_t physfree)
 	 * output is required. If it's grossly incorrect the kernel will never
 	 * make it this far.
 	 */
-	if ((boothowto & RB_VERBOSE) &&
-	    getenv_is_true("debug.dump_modinfo_at_boot"))
+	if (getenv_is_true("debug.dump_modinfo_at_boot"))
 		preload_dump();
 
 #ifdef DEV_ISA

Modified: head/sys/arm/arm/machdep.c
==============================================================================
--- head/sys/arm/arm/machdep.c	Thu Oct 15 18:03:14 2020	(r366736)
+++ head/sys/arm/arm/machdep.c	Thu Oct 15 20:21:15 2020	(r366737)
@@ -1032,8 +1032,7 @@ initarm(struct arm_boot_params *abp)
 	 * output is required. If it's grossly incorrect the kernel will never
 	 * make it this far.
 	 */
-	if ((boothowto & RB_VERBOSE) &&
-	    getenv_is_true("debug.dump_modinfo_at_boot"))
+	if (getenv_is_true("debug.dump_modinfo_at_boot"))
 		preload_dump();
 
 	env = kern_getenv("kernelname");

Modified: head/sys/arm64/arm64/machdep.c
==============================================================================
--- head/sys/arm64/arm64/machdep.c	Thu Oct 15 18:03:14 2020	(r366736)
+++ head/sys/arm64/arm64/machdep.c	Thu Oct 15 20:21:15 2020	(r366737)
@@ -1247,8 +1247,7 @@ initarm(struct arm64_bootparams *abp)
 	 * output is required. If it's grossly incorrect the kernel will never
 	 * make it this far.
 	 */
-	if ((boothowto & RB_VERBOSE) &&
-	    getenv_is_true("debug.dump_modinfo_at_boot"))
+	if (getenv_is_true("debug.dump_modinfo_at_boot"))
 		preload_dump();
 
 	init_proc0(abp->kern_stack);

Modified: head/sys/riscv/riscv/machdep.c
==============================================================================
--- head/sys/riscv/riscv/machdep.c	Thu Oct 15 18:03:14 2020	(r366736)
+++ head/sys/riscv/riscv/machdep.c	Thu Oct 15 20:21:15 2020	(r366737)
@@ -954,8 +954,7 @@ initriscv(struct riscv_bootparams *rvbp)
 	 * output is required. If it's grossly incorrect the kernel will never
 	 * make it this far.
 	 */
-	if ((boothowto & RB_VERBOSE) &&
-	    getenv_is_true("debug.dump_modinfo_at_boot"))
+	if (getenv_is_true("debug.dump_modinfo_at_boot"))
 		preload_dump();
 
 	init_proc0(rvbp->kern_stack);



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202010152021.09FKLGsb016515>