* [PATCH v3] Revert "serial: 8250_dma: don't bother DMA with small transfers"
@ 2015-10-09 15:14 Frederic Danis
2015-10-13 11:22 ` Heikki Krogerus
2015-10-13 14:10 ` Andy Shevchenko
0 siblings, 2 replies; 3+ messages in thread
From: Frederic Danis @ 2015-10-09 15:14 UTC (permalink / raw)
To: linux-serial, andriy.shevchenko, peter, gregkh
Cc: linux-bluetooth, heikki.krogerus, marcel
This reverts commit 9119fba0cfeda6d415c9f068df66838a104b87cb.
This commit prevents from sending "big" file using Bluetooth.
When sending a lot of data quickly through the Bluetooth interface, and
after a variable amount of data sent, transfer fails with error:
kernel: [ 415.247453] Bluetooth: hci0 hardware error 0x00
Found on T100TA.
After reverting this commit, send works fine for any file size.
Signed-off-by: Frederic Danis <frederic.danis@linux.intel.com>
Fixes: 9119fba0cfed (serial: 8250_dma: don't bother DMA with small
transfers)
Cc: stable@vger.kernel.org
---
v3: Add Fixes and Cc tags
v2: Re-send to correct people
drivers/tty/serial/8250/8250_dma.c | 4 ----
1 file changed, 4 deletions(-)
diff --git a/drivers/tty/serial/8250/8250_dma.c b/drivers/tty/serial/8250/8250_dma.c
index b503071..78259d3 100644
--- a/drivers/tty/serial/8250/8250_dma.c
+++ b/drivers/tty/serial/8250/8250_dma.c
@@ -77,10 +77,6 @@ int serial8250_tx_dma(struct uart_8250_port *p)
return 0;
dma->tx_size = CIRC_CNT_TO_END(xmit->head, xmit->tail, UART_XMIT_SIZE);
- if (dma->tx_size < p->port.fifosize) {
- ret = -EINVAL;
- goto err;
- }
desc = dmaengine_prep_slave_single(dma->txchan,
dma->tx_addr + xmit->tail,
--
1.9.1
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [PATCH v3] Revert "serial: 8250_dma: don't bother DMA with small transfers"
2015-10-09 15:14 [PATCH v3] Revert "serial: 8250_dma: don't bother DMA with small transfers" Frederic Danis
@ 2015-10-13 11:22 ` Heikki Krogerus
2015-10-13 14:10 ` Andy Shevchenko
1 sibling, 0 replies; 3+ messages in thread
From: Heikki Krogerus @ 2015-10-13 11:22 UTC (permalink / raw)
To: Frederic Danis
Cc: linux-serial, andriy.shevchenko, peter, gregkh, linux-bluetooth,
marcel
On Fri, Oct 09, 2015 at 05:14:56PM +0200, Frederic Danis wrote:
> This reverts commit 9119fba0cfeda6d415c9f068df66838a104b87cb.
>
> This commit prevents from sending "big" file using Bluetooth.
> When sending a lot of data quickly through the Bluetooth interface, and
> after a variable amount of data sent, transfer fails with error:
> kernel: [ 415.247453] Bluetooth: hci0 hardware error 0x00
>
> Found on T100TA.
>
> After reverting this commit, send works fine for any file size.
>
> Signed-off-by: Frederic Danis <frederic.danis@linux.intel.com>
> Fixes: 9119fba0cfed (serial: 8250_dma: don't bother DMA with small
> transfers)
> Cc: stable@vger.kernel.org
FWIW:
Reviewed-by: Heikki Krogerus <heikki.krogerus@linux.intel.com>
> ---
> v3: Add Fixes and Cc tags
> v2: Re-send to correct people
>
> drivers/tty/serial/8250/8250_dma.c | 4 ----
> 1 file changed, 4 deletions(-)
>
> diff --git a/drivers/tty/serial/8250/8250_dma.c b/drivers/tty/serial/8250/8250_dma.c
> index b503071..78259d3 100644
> --- a/drivers/tty/serial/8250/8250_dma.c
> +++ b/drivers/tty/serial/8250/8250_dma.c
> @@ -77,10 +77,6 @@ int serial8250_tx_dma(struct uart_8250_port *p)
> return 0;
>
> dma->tx_size = CIRC_CNT_TO_END(xmit->head, xmit->tail, UART_XMIT_SIZE);
> - if (dma->tx_size < p->port.fifosize) {
> - ret = -EINVAL;
> - goto err;
> - }
>
> desc = dmaengine_prep_slave_single(dma->txchan,
> dma->tx_addr + xmit->tail,
> --
> 1.9.1
--
heikki
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH v3] Revert "serial: 8250_dma: don't bother DMA with small transfers"
2015-10-09 15:14 [PATCH v3] Revert "serial: 8250_dma: don't bother DMA with small transfers" Frederic Danis
2015-10-13 11:22 ` Heikki Krogerus
@ 2015-10-13 14:10 ` Andy Shevchenko
1 sibling, 0 replies; 3+ messages in thread
From: Andy Shevchenko @ 2015-10-13 14:10 UTC (permalink / raw)
To: Frederic Danis, linux-serial, peter, gregkh
Cc: linux-bluetooth, heikki.krogerus, marcel
On Fri, 2015-10-09 at 17:14 +0200, Frederic Danis wrote:
> This reverts commit 9119fba0cfeda6d415c9f068df66838a104b87cb.
>
> This commit prevents from sending "big" file using Bluetooth.
> When sending a lot of data quickly through the Bluetooth interface,
> and
> after a variable amount of data sent, transfer fails with error:
> kernel: [ 415.247453] Bluetooth: hci0 hardware error 0x00
>
> Found on T100TA.
>
> After reverting this commit, send works fine for any file size.
>
> Signed-off-by: Frederic Danis <frederic.danis@linux.intel.com>
> Fixes: 9119fba0cfed (serial: 8250_dma: don't bother DMA with small
> transfers)
> Cc: stable@vger.kernel.org
Acked-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> ---
> v3: Add Fixes and Cc tags
> v2: Re-send to correct people
>
> drivers/tty/serial/8250/8250_dma.c | 4 ----
> 1 file changed, 4 deletions(-)
>
> diff --git a/drivers/tty/serial/8250/8250_dma.c
> b/drivers/tty/serial/8250/8250_dma.c
> index b503071..78259d3 100644
> --- a/drivers/tty/serial/8250/8250_dma.c
> +++ b/drivers/tty/serial/8250/8250_dma.c
> @@ -77,10 +77,6 @@ int serial8250_tx_dma(struct uart_8250_port *p)
> return 0;
>
> dma->tx_size = CIRC_CNT_TO_END(xmit->head, xmit->tail,
> UART_XMIT_SIZE);
> - if (dma->tx_size < p->port.fifosize) {
> - ret = -EINVAL;
> - goto err;
> - }
>
> desc = dmaengine_prep_slave_single(dma->txchan,
> dma->tx_addr + xmit
> ->tail,
--
Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Intel Finland Oy
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2015-10-13 14:10 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-10-09 15:14 [PATCH v3] Revert "serial: 8250_dma: don't bother DMA with small transfers" Frederic Danis
2015-10-13 11:22 ` Heikki Krogerus
2015-10-13 14:10 ` Andy Shevchenko
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).