<feed xmlns='http://www.w3.org/2005/Atom'>
<title>kernel.git/drivers/nfc/pn533.c, branch linux-3.8.y</title>
<subtitle>Hosts the 0x221E linux distro kernel.
</subtitle>
<id>https://git.0xinfinity.dev/distro/kernel.git/atom?h=linux-3.8.y</id>
<link rel='self' href='https://git.0xinfinity.dev/distro/kernel.git/atom?h=linux-3.8.y'/>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/'/>
<updated>2012-11-25T17:49:17Z</updated>
<entry>
<title>Merge git://git.kernel.org/pub/scm/linux/kernel/git/davem/net</title>
<updated>2012-11-25T17:49:17Z</updated>
<author>
<name>David S. Miller</name>
</author>
<published>2012-11-25T17:49:17Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=24bc518a6888e4c97add5a5ebbff11c1ccac219f'/>
<id>urn:sha1:24bc518a6888e4c97add5a5ebbff11c1ccac219f</id>
<content type='text'>
Conflicts:
	drivers/net/wireless/iwlwifi/pcie/tx.c

Minor iwlwifi conflict in TX queue disabling between 'net', which
removed a bogus warning, and 'net-next' which added some status
register poking code.

Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</content>
</entry>
<entry>
<title>NFC: Fix pn533 target mode memory leak</title>
<updated>2012-11-19T23:09:26Z</updated>
<author>
<name>Thierry Escande</name>
</author>
<published>2012-11-15T17:24:28Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=5b412fd11c918171c98a253d8a3484afa9f69ca5'/>
<id>urn:sha1:5b412fd11c918171c98a253d8a3484afa9f69ca5</id>
<content type='text'>
In target mode, sent sk_buff were not freed in pn533_tm_send_complete

Signed-off-by: Thierry Escande &lt;thierry.escande@linux.intel.com&gt;
Signed-off-by: Samuel Ortiz &lt;sameo@linux.intel.com&gt;
</content>
</entry>
<entry>
<title>NFC: pn533: Fix mem leak in pn533_in_dep_link_up</title>
<updated>2012-11-19T23:09:26Z</updated>
<author>
<name>Waldemar Rymarkiewicz</name>
</author>
<published>2012-10-11T12:04:00Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=70418e6efcf4f8652cc08e3f2ab8ae35f0948fd9'/>
<id>urn:sha1:70418e6efcf4f8652cc08e3f2ab8ae35f0948fd9</id>
<content type='text'>
cmd is allocated in pn533_dep_link_up and passed as an arg to
pn533_send_cmd_frame_async together with a complete cb.

arg is passed to the cb and must be kfreed there.

Signed-off-by: Waldemar Rymarkiewicz &lt;waldemar.rymarkiewicz@tieto.com&gt;
Signed-off-by: Samuel Ortiz &lt;sameo@linux.intel.com&gt;
</content>
</entry>
<entry>
<title>NFC: pn533: Fix use after free</title>
<updated>2012-11-19T23:09:26Z</updated>
<author>
<name>Szymon Janc</name>
</author>
<published>2012-10-29T13:04:43Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=770f750bc2b8312489c8e45306f551d08a319d3c'/>
<id>urn:sha1:770f750bc2b8312489c8e45306f551d08a319d3c</id>
<content type='text'>
cmd was freed in pn533_dep_link_up regardless of
pn533_send_cmd_frame_async return code. Cmd is passed as argument to
pn533_in_dep_link_up_complete callback and should be freed there.

Signed-off-by: Szymon Janc &lt;szymon.janc@tieto.com&gt;
Signed-off-by: Samuel Ortiz &lt;sameo@linux.intel.com&gt;
</content>
</entry>
<entry>
<title>NFC: pn533: Fix missing lock while operating on commands list</title>
<updated>2012-11-19T23:09:25Z</updated>
<author>
<name>Szymon Janc</name>
</author>
<published>2012-10-25T15:29:45Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=60ad07ab6bc86f48b6ebda1788d79ca5f88d824c'/>
<id>urn:sha1:60ad07ab6bc86f48b6ebda1788d79ca5f88d824c</id>
<content type='text'>
In pn533_wq_cmd command was removed from list without cmd_lock held
(race with pn533_send_cmd_frame_async) which could lead to list
corruption. Delete command from list before releasing lock.

Signed-off-by: Szymon Janc &lt;szymon.janc@tieto.com&gt;
Signed-off-by: Samuel Ortiz &lt;sameo@linux.intel.com&gt;
</content>
</entry>
<entry>
<title>NFC: pn533: Remove unused arg parameter</title>
<updated>2012-10-26T16:26:51Z</updated>
<author>
<name>Waldemar Rymarkiewicz</name>
</author>
<published>2012-10-11T12:04:01Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=ab34a1813ca16bfbc672a21ca354164c8b941922'/>
<id>urn:sha1:ab34a1813ca16bfbc672a21ca354164c8b941922</id>
<content type='text'>
Get rid of unused arg param in pn533_init_target_complete and
in pn533_start_poll_complete.

Signed-off-by: Waldemar Rymarkiewicz &lt;waldemar.rymarkiewicz@tieto.com&gt;
Signed-off-by: Samuel Ortiz &lt;sameo@linux.intel.com&gt;
</content>
</entry>
<entry>
<title>NFC: pn533: Fix in/out frame buffer allocation</title>
<updated>2012-10-26T16:26:51Z</updated>
<author>
<name>Waldemar Rymarkiewicz</name>
</author>
<published>2012-10-11T12:03:58Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=82dec34df64620a0e06ea6f7020092e41244c3c2'/>
<id>urn:sha1:82dec34df64620a0e06ea6f7020092e41244c3c2</id>
<content type='text'>
Max frame size should be 264 bytes as per spec and not limited to
endpoint MaxPacketSize which is 64 in my case (acr122 reader).

Signed-off-by: Waldemar Rymarkiewicz &lt;waldemar.rymarkiewicz@tieto.com&gt;
Signed-off-by: Samuel Ortiz &lt;sameo@linux.intel.com&gt;
</content>
</entry>
<entry>
<title>NFC: Fix missing mutex unlock in pn533_send_cmd_frame_async</title>
<updated>2012-09-27T08:47:48Z</updated>
<author>
<name>Szymon Janc</name>
</author>
<published>2012-09-27T07:16:54Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=ee5e8d812ca25bf95a97b4368aec4e680678e75e'/>
<id>urn:sha1:ee5e8d812ca25bf95a97b4368aec4e680678e75e</id>
<content type='text'>
If command allocation failed cmd_lock was not released and deadlock
would occur.

Signed-off-by: Szymon Janc &lt;szymon.janc@tieto.com&gt;
Signed-off-by: Samuel Ortiz &lt;sameo@linux.intel.com&gt;
</content>
</entry>
<entry>
<title>NFC: Don't use WQ_MEM_RECLAIM for pn533</title>
<updated>2012-09-24T22:17:24Z</updated>
<author>
<name>Tejun Heo</name>
</author>
<published>2012-08-22T23:28:46Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=58637c9b65300cbdf9fb7c037978daadd136be86'/>
<id>urn:sha1:58637c9b65300cbdf9fb7c037978daadd136be86</id>
<content type='text'>
NFC driver doesn't sit in memory reclaim path and has no reason to use
WQ_MEM_RECLAIM.  Drop WQ_MEM_RECLAIM from pn533-&gt;wq and use
alloc_ordered_workqueue() instead of WQ_UNBOUND w/ max_active == 1.

Signed-off-by: Tejun Heo &lt;tj@kernel.org&gt;
Signed-off-by: Samuel Ortiz &lt;sameo@linux.intel.com&gt;
</content>
</entry>
<entry>
<title>NFC: Queue pn533 commands</title>
<updated>2012-09-24T22:17:23Z</updated>
<author>
<name>Samuel Ortiz</name>
</author>
<published>2012-08-17T21:47:54Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=5d50b364e61e85eb41938d25770db3aab5e07d82'/>
<id>urn:sha1:5d50b364e61e85eb41938d25770db3aab5e07d82</id>
<content type='text'>
Instead of returning EBUSY when getting a command while another one is
running, we queue them. Upon completion of the pending command, the next
one is processed.
Besides the fact that it simplifies the pn533 locking scheme, it also
comes with the nice side effect of fixing the following warning:

[   82.274297] =====================================
[   82.274297] [ BUG: bad unlock balance detected! ]
[   82.274298] 3.5.0-rc1+ #1 Not tainted
[   82.274299] -------------------------------------
[   82.274300] kworker/u:1/16 is trying to release lock (&amp;dev-&gt;cmd_lock) at:
[   82.274305] [&lt;ffffffff8144f246&gt;] mutex_unlock+0x9/0xb
[   82.274305] but there are no more locks to release!
[   82.274306]
[   82.274306] other info that might help us debug this:
[   82.274306] 2 locks held by kworker/u:1/16:
[   82.274311]  #0:  (pn533){.+.+..}, at: [&lt;ffffffff8103a67d&gt;]
+process_one_work+0x145/0x2e2
[   82.274314]  #1:  ((&amp;dev-&gt;cmd_work)){+.+...}, at: [&lt;ffffffff8103a67d&gt;]
+process_one_work+0x145/0x2e2
[   82.274314]
[   82.274314] stack backtrace:
[   82.274315] Pid: 16, comm: kworker/u:1 Not tainted 3.5.0-rc1+ #1
[   82.274315] Call Trace:
[   82.274317]  [&lt;ffffffff8144f246&gt;] ? mutex_unlock+0x9/0xb
[   82.274321]  [&lt;ffffffff81059841&gt;] print_unlock_inbalance_bug+0xda/0xe4
[   82.274323]  [&lt;ffffffff8105c74c&gt;] lock_release_non_nested+0xb2/0x232
[   82.274325]  [&lt;ffffffff8105a61e&gt;] ? mark_held_locks+0x6d/0x95
[   82.274326]  [&lt;ffffffff8144f246&gt;] ? mutex_unlock+0x9/0xb
[   82.274328]  [&lt;ffffffff81451105&gt;] ? _raw_spin_unlock_irqrestore+0x40/0x5c
[   82.274329]  [&lt;ffffffff8144f246&gt;] ? mutex_unlock+0x9/0xb
[   82.274330]  [&lt;ffffffff8105ca42&gt;] lock_release+0x176/0x1ac
[   82.274333]  [&lt;ffffffff8123de14&gt;] ? pn533_send_complete+0xa8/0xa8
[   82.274334]  [&lt;ffffffff8144f1d6&gt;] __mutex_unlock_slowpath+0xb0/0x117
[   82.274336]  [&lt;ffffffff8144f246&gt;] mutex_unlock+0x9/0xb
[   82.274337]  [&lt;ffffffff8123de65&gt;] pn533_wq_cmd_complete+0x51/0x55
[   82.274338]  [&lt;ffffffff8103a6db&gt;] process_one_work+0x1a3/0x2e2
[   82.274340]  [&lt;ffffffff8103a67d&gt;] ? process_one_work+0x145/0x2e2
[   82.274341]  [&lt;ffffffff8103b119&gt;] worker_thread+0xcf/0x153
[   82.274343]  [&lt;ffffffff8103b04a&gt;] ? manage_workers.isra.22+0x16b/0x16b
[   82.274344]  [&lt;ffffffff8103b04a&gt;] ? manage_workers.isra.22+0x16b/0x16b
[   82.274346]  [&lt;ffffffff8103eb11&gt;] kthread+0x95/0x9d
[   82.274348]  [&lt;ffffffff81452ef4&gt;] kernel_thread_helper+0x4/0x10
[   82.274351]  [&lt;ffffffff81046561&gt;] ? finish_task_switch+0x45/0xc3
[   82.274352]  [&lt;ffffffff814514f0&gt;] ? retint_restore_args+0x13/0x13
[   82.274353]  [&lt;ffffffff8103ea7c&gt;] ? __init_kthread_worker+0x55/0x55
[   82.274354]  [&lt;ffffffff81452ef0&gt;] ? gs_change+0x13/0x13

Signed-off-by: Samuel Ortiz &lt;sameo@linux.intel.com&gt;
</content>
</entry>
</feed>
