diff options
| author | Sampo Savolainen <v2@iki.fi> | 2008-09-22 20:42:15 +0000 |
|---|---|---|
| committer | Sampo Savolainen <v2@iki.fi> | 2008-09-22 20:42:15 +0000 |
| commit | 5315c64acf24517289cf62feeefb7dd3e527a92c (patch) | |
| tree | bccd17be990b40fcff01e04d5b404646647b5de5 /libs | |
| parent | 8623302340b091fd65a5ad2ff5dc25915ea297ca (diff) | |
Fixed overflow issue. Code originally meant to truncate the 64 bit integer did not work as assembler doesn't "cast" variables like C does. get_cycles() was passing the memory address of a 32 bit unsigned int to the x86 call which required 64 bits of storage space. The extra 32 bits corrupted whatever was on the stack next to the variable.
git-svn-id: svn://localhost/ardour2/branches/2.0-ongoing@3791 d708f5d6-7413-0410-9779-e7cbd77b26cf
Diffstat (limited to 'libs')
| -rw-r--r-- | libs/ardour/ardour/cycles.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/libs/ardour/ardour/cycles.h b/libs/ardour/ardour/cycles.h index 0d1ac154dd..a18ea23ba1 100644 --- a/libs/ardour/ardour/cycles.h +++ b/libs/ardour/ardour/cycles.h @@ -48,10 +48,10 @@ extern cycles_t cacheflush_time; static inline cycles_t get_cycles (void) { - uint32_t long ret; + cycles_t long ret; rdtscll(ret); - return ret; + return ret & 0xffffffff; } #elif defined(__powerpc__) |