Author: bz
Date: Tue Jan 17 22:09:33 2012
New Revision: 230285
URL: http://svn.freebsd.org/changeset/base/230285

Log:
  MFC r225048:
  
   In this branch when doing no further checkes there is no reason use
   the temporary variable and check with if as TUNABLE_*_FETCH do not
   alter values unless successfully found the tunable.

Modified:
  stable/7/sys/amd64/amd64/machdep.c
  stable/7/sys/i386/i386/machdep.c
  stable/7/sys/pc98/pc98/machdep.c
Directory Properties:
  stable/7/sys/   (props changed)

Modified: stable/7/sys/amd64/amd64/machdep.c
==============================================================================
--- stable/7/sys/amd64/amd64/machdep.c  Tue Jan 17 22:08:58 2012        
(r230284)
+++ stable/7/sys/amd64/amd64/machdep.c  Tue Jan 17 22:09:33 2012        
(r230285)
@@ -956,7 +956,7 @@ getmemsize(caddr_t kmdp, u_int64_t first
 {
        int i, off, physmap_idx, pa_indx, da_indx;
        vm_paddr_t pa, physmap[PHYSMAP_SIZE];
-       u_long physmem_tunable, memtest, tmpul;
+       u_long physmem_tunable, memtest;
        pt_entry_t *pte;
        struct bios_smap *smapbase, *smap, *smapend;
        u_int32_t smapsize;
@@ -1023,8 +1023,7 @@ getmemsize(caddr_t kmdp, u_int64_t first
         * one could eventually do more with the code than just disable it.
         */
        memtest = 1;
-       if (TUNABLE_ULONG_FETCH("hw.memtest.tests", &tmpul))
-               memtest = tmpul;
+       TUNABLE_ULONG_FETCH("hw.memtest.tests", &memtest);
 
        /*
         * Don't allow MAXMEM or hw.physmem to extend the amount of memory

Modified: stable/7/sys/i386/i386/machdep.c
==============================================================================
--- stable/7/sys/i386/i386/machdep.c    Tue Jan 17 22:08:58 2012        
(r230284)
+++ stable/7/sys/i386/i386/machdep.c    Tue Jan 17 22:09:33 2012        
(r230285)
@@ -1809,7 +1809,7 @@ static void
 getmemsize(int first)
 {
        int has_smap, off, physmap_idx, pa_indx, da_indx;
-       u_long physmem_tunable, memtest, tmpul;
+       u_long physmem_tunable, memtest;
        vm_paddr_t physmap[PHYSMAP_SIZE];
        pt_entry_t *pte;
        quad_t dcons_addr, dcons_size;
@@ -2010,8 +2010,7 @@ physmap_done:
         * one could eventually do more with the code than just disable it.
         */
        memtest = 1;
-       if (TUNABLE_ULONG_FETCH("hw.memtest.tests", &tmpul))
-               memtest = tmpul;
+       TUNABLE_ULONG_FETCH("hw.memtest.tests", &memtest);
 
        if (atop(physmap[physmap_idx + 1]) != Maxmem &&
            (boothowto & RB_VERBOSE))

Modified: stable/7/sys/pc98/pc98/machdep.c
==============================================================================
--- stable/7/sys/pc98/pc98/machdep.c    Tue Jan 17 22:08:58 2012        
(r230284)
+++ stable/7/sys/pc98/pc98/machdep.c    Tue Jan 17 22:09:33 2012        
(r230285)
@@ -1697,7 +1697,7 @@ static void
 getmemsize(int first)
 {
        int off, physmap_idx, pa_indx, da_indx;
-       u_long physmem_tunable, memtest, tmpul;
+       u_long physmem_tunable, memtest;
        vm_paddr_t physmap[PHYSMAP_SIZE];
        pt_entry_t *pte;
        quad_t dcons_addr, dcons_size;
@@ -1762,8 +1762,7 @@ getmemsize(int first)
         * one could eventually do more with the code than just disable it.
         */
        memtest = 1;
-       if (TUNABLE_ULONG_FETCH("hw.memtest.tests", &tmpul))
-               memtest = tmpul;
+       TUNABLE_ULONG_FETCH("hw.memtest.tests", &memtest);
 
        if (atop(physmap[physmap_idx + 1]) != Maxmem &&
            (boothowto & RB_VERBOSE))
_______________________________________________
[email protected] mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "[email protected]"

Reply via email to