Re: error: 'subbuf_idx' undeclared after enablingCONFIG_LTT_RELAY_LOCKED=y

From: Mathieu Desnoyers
Date: Mon Feb 22 2010 - 10:35:36 EST


* naresh kamboju (naresh.kernel@xxxxxxxxx) wrote:
> Hi,
>
> I have noticed below error after enabling CONFIG_LTT_RELAY_LOCKED=y

Hi Naresh,

A few LTTng versions after the one you are working on, I flagged
LTT_RELAY_LOCKED and LTT_RELAY_IRQOFF as "CONFIG_BROKEN", because I did
not want to go through the trouble of duplicating the changes I did to
the lockless ring buffer to the two other flavors. I recommend using the
lockless ring buffer anyway.

So I recommend that you simply do the same with your old LTTng version
and flag LTT_RELAY_LOCKED and LTT_RELAY_IRQOFF as "CONFIG_BROKEN", as I
don't see the point in spending time on these at the moment.

Thanks,

Mathieu

>
> CC ltt/ltt-relay-locked.o
> In file included from ltt/ltt-relay-locked.c:56:
> ltt/ltt-relay-locked.h: In function 'ltt_check_deliver':
> ltt/ltt-relay-locked.h:167: (first use in this function)
> ltt/ltt-relay-locked.h:167: error: (Each undeclared identifier is
> reported only once
> ltt/ltt-relay-locked.h:167: error: for each function it appears in.)
>
> After my work around I have found error is caused from the
> lttng-relay-locked-writer-use-noref-flag.patch
>
> Below patch will fix the compilation issues.
> Please review this patch.
>
> ---
> ltt/ltt-relay-locked.h | 2 1 + 1 - 0 !
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> Index: b/ltt/ltt-relay-locked.h
> ===================================================================
> --- a/ltt/ltt-relay-locked.h
> +++ b/ltt/ltt-relay-locked.h
> @@ -164,7 +164,7 @@ static __inline__ void ltt_check_deliver
> */
> ltt_set_noref_flag(rchan, buf, idx);
> #ifdef CONFIG_LTT_VMCORE
> - ltt_buf->commit_seq[subbuf_idx] = commit_count;
> + ltt_buf->commit_seq[idx] = commit_count;
> #endif
> ltt_buf->wakeup_readers = 1;
> }
>
> For your information LTTng configuration. dotconfig
>
>
> CONFIG_LTT=y
> CONFIG_LTT_FILTER=m
> CONFIG_HAVE_LTT_DUMP_TABLES=y
> CONFIG_LTT_RELAY_ALLOC=y
> # CONFIG_LTT_RELAY_LOCKLESS is not set
> # CONFIG_LTT_RELAY_IRQOFF is not set
> CONFIG_LTT_RELAY_LOCKED=y
> CONFIG_LTT_SERIALIZE=y
> CONFIG_LTT_FAST_SERIALIZE=y
> CONFIG_LTT_TRACEPROBES=m
> CONFIG_LTT_TRACE_CONTROL=y
> CONFIG_LTT_TRACER=y
> CONFIG_LTT_ALIGNMENT=y
> CONFIG_LTT_CHECK_ARCH_EFFICIENT_UNALIGNED_ACCESS=y
> # CONFIG_LTT_DEBUG_EVENT_SIZE is not set
> CONFIG_LTT_USERSPACE_EVENT=m
> CONFIG_LTT_VMCORE=y
> CONFIG_LTT_STATEDUMP=m
> CONFIG_LTT_KPROBES=m
>
> Please review the patch and provide your comments.
>
> Kernel: 2.6.29-RT
> Lttng: 0.158
> ARCH: ARM
>
> Best regards
> Naresh Kamboju
>
> _______________________________________________
> ltt-dev mailing list
> ltt-dev@xxxxxxxxxxxxxxxxxxxxx
> http://lists.casi.polymtl.ca/cgi-bin/mailman/listinfo/ltt-dev
>

--
Mathieu Desnoyers
OpenPGP key fingerprint: 8CD5 52C3 8E3C 4140 715F BA06 3F25 A8FE 3BAE 9A68
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/