From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Message-Id: <20130603221159.809766434@goodmis.org> Date: Mon, 03 Jun 2013 18:11:00 -0400 From: Steven Rostedt To: linux-kernel@vger.kernel.org, stable@vger.kernel.org Cc: Tomoya MORINAGA , Vinod Koul Subject: [44/65] pch_dma: Use GFP_ATOMIC because called from interrupt context References: <20130603221016.243553628@goodmis.org> Content-Disposition: inline; filename=0044-pch_dma-Use-GFP_ATOMIC-because-called-from-interrupt.patch Sender: linux-kernel-owner@vger.kernel.org List-ID: 3.6.11.5 stable review patch. If anyone has any objections, please let me know. ------------------ From: Tomoya MORINAGA [ Upstream commit 5c1ef59168c485318e40ba485c1eba57d81d0faa ] pdc_desc_get() is called from pd_prep_slave_sg, and the function is called from interrupt context(e.g. Uart driver "pch_uart.c"). In fact, I saw kernel error message. So, GFP_ATOMIC must be used not GFP_NOIO. Signed-off-by: Tomoya MORINAGA Signed-off-by: Vinod Koul Signed-off-by: Steven Rostedt --- drivers/dma/pch_dma.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/dma/pch_dma.c b/drivers/dma/pch_dma.c index 987ab5c..d5cbd44 100644 --- a/drivers/dma/pch_dma.c +++ b/drivers/dma/pch_dma.c @@ -476,7 +476,7 @@ static struct pch_dma_desc *pdc_desc_get(struct pch_dma_chan *pd_chan) dev_dbg(chan2dev(&pd_chan->chan), "scanned %d descriptors\n", i); if (!ret) { - ret = pdc_alloc_desc(&pd_chan->chan, GFP_NOIO); + ret = pdc_alloc_desc(&pd_chan->chan, GFP_ATOMIC); if (ret) { spin_lock(&pd_chan->lock); pd_chan->descs_allocated++; -- 1.7.10.4