powerpc/nvram: More flexible nvram_create_partition()
authorBenjamin Herrenschmidt <benh@kernel.crashing.org>
Thu, 29 Jul 2010 05:28:20 +0000 (15:28 +1000)
committerBenjamin Herrenschmidt <benh@kernel.crashing.org>
Tue, 30 Nov 2010 04:31:51 +0000 (15:31 +1100)
Replace nvram_create_os_partition() with a variant that takes
the partition name, signature and size as arguments.

Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
arch/powerpc/kernel/nvram_64.c
arch/powerpc/platforms/pseries/nvram.c

index a8154f1..9e13335 100644 (file)
@@ -307,13 +307,15 @@ static int __init nvram_remove_os_partition(void)
        return 0;
 }
 
-/* nvram_create_os_partition
- *
- * Create a OS linux partition to buffer error logs.
- * Will create a partition starting at the first free
- * space found if space has enough room.
+/**
+ * nvram_create_partition - Create a partition in nvram
+ * @name: name of the partition to create
+ * @sig: signature of the partition to create
+ * @req_size: size to allocate preferrably
+ * @min_size: minimum acceptable size (0 means req_size)
  */
-static int __init nvram_create_os_partition(void)
+static int __init nvram_create_partition(const char *name, int sig,
+                                        int req_size, int min_size)
 {
        struct nvram_partition *part;
        struct nvram_partition *new_part;
@@ -322,20 +324,27 @@ static int __init nvram_create_os_partition(void)
        loff_t tmp_index;
        long size = 0;
        int rc;
-       
+
+       /* If no minimum size specified, make it the same as the
+        * requested size
+        */
+       if (min_size == 0)
+               min_size = req_size;
+
        /* Find a free partition that will give us the maximum needed size 
           If can't find one that will give us the minimum size needed */
        list_for_each_entry(part, &nvram_part->partition, partition) {
                if (part->header.signature != NVRAM_SIG_FREE)
                        continue;
 
-               if (part->header.length >= NVRAM_MAX_REQ) {
-                       size = NVRAM_MAX_REQ;
+               if (part->header.length >= req_size) {
+                       size = req_size;
                        free_part = part;
                        break;
                }
-               if (!size && part->header.length >= NVRAM_MIN_REQ) {
-                       size = NVRAM_MIN_REQ;
+               if (part->header.length > size &&
+                   part->header.length >= min_size) {
+                       size = part->header.length;
                        free_part = part;
                }
        }
@@ -350,9 +359,9 @@ static int __init nvram_create_os_partition(void)
        }
 
        new_part->index = free_part->index;
-       new_part->header.signature = NVRAM_SIG_OS;
+       new_part->header.signature = sig;
        new_part->header.length = size;
-       strcpy(new_part->header.name, "ppc64,linux");
+       strncpy(new_part->header.name, name, 12);
        new_part->header.checksum = nvram_checksum(&new_part->header);
 
        rc = nvram_write_header(new_part);
@@ -451,10 +460,10 @@ static int __init nvram_setup_partition(void)
        }
        
        /* try creating a partition with the free space we have */
-       rc = nvram_create_partition("ppc64,linux", );
-       if (!rc) {
+       rc =    nvram_create_partition("ppc64,linux", NVRAM_SIG_OS,
+                                      NVRAM_MAX_REQ, NVRAM_MIN_REQ);
+       if (!rc)
                return 0;
-       }
                
        /* need to free up some space */
        rc = nvram_remove_os_partition();
@@ -463,9 +472,10 @@ static int __init nvram_setup_partition(void)
        }
        
        /* create a partition in this new space */
-       rc = nvram_create_os_partition();
+       rc =    nvram_create_partition("ppc64,linux", NVRAM_SIG_OS,
+                                      NVRAM_MAX_REQ, NVRAM_MIN_REQ);
        if (rc) {
-               printk(KERN_ERR "nvram_create_os_partition: Could not find a "
+               printk(KERN_ERR "nvram_create_partition: Could not find a "
                       "NVRAM partition large enough\n");
                return rc;
        }
index f4e4c06..2a1ef5c 100644 (file)
 #include <asm/prom.h>
 #include <asm/machdep.h>
 
+/* Max bytes to read/write in one go */
+#define NVRW_CNT 0x20
+
 static unsigned int nvram_size;
 static int nvram_fetch, nvram_store;
 static char nvram_buf[NVRW_CNT];       /* assume this is in the first 4GB */
 static DEFINE_SPINLOCK(nvram_lock);
 
-/* Max bytes to read/write in one go */
-#define NVRW_CNT 0x20
-
 static ssize_t pSeries_nvram_read(char *buf, size_t count, loff_t *index)
 {
        unsigned int i;