<feed xmlns='http://www.w3.org/2005/Atom'>
<title>user/sven/linux.git/include/linux/kfifo.h, branch v3.12.6</title>
<subtitle>Linux Kernel
</subtitle>
<id>https://git.stealer.net/cgit.cgi/user/sven/linux.git/atom?h=v3.12.6</id>
<link rel='self' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/atom?h=v3.12.6'/>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/'/>
<updated>2010-10-28T01:03:18Z</updated>
<entry>
<title>kfifo: fix kfifo_alloc() to return a signed int value</title>
<updated>2010-10-28T01:03:18Z</updated>
<author>
<name>Stefani Seibold</name>
<email>stefani@seibold.net</email>
</author>
<published>2010-10-27T22:34:50Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=144ecf310eb52d9df607b9b7eeb096743e232a96'/>
<id>urn:sha1:144ecf310eb52d9df607b9b7eeb096743e232a96</id>
<content type='text'>
Add a new __kfifo_int_must_check_helper() helper function, which is needed
for kfifo_alloc() to return the right signed integer value.

The origin __kfifo_must_check_helper() helper was renamed into
__kfifo_uint_must_check_helper() to show the sign which is expected and
returned.

(And revert the temporary disabling of __kfifo_must_check_helper())

Signed-off-by: Stefani Seibold &lt;stefani@seibold.net&gt;
Acked-by: Randy Dunlap &lt;randy.dunlap@oracle.com&gt;
Signed-off-by: Andrew Morton &lt;akpm@linux-foundation.org&gt;
Signed-off-by: Linus Torvalds &lt;torvalds@linux-foundation.org&gt;
</content>
</entry>
<entry>
<title>kfifo: disable __kfifo_must_check_helper()</title>
<updated>2010-10-26T23:52:04Z</updated>
<author>
<name>Andrew Morton</name>
<email>akpm@linux-foundation.org</email>
</author>
<published>2010-10-26T21:21:19Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=52c5171214ff3327961d0ce0db7e8d2ce55004fd'/>
<id>urn:sha1:52c5171214ff3327961d0ce0db7e8d2ce55004fd</id>
<content type='text'>
This helper is wrong: it coerces signed values into unsigned ones, so code
such as

	if (kfifo_alloc(...) &lt; 0) {
		error
	}

will fail to detect the error.

So let's disable __kfifo_must_check_helper() for 2.6.36.

Cc: Randy Dunlap &lt;randy.dunlap@oracle.com&gt;
Cc: Stefani Seibold &lt;stefani@seibold.net&gt;
Cc: &lt;stable@kernel.org&gt;
Signed-off-by: Andrew Morton &lt;akpm@linux-foundation.org&gt;
Signed-off-by: Linus Torvalds &lt;torvalds@linux-foundation.org&gt;
</content>
</entry>
<entry>
<title>kfifo: add parenthesis for macro parameter reference</title>
<updated>2010-09-10T01:57:22Z</updated>
<author>
<name>Huang Ying</name>
<email>ying.huang@intel.com</email>
</author>
<published>2010-09-09T23:37:26Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=e0bf1024b36be90da241af3c2767311e055b612c'/>
<id>urn:sha1:e0bf1024b36be90da241af3c2767311e055b612c</id>
<content type='text'>
Some macro parameter references inside typeof() operator are not enclosed
with parenthesis.  It should be safer to add them.

Signed-off-by: Huang Ying &lt;ying.huang@intel.com&gt;
Acked-by: Stefani Seibold &lt;stefani@seibold.net&gt;
Signed-off-by: Andrew Morton &lt;akpm@linux-foundation.org&gt;
Signed-off-by: Linus Torvalds &lt;torvalds@linux-foundation.org&gt;
</content>
</entry>
<entry>
<title>kfifo: implement missing __kfifo_skip_r()</title>
<updated>2010-08-20T16:34:54Z</updated>
<author>
<name>Andrea Righi</name>
<email>arighi@develer.com</email>
</author>
<published>2010-08-19T21:13:27Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=b35de43b31040828f83046f40fd34ba33146409d'/>
<id>urn:sha1:b35de43b31040828f83046f40fd34ba33146409d</id>
<content type='text'>
kfifo_skip() is currently broken, due to the missing of the internal
helper function.  Add it.

Signed-off-by: Andrea Righi &lt;arighi@develer.com&gt;
Cc: Greg KH &lt;greg@kroah.com&gt;
Acked-by: Stefani Seibold &lt;stefani@seibold.net&gt;
Signed-off-by: Andrew Morton &lt;akpm@linux-foundation.org&gt;
Signed-off-by: Linus Torvalds &lt;torvalds@linux-foundation.org&gt;
</content>
</entry>
<entry>
<title>kfifo: replace the old non generic API</title>
<updated>2010-08-11T15:59:23Z</updated>
<author>
<name>Stefani Seibold</name>
<email>stefani@seibold.net</email>
</author>
<published>2010-08-11T01:03:38Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=2e956fb320568cc70861761483e2f0e2db75fd66'/>
<id>urn:sha1:2e956fb320568cc70861761483e2f0e2db75fd66</id>
<content type='text'>
Simply replace the whole kfifo.c and kfifo.h files with the new generic
version and fix the kerneldoc API template file.

Signed-off-by: Stefani Seibold &lt;stefani@seibold.net&gt;
Cc: Greg KH &lt;greg@kroah.com&gt;
Signed-off-by: Andrew Morton &lt;akpm@linux-foundation.org&gt;
Signed-off-by: Linus Torvalds &lt;torvalds@linux-foundation.org&gt;
</content>
</entry>
<entry>
<title>kfifo: kfifo_is_{full,empty} should return bools, not ints</title>
<updated>2010-08-11T15:59:23Z</updated>
<author>
<name>Robert P. J. Day</name>
<email>rpjday@crashcourse.ca</email>
</author>
<published>2010-08-11T01:03:34Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=ad9c7ed0685406fe3cd7f0749b82bf433a39dd9c'/>
<id>urn:sha1:ad9c7ed0685406fe3cd7f0749b82bf433a39dd9c</id>
<content type='text'>
For consistency with other kfifo routines, return bool, not int.

Signed-off-by: Robert P. J. Day &lt;rpjday@crashcourse.ca&gt;
Cc: Stefani Seibold &lt;stefani@seibold.net&gt;
Signed-off-by: Andrew Morton &lt;akpm@linux-foundation.org&gt;
Signed-off-by: Linus Torvalds &lt;torvalds@linux-foundation.org&gt;
</content>
</entry>
<entry>
<title>Merge branch 'master' into for-next</title>
<updated>2010-04-23T00:08:44Z</updated>
<author>
<name>Jiri Kosina</name>
<email>jkosina@suse.cz</email>
</author>
<published>2010-04-23T00:08:44Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=6c9468e9eb1252eaefd94ce7f06e1be9b0b641b1'/>
<id>urn:sha1:6c9468e9eb1252eaefd94ce7f06e1be9b0b641b1</id>
<content type='text'>
</content>
</entry>
<entry>
<title>include/linux/kfifo.h: fix INIT_KFIFO()</title>
<updated>2010-04-07T15:38:02Z</updated>
<author>
<name>David Härdeman</name>
<email>david@hardeman.nu</email>
</author>
<published>2010-04-06T21:34:43Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=530cd330dc3865e3107304a6e84fdc332aa72f7d'/>
<id>urn:sha1:530cd330dc3865e3107304a6e84fdc332aa72f7d</id>
<content type='text'>
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 &lt;david@hardeman.nu&gt;
Acked-by: Stefani Seibold &lt;stefani@seibold.net&gt;
Cc: &lt;stable@kernel.org&gt;
Signed-off-by: Andrew Morton &lt;akpm@linux-foundation.org&gt;
Signed-off-by: Linus Torvalds &lt;torvalds@linux-foundation.org&gt;
</content>
</entry>
<entry>
<title>kfifo: fix KFIFO_INIT in include/linux/kfifo.h</title>
<updated>2010-03-24T23:31:20Z</updated>
<author>
<name>David Härdeman</name>
<email>david@hardeman.nu</email>
</author>
<published>2010-03-23T20:35:22Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=4c87684d32e8f95715d53039dcd2d998dc63d1eb'/>
<id>urn:sha1:4c87684d32e8f95715d53039dcd2d998dc63d1eb</id>
<content type='text'>
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 &lt;david@hardeman.nu&gt;
Acked-by: Stefani Seibold &lt;stefani@seibold.net&gt;
Cc: &lt;stable@kernel.org&gt;
Signed-off-by: Andrew Morton &lt;akpm@linux-foundation.org&gt;
Signed-off-by: Linus Torvalds &lt;torvalds@linux-foundation.org&gt;
</content>
</entry>
<entry>
<title>kfifo: fix kerneldoc for kfifo_avail and kfifo_in_locked</title>
<updated>2010-03-18T00:24:38Z</updated>
<author>
<name>Viral Mehta</name>
<email>Viral.Mehta@lntinfotech.com</email>
</author>
<published>2010-03-17T14:01:17Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=a8d89807373f95f3f7637e3987710a4500a7a857'/>
<id>urn:sha1:a8d89807373f95f3f7637e3987710a4500a7a857</id>
<content type='text'>
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 &lt;viral.mehta@lntinfotech.com&gt;
Signed-off-by: Jiri Kosina &lt;jkosina@suse.cz&gt;
</content>
</entry>
</feed>
