> -----Original Message----- > From: dmaengine-owner@xxxxxxxxxxxxxxx [mailto:dmaengine-owner@xxxxxxxxxxxxxxx] On Behalf Of Vinod Koul > Sent: Thursday, February 11, 2016 1:52 AM > To: Jiang, Dave <dave.jiang@xxxxxxxxx> > Cc: dmaengine@xxxxxxxxxxxxxxx; Williams, Dan J <dan.j.williams@xxxxxxxxx> > Subject: Re: [PATCH v2] IOATDMA: fix timer code that continues to restart channels during idle > > On Wed, Feb 10, 2016 at 05:13:49AM +0530, Jiang, Dave wrote: > > On Sat, 2016-01-30 at 16:59 +0000, Koul, Vinod wrote: > > > On Tue, Jan 19, 2016 at 08:57:48AM -0700, Dave Jiang wrote: > > > > The timer_event() function seems to have a bug where it ends up > > > > marking the > > > > last entry as non-responding and eventually attempts to restart the > > > > channel. This also continuously happen when idle. What needs to > > > > happen is > > > > for us to make sure there are no descriptors active and then handle > > > > that > > > > case properly. We should only hit the "cleanup" stage if there are > > > > still > > > > active descriptors. > > > > > > Applied now and for Nth time pls use the right subsystem name > > > > > > > Vinod, > > I'm not seeing this patch in the slave-dma/topic/ioatdma branch. > > Its in -next and -fixes. Which branch should I be basing ioat patches against? > > -- > ~Vinod > -- > To unsubscribe from this list: send the line "unsubscribe dmaengine" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe dmaengine" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
