{"action":"opened","issue":{"url":"https://api.github.com/repos/projectNe10/Ne10/issues/129","repository_url":"https://api.github.com/repos/projectNe10/Ne10","labels_url":"https://api.github.com/repos/projectNe10/Ne10/issues/129/labels{/name}","comments_url":"https://api.github.com/repos/projectNe10/Ne10/issues/129/comments","events_url":"https://api.github.com/repos/projectNe10/Ne10/issues/129/events","html_url":"https://github.com/projectNe10/Ne10/issues/129","id":137291273,"number":129,"title":"Ne10 DSP Performance Test - Incorrect times reported","user":{"login":"benjmarshall","id":6310728,"avatar_url":"https://avatars.githubusercontent.com/u/6310728?v=3","gravatar_id":"","url":"https://api.github.com/users/benjmarshall","html_url":"https://github.com/benjmarshall","followers_url":"https://api.github.com/users/benjmarshall/followers","following_url":"https://api.github.com/users/benjmarshall/following{/other_user}","gists_url":"https://api.github.com/users/benjmarshall/gists{/gist_id}","starred_url":"https://api.github.com/users/benjmarshall/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/benjmarshall/subscriptions","organizations_url":"https://api.github.com/users/benjmarshall/orgs","repos_url":"https://api.github.com/users/benjmarshall/repos","events_url":"https://api.github.com/users/benjmarshall/events{/privacy}","received_events_url":"https://api.github.com/users/benjmarshall/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":0,"created_at":"2016-02-29T15:48:59Z","updated_at":"2016-02-29T15:48:59Z","closed_at":null,"body":"All performance tests in the DSP modules use a macro 'GET_TIME' to wrap function tests in timer code. This macro uses the function 'GetTickCount()' defined in 'seatest.c' to return the time at the start and end of the test. 'GetTickCount()' uses to the standard 'gettimeofday' function available in time.h to return the current time in **microseconds**.\r\n\r\nThe code used throughout the performance tests to log to the console specify that results are reported in **milliseconds**, however they all use the returned value from 'GetTickCount' directly which is in microseconds, thus reporting all times incorrectly.\r\n\r\nThe performance tests should be modified to specify that times are reported in microseconds to resolve this.\r\n\r\nAdditionally I would suggest the function 'GetTickCount' be replaced or renamed to more clearly show that the the returned value is a time and not a number of clock ticks. This was a major source of confusion when attempting to debug why the reported times seemed to be incorrect."}}