On 11/20/24 06:20, Bart Van Assche wrote: > On 11/18/24 11:55 PM, Damien Le Moal wrote: >> I do not really see the point... Anyway, looks OK. > Hi Damien, > > One of the approaches I used to debug this patch series is to add > trace_printk() calls to track zwplug state changes. trace_printk() > reports the function name in front of the information in its argument > list. I noticed that the function name reported by trace_printk() is > incorrect for inlined functions. Hence this patch. Maybe add this explanation to the commit message ? It does make sense to no inline these function with tracing. > > Thanks, > > Bart. -- Damien Le Moal Western Digital Research