sparc32: More memory probing consolidation.
authorDavid S. Miller <davem@davemloft.net>
Fri, 2 May 2008 05:28:17 +0000 (22:28 -0700)
committerDavid S. Miller <davem@davemloft.net>
Fri, 2 May 2008 12:22:53 +0000 (05:22 -0700)
The PROM library function prom_meminit() builds a table,
prom_phys_avail[], just so that probe_memory() in
arch/sparc/mm/fault.c can copy it into sp_banks[].

Just have prom_meminit() fill in the sp_banks[] array directly, and
remove duplicated sort() function.

Signed-off-by: David S. Miller <davem@davemloft.net>
arch/sparc/kernel/setup.c
arch/sparc/mm/fault.c
arch/sparc/prom/memory.c
include/asm-sparc/oplib.h
include/asm-sparc/page.h

index 3c13137..8a55c4f 100644 (file)
@@ -180,11 +180,9 @@ static void __init boot_flags_init(char *commands)
 
 /* This routine will in the future do all the nasty prom stuff
  * to probe for the mmu type and its parameters, etc. This will
- * also be where SMP things happen plus the Sparc specific memory
- * physical memory probe as on the alpha.
+ * also be where SMP things happen.
  */
 
-extern int prom_probe_memory(void);
 extern void sun4c_probe_vac(void);
 extern char cputypval;
 extern unsigned long start, end;
@@ -268,7 +266,6 @@ void __init setup_arch(char **cmdline_p)
        if (ARCH_SUN4C_SUN4)
                sun4c_probe_vac();
        load_mmu();
-       (void) prom_probe_memory();
 
        phys_base = 0xffffffffUL;
        highest_paddr = 0UL;
index f25876a..abd5079 100644 (file)
@@ -47,64 +47,15 @@ int vac_size, vac_linesize, vac_do_hw_vac_flushes;
 int vac_entries_per_context, vac_entries_per_segment;
 int vac_entries_per_page;
 
-/* Nice, simple, prom library does all the sweating for us. ;) */
-int prom_probe_memory (void)
+/* Return how much physical memory we have.  */
+unsigned long probe_memory(void)
 {
-       register struct linux_mlist_v0 *mlist;
-       register unsigned long bytes, base_paddr, tally;
-       register int i;
-
-       i = 0;
-       mlist= prom_meminfo();
-       bytes = tally = mlist->num_bytes;
-       base_paddr = (unsigned long) mlist->start_adr;
-  
-       sp_banks[0].base_addr = base_paddr;
-       sp_banks[0].num_bytes = bytes;
-
-       while (mlist->theres_more != (void *) 0){
-               i++;
-               mlist = mlist->theres_more;
-               bytes = mlist->num_bytes;
-               tally += bytes;
-               if (i > SPARC_PHYS_BANKS-1) {
-                       printk ("The machine has more banks than "
-                               "this kernel can support\n"
-                               "Increase the SPARC_PHYS_BANKS "
-                               "setting (currently %d)\n",
-                               SPARC_PHYS_BANKS);
-                       i = SPARC_PHYS_BANKS-1;
-                       break;
-               }
-    
-               sp_banks[i].base_addr = (unsigned long) mlist->start_adr;
-               sp_banks[i].num_bytes = mlist->num_bytes;
-       }
-
-       i++;
-       sp_banks[i].base_addr = 0xdeadbeef;
-       sp_banks[i].num_bytes = 0;
-
-       /* Now mask all bank sizes on a page boundary, it is all we can
-        * use anyways.
-        */
-       for(i=0; sp_banks[i].num_bytes != 0; i++)
-               sp_banks[i].num_bytes &= PAGE_MASK;
-
-       return tally;
-}
-
-/* Traverse the memory lists in the prom to see how much physical we
- * have.
- */
-unsigned long
-probe_memory(void)
-{
-       int total;
+       unsigned long total = 0;
+       int i;
 
-       total = prom_probe_memory();
+       for (i = 0; sp_banks[i].num_bytes; i++)
+               total += sp_banks[i].num_bytes;
 
-       /* Oh man, much nicer, keep the dirt in promlib. */
        return total;
 }
 
index 08ac1bf..947f047 100644 (file)
 /* memory.c: Prom routine for acquiring various bits of information
  *           about RAM on the machine, both virtual and physical.
  *
- * Copyright (C) 1995 David S. Miller (davem@caip.rutgers.edu)
+ * Copyright (C) 1995, 2008 David S. Miller (davem@davemloft.net)
  * Copyright (C) 1997 Michael A. Griffith (grif@acm.org)
  */
 
 #include <linux/kernel.h>
+#include <linux/sort.h>
 #include <linux/init.h>
 
 #include <asm/openprom.h>
 #include <asm/sun4prom.h>
 #include <asm/oplib.h>
+#include <asm/page.h>
 
-/* This routine, for consistency, returns the ram parameters in the
- * V0 prom memory descriptor format.  I choose this format because I
- * think it was the easiest to work with.  I feel the religious
- * arguments now... ;)  Also, I return the linked lists sorted to
- * prevent paging_init() upset stomach as I have not yet written
- * the pepto-bismol kernel module yet.
- */
+static int __init prom_meminit_v0(void)
+{
+       struct linux_mlist_v0 *p;
+       int index;
 
-struct linux_prom_registers prom_reg_memlist[64];
+       index = 0;
+       for (p = *(romvec->pv_v0mem.v0_available); p; p = p->theres_more) {
+               sp_banks[index].base_addr = (unsigned long) p->start_adr;
+               sp_banks[index].num_bytes = p->num_bytes;
+               index++;
+       }
 
-struct linux_mlist_v0 prom_phys_avail[64];
+       return index;
+}
 
-/* Internal Prom library routine to sort a linux_mlist_v0 memory
- * list.  Used below in initialization.
- */
-static void __init
-prom_sortmemlist(struct linux_mlist_v0 *thislist)
+static int __init prom_meminit_v2(void)
 {
-       int swapi = 0;
-       int i, mitr, tmpsize;
-       char *tmpaddr;
-       char *lowest;
-
-       for(i=0; thislist[i].theres_more; i++) {
-               lowest = thislist[i].start_adr;
-               for(mitr = i+1; thislist[mitr-1].theres_more; mitr++)
-                       if(thislist[mitr].start_adr < lowest) {
-                               lowest = thislist[mitr].start_adr;
-                               swapi = mitr;
-                       }
-               if(lowest == thislist[i].start_adr) continue;
-               tmpaddr = thislist[swapi].start_adr;
-               tmpsize = thislist[swapi].num_bytes;
-               for(mitr = swapi; mitr > i; mitr--) {
-                       thislist[mitr].start_adr = thislist[mitr-1].start_adr;
-                       thislist[mitr].num_bytes = thislist[mitr-1].num_bytes;
-               }
-               thislist[i].start_adr = tmpaddr;
-               thislist[i].num_bytes = tmpsize;
+       struct linux_prom_registers reg[64];
+       int node, size, num_ents, i;
+
+       node = prom_searchsiblings(prom_getchild(prom_root_node), "memory");
+       size = prom_getproperty(node, "available", (char *) reg, sizeof(reg));
+       num_ents = size / sizeof(struct linux_prom_registers);
+
+       for (i = 0; i < num_ents; i++) {
+               sp_banks[i].base_addr = reg[i].phys_addr;
+               sp_banks[i].num_bytes = reg[i].reg_size;
        }
 
-       return;
+       return num_ents;
+}
+
+static int __init prom_meminit_sun4(void)
+{
+#ifdef CONFIG_SUN4
+       sp_banks[0].base_addr = 0;
+       sp_banks[0].num_bytes = *(sun4_romvec->memoryavail);
+#endif
+       return 1;
+}
+
+static int sp_banks_cmp(const void *a, const void *b)
+{
+       const struct sparc_phys_banks *x = a, *y = b;
+
+       if (x->base_addr > y->base_addr)
+               return 1;
+       if (x->base_addr < y->base_addr)
+               return -1;
+       return 0;
 }
 
 /* Initialize the memory lists based upon the prom version. */
 void __init prom_meminit(void)
 {
-       int node = 0;
-       unsigned int iter, num_regs;
-       struct linux_mlist_v0 *mptr;  /* ptr for traversal */
+       int i, num_ents = 0;
 
-       switch(prom_vers) {
+       switch (prom_vers) {
        case PROM_V0:
-               /* Nice, kind of easier to do in this case. */
-               for(mptr = (*(romvec->pv_v0mem.v0_available)), iter=0;
-                   mptr; mptr=mptr->theres_more, iter++) {
-                       prom_phys_avail[iter].start_adr = mptr->start_adr;
-                       prom_phys_avail[iter].num_bytes = mptr->num_bytes;
-                       prom_phys_avail[iter].theres_more = &prom_phys_avail[iter+1];
-               }
-               prom_phys_avail[iter-1].theres_more = NULL;
-               prom_sortmemlist(prom_phys_avail);
+               num_ents = prom_meminit_v0();
                break;
+
        case PROM_V2:
        case PROM_V3:
-               /* Grrr, have to traverse the prom device tree ;( */
-               node = prom_getchild(prom_root_node);
-               node = prom_searchsiblings(node, "memory");
-               num_regs = prom_getproperty(node, "available",
-                                           (char *) prom_reg_memlist,
-                                           sizeof(prom_reg_memlist));
-               num_regs = (num_regs/sizeof(struct linux_prom_registers));
-               for(iter=0; iter<num_regs; iter++) {
-                       prom_phys_avail[iter].start_adr =
-                               (char *) prom_reg_memlist[iter].phys_addr;
-                       prom_phys_avail[iter].num_bytes =
-                               (unsigned long) prom_reg_memlist[iter].reg_size;
-                       prom_phys_avail[iter].theres_more =
-                               &prom_phys_avail[iter+1];
-               }
-               prom_phys_avail[iter-1].theres_more = NULL;
-               prom_sortmemlist(prom_phys_avail);
+               num_ents = prom_meminit_v2();
                break;
 
        case PROM_SUN4:
-#ifdef CONFIG_SUN4     
-               /* how simple :) */
-               prom_phys_avail[0].start_adr = NULL;
-               prom_phys_avail[0].num_bytes = *(sun4_romvec->memoryavail);
-               prom_phys_avail[0].theres_more = NULL;
-#endif
+               num_ents = prom_meminit_sun4();
                break;
 
        default:
                break;
-       };
-}
+       }
+       sort(sp_banks, num_ents, sizeof(struct sparc_phys_banks),
+            sp_banks_cmp, NULL);
 
-/* This returns a pointer to our libraries internal v0 format
- * available memory list.
- */
-struct linux_mlist_v0 *
-prom_meminfo(void)
-{
-       return prom_phys_avail;
+       /* Sentinel.  */
+       sp_banks[num_ents].base_addr = 0xdeadbeef;
+       sp_banks[num_ents].num_bytes = 0;
+
+       for (i = 0; i < num_ents; i++)
+               sp_banks[i].num_bytes &= PAGE_MASK;
 }
index 6ff1a3b..22dc39d 100644 (file)
@@ -84,13 +84,6 @@ extern int prom_devclose(int device_handle);
 extern void prom_seek(int device_handle, unsigned int seek_hival,
                      unsigned int seek_lowval);
 
-/* Machine memory configuration routine. */
-
-/* This function returns a V0 format available memory descriptor entry.
- * This list is pre-sorted,
- */
-extern struct linux_mlist_v0 *prom_meminfo(void);
-
 /* Miscellaneous routines, don't really fit in any category per se. */
 
 /* Reboot the machine with the command line passed. */
index 39ccf2d..1625a8c 100644 (file)
 
 /* The following structure is used to hold the physical
  * memory configuration of the machine.  This is filled in
- * probe_memory() and is later used by mem_init() to set up
- * mem_map[].  We statically allocate SPARC_PHYS_BANKS of
+ * prom_meminit() and is later used by mem_init() to set up
+ * mem_map[].  We statically allocate SPARC_PHYS_BANKS+1 of
  * these structs, this is arbitrary.  The entry after the
  * last valid one has num_bytes==0.
  */
-
 struct sparc_phys_banks {
   unsigned long base_addr;
   unsigned long num_bytes;