[lttng-dev] LTTng wrapper cleanup
Greg KH
greg at kroah.com
Wed Nov 30 13:47:17 EST 2011
On Wed, Nov 30, 2011 at 01:34:13PM -0500, Mathieu Desnoyers wrote:
>
> Hi Greg,
>
> Here is the cleanup of the LTTng wrappers that used kallsyms to lookup
> some kernel symbols. I introduce the missing symbols in the mainline
> kernel, trying to follow the local exports already in place when it
> comes to choosing between EXPORT_SYMBOL and EXPORT_SYMBOL_GPL. I also
> removed wrappers that deal with API changes across kernel versions (for
> older kernels). Only few wrapper files are left at this point (TODO),
> but those are less repulsive than those that used kallsyms:
>
> - inline_memcpy.h: ensures that inline_memcpy is declared on all
> architectures.
> - poll.h: see TODO file, "Features" section.
> - trace-clock.h: see TODO file, "Features" section.
> - ringbuffer: those are internal to the driver. They were there to make
> it easier to switch between the ringbuffer deployed in my own kernel
> tree and the module-specific ringbuffer.
>
> The 3 first patches are small cleanups recommended to me by a static
> checker.
>
> Diffstat:
>
> b/drivers/staging/lttng/lib/ringbuffer/ring_buffer_frontend.c | 6
> b/drivers/staging/lttng/lib/ringbuffer/ring_buffer_mmap.c | 3
> drivers/staging/lttng/lib/ringbuffer/ring_buffer_mmap.c | 3
Hm, try 'diffstat -p1' next time please, for a cleaner "real" version of
what is going on here.
greg k-h
More information about the lttng-dev
mailing list