3c59x with linuxppc

Bogdan Costescu Bogdan.Costescu@IWR.Uni-Heidelberg.De
Tue May 9 08:28:04 2000


On Tue, 9 May 2000, Andreas Tobler wrote:

> What about when I put in here:
> -------->   next_tick = media_tbl[dev->if_port].wait
> ...
> And what about the approach from Don, in his N,L series he sets the
> next_tick at init to 60*HZ?

That's what I also noticed 30 seconds after sending my reply...

Generally speaking, you can have 2 approaches:
a) you set the variable at the beginning and only modify it if it's really
necessary (so you may have "treated cases" < "total cases")
b) you initialize the variable to 0 (zero) and then be sure that you have
"treated cases" = "total cases" (and you can check at the end of the
function for 0 which means that there are still untreated cases).

The current source of vortex_timer seems to be somehow at the middle.
Probably, if you set next_tick = 60*HZ at the beginning, you will be more
or less in the a) case; having a default non-zero value means that
you don't need to check it and that was the reason why the 'if' line was
removed...
AFAIK, the code was gradually modified from Mr. Becker's 0.99H, so either:
1. somebody didn't understand what the initialization to 60*HZ was meant
for
2. somebody was trying to implement case b), but forgot to treat all
cases
3. something else that I don't understand. 8-/

Sincerely,

Bogdan Costescu

IWR - Interdisziplinaeres Zentrum fuer Wissenschaftliches Rechnen
Universitaet Heidelberg, INF 368, D-69120 Heidelberg, GERMANY
Telephone: +49 6221 54 8869, Telefax: +49 6221 54 8868
E-mail: Bogdan.Costescu@IWR.Uni-Heidelberg.De

-------------------------------------------------------------------
To unsubscribe send a message body containing "unsubscribe"
to linux-vortex-request@beowulf.org