Merge pull request #520 from Ziemas/threadman-fixes2

uyjulian · web-flow · commit 6f160a090f13 · 2024-01-29T12:15:57.000-06:00
threadman: Fix a typo and a bug
diff --git a/iop/system/threadman/src/thbase.c b/iop/system/threadman/src/thbase.c
@@ -763,7 +763,7 @@ int WakeupThread(int thid)
         return thread_start(thread, state);
     }
 
-    thread->wakeup_count--;
+    thread->wakeup_count++;
 
     CpuResumeIntr(state);
 
diff --git a/iop/system/threadman/src/thmsgbx.c b/iop/system/threadman/src/thmsgbx.c
@@ -15,7 +15,7 @@ int CreateMbx(iop_mbx_t *mbx_param)
         return KE_ILLEGAL_CONTEXT;
     }
 
-    if (mbx_param->attr & ~(MBA_MSFIFO | MBA_THPRI | MBA_MSFIFO | MBA_MSPRI)) {
+    if (mbx_param->attr & ~(MBA_THFIFO | MBA_THPRI | MBA_MSFIFO | MBA_MSPRI)) {
         return KE_ILLEGAL_ATTR;
     }
 

Original file line number	Diff line number	Diff line change
`@@ -763,7 +763,7 @@ int WakeupThread(int thid)`
`763`	`763`	`return thread_start(thread, state);`
`764`	`764`	`}`
`765`	`765`
`766`		`- thread->wakeup_count--;`
	`766`	`+ thread->wakeup_count++;`
`767`	`767`
`768`	`768`	`CpuResumeIntr(state);`
`769`	`769`
Original file line number	Diff line number	Diff line change
`@@ -15,7 +15,7 @@ int CreateMbx(iop_mbx_t *mbx_param)`
`15`	`15`	`return KE_ILLEGAL_CONTEXT;`
`16`	`16`	`}`
`17`	`17`
`18`		`- if (mbx_param->attr & ~(MBA_MSFIFO \| MBA_THPRI \| MBA_MSFIFO \| MBA_MSPRI)) {`
	`18`	`+ if (mbx_param->attr & ~(MBA_THFIFO \| MBA_THPRI \| MBA_MSFIFO \| MBA_MSPRI)) {`
`19`	`19`	`return KE_ILLEGAL_ATTR;`
`20`	`20`	`}`
`21`	`21`