Re: [PATCH] input: pcap2: avoid calling mutex_lock() in irq handler

2014-05-21 Thread Antonio Ospite
On Sun, 6 Apr 2014 13:24:36 -0700 Dmitry Torokhov dmitry.torok...@gmail.com wrote: On Sun, Apr 06, 2014 at 12:54:50AM +0400, Alexey Khoroshilov wrote: pcap_keys_handler() calls ezx_pcap_read() that calls mutex_lock(). pcap_keys_handler() is registered as nonthreaded irq handler, that means

Re: [PATCH] input: pcap2: avoid calling mutex_lock() in irq handler

2014-05-13 Thread Antonio Ospite
On Mon, 12 May 2014 19:14:56 -0700 Dmitry Torokhov dmitry.torok...@gmail.com wrote: On Sun, Apr 06, 2014 at 11:37:44PM +0200, Antonio Ospite wrote: On Sun, 6 Apr 2014 13:24:36 -0700 Dmitry Torokhov dmitry.torok...@gmail.com wrote: On Sun, Apr 06, 2014 at 12:54:50AM +0400, Alexey

Re: [PATCH] input: pcap2: avoid calling mutex_lock() in irq handler

2014-05-12 Thread Dmitry Torokhov
On Sun, Apr 06, 2014 at 11:37:44PM +0200, Antonio Ospite wrote: On Sun, 6 Apr 2014 13:24:36 -0700 Dmitry Torokhov dmitry.torok...@gmail.com wrote: On Sun, Apr 06, 2014 at 12:54:50AM +0400, Alexey Khoroshilov wrote: pcap_keys_handler() calls ezx_pcap_read() that calls mutex_lock().

Re: [PATCH] input: pcap2: avoid calling mutex_lock() in irq handler

2014-04-06 Thread Dmitry Torokhov
On Sun, Apr 06, 2014 at 12:54:50AM +0400, Alexey Khoroshilov wrote: pcap_keys_handler() calls ezx_pcap_read() that calls mutex_lock(). pcap_keys_handler() is registered as nonthreaded irq handler, that means sleeping function is called in irq handler. The patch makes a switch to threaded irq

Re: [PATCH] input: pcap2: avoid calling mutex_lock() in irq handler

2014-04-06 Thread Antonio Ospite
On Sun, 6 Apr 2014 13:24:36 -0700 Dmitry Torokhov dmitry.torok...@gmail.com wrote: On Sun, Apr 06, 2014 at 12:54:50AM +0400, Alexey Khoroshilov wrote: pcap_keys_handler() calls ezx_pcap_read() that calls mutex_lock(). pcap_keys_handler() is registered as nonthreaded irq handler, that means

[PATCH] input: pcap2: avoid calling mutex_lock() in irq handler

2014-04-05 Thread Alexey Khoroshilov
pcap_keys_handler() calls ezx_pcap_read() that calls mutex_lock(). pcap_keys_handler() is registered as nonthreaded irq handler, that means sleeping function is called in irq handler. The patch makes a switch to threaded irq handling. Compile tested only. Found by Linux Driver Verification