Re: [paulmck-rcu:dev.2026.09.03a 43/55] drivers/usb/mtu3/./mtu3_trace.h:239:27: error: cannot take the address of an rvalue of type 'dma_addr_t' (aka 'unsigned long long')

From: Paul E. McKenney

Date: Thu Sep 10 2026 - 12:32:19 EST

On Thu, Sep 10, 2026 at 10:03:48PM +0800, kernel test robot wrote:
> tree: https://git.kernel.org/pub/scm/linux/kernel/git/paulmck/linux-rcu.git dev.2026.09.03a
> head: 6c5c244f4b239bc4d284e58a2f79a143648557cb
> commit: afda2321e6e93a5bcc28bfb4717fbc3a1768446c [43/55] EXP drivers/usb/mtu3: Work around mtu3_log_ep double-indirection issue
> config: riscv-allyesconfig (https://download.01.org/0day-ci/archive/20260910/202609101926.cc5w6bhk-lkp@xxxxxxxxx/config)
> compiler: clang version 24.0.0git (https://github.com/llvm/llvm-project c1872e7fd3307679a2da53f9d83de0ccf234b4ad)
> reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260910/202609101926.cc5w6bhk-lkp@xxxxxxxxx/reproduce)
>
> If you fix the issue in a separate patch/commit (i.e. not just a new version of
> the same patch/commit), kindly add following tags
> | Reported-by: kernel test robot <lkp@xxxxxxxxx>
> | Closes: https://lore.kernel.org/oe-kbuild-all/202609101926.cc5w6bhk-lkp@xxxxxxxxx/

Good catch, but this has been reverted to the previous form.

Nevertheless, thank you for your testing efforts!

Thanx, Paul

> Note: the paulmck-rcu/dev.2026.09.03a HEAD 6c5c244f4b239bc4d284e58a2f79a143648557cb builds fine.
> It only hurts bisectability.
>
> All errors (new ones prefixed by >>):
>
> In file included from drivers/usb/mtu3/mtu3_trace.c:12:
> In file included from drivers/usb/mtu3/mtu3_trace.h:279:
> In file included from include/trace/define_trace.h:132:
> In file included from include/trace/trace_events.h:489:
> drivers/usb/mtu3/./mtu3_trace.h:239:41: error: member reference type 'struct mtu3_gpd_ring' is not a pointer; did you mean to use '.'?
> 239 | __entry->gpd_ring_dma = &mep->gpd_ring->dma;
> | ~~~~~~~~~~~~~^~
> | .
> include/trace/stages/stage6_event_callback.h:133:33: note: expanded from macro 'TP_fast_assign'
> 133 | #define TP_fast_assign(args...) args
> | ^~~~
> include/trace/trace_events.h:456:16: note: expanded from macro 'DECLARE_EVENT_CLASS'
> 456 | PARAMS(assign), PARAMS(print)) \
> | ^~~~~~
> include/linux/tracepoint.h:161:25: note: expanded from macro 'PARAMS'
> 161 | #define PARAMS(args...) args
> | ^~~~
> include/trace/trace_events.h:448:4: note: expanded from macro '\
> __DECLARE_EVENT_CLASS'
> 448 | { assign; } \
> | ^~~~~~
> In file included from drivers/usb/mtu3/mtu3_trace.c:12:
> In file included from drivers/usb/mtu3/mtu3_trace.h:279:
> In file included from include/trace/define_trace.h:132:
> In file included from include/trace/trace_events.h:489:
> >> drivers/usb/mtu3/./mtu3_trace.h:239:27: error: cannot take the address of an rvalue of type 'dma_addr_t' (aka 'unsigned long long')
> 239 | __entry->gpd_ring_dma = &mep->gpd_ring->dma;
> | ^~~~~~~~~~~~~~~~~~~
> include/trace/stages/stage6_event_callback.h:133:33: note: expanded from macro 'TP_fast_assign'
> 133 | #define TP_fast_assign(args...) args
> | ^~~~
> include/trace/trace_events.h:456:16: note: expanded from macro 'DECLARE_EVENT_CLASS'
> 456 | PARAMS(assign), PARAMS(print)) \
> | ^~~~~~
> include/linux/tracepoint.h:161:25: note: expanded from macro 'PARAMS'
> 161 | #define PARAMS(args...) args
> | ^~~~
> include/trace/trace_events.h:448:4: note: expanded from macro '\
> __DECLARE_EVENT_CLASS'
> 448 | { assign; } \
> | ^~~~~~
> In file included from drivers/usb/mtu3/mtu3_trace.c:12:
> In file included from drivers/usb/mtu3/mtu3_trace.h:279:
> In file included from include/trace/define_trace.h:133:
> In file included from include/trace/perf.h:110:
> drivers/usb/mtu3/./mtu3_trace.h:239:41: error: member reference type 'struct mtu3_gpd_ring' is not a pointer; did you mean to use '.'?
> 239 | __entry->gpd_ring_dma = &mep->gpd_ring->dma;
> | ~~~~~~~~~~~~~^~
> | .
> include/trace/stages/stage6_event_callback.h:133:33: note: expanded from macro 'TP_fast_assign'
> 133 | #define TP_fast_assign(args...) args
> | ^~~~
> include/trace/perf.h:67:16: note: expanded from macro 'DECLARE_EVENT_CLASS'
> 67 | PARAMS(assign), PARAMS(print)) \
> | ^~~~~~
> include/linux/tracepoint.h:161:25: note: expanded from macro 'PARAMS'
> 161 | #define PARAMS(args...) args
> | ^~~~
> include/trace/perf.h:51:4: note: expanded from macro '\
> __DECLARE_EVENT_CLASS'
> 51 | { assign; } \
> | ^~~~~~
> In file included from drivers/usb/mtu3/mtu3_trace.c:12:
> In file included from drivers/usb/mtu3/mtu3_trace.h:279:
> In file included from include/trace/define_trace.h:133:
> In file included from include/trace/perf.h:110:
> >> drivers/usb/mtu3/./mtu3_trace.h:239:27: error: cannot take the address of an rvalue of type 'dma_addr_t' (aka 'unsigned long long')
> 239 | __entry->gpd_ring_dma = &mep->gpd_ring->dma;
> | ^~~~~~~~~~~~~~~~~~~
> include/trace/stages/stage6_event_callback.h:133:33: note: expanded from macro 'TP_fast_assign'
> 133 | #define TP_fast_assign(args...) args
> | ^~~~
> include/trace/perf.h:67:16: note: expanded from macro 'DECLARE_EVENT_CLASS'
> 67 | PARAMS(assign), PARAMS(print)) \
> | ^~~~~~
> include/linux/tracepoint.h:161:25: note: expanded from macro 'PARAMS'
> 161 | #define PARAMS(args...) args
> | ^~~~
> include/trace/perf.h:51:4: note: expanded from macro '\
> __DECLARE_EVENT_CLASS'
> 51 | { assign; } \
> | ^~~~~~
> 4 errors generated.
>
>
> vim +239 drivers/usb/mtu3/./mtu3_trace.h
>
> 213
> 214 DECLARE_EVENT_CLASS(mtu3_log_ep,
> 215 TP_PROTO(struct mtu3_ep *mep),
> 216 TP_ARGS(mep),
> 217 TP_STRUCT__entry(
> 218 __string(name, mep->name)
> 219 __field(unsigned int, type)
> 220 __field(unsigned int, slot)
> 221 __field(unsigned int, maxp)
> 222 __field(unsigned int, mult)
> 223 __field(unsigned int, maxburst)
> 224 __field(unsigned int, flags)
> 225 __field(unsigned int, direction)
> 226 __field(struct mtu3_gpd_ring *, gpd_ring)
> 227 __field(dma_addr_t *, gpd_ring_dma)
> 228 ),
> 229 TP_fast_assign(
> 230 __assign_str(name);
> 231 __entry->type = mep->type;
> 232 __entry->slot = mep->slot;
> 233 __entry->maxp = mep->ep.maxpacket;
> 234 __entry->mult = mep->ep.mult;
> 235 __entry->maxburst = mep->ep.maxburst;
> 236 __entry->flags = mep->flags;
> 237 __entry->direction = mep->is_in;
> 238 __entry->gpd_ring = &mep->gpd_ring;
> > 239 __entry->gpd_ring_dma = &mep->gpd_ring->dma;
> 240 ),
> 241 TP_printk("%s: type %s maxp %d slot %d mult %d burst %d ring %p/%pad flags %c:%c%c%c:%c",
> 242 __get_str(name), usb_ep_type_string(__entry->type),
> 243 __entry->maxp, __entry->slot,
> 244 __entry->mult, __entry->maxburst,
> 245 __entry->gpd_ring, __entry->gpd_ring_dma,
> 246 __entry->flags & MTU3_EP_ENABLED ? 'E' : 'e',
> 247 __entry->flags & MTU3_EP_STALL ? 'S' : 's',
> 248 __entry->flags & MTU3_EP_WEDGE ? 'W' : 'w',
> 249 __entry->flags & MTU3_EP_BUSY ? 'B' : 'b',
> 250 __entry->direction ? '<' : '>'
> 251 )
> 252 );
> 253
>
> --
> 0-DAY CI Kernel Test Service
> https://github.com/intel/lkp-tests/wiki