* [lttng-dev] DEBUG_RCU vs RCU_DEBUG
@ 2015-04-20 8:07 Duncan Sands
2015-04-23 19:44 ` Mathieu Desnoyers
0 siblings, 1 reply; 3+ messages in thread
From: Duncan Sands @ 2015-04-20 8:07 UTC (permalink / raw)
I noticed that urcu-qsbr.h checks for RCU_DEBUG being defined while the rest of
userspace-rcu (0.8.6) checks DEBUG_RCU. Is this a mistake or a cunning plan?
Ciao, Duncan.
^ permalink raw reply [flat|nested] 3+ messages in thread
* [lttng-dev] DEBUG_RCU vs RCU_DEBUG
2015-04-20 8:07 [lttng-dev] DEBUG_RCU vs RCU_DEBUG Duncan Sands
@ 2015-04-23 19:44 ` Mathieu Desnoyers
2015-04-23 19:47 ` Mathieu Desnoyers
0 siblings, 1 reply; 3+ messages in thread
From: Mathieu Desnoyers @ 2015-04-23 19:44 UTC (permalink / raw)
----- Original Message -----
> I noticed that urcu-qsbr.h checks for RCU_DEBUG being defined while the rest
> of
> userspace-rcu (0.8.6) checks DEBUG_RCU. Is this a mistake or a cunning plan?
Good catch! Even urcu/static/urcu-qsbr.h has a ifdef based on
DEBUG_RCU.
Will rename the urcu-qsbr.[ch] RCU_DEBUG to DEBUG_RCU, and I will
define RCU_DEBUG if DEBUG_RCU is defined in qsbr (for backward compat).
Fixes pushed into master, stable-0.8 and stable-0.7.
Thanks!
Mathieu
>
> Ciao, Duncan.
>
> _______________________________________________
> lttng-dev mailing list
> lttng-dev at lists.lttng.org
> http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev
>
--
Mathieu Desnoyers
EfficiOS Inc.
http://www.efficios.com
^ permalink raw reply [flat|nested] 3+ messages in thread
* [lttng-dev] DEBUG_RCU vs RCU_DEBUG
2015-04-23 19:44 ` Mathieu Desnoyers
@ 2015-04-23 19:47 ` Mathieu Desnoyers
0 siblings, 0 replies; 3+ messages in thread
From: Mathieu Desnoyers @ 2015-04-23 19:47 UTC (permalink / raw)
----- Original Message -----
> ----- Original Message -----
> > I noticed that urcu-qsbr.h checks for RCU_DEBUG being defined while the
> > rest
> > of
> > userspace-rcu (0.8.6) checks DEBUG_RCU. Is this a mistake or a cunning
> > plan?
>
> Good catch! Even urcu/static/urcu-qsbr.h has a ifdef based on
> DEBUG_RCU.
>
> Will rename the urcu-qsbr.[ch] RCU_DEBUG to DEBUG_RCU, and I will
> define RCU_DEBUG if DEBUG_RCU is defined in qsbr (for backward compat).
I rather mean the opposite: we define DEBUG_RCU if RCU_DEBUG
is defined.
Thanks,
Mathieu
> Fixes pushed into master, stable-0.8 and stable-0.7.
>
> Thanks!
>
> Mathieu
>
> >
> > Ciao, Duncan.
> >
> > _______________________________________________
> > lttng-dev mailing list
> > lttng-dev at lists.lttng.org
> > http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev
> >
>
> --
> Mathieu Desnoyers
> EfficiOS Inc.
> http://www.efficios.com
>
> _______________________________________________
> lttng-dev mailing list
> lttng-dev at lists.lttng.org
> http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev
>
--
Mathieu Desnoyers
EfficiOS Inc.
http://www.efficios.com
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2015-04-23 19:47 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-04-20 8:07 [lttng-dev] DEBUG_RCU vs RCU_DEBUG Duncan Sands
2015-04-23 19:44 ` Mathieu Desnoyers
2015-04-23 19:47 ` Mathieu Desnoyers
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox