<feed xmlns='http://www.w3.org/2005/Atom'>
<title>linux-toradex.git/net/irda, branch v3.0.44</title>
<subtitle>Linux kernel for Apalis and Colibri modules</subtitle>
<link rel='alternate' type='text/html' href='https://git.toradex.cn/cgit/linux-toradex.git/'/>
<entry>
<title>irda: iriap: Use seperate lockdep class for irias_objects-&gt;hb_spinlock</title>
<updated>2011-06-07T00:00:35+00:00</updated>
<author>
<name>David S. Miller</name>
<email>davem@davemloft.net</email>
</author>
<published>2011-06-07T00:00:35+00:00</published>
<link rel='alternate' type='text/html' href='https://git.toradex.cn/cgit/linux-toradex.git/commit/?id=79b3891587741dfac72cdfead1f2764b56a567b0'/>
<id>79b3891587741dfac72cdfead1f2764b56a567b0</id>
<content type='text'>
The SEQ output functions grab the obj-&gt;attrib-&gt;hb_spinlock lock of
sub-objects found in the hash traversal.  These locks are in a different
realm than the one used for the irias_objects hash table itself.

So put the latter into it's own lockdep class.

Reported-by: Dave Jones &lt;davej@redhat.com&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
The SEQ output functions grab the obj-&gt;attrib-&gt;hb_spinlock lock of
sub-objects found in the hash traversal.  These locks are in a different
realm than the one used for the irias_objects hash table itself.

So put the latter into it's own lockdep class.

Reported-by: Dave Jones &lt;davej@redhat.com&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>irda: Fix error propagation in ircomm_lmp_connect_response()</title>
<updated>2011-05-19T22:58:39+00:00</updated>
<author>
<name>David S. Miller</name>
<email>davem@davemloft.net</email>
</author>
<published>2011-05-19T22:58:39+00:00</published>
<link rel='alternate' type='text/html' href='https://git.toradex.cn/cgit/linux-toradex.git/commit/?id=8ce6eb1a4a4bc46820f23e927377d386d3ec5404'/>
<id>8ce6eb1a4a4bc46820f23e927377d386d3ec5404</id>
<content type='text'>
The variable 'ret' is set but unused, and this pointed out that
errors from irlmp_connect_response() are not propagated to the
caller.

Note that this is currently academic since irlmp_connect_response()
always returns 0. :-)

Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
The variable 'ret' is set but unused, and this pointed out that
errors from irlmp_connect_response() are not propagated to the
caller.

Note that this is currently academic since irlmp_connect_response()
always returns 0. :-)

Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>irda: Kill set but unused variable 'bytes' in irlan_check_command_param()</title>
<updated>2011-05-19T22:53:58+00:00</updated>
<author>
<name>David S. Miller</name>
<email>davem@davemloft.net</email>
</author>
<published>2011-05-19T22:53:58+00:00</published>
<link rel='alternate' type='text/html' href='https://git.toradex.cn/cgit/linux-toradex.git/commit/?id=ad6376434c945495ea8c4082ad8a392cc58c3406'/>
<id>ad6376434c945495ea8c4082ad8a392cc58c3406</id>
<content type='text'>
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>irda: Kill set but unused variable 'clen' in ircomm_connect_indication()</title>
<updated>2011-05-19T22:52:59+00:00</updated>
<author>
<name>David S. Miller</name>
<email>davem@davemloft.net</email>
</author>
<published>2011-05-19T22:52:59+00:00</published>
<link rel='alternate' type='text/html' href='https://git.toradex.cn/cgit/linux-toradex.git/commit/?id=bfad6ef6d29e776472b3ead1f5bc920c5410ed2c'/>
<id>bfad6ef6d29e776472b3ead1f5bc920c5410ed2c</id>
<content type='text'>
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>irda: Kill set but unused vars 'saddr' and 'daddr' in irlan_provider_connect_indication()</title>
<updated>2011-05-19T22:46:41+00:00</updated>
<author>
<name>David S. Miller</name>
<email>davem@davemloft.net</email>
</author>
<published>2011-05-19T22:46:41+00:00</published>
<link rel='alternate' type='text/html' href='https://git.toradex.cn/cgit/linux-toradex.git/commit/?id=fa7479cf48103ee9261398034528495966bd7d2d'/>
<id>fa7479cf48103ee9261398034528495966bd7d2d</id>
<content type='text'>
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>net/irda/ircomm_tty.c: Use flip buffers to deliver data</title>
<updated>2011-05-12T22:02:51+00:00</updated>
<author>
<name>Amit Virdi</name>
<email>amit.virdi@st.com</email>
</author>
<published>2011-05-12T01:04:40+00:00</published>
<link rel='alternate' type='text/html' href='https://git.toradex.cn/cgit/linux-toradex.git/commit/?id=cbfd1526f6076ece92b4e2dcc0a2f1e89041b3bd'/>
<id>cbfd1526f6076ece92b4e2dcc0a2f1e89041b3bd</id>
<content type='text'>
use tty_insert_flip_string and tty_flip_buffer_push to deliver incoming data
packets from the IrDA device instead of delivering the packets directly to the
line discipline. Following later approach resulted in warning "Sleeping function
called from invalid context".

Signed-off-by: Amit Virdi &lt;amit.virdi@st.com&gt;
Acked-by: Alan Cox &lt;alan@linux.intel.com&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
use tty_insert_flip_string and tty_flip_buffer_push to deliver incoming data
packets from the IrDA device instead of delivering the packets directly to the
line discipline. Following later approach resulted in warning "Sleeping function
called from invalid context".

Signed-off-by: Amit Virdi &lt;amit.virdi@st.com&gt;
Acked-by: Alan Cox &lt;alan@linux.intel.com&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>Merge branch 'master' of master.kernel.org:/pub/scm/linux/kernel/git/davem/net-2.6</title>
<updated>2011-04-19T07:21:33+00:00</updated>
<author>
<name>David S. Miller</name>
<email>davem@davemloft.net</email>
</author>
<published>2011-04-19T07:21:33+00:00</published>
<link rel='alternate' type='text/html' href='https://git.toradex.cn/cgit/linux-toradex.git/commit/?id=e1943424e43974f85b82bb31eaf832823bf49ce7'/>
<id>e1943424e43974f85b82bb31eaf832823bf49ce7</id>
<content type='text'>
Conflicts:
	drivers/net/bnx2x/bnx2x_ethtool.c
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Conflicts:
	drivers/net/bnx2x/bnx2x_ethtool.c
</pre>
</div>
</content>
</entry>
<entry>
<title>irda: irproc: Fix set-but-unused variables.</title>
<updated>2011-04-17T23:59:50+00:00</updated>
<author>
<name>David S. Miller</name>
<email>davem@davemloft.net</email>
</author>
<published>2011-04-17T23:59:50+00:00</published>
<link rel='alternate' type='text/html' href='https://git.toradex.cn/cgit/linux-toradex.git/commit/?id=6385969b3297287e6259a012671be09c78c20620'/>
<id>6385969b3297287e6259a012671be09c78c20620</id>
<content type='text'>
The variable 'd' is set but unused in irda_proc_register().

Just kill it off.

Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
The variable 'd' is set but unused in irda_proc_register().

Just kill it off.

Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>irda: irlap_event: Fix set-but-unused variables.</title>
<updated>2011-04-17T23:59:09+00:00</updated>
<author>
<name>David S. Miller</name>
<email>davem@davemloft.net</email>
</author>
<published>2011-04-17T23:59:09+00:00</published>
<link rel='alternate' type='text/html' href='https://git.toradex.cn/cgit/linux-toradex.git/commit/?id=1627ea35cb10af2f71d38a82de6f6dfb910771ed'/>
<id>1627ea35cb10af2f71d38a82de6f6dfb910771ed</id>
<content type='text'>
The variable 'ret' is set but unused in irlap_state_sclose().

Just kill it off.

Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
The variable 'ret' is set but unused in irlap_state_sclose().

Just kill it off.

Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>irda: fix locking unbalance in irda_sendmsg</title>
<updated>2011-04-12T22:29:54+00:00</updated>
<author>
<name>Dave Jones</name>
<email>davej@redhat.com</email>
</author>
<published>2011-04-12T22:29:54+00:00</published>
<link rel='alternate' type='text/html' href='https://git.toradex.cn/cgit/linux-toradex.git/commit/?id=020318d0d2af51e0fd59ba654ede9b2171558720'/>
<id>020318d0d2af51e0fd59ba654ede9b2171558720</id>
<content type='text'>
5b40964eadea40509d353318d2c82e8b7bf5e8a5 ("irda: Remove BKL instances
from af_irda.c") introduced a path where we have a locking unbalance.
If we pass invalid flags, we unlock a socket we never locked,
resulting in this...

=====================================
[ BUG: bad unlock balance detected! ]
-------------------------------------
trinity/20101 is trying to release lock (sk_lock-AF_IRDA) at:
[&lt;ffffffffa057f001&gt;] irda_sendmsg+0x207/0x21d [irda]
but there are no more locks to release!

other info that might help us debug this:
no locks held by trinity/20101.

stack backtrace:
Pid: 20101, comm: trinity Not tainted 2.6.39-rc3+ #3
Call Trace:
 [&lt;ffffffffa057f001&gt;] ? irda_sendmsg+0x207/0x21d [irda]
 [&lt;ffffffff81085041&gt;] print_unlock_inbalance_bug+0xc7/0xd2
 [&lt;ffffffffa057f001&gt;] ? irda_sendmsg+0x207/0x21d [irda]
 [&lt;ffffffff81086aca&gt;] lock_release+0xcf/0x18e
 [&lt;ffffffff813ed190&gt;] release_sock+0x2d/0x155
 [&lt;ffffffffa057f001&gt;] irda_sendmsg+0x207/0x21d [irda]
 [&lt;ffffffff813e9f8c&gt;] __sock_sendmsg+0x69/0x75
 [&lt;ffffffff813ea105&gt;] sock_sendmsg+0xa1/0xb6
 [&lt;ffffffff81100ca3&gt;] ? might_fault+0x5c/0xac
 [&lt;ffffffff81086b7c&gt;] ? lock_release+0x181/0x18e
 [&lt;ffffffff81100cec&gt;] ? might_fault+0xa5/0xac
 [&lt;ffffffff81100ca3&gt;] ? might_fault+0x5c/0xac
 [&lt;ffffffff81133b94&gt;] ? fcheck_files+0xb9/0xf0
 [&lt;ffffffff813f387a&gt;] ? copy_from_user+0x2f/0x31
 [&lt;ffffffff813f3b70&gt;] ? verify_iovec+0x52/0xa6
 [&lt;ffffffff813eb4e3&gt;] sys_sendmsg+0x23a/0x2b8
 [&lt;ffffffff81086b7c&gt;] ? lock_release+0x181/0x18e
 [&lt;ffffffff810773c6&gt;] ? up_read+0x28/0x2c
 [&lt;ffffffff814bec3d&gt;] ? do_page_fault+0x360/0x3b4
 [&lt;ffffffff81087043&gt;] ? trace_hardirqs_on_caller+0x10b/0x12f
 [&lt;ffffffff810458aa&gt;] ? finish_task_switch+0xb2/0xe3
 [&lt;ffffffff8104583e&gt;] ? finish_task_switch+0x46/0xe3
 [&lt;ffffffff8108364a&gt;] ? trace_hardirqs_off_caller+0x33/0x90
 [&lt;ffffffff814bbaf9&gt;] ? retint_swapgs+0x13/0x1b
 [&lt;ffffffff81087043&gt;] ? trace_hardirqs_on_caller+0x10b/0x12f
 [&lt;ffffffff810a9dd3&gt;] ? audit_syscall_entry+0x11c/0x148
 [&lt;ffffffff8125609e&gt;] ? trace_hardirqs_on_thunk+0x3a/0x3f
 [&lt;ffffffff814c22c2&gt;] system_call_fastpath+0x16/0x1b

Signed-off-by: Dave Jones &lt;davej@redhat.com&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
5b40964eadea40509d353318d2c82e8b7bf5e8a5 ("irda: Remove BKL instances
from af_irda.c") introduced a path where we have a locking unbalance.
If we pass invalid flags, we unlock a socket we never locked,
resulting in this...

=====================================
[ BUG: bad unlock balance detected! ]
-------------------------------------
trinity/20101 is trying to release lock (sk_lock-AF_IRDA) at:
[&lt;ffffffffa057f001&gt;] irda_sendmsg+0x207/0x21d [irda]
but there are no more locks to release!

other info that might help us debug this:
no locks held by trinity/20101.

stack backtrace:
Pid: 20101, comm: trinity Not tainted 2.6.39-rc3+ #3
Call Trace:
 [&lt;ffffffffa057f001&gt;] ? irda_sendmsg+0x207/0x21d [irda]
 [&lt;ffffffff81085041&gt;] print_unlock_inbalance_bug+0xc7/0xd2
 [&lt;ffffffffa057f001&gt;] ? irda_sendmsg+0x207/0x21d [irda]
 [&lt;ffffffff81086aca&gt;] lock_release+0xcf/0x18e
 [&lt;ffffffff813ed190&gt;] release_sock+0x2d/0x155
 [&lt;ffffffffa057f001&gt;] irda_sendmsg+0x207/0x21d [irda]
 [&lt;ffffffff813e9f8c&gt;] __sock_sendmsg+0x69/0x75
 [&lt;ffffffff813ea105&gt;] sock_sendmsg+0xa1/0xb6
 [&lt;ffffffff81100ca3&gt;] ? might_fault+0x5c/0xac
 [&lt;ffffffff81086b7c&gt;] ? lock_release+0x181/0x18e
 [&lt;ffffffff81100cec&gt;] ? might_fault+0xa5/0xac
 [&lt;ffffffff81100ca3&gt;] ? might_fault+0x5c/0xac
 [&lt;ffffffff81133b94&gt;] ? fcheck_files+0xb9/0xf0
 [&lt;ffffffff813f387a&gt;] ? copy_from_user+0x2f/0x31
 [&lt;ffffffff813f3b70&gt;] ? verify_iovec+0x52/0xa6
 [&lt;ffffffff813eb4e3&gt;] sys_sendmsg+0x23a/0x2b8
 [&lt;ffffffff81086b7c&gt;] ? lock_release+0x181/0x18e
 [&lt;ffffffff810773c6&gt;] ? up_read+0x28/0x2c
 [&lt;ffffffff814bec3d&gt;] ? do_page_fault+0x360/0x3b4
 [&lt;ffffffff81087043&gt;] ? trace_hardirqs_on_caller+0x10b/0x12f
 [&lt;ffffffff810458aa&gt;] ? finish_task_switch+0xb2/0xe3
 [&lt;ffffffff8104583e&gt;] ? finish_task_switch+0x46/0xe3
 [&lt;ffffffff8108364a&gt;] ? trace_hardirqs_off_caller+0x33/0x90
 [&lt;ffffffff814bbaf9&gt;] ? retint_swapgs+0x13/0x1b
 [&lt;ffffffff81087043&gt;] ? trace_hardirqs_on_caller+0x10b/0x12f
 [&lt;ffffffff810a9dd3&gt;] ? audit_syscall_entry+0x11c/0x148
 [&lt;ffffffff8125609e&gt;] ? trace_hardirqs_on_thunk+0x3a/0x3f
 [&lt;ffffffff814c22c2&gt;] system_call_fastpath+0x16/0x1b

Signed-off-by: Dave Jones &lt;davej@redhat.com&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</pre>
</div>
</content>
</entry>
</feed>
