Change the way thread priority is handled in binfmt/ to better match the way that priority is set up for the builtin tasks

git-svn-id: svn://svn.code.sf.net/p/nuttx/code/trunk@5527 42af7a65-404d-4744-a932-0658087f49c3

binfmt/binfmt_exec.c

@@ -40,7 +40,6 @@
 #include <nuttx/config.h>
 
 #include <string.h>
-#include <sched.h>
 #include <debug.h>
 #include <errno.h>
 
@@ -75,8 +74,7 @@
  *
  * Description:
  *   This is a convenience function that wraps load_ and exec_module into
- *   one call.  The priority of the executed program is set to be the
- *   same as the priority of the calling thread.
+ *   one call.
  *
  * Input Parameter:
  *   filename - Fulll path to the binary to be loaded
@@ -95,18 +93,8 @@ int exec(FAR const char *filename, FAR const char **argv,
          FAR const struct symtab_s *exports, int nexports)
 {
   struct binary_s bin;
-  struct sched_param param;
   int ret;
 
-  /* Get the priority of this thread */
-
-  ret = sched_getparam(0, &param);
-  if (ret < 0)
-    {
-      bdbg("ERROR: sched_getparam failed: %d\n", errno);
-      return ERROR;
-    }
-  
   /* Load the module into memory */
 
   memset(&bin, 0, sizeof(struct binary_s));
@@ -121,9 +109,9 @@ int exec(FAR const char *filename, FAR const char **argv,
       return ERROR;
     }
 
-  /* Then start the module at the priority of this thread */
+  /* Then start the module */
 
-  ret = exec_module(&bin, param.sched_priority);
+  ret = exec_module(&bin);
   if (ret < 0)
     {
       bdbg("ERROR: Failed to execute program '%s'\n", filename);

binfmt/binfmt_execmodule.c

@@ -144,7 +144,7 @@ static inline int exec_ctors(FAR const struct binary_s *binp)
  *
  ****************************************************************************/
 
-int exec_module(FAR const struct binary_s *binp, int priority)
+int exec_module(FAR const struct binary_s *binp)
 {
   FAR _TCB     *tcb;
 #ifndef CONFIG_CUSTOM_STACK
@@ -187,12 +187,12 @@ int exec_module(FAR const struct binary_s *binp, int priority)
 
   /* Initialize the task */
 
-  ret = task_init(tcb, binp->filename, priority, stack,
+  ret = task_init(tcb, binp->filename, binp->priority, stack,
                   binp->stacksize, binp->entrypt, binp->argv);
 #else
   /* Initialize the task */
 
-  ret = task_init(tcb, binp->filename, priority, stack,
+  ret = task_init(tcb, binp->filename, binp->priority, stack,
                   binp->entrypt, binp->argv);
 #endif
   if (ret < 0)

binfmt/binfmt_loadmodule.c

@@ -66,6 +66,39 @@
  * Private Functions
  ****************************************************************************/
 
+/****************************************************************************
+ * Name: load_default_priority
+ *
+ * Description:
+ *   Set the default priority of the module to be loaded.  This may be
+ *   changed (1) by the actions of the binary format's load() method if
+ *   the binary format contains priority informaition, or (2) by the user
+ *   between calls to load_module() and exec_module().
+ *
+ * Returned Value:
+ *   Zero (OK) is returned on success; Otherwise, -1 (ERROR) is returned and
+ *   the errno variable is set appropriately.
+ *
+ ****************************************************************************/
+
+static int load_default_priority(FAR struct binary_s *bin)
+{
+  struct sched_param param;
+
+  /* Get the priority of this thread */
+
+  ret = sched_getparam(0, &param);
+  if (ret < 0)
+    {
+      bdbg("ERROR: sched_getparam failed: %d\n", errno);
+      return ERROR;
+    }
+  
+  /* Save that as the priority of child thread */
+
+  bin->priority = param.sched_priority;
+}
+
 /****************************************************************************
  * Name: load_absmodule
  *
@@ -145,6 +178,16 @@ int load_module(FAR struct binary_s *bin)
   if (bin && bin->filename)
 #endif
     {
+      /* Set the default priority of the new program. */
+
+      ret = load_default_priority(bin)
+      if (ret < 0)
+        {
+          /* The errno is already set in this case */
+
+          return ERROR;
+        }
+
       /* Were we given a relative path?  Or an absolute path to the file to
        * be loaded?  Absolute paths start with '/'.
        */

binfmt/builtin.c

@@ -136,6 +136,7 @@ static int builtin_loadbinary(struct binary_s *binp)
 
   binp->entrypt   = g_builtins[index].main;
   binp->stacksize = g_builtins[index].stacksize;
+  binp->priority  = g_builtins[index].priority;
   return OK;
 }
 

binfmt/libbuiltin/Make.defs

@@ -41,9 +41,9 @@ BINFMT_CSRCS += builtin.c
 
 # Builtin library interfaces
 
-BINFMT_CSRCS += libbuiltin_utils.c
+BINFMT_CSRCS += libbuiltin_getname.c libbuiltin_isavail.c
 
-# Hook the libelf subdirectory into the build
+# Hook the libbuiltin subdirectory into the build
 
 VPATH += libbuiltin
 SUBDIRS += libbuiltin

binfmt/libbuiltin/libbuiltin_getname.c

@@ -0,0 +1,92 @@
+/****************************************************************************
+ * binfmt/libbuiltin/libbuiltin_getname.c
+ *
+ * Originally by:
+ *
+ *   Copyright (C) 2011 Uros Platise. All rights reserved.
+ *   Author: Uros Platise <uros.platise@isotel.eu>
+ *
+ * With subsequent updates, modifications, and general maintenance by:
+ *
+ *   Copyright (C) 2012-2013 Gregory Nutt.  All rights reserved.
+ *   Author: Gregory Nutt <gnutt@nuttx.org>
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ *
+ * 1. Redistributions of source code must retain the above copyright
+ *    notice, this list of conditions and the following disclaimer.
+ * 2. Redistributions in binary form must reproduce the above copyright
+ *    notice, this list of conditions and the following disclaimer in
+ *    the documentation and/or other materials provided with the
+ *    distribution.
+ * 3. Neither the name NuttX nor the names of its contributors may be
+ *    used to endorse or promote products derived from this software
+ *    without specific prior written permission.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+ * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+ * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS
+ * FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE
+ * COPYRIGHT OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT,
+ * INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING,
+ * BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS
+ * OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED
+ * AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
+ * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN
+ * ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
+ * POSSIBILITY OF SUCH DAMAGE.
+ *
+ ****************************************************************************/
+
+/****************************************************************************
+ * Included Files
+ ****************************************************************************/
+
+#include <nuttx/config.h>
+
+#include <nuttx/binfmt/builtin.h>
+
+/****************************************************************************
+ * Pre-processor Definitions
+ ****************************************************************************/
+
+/****************************************************************************
+ * Private Types
+ ****************************************************************************/
+
+/****************************************************************************
+ * Private Function Prototypes
+ ****************************************************************************/
+
+/****************************************************************************
+ * Private Data
+ ****************************************************************************/
+
+/****************************************************************************
+ * Private Functions
+ ****************************************************************************/
+
+/****************************************************************************
+ * Public Functions
+ ****************************************************************************/
+
+/****************************************************************************
+ * Name: builtin_getname
+ *
+ * Description:
+ *   Return the name of the application at index in the table of builtin
+ *   applications.
+ *
+ ****************************************************************************/
+
+FAR const char *builtin_getname(int index)
+{
+  if (index < 0 || index >= number_builtins())
+   {
+     return NULL;
+   }
+
+  return g_builtins[index].name;
+}

binfmt/libbuiltin/libbuiltin_isavail.c

@@ -1,5 +1,5 @@
 /****************************************************************************
- * binfmt/libbuiltin/libbuiltin_utils.c
+ * binfmt/libbuiltin/libbuiltin_isavail.c
  *
  * Originally by:
  *
@@ -76,25 +76,6 @@
  * Public Functions
  ****************************************************************************/
 
-/****************************************************************************
- * Name: builtin_getname
- *
- * Description:
- *   Return the name of the application at index in the table of builtin
- *   applications.
- *
- ****************************************************************************/
-
-FAR const char *builtin_getname(int index)
-{
-  if (index < 0 || index >= number_builtins())
-   {
-     return NULL;
-   }
-
-  return g_builtins[index].name;
-}
-
 /****************************************************************************
  * Name: builtin_isavail
  *