* [PATCH] dma:xgene-dma:move spin_lock_bh to spin_lock in tasklet
@ 2019-03-19 16:45 Jeff Xie
2019-03-25 5:52 ` Vinod Koul
0 siblings, 1 reply; 2+ messages in thread
From: Jeff Xie @ 2019-03-19 16:45 UTC (permalink / raw)
To: dan.j.williams, vkoul; +Cc: dmaengine, chongguiguzi, linux-kernel
It is unnecessary to call spin_lock_bh in a tasklet.
Signed-off-by: Jeff Xie <chongguiguzi@gmail.com>
---
drivers/dma/xgene-dma.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/dma/xgene-dma.c b/drivers/dma/xgene-dma.c
index eafd6c4..0023236 100644
--- a/drivers/dma/xgene-dma.c
+++ b/drivers/dma/xgene-dma.c
@@ -703,7 +703,7 @@ static void xgene_dma_cleanup_descriptors(struct xgene_dma_chan *chan)
INIT_LIST_HEAD(&ld_completed);
- spin_lock_bh(&chan->lock);
+ spin_lock(&chan->lock);
/* Clean already completed and acked descriptors */
xgene_dma_clean_completed_descriptor(chan);
@@ -772,7 +772,7 @@ static void xgene_dma_cleanup_descriptors(struct xgene_dma_chan *chan)
*/
xgene_chan_xfer_ld_pending(chan);
- spin_unlock_bh(&chan->lock);
+ spin_unlock(&chan->lock);
/* Run the callback for each descriptor, in order */
list_for_each_entry_safe(desc_sw, _desc_sw, &ld_completed, node) {
--
2.7.4
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] dma:xgene-dma:move spin_lock_bh to spin_lock in tasklet
2019-03-19 16:45 [PATCH] dma:xgene-dma:move spin_lock_bh to spin_lock in tasklet Jeff Xie
@ 2019-03-25 5:52 ` Vinod Koul
0 siblings, 0 replies; 2+ messages in thread
From: Vinod Koul @ 2019-03-25 5:52 UTC (permalink / raw)
To: Jeff Xie; +Cc: dan.j.williams, dmaengine, linux-kernel
On 20-03-19, 00:45, Jeff Xie wrote:
> It is unnecessary to call spin_lock_bh in a tasklet.
1. Please use the right subsystem name, dmaengine: xxx (you cna find
that by git log on that subsystem)
2. Space after each :, like dmaengine: xgene-dma: move...
3. Please explain a bit more about the change on why it is unnecessary
:)
Nevertheless, I have fixed 1 & 2 above and applied it
--
~Vinod
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2019-03-25 5:52 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-03-19 16:45 [PATCH] dma:xgene-dma:move spin_lock_bh to spin_lock in tasklet Jeff Xie
2019-03-25 5:52 ` Vinod Koul
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).