The attached patch is sufficient to make coreinfo run on a machine without TSC, however while trying to get FILO to run I've realised this involves more changes than I originally expected.
cpu_khz gets exposed as a global, so it's going to be very difficult to support CPUs without TSC, without changing the mechanisms some payloads use to do timing. If you don't have TSC, then cpu_khz is largely irrelevant, but FILO accesses it directly.
I think this global needs to be removed and some other mechanism introduced for handling timeouts.
regards, Biff.
On Fri, Feb 5, 2010 at 3:12 AM, bifferos bifferos@yahoo.co.uk wrote:
cpu_khz gets exposed as a global, so it's going to be very difficult to support CPUs without TSC, without changing the mechanisms some payloads use to do timing. If you don't have TSC, then cpu_khz is largely irrelevant, but FILO accesses it directly.
There's no way to just make it relevant? Can't a non-TSC CPU support something meaningful in that variable? (I don't know,just asking)
ron
On 2/5/10 5:23 PM, ron minnich wrote:
On Fri, Feb 5, 2010 at 3:12 AM, bifferos bifferos@yahoo.co.uk wrote:
cpu_khz gets exposed as a global, so it's going to be very difficult to support CPUs without TSC, without changing the mechanisms some payloads use to do timing. If you don't have TSC, then cpu_khz is largely irrelevant, but FILO accesses it directly.
There's no way to just make it relevant? Can't a non-TSC CPU support something meaningful in that variable? (I don't know,just asking)
FILO uses this for its timeouts in various places. If there is a better way to do what FILO does, FILO should be fixed
--- On Fri, 5/2/10, Stefan Reinauer stepan@coresystems.de wrote:
There's no way to just make it relevant? Can't a
non-TSC CPU support
something meaningful in that variable? (I don't know,just asking)
FILO uses this for its timeouts in various places. If there is a better way to do what FILO does, FILO should be fixed
It's not hard, at least for the user-visible delays - I'll try to post a patch for that too.
regards, Biff.