Sebastian Smolorz wrote:
> Hi Gilles,
>
> Gilles Chanteperdrix wrote:
>
>>Sebastian Smolorz wrote:
>>
>>>Hi,
>>>
>>>here comes a preliminary version of the missing ARM-parts for the I-pipe
>>>tracer. It's not yet complete, some features for the tracing of IRQs-off
>>>times are missing, but will soon be added. Besides of that, the tracer
>>>seems to work. ;-)
>>
>>That's nice.
>>
>>
>>>The following traces were recorded on a S3C2440. It would be nice if
>>>someone could have a look at them and determine if everything looks sane
>>>so far.
>>
>>The patch looks ok, a few remarks however:
>>- you include the full head.S in the patch whereas you most likely just
>>added the empty mcount;
>
>
> I diff'ed over a ipipe cvs repository where head.S does not exist. Of course,
> if I would create the patch over a ipipe'd linux kernel the empty mcount
> would be the only change in head.S
>
>
>>- I see no patch of any Kconfig, did you forget them ?
>
>
> As far as I remember no change in any Kconfig file was needed.
>
>
>>- the arm_return_address function seem to rely on frame pointers, so
>>should not the tracer option require the CONFIG_FRAME_POINTER option to
>>be enabled ?
>
>
> You mean the existing contents of kernel/ipipe/Kconfig.debug are not enough?
>
> ...
> config IPIPE_TRACE
> bool "Latency tracing"
> depends on IPIPE_DEBUG
> select FRAME_POINTER
> ...
Sorry for the noise, I was not looking at Adeos cvs.
--
Gilles Chanteperdrix
_______________________________________________
Adeos-main mailing list
[email protected]
https://mail.gna.org/listinfo/adeos-main
