Re: Thread support plan with initial patch

[Mikael Djurfeldt]

NIIBE Yutaka <address@hidden> writes:

> NIIBE Yutaka wrote:
>  >  > It actually looks like you're moving the thread switching code to a
>  >  > place where scm_ints_disabled can have a different value than now.
>  >  > I don't have time to verify this, but if that is true, it could cause
>  >  > strange things to happen to the current COOP threads...
>  > 
>  > Yes, I moved it.  I dont' think it causes bad behavior, but I agree to
>  > verify and double-check it.
> 
> I've checked.  Current code is buggy.  Even after my change, the code
> is still buggy.

[...]

> I think that the switching at REALLOW_INTS should be removed.
> 
> After we remove the switching at REALLOW_INTS, semantics is clearer
> (we only switch at ALLOW_INTS).  Then, it's ovious that my change has
> no effect, and it looks better IMHO.

I'll look at this ASAP (which may be during the weekend).

Thanks,
/mdj