Re: LTTng wrapper cleanup

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel


[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux