[ltt-dev] lttv unable to execute textDump on MIPS multicore target

[nareshgbhat@gmail.com (Naresh Bhat)]

Hi Mathieu/David,

Thank you very much. The patch is applied without any issues. But facing the
linking errors. Is there any configuration missing in Kconfig ?
*
Thanks and Regards
-Naresh Bhat*
*
Linking error log:*

========================================================================================================
..................
......................
..............................
AR      arch/mips/lib/lib.a
  LD      vmlinux.o
  MODPOST vmlinux.o
  GEN     .version
  CHK     include/linux/compile.h
  UPD     include/linux/compile.h
  CC      init/version.o
  LD      init/built-in.o
  LD      .tmp_vmlinux1
ltt/built-in.o: In function `ltt_trace_destroy':
(.text.ltt_trace_destroy+0x6c): undefined reference to `put_synthetic_tsc'
ltt/built-in.o: In function `ltt_trace_alloc':
(.text.ltt_trace_alloc+0x5c): undefined reference to `get_synthetic_tsc'
ltt/built-in.o: In function `ltt_trace_alloc':
(.text.ltt_trace_alloc+0x164): undefined reference to
`trace_clock_read_synthetic_tsc'
ltt/built-in.o: In function `ltt_trace_alloc':
(.text.ltt_trace_alloc+0x454): undefined reference to `put_synthetic_tsc'
ltt/built-in.o: In function `ltt_trace_alloc':
(.text.ltt_trace_alloc+0x4b4): undefined reference to `put_synthetic_tsc'
ltt/built-in.o: In function `ltt_trace_alloc':
(.text.ltt_trace_alloc+0x4f8): undefined reference to `put_synthetic_tsc'
ltt/built-in.o: In function `ltt_reserve_slot_lockless_slow':
(.text.ltt_reserve_slot_lockless_slow+0xa8): undefined reference to
`trace_clock_read_synthetic_tsc'
ltt/built-in.o: In function `ltt_force_switch_lockless_slow':
(.text.ltt_force_switch_lockless_slow+0x44): undefined reference to
`trace_clock_read_synthetic_tsc'
ltt/built-in.o: In function `_ltt_specialized_trace':
(.text._ltt_specialized_trace+0x1cc): undefined reference to
`trace_clock_read_synthetic_tsc'
make: *** [.tmp_vmlinux1] Error 1
#
==============================================================================================
===


On Tue, Feb 15, 2011 at 9:02 AM, Mathieu Desnoyers <
compudj at krystal.dyndns.org> wrote:

> Is the following patch less confusing ?
>
>
> MIPS: octeon fix get_cycles
>
> Make sure get_cycles(), used by kernel/time/tsc-sync.c TSC synchronicity
> checker, works fine on octeon by using the full 64-bits.
>
> Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers at efficios.com>
> ---
>  arch/mips/Kconfig             |   11 +++++++++--
>  arch/mips/include/asm/timex.h |   31 ++++++++++++++++++++++++++++++-
>  2 files changed, 39 insertions(+), 3 deletions(-)
>
> Index: linux-2.6-lttng/arch/mips/Kconfig
> ===================================================================
> --- linux-2.6-lttng.orig/arch/mips/Kconfig
> +++ linux-2.6-lttng/arch/mips/Kconfig
> @@ -1949,9 +1949,16 @@ config CPU_R4400_WORKAROUNDS
>  config HAVE_GET_CYCLES_32
>         def_bool y
>         depends on !CPU_R4400_WORKAROUNDS
> +       depends on !CPU_CAVIUM_OCTEON
>        select HAVE_TRACE_CLOCK
> -       select HAVE_TRACE_CLOCK_32_TO_64 if (!CPU_CAVIUM_OCTEON)
> -       select HAVE_UNSYNCHRONIZED_TSC if (!CPU_CAVIUM_OCTEON)
> +       select HAVE_TRACE_CLOCK_32_TO_64
> +       select HAVE_UNSYNCHRONIZED_TSC
> +
> +config HAVE_GET_CYCLES
> +       def_bool y
> +       depends on CPU_CAVIUM_OCTEON
> +       select HAVE_TRACE_CLOCK
> +       select HAVE_UNSYNCHRONIZED_TSC
>
>  #
>  # - Highmem only makes sense for the 32-bit kernel.
> 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,36 @@ extern unsigned int mips_hpt_frequency;
>   * will result in the timer interrupt getting lost.
>  */
>
> +#ifdef CONFIG_HAVE_GET_CYCLES
> +# ifdef CONFIG_CPU_CAVIUM_OCTEON
> +typedef unsigned int cycles_t;
> +
> +static inline cycles_t get_cycles(void)
> +{
> +       return read_c0_cvmcount();
> +}
> +
> +static inline void get_cycles_barrier(void)
> +{
> +}
> +
> +static inline cycles_t get_cycles_rate(void)
> +{
> +       return mips_hpt_frequency;
> +}
> +
> +extern int test_tsc_synchronization(void);
> +extern int _tsc_is_sync;
> +static inline int tsc_is_sync(void)
> +{
> +       return _tsc_is_sync;
> +}
> +# else /* #ifdef CONFIG_CPU_CAVIUM_OCTEON */
> +#  error "64-bit get_cycles() supported only on Cavium Octeon MIPS
> architectures"
> +# endif /* #else #ifdef CONFIG_CPU_CAVIUM_OCTEON */
> +#elif defined(CONFIG_HAVE_GET_CYCLES_32)
>  typedef unsigned int cycles_t;
>
> -#ifdef CONFIG_HAVE_GET_CYCLES_32
>  static inline cycles_t get_cycles(void)
>  {
>         return read_c0_count();
> @@ -66,6 +93,8 @@ static inline int tsc_is_sync(void)
>         return _tsc_is_sync;
>  }
>  #else
> +typedef unsigned int cycles_t;
> +
>  static inline cycles_t get_cycles(void)
>  {
>         return 0;
> --
> Mathieu Desnoyers
> Operating System Efficiency R&D Consultant
> EfficiOS Inc.
> http://www.efficios.com
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.casi.polymtl.ca/pipermail/lttng-dev/attachments/20110215/1ee45571/attachment-0003.htm>