[FFmpeg-devel] [PATCH] read_time() for SPARC

Måns Rullgård mans
Thu Sep 9 23:01:44 CEST 2010


Michael Niedermayer <michaelni at gmx.at> writes:

> On Wed, Sep 08, 2010 at 01:08:57AM +0400, Michael Kostylev wrote:
>> 
>> The attached patch provides access to a 63-bit tick counter which is
>> available on the v9 cpus in the manner compatible with the v8+ abi.
>> 
>> Michael
>
>>  sparc/timer.h |   41 +++++++++++++++++++++++++++++++++++++++++
>>  timer.h       |    2 ++
>>  2 files changed, 43 insertions(+)
>> 555a36c6bda504671f232b6e9433e4f429a2e3fd  sparc-timer.diff
>
> if the only issue left is the sometimes unneeded stx instruction
> then i approve the patch with this. If there are other issues they
> of course have to be fixed first

Apparently the ifdef test in the patch is not reliable.

> but 1 unneeded instruction in code that is used primarely for
> debuging and otherwise is in no way speed critical and on a rare
> arch is not worth the work&discussion here

If two or three extra lines can make it optimal, I think that should
be done.

-- 
M?ns Rullg?rd
mans at mansr.com



More information about the ffmpeg-devel mailing list