Re: [PATCH] eata: remove driver_lock

2014-08-10 Thread Arthur Marsh
Christoph Hellwig wrote, on 14/07/14 17:56: port_detect is only called from the module_init routine and thus implicitly serialized, so remove the driver lock which was held over potentially sleeping function calls. Signed-off-by: Christoph Hellwig h...@lst.de Reported-by: Arthur Marsh

Re: [PATCH] eata: remove driver_lock

2014-08-05 Thread Hannes Reinecke
On 07/14/2014 10:26 AM, Christoph Hellwig wrote: port_detect is only called from the module_init routine and thus implicitly serialized, so remove the driver lock which was held over potentially sleeping function calls. Signed-off-by: Christoph Hellwig h...@lst.de Reported-by: Arthur Marsh

Re: [PATCH] eata: remove driver_lock

2014-07-31 Thread Christoph Hellwig
Can I also get a second review? On Sat, Jul 26, 2014 at 09:25:59AM -0700, Christoph Hellwig wrote: Can I get a quick review for this one? On Mon, Jul 14, 2014 at 10:26:33AM +0200, Christoph Hellwig wrote: port_detect is only called from the module_init routine and thus implicitly

Re: [PATCH] eata: remove driver_lock

2014-07-26 Thread Christoph Hellwig
Can I get a quick review for this one? On Mon, Jul 14, 2014 at 10:26:33AM +0200, Christoph Hellwig wrote: port_detect is only called from the module_init routine and thus implicitly serialized, so remove the driver lock which was held over potentially sleeping function calls. Signed-off-by:

Re: [PATCH] eata: remove driver_lock

2014-07-26 Thread Martin K. Petersen
Christoph == Christoph Hellwig h...@infradead.org writes: Christoph Can I get a quick review for this one? Reviewed-by: Martin K. Petersen martin.peter...@oracle.com -- Martin K. Petersen Oracle Linux Engineering -- To unsubscribe from this list: send the line unsubscribe linux-scsi in

[PATCH] eata: remove driver_lock

2014-07-14 Thread Christoph Hellwig
port_detect is only called from the module_init routine and thus implicitly serialized, so remove the driver lock which was held over potentially sleeping function calls. Signed-off-by: Christoph Hellwig h...@lst.de Reported-by: Arthur Marsh arthur.ma...@internode.on.net Tested-by: Arthur Marsh