Messages in this thread | | | Subject | Re: [PATCH] tsc-disable_B9 | From | john stultz <> | Date | 09 Aug 2002 13:58:01 -0700 |
| |
On Fri, 2002-08-09 at 11:49, george anzinger wrote:
> An interesting approach, however, could you take a look at > the high-res-timers patch (see signature). In that code (in
I skimmed the patch, and I like alot of the cleanups and tweaks you have make to arch/i386/kernel/time.c. As the number of time sources are growing with the TSC, PIT, and now the cyclone counter and ACPI timer (also, doesn't MS have some sort of high res performance counter standard they're pushing for in hardware?), I'm thinking of rearranging and renaming some of the code to better allow these changes. The current do_slow/fast_gettimeoffset plus in-function 'if(use_tsc)' switches are getting to be messy with the new additions.
However, this may be a 2.5 issue, so before I take my 2.4 fixes and port them to a new infrastructure, I'd like it if someone chimed in and said "do it" or "skip it" for 2.4.
> the TSC version), we use the TSC to update jiffies and > _sub_jiffie (which is TSC counts into the next jiffie). We > also want to be able to "grab" a new TSC and figure the time > quickly, without updating either jiffies or _sub_jiffie. > Your approach would, I think, mean that both jiffies and > _sub_jiffie would be per cpu values, not impossible, but, > well, hard.
I need to look over you code a bit more. It looks interesting though.
> On the other hand, the high-res-timers patch also allows one > to use the ACPI pm timer, and ignore TSC completely :)
Yea, having another source of time is always nice, as long as it works reliably. After all this time related work I'm not sure if I'm going to start wearing 3-4 watches or just screw it and wear none. :)
thank for the pointer and feedback. -john
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |