linux-toradex.git/include/linux/kfifo.h, branch v2.6.35.3

Merge branch 'master' into for-next

2010-04-23T00:08:44+00:00

include/linux/kfifo.h: fix INIT_KFIFO()

2010-04-07T15:38:02+00:00

DECLARE_KFIFO creates a union with a struct kfifo and a buffer array with
size [size + sizeof(struct kfifo)].

INIT_KFIFO then sets the buffer pointer in struct kfifo to point to the
beginning of the buffer array which means that the first call to kfifo_in
will overwrite members of the struct kfifo.

Signed-off-by: David Härdeman 
Acked-by: Stefani Seibold 
Cc: 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

kfifo: fix KFIFO_INIT in include/linux/kfifo.h

2010-03-24T23:31:20+00:00

include/linux/kfifo.h first defines and then undefines __kfifo_initializer
which is used by INIT_KFIFO (which is also a macro, so building a module
which uses INIT_KFIFO will fail).

Signed-off-by: David Härdeman 
Acked-by: Stefani Seibold 
Cc: 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

kfifo: fix kerneldoc for kfifo_avail and kfifo_in_locked

2010-03-18T00:24:38+00:00

Function argument is not having "len" anywhere;
"len" is later used in kfifo_{in|out}.

So here, it would be appropriate to say,
it copies @n bytes and not @len

Signed-off-by: Viral Mehta 
Signed-off-by: Jiri Kosina

kfifo: Don't use integer as NULL pointer

2010-02-16T23:11:08+00:00

This patch fixes following sparse warnings:

include/linux/kfifo.h:127:25: warning: Using plain integer as NULL pointer
kernel/kfifo.c:83:21: warning: Using plain integer as NULL pointer

Signed-off-by: Anton Vorontsov 
Acked-by: Stefani Seibold 
Signed-off-by: Greg Kroah-Hartman

kfifo: document everywhere that size has to be power of two

2010-01-16T20:15:38+00:00

On my first try using them I missed that the fifos need to be power of
two, resulting in a runtime bug.  Document that requirement everywhere
(and fix one grammar bug)

Signed-off-by: Andi Kleen 
Acked-by: Stefani Seibold 
Cc: Roland Dreier 
Cc: Dmitry Torokhov 
Cc: Andy Walls 
Cc: Vikram Dhillon 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

kfifo: add kfifo_initialized

2010-01-16T20:15:38+00:00

Simple inline that checks if kfifo_init() has been executed on a fifo.

This is useful for walking all per CPU fifos, when some of them might not
have been brought up yet.

Signed-off-by: Andi Kleen 
Acked-by: Stefani Seibold 
Cc: Roland Dreier 
Cc: Dmitry Torokhov 
Cc: Andy Walls 
Cc: Vikram Dhillon 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

kfifo: add kfifo_out_peek

2010-01-16T20:15:38+00:00

In some upcoming code it's useful to peek into a FIFO without permanentely
removing data.  This patch implements a new kfifo_out_peek() to do this.

Signed-off-by: Andi Kleen 
Acked-by: Stefani Seibold 
Cc: Roland Dreier 
Cc: Dmitry Torokhov 
Cc: Andy Walls 
Cc: Vikram Dhillon 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

kfifo: sanitize *_user error handling

2010-01-16T20:15:38+00:00

Right now for kfifo_*_user it's not easily possible to distingush between
a user copy failing and the FIFO not containing enough data.  The problem
is that both conditions are multiplexed into the same return code.

Avoid this by moving the "copy length" into a separate output parameter
and only return 0/-EFAULT in the main return value.

I didn't fully adapt the weird "record" variants, those seem
to be unused anyways and were rather messy (should they be just removed?)

I would appreciate some double checking if I did all the conversions
correctly.

Signed-off-by: Andi Kleen 
Cc: Stefani Seibold 
Cc: Roland Dreier 
Cc: Dmitry Torokhov 
Cc: Andy Walls 
Cc: Vikram Dhillon 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

kfifo: use void * pointers for user buffers

2010-01-16T20:15:38+00:00

The pointers to user buffers are currently unsigned char *, which requires
a lot of casting in the caller for any non-char typed buffers.  Use void *
instead.

Signed-off-by: Andi Kleen 
Acked-by: Stefani Seibold 
Cc: Roland Dreier 
Cc: Dmitry Torokhov 
Cc: Andy Walls 
Cc: Vikram Dhillon 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds