|
Rockbox mail archive
Subject: Re: cvs: firmware thread.c,1.27,1.28
From: Linus Nielsen Feltzing (linus_at_haxx.se)
Date: 2004-05-01
- Next message: [IDC]Dragon: "Re: cvs: firmware thread.c,1.27,1.28"
- Previous message: [IDC]Dragon: "Re: cvs: apps plugin.c,1.18,1.19"
- In reply to: [IDC]Dragon: "Re: cvs: firmware thread.c,1.27,1.28"
- Next in thread: [IDC]Dragon: "Re: cvs: firmware thread.c,1.27,1.28"
- Reply: [IDC]Dragon: "Re: cvs: firmware thread.c,1.27,1.28"
- Messages sorted by:
[ date ]
[ thread ]
[ subject ]
[ author ]
[ attachment ]
[IDC]Dragon wrote:
> Good point, it could be the last one, which is just becoming out ouf range
> if the number of threads is reduced. But since switch_thread() would wrap it
> to zero anyhow, I think it's not necessary.
See below.
>>b) is pointing to the currently running thread
>
>
> If not, it would temporarily be not strictly round-robin, but is this a
> problem?
It is indeed a problem. The current_thread variable determines where to
store the context of the current thread. If you don't adjust it,
switch_thread() will store the current context in the wrong place.
> I think this is not necessary. A thread can delete itself, with the next
> yield it just wouldn't be called again.
See above.
Linus
_______________________________________________
http://cool.haxx.se/mailman/listinfo/rockbox
- Next message: [IDC]Dragon: "Re: cvs: firmware thread.c,1.27,1.28"
- Previous message: [IDC]Dragon: "Re: cvs: apps plugin.c,1.18,1.19"
- In reply to: [IDC]Dragon: "Re: cvs: firmware thread.c,1.27,1.28"
- Next in thread: [IDC]Dragon: "Re: cvs: firmware thread.c,1.27,1.28"
- Reply: [IDC]Dragon: "Re: cvs: firmware thread.c,1.27,1.28"
- Messages sorted by:
[ date ]
[ thread ]
[ subject ]
[ author ]
[ attachment ]
Page was last modified "Mar 9 2008" The Rockbox Crew
|