From mboxrd@z Thu Jan  1 00:00:00 1970
From: compudj@krystal.dyndns.org (Mathieu Desnoyers)
Date: Mon, 14 Feb 2011 14:56:22 -0500
Subject: [ltt-dev] lttv unable to execute textDump on MIPS multicore
	target
In-Reply-To: <4D59876C.5090409@caviumnetworks.com>
References: <AANLkTinh8O5aupe2Fz8oRaiOZjwurP8k9y_dnojC44=Y@mail.gmail.com>
	<BLU0-SMTP48F24DF0C4B7D6CF250B2696EF0@phx.gbl>
	<AANLkTim2AfBPSMyCruU9dsRefZ+t4NPtMkRgcYLax9ei@mail.gmail.com>
	<BLU0-SMTP19709C87A494AE8507C77296EF0@phx.gbl>
	<AANLkTik=LoXHOgsKxB_jkd1xa8dhfXeXdUkbj_e=wLTh@mail.gmail.com>
	<BLU0-SMTP86B1C8A867AFBF486A0DEB96D00@phx.gbl>
	<4D596AF7.9030607@caviumnetworks.com>
	<BLU0-SMTP381C9DC048B0AF55E0C00596D00@phx.gbl>
	<4D59876C.5090409@caviumnetworks.com>
Message-ID: <BLU0-SMTP53AB418CB6B7155C837A7B96D00@phx.gbl>

* David Daney (ddaney at caviumnetworks.com) wrote:
> On 02/14/2011 11:43 AM, Mathieu Desnoyers wrote:
>
>> Index: linux-2.6-lttng/arch/mips/include/asm/timex.h
>> ===================================================================
>> --- linux-2.6-lttng.orig/arch/mips/include/asm/timex.h
>> +++ linux-2.6-lttng/arch/mips/include/asm/timex.h
>> @@ -42,9 +42,32 @@ extern unsigned int mips_hpt_frequency;
>>    * will result in the timer interrupt getting lost.
>>    */
>>
>> +#ifdef CONFIG_HAVE_GET_CYCLES
>> +typedef unsigned int cycles_t;
>> +
>> +static inline cycles_t get_cycles(void)
>> +{
>> +	return read_c0_cvmcount();
>> +}
>
>
> c0_cvmcount is not a standard register.  It is an implementation defined  
> extension to mips64r2, therefore its use should be gated by:
>
> #ifdef CONFIG_CPU_CAVIUM_OCTEON

It actually is, thanks to the following Kconfig dep:

+config HAVE_GET_CYCLES
+       def_bool y
+       depends on CPU_CAVIUM_OCTEON
[...]

But maybe is that a bit misleading for the future ?

Thanks,

Mathieu


>
> David Daney
>

-- 
Mathieu Desnoyers
Operating System Efficiency R&D Consultant
EfficiOS Inc.
http://www.efficios.com