From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sergei Shtylyov Date: Tue, 27 Jan 2015 18:19:04 +0000 Subject: Re: [PATCH 2/6] dmaengine: rcar-dmac: Fix spinlock issues in interrupt Message-Id: <54C7D698.1070201@cogentembedded.com> List-Id: References: <1422379516-1633-3-git-send-email-laurent.pinchart+renesas@ideasonboard.com> In-Reply-To: <1422379516-1633-3-git-send-email-laurent.pinchart+renesas@ideasonboard.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: linux-sh@vger.kernel.org Hello. On 01/27/2015 08:25 PM, Laurent Pinchart wrote: [...] > Furthermore, as the driver can be used on SMP systems, the IRQ handlers > must use spin_lock_irqsave() instead of spin_lock(). Hmm, care to explain why? It's the first time I'm hearing that... This looks especially strange in the context of the current genirq code calling IRQ handlers with interrupts always disabled. > Signed-off-by: Laurent Pinchart WBR, Sergei