From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752280AbbHCAh3 (ORCPT ); Sun, 2 Aug 2015 20:37:29 -0400 Received: from mailout4.w1.samsung.com ([210.118.77.14]:49248 "EHLO mailout4.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751071AbbHCAh1 (ORCPT ); Sun, 2 Aug 2015 20:37:27 -0400 X-AuditID: cbfec7f5-f794b6d000001495-ae-55beb7c47bcb Message-id: <55BEB7BE.9030606@samsung.com> Date: Mon, 03 Aug 2015 09:37:18 +0900 From: Krzysztof Kozlowski User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.8.0 MIME-version: 1.0 To: Robert Baldyga , gregkh@linuxfoundation.org Cc: akpm@linux-foundation.org, jslaby@suse.cz, linux-serial@vger.kernel.org, linux-kernel@vger.kernel.org, m.szyprowski@samsung.com, kmpark@infradead.org, stable@vger.kernel.org Subject: Re: [PATCH v2 0/2] serial: samsung: fix DMA for small FIFO sizes References: <1438333108-25822-1-git-send-email-r.baldyga@samsung.com> In-reply-to: <1438333108-25822-1-git-send-email-r.baldyga@samsung.com> Content-type: text/plain; charset=windows-1252 Content-transfer-encoding: 7bit X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFnrDLMWRmVeSWpSXmKPExsVy+t/xq7pHtu8LNeh9JmgxZ/0aNovmxevZ LD7/3sVq8fqFocWty6tYLC7vmsNmcWZxL7vF2iN32S0eHN7JbrFg4yNGBy6PzSu0PE7M+M3i sX/uGnaPvi2rGD3OLDjC7vF5k1wAWxSXTUpqTmZZapG+XQJXxu6vU1gKvvNUXHpwgbmBcTNX FyMnh4SAicTfi7PZIGwxiQv31gPZXBxCAksZJfpb7jFCOF8YJdZ9/MIOUsUroCVxpfs4E4jN IqAqceDsRGYQm03AWGLz8iVgk0QFIiSWrz7JCFEvKPFj8j0WEFtEwE1i0b+7YDXMAlsZJaa2 RYPYwgKeEoferQGbLyTgKvH50X2wXk6g+uMXWoHmcwDV60ncv6gF0SovsXnNW+YJjAKzkGyY hVA1C0nVAkbmVYyiqaXJBcVJ6blGesWJucWleel6yfm5mxghcfB1B+PSY1aHGAU4GJV4eD8s 2BcqxJpYVlyZe4hRgoNZSYT3YjpQiDclsbIqtSg/vqg0J7X4EKM0B4uSOO/MXe9DhATSE0tS s1NTC1KLYLJMHJxSDYwuj2VWRAmc7lLzm/CCKYonKf1HW9Hp6yV3ND9PkH0bZay8PVlq6uka yea3pycs3pBxJlzsWf0Zi18LJrxx2qv4y+bYn4W1TPecFrtpPi3gqZ+oEBoyXT7sMeuD1/JR Cda8ToEej5UYuGwWvdZc+Fys9qb7dgW/5uarARJ15xUWfN6+y/T977dKLMUZiYZazEXFiQB7 qjsVfwIAAA== Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 31.07.2015 17:58, Robert Baldyga wrote: > Hello, > > This patch set fixes bug causing serial hang in DMA mode for FIFO sizes > smaller than cache alignment. The first patch fixes DMA mode entering > condition to avoid starting with buffer smaller than cache line size. > Second patch fixes the serial hang bug, which was caused by unproper > buffer aligning algorithm which assumed that there is always enough > free space in FIFO for excessive bytes of buffer that is being alligned. > > Best regards, > Robert Baldyga > > Changelog: > > v2: > - Add CC to stable > - Add Reported-by: Krzysztof Kozlowski > - Change title of the first patch to more relevant > > v1: http://permalink.gmane.org/gmane.linux.kernel/2008281 > > Marek Szyprowski (1): > serial: samsung: fix DMA mode enter condition for small FIFO sizes > > Robert Baldyga (1): > serial: samsung: fix DMA for FIFO smaller than cache line size > > drivers/tty/serial/samsung.c | 47 +++++++++++++++++++++++++++++--------------- > drivers/tty/serial/samsung.h | 1 + > 2 files changed, 32 insertions(+), 16 deletions(-) > Tested on Trats2 (Exynos4412 with pl330, serial output on ttySAC2). Patchset fixes the reported issue (hang after applying "ARM: dts: exynos4: add DMA support for serial ports" [0]). However with [0] dmatest cannot execute tests on dma0 channels. Just silently passes but does not execute memcpy on any of dma0 channels. I wonder why... the serial ports use only some of dma0 channels, not all of them. Any ideas? [0] http://www.spinics.net/lists/linux-samsung-soc/msg45700.html Best regards, Krzysztof