Ticket #344 (closed defect: wontfix)

Opened 12 years ago

Last modified 12 years ago

neo doesn't build on Jaunty

Reported by: jdreed Owned by:
Priority: normal Milestone: Upstream Utopia
Component: -- Keywords:
Cc: Fixed in version:
Upstream bug:

Description

In its sleeping code, neo uses the obsolete CLK_TCK constant. /usr/include/time.h and time.h(7posix) claim that CLOCKS_PER_SEC should be used instead, but go on to warn that CLOCKS_PER_SEC should not be assumed to be a compile time constant. CLOCKS_PER_SEC is required to be one million (1000000l) on XSI-compliant systems, and indeed is on both RHEL4 and Ubuntu. However, on RHEL4, CLK_TCK is 100, and is undefined on Ubuntu. Defining CLK_TCK as CLOCKS_PER_SEC makes neo sleep for significantly longer (1000 times longer, in fact) than it should between SNMP queries when getting device stats. Defining CLK_TCK as 100 allows neo to function correctly, but this is almost certainly the wrong answer.

I've reported this "upstream" to bug-ktools, but can't even reference a transaction number in the discuss meeting because it's private. Yay.

Change History

comment:1 Changed 12 years ago by jdreed

Bug report resent to kretch@… at kchen's suggestion.

comment:2 Changed 12 years ago by jdreed

kretch said he'll look at this at some point this coming week.

comment:3 Changed 12 years ago by jdreed

  • Status changed from new to closed
  • Resolution set to wontfix

This is not our problem, because:
-it's in a locker.
-it's dealt with for the RCCs
-neo will soon be useless anyway once all buildings are upgraded

Note: See TracTickets for help on using tickets.