Hi,
On Thu, 16 Feb 2006, john stultz wrote:
> I'll admit I'm slow, but since it has taken me a number of weeks to sort
> out exactly the details of what is being done in your implementation,
> and I *still* have bugs after re-implementing it, I'd not claim your
> code is simple. The potential to be very precise and efficient: yes, but
> not so trivial to follow.
Wow, I didn't expect it to be that difficult, I'm sorry about that. :)
Next time I'll split it apart into the basic parts, so it should be easier
to read and follow.
> (This is why I cringe at the idea of trying to implement it for each
> clock like you're prototype suggested.)
I didn't suggest that, the function itself is already quite generic and
could be called like "clock_update(cycles);". What I'm suggesting is to
make it a template function, so that one create a optimized version based
on some parameters (e.g. it's possible to optimize some parts away by
making them constant).
That's not really necessary for the first version, only that a clock can
call the "clock_update(cycles);" directly from it's interrupt handler,
later it can be replaced with an optimized version.
> Maybe when I send out the patch you can suggest improvements to the
> comments or other ways to better clarify the code as you suggested
> above.
Now I'm really curious. :)
> I'd be very much open to it, although I haven't seen any further updates
> to the code since I mailed you some feedback on them. Have you had a
> chance to follow up on those?
Not yet, but it would only minor updates (mostly documenting it better and
cleanups as you suggested), the basic stuff is still the same.
> > In the end the simplification of my patches should also
> > make your patches simpler, as it precalculates as much as possible and
> > reduces the work done in the fast paths. It would avoid a lot of extra
> > work, which you currently do.
>
> Well, I'm still cautious, since it still has some dependencies on HZ
> (see equation below), which I'm trying to avoid.
There is no real dependency on HZ, it's just that the synchronisations
steps and incremental updates are done in fixed intervals. The interval
could easily be independent of HZ.
bye, Roman
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
