From mboxrd@z Thu Jan 1 00:00:00 1970 From: agust@denx.de (Anatolij Gustschin) Date: Mon, 7 Feb 2011 12:21:47 +0100 Subject: [PATCH 2/2 v2] dma: ipu_idmac: do not lose valid received data in the irq handler In-Reply-To: References: <1296031789-1721-3-git-send-email-agust@denx.de> <1296476549-10421-1-git-send-email-agust@denx.de> <20110205143505.0b300a3a@wker> <20110205210457.7218ecdc@wker> Message-ID: <20110207122147.4081f47d@wker> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Mon, 7 Feb 2011 12:09:15 +0100 (CET) Guennadi Liakhovetski wrote: ... > > I can't try mplayer since I don't have mplayer setup for this. > > But looking at the mplayer source I don't see why it should > > behave differently. Depending on mode mplayer queues 2 or 6 > > buffers. Testing with my test app with 6 queued buffers shows > > no issues, here the buffer numbers toggle correctly, too. > > Ok, I've done a couple more tests. With larger frames, and, therefore > lower fps - yes, with your patch buffers toggle correctly. Whereas in my > tests with smaller frames and higher fps either only one buffer is used, > or one is used much more often, than the other, e.g., 0 0 0 1 0 0 0 1 0... > Could you try to verify? Without your patch with any fps buffers toggle > consistently. How small are the frames in you test? What is the highest fps value in your test? Thanks, Anatolij