bugCERTI - Bugs: bug #23667, Intel x86 assembly instruction in...

 
 

bug #23667: Intel x86 assembly instruction in libCERTI/TSCClock.hh:53

Submitter:  Petr Gotthard <gotthardp>
Submitted:  Sat 21 Jun 2008 05:31:10 PM UTC
   
 
Category:  CERTI Severity:  * 3 - Normal
Item Group:  Software error Status:  Wont Fix
Privacy:  Public Assigned to:  erk
Open/Closed:  Closed Release:  * CVS
Reproducibility:  Every Time Fixed Release:  * Unknown
* Mandatory Fields

Add a New Comment Rich Markup
   

Mon 23 Jun 2008 01:32:33 PM UTC, comment #2: 

Just some more information about
ClockXXX class hierarchy.

The idea of the Clock class hierarchy is to
offer best-available clock for CERTI internal timing
on various platform.

CMake build will check for available HW support and
add to the build specific class (e.g. TSCClock.hh)
if it is available.

Clock is the base abstract class,
PosixClock use Posix clock API
TSCClock use IA-32 Time Stamp Counter

others clock may be useful
see
https://savannah.nongnu.org/task/?7929

I leave the bug open until you (or someone else )confirm
TSCClock is efficiently excluded from a Sparc/Solaris build.

Eric NOULARD <erk>
Group administrator
Sun 22 Jun 2008 07:16:34 PM UTC, comment #1: 

This file should be excluded from the compilation
by the CMake script on unsupported platform:

You may found this in the
certi/CMakeLists.txt

SET(TSC_ENABLED_PROCESSOR_REGEX ".*x86_64.*|.*i686.*")
IF (CMAKE_SYSTEM_PROCESSOR MATCHES "${TSC_ENABLED_PROCESSOR_REGEX}")
    SET(HAVE_TSC_CLOCK 1)
ENDIF (CMAKE_SYSTEM_PROCESSOR MATCHES "${TSC_ENABLED_PROCESSOR_REGEX}")

Then in
libCERTI/CMakeLists.txt
IF (HAVE_TSC_CLOCK)
    SET(CERTI_SUPPORT_SRCS
           ${CERTI_SUPPORT_SRCS}
           TSCClock.cc TSCClock.hh)             
ENDIF(HAVE_TSC_CLOCK)

Or may be the file is included in a file it shouldnt?

Did you compile using CMake?

Eric NOULARD <erk>
Group administrator
Sat 21 Jun 2008 05:31:10 PM UTC, original submission:  

I guess CERTI does not run on Sparc. There is an Intel x86 assembly instruction in libCERTI/TSCClock.hh:53.

Could you possibly get rid of it? (Just this one, no other.)

Petr Gotthard <gotthardp>
Group Member

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

No files currently attached

 

Digest:
   task dependencies.

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by erk (Posted a comment)
  • -email is unavailable- added by gotthardp (Submitted the item)
  •  

    Follow 4 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2008-07-02 erk Open/ClosedOpen Closed
    2008-06-23 erk Dependencies- Depends on task #7929
    2008-06-23 erk StatusNone Wont Fix
        Assigned toNone erk

    Back to the top

    Powered by Savane 3.13-f8d8.
    Corresponding source code