Removes __proc_set_allcores()
authorBarret Rhoden <brho@cs.berkeley.edu>
Wed, 11 Jan 2012 22:28:50 +0000 (14:28 -0800)
committerBarret Rhoden <brho@cs.berkeley.edu>
Wed, 11 Jan 2012 22:29:33 +0000 (14:29 -0800)
Never was used, and I'm cleaning up the proc core management.

kern/include/process.h
kern/src/process.c

index ca64ae5..871c86f 100644 (file)
@@ -104,18 +104,9 @@ struct vcore *vcoreid2vcore(struct proc *p, uint32_t vcoreid);
  * shouldn't call these functions with parameters you are not sure about (like
  * an invalid corelist).  
  *
- * They also may cause an IPI to be sent to core it is called on.  If so, the
- * return value will be true.  Once you unlock (and enable interrupts) you will
- * be preempted, and usually lose your stack.  There is a helper to unlock and
- * handle the refcnt.
- *
  * WARNING: YOU MUST HOLD THE PROC_LOCK BEFORE CALLING THESE! */
 /* Gives process p the additional num cores listed in corelist */
 void __proc_give_cores(struct proc *SAFE p, uint32_t *pcorelist, size_t num);
-/* Makes process p's coremap look like corelist (add, remove, etc). Not used */
-void __proc_set_allcores(struct proc *SAFE p, uint32_t *pcorelist,
-                         size_t *num, amr_t message, TV(a0t) arg0,
-                         TV(a1t) arg1, TV(a2t) arg2);
 /* Takes from process p the num cores listed in corelist */
 void __proc_take_cores(struct proc *p, uint32_t *pcorelist, size_t num,
                        amr_t message, long arg0, long arg1, long arg2);
index 98a0eea..4aa0046 100644 (file)
@@ -1210,24 +1210,6 @@ void __proc_give_cores(struct proc *SAFE p, uint32_t *pcorelist, size_t num)
        p->resources[RES_CORES].amt_granted += num;
 }
 
-/* Makes process p's coremap look like pcorelist (add, remove, etc).  Caller
- * needs to know what cores are free after this call (removed, failed, etc).
- * This info will be returned via corelist and *num.  This will send message to
- * any cores that are getting removed.
- *
- * Before implementing this, we should probably think about when this will be
- * used.  Implies preempting for the message.  The more that I think about this,
- * the less I like it.  For now, don't use this, and think hard before
- * implementing it.
- *
- * WARNING: You must hold the proc_lock before calling this! */
-void __proc_set_allcores(struct proc *SAFE p, uint32_t *pcorelist,
-                         size_t *num, amr_t message,TV(a0t) arg0,
-                         TV(a1t) arg1, TV(a2t) arg2)
-{
-       panic("Set all cores not implemented.\n");
-}
-
 /* Helper for the take_cores calls: takes a specific vcore from p, optionally
  * sending the message (or just unmapping), gives the pcore to the idlecoremap.
  */