Fixed compiler warnings about signed/unsigned comparisons, unused and shadowed identifiers and use of void * arithmetic
Remove __attribute__((unused)) use for now.

Likely not portable enough. Would need better ifdef code first

Signed-off-by: Andi Kleen <ak@linux.intel.com>
Fixed unused argument in sink_peek using a wrapper macro, removing the need for __attribute__((unused))