diff options
| author | Stafford Horne <shorne@gmail.com> | 2017-01-12 00:16:05 +0900 |
|---|---|---|
| committer | Shuah Khan <shuahkh@osg.samsung.com> | 2017-01-19 10:30:34 -0700 |
| commit | aff985fd07cf785fbd1544194f8fae08333e523e (patch) | |
| tree | e0eac4aad5df475e6ffe743abca8a48e0eb0ac85 /tools/testing | |
| parent | 4da39ceb269cee9f96815a54f97931a7e59f9e7f (diff) | |
selftests/intel_pstate: Fix warning on loop index overflow
The build was showing the warning:
aperf.c:60:27: warning: iteration 2147483647 invokes undefined behavior
[-Waggressive-loop-optimizations]
for (i=0; i<0x8fffffff; i++) {
This change sets i, cpu and fd to unsigned int as they should not need
to be signed.
Cc: Prarit Bhargava <prarit@redhat.com>
Signed-off-by: Stafford Horne <shorne@gmail.com>
Signed-off-by: Shuah Khan <shuahkh@osg.samsung.com>
Diffstat (limited to 'tools/testing')
| -rw-r--r-- | tools/testing/selftests/intel_pstate/aperf.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/testing/selftests/intel_pstate/aperf.c b/tools/testing/selftests/intel_pstate/aperf.c index 6046e183f4ad..cd72f3dc83e9 100644 --- a/tools/testing/selftests/intel_pstate/aperf.c +++ b/tools/testing/selftests/intel_pstate/aperf.c @@ -14,7 +14,7 @@ void usage(char *name) { } int main(int argc, char **argv) { - int i, cpu, fd; + unsigned int i, cpu, fd; char msr_file_name[64]; long long tsc, old_tsc, new_tsc; long long aperf, old_aperf, new_aperf; |