Thanks Will, please apply!<div><br></div><div>I don't think we should keep the two methods with the same name though, too much confusion. We should probably rename them to enterUncooperativeCodeAtLevel and enterUncooperativeCodeWithSP.</div>
<div><br></div><div>Nicolas<br><br><div class="gmail_quote">On Mon, Oct 24, 2011 at 12:57 AM, Will Dietz <span dir="ltr"><<a href="mailto:wdietz2@illinois.edu">wdietz2@illinois.edu</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
Small patch, and since in all cases of the code level is zero, the<br>
type size reduction seems easiest fix.<br>
<br>
(Leaves ability to have non-zero, but small, levels in the future)<br>
<br>
Without this patch, 'unsigned' is the same as 'word_t' on 32bit so the two<br>
prototypes conflict.<br>
<br>
Attached, and copied below.<br>
<br>
~Will<br>
<br>
>From a6772df268b2a4f23a3a83b6adfee57f38405dba Mon Sep 17 00:00:00 2001<br>
From: Will Dietz <<a href="mailto:w@wdtz.org">w@wdtz.org</a>><br>
Date: Fri, 14 Oct 2011 03:02:49 -0500<br>
Subject: [PATCH] Change enterUncooperativeCode prototype to fix build on<br>
 32bit.<br>
<br>
---<br>
 include/mvm/Threads/Thread.h      |    2 +-<br>
 lib/Mvm/CommonThread/ctthread.cpp |    2 +-<br>
 2 files changed, 2 insertions(+), 2 deletions(-)<br>
<br>
diff --git a/include/mvm/Threads/Thread.h b/include/mvm/Threads/Thread.h<br>
index 73c63f8..03be9cd 100644<br>
--- a/include/mvm/Threads/Thread.h<br>
+++ b/include/mvm/Threads/Thread.h<br>
@@ -208,7 +208,7 @@ public:<br>
  void joinRVBeforeEnter();<br>
  void joinRVAfterLeave(word_t savedSP);<br>
<br>
-  void enterUncooperativeCode(unsigned level = 0) __attribute__ ((noinline));<br>
+  void enterUncooperativeCode(uint16_t level = 0) __attribute__ ((noinline));<br>
  void enterUncooperativeCode(word_t SP);<br>
  void leaveUncooperativeCode();<br>
  word_t waitOnSP();<br>
diff --git a/lib/Mvm/CommonThread/ctthread.cpp<br>
b/lib/Mvm/CommonThread/ctthread.cpp<br>
index 739ad2f..1c3b1fd 100644<br>
--- a/lib/Mvm/CommonThread/ctthread.cpp<br>
+++ b/lib/Mvm/CommonThread/ctthread.cpp<br>
@@ -179,7 +179,7 @@ void Thread::scanStack(word_t closure) {<br>
  }<br>
 }<br>
<br>
-void Thread::enterUncooperativeCode(unsigned level) {<br>
+void Thread::enterUncooperativeCode(uint16_t level) {<br>
  if (isMvmThread()) {<br>
    if (!inRV) {<br>
      assert(!lastSP && "SP already set when entering uncooperative code");<br>
<font color="#888888">--<br>
1.7.5.1<br>
</font><br>_______________________________________________<br>
vmkit-commits mailing list<br>
<a href="mailto:vmkit-commits@cs.uiuc.edu">vmkit-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/vmkit-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/vmkit-commits</a><br>
<br></blockquote></div><br></div>