All of lore.kernel.org
 help / color / mirror / Atom feed
From: holgerschurig@gmail.com (Holger Schurig)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH net-next V3 00/16] net: fec: cleanup and fixes
Date: Fri, 15 Apr 2016 10:33:37 +0200	[thread overview]
Message-ID: <87bn5bp8ym.fsf@gmail.com> (raw)
In-Reply-To: <570FB998.2080900@boundarydevices.com> (Troy Kisky's message of "Thu, 14 Apr 2016 08:39:04 -0700")

> I think I've already fixed this, but I've only submitted once.
>
> commit 466cb4a2e5583d2e18470f30d5948edcf4b947f5
> Author: Troy Kisky <troy.kisky@boundarydevices.com>
> Date:   Wed Jan 20 12:52:10 2016 -0700
>
>     net: fec: update dirty_tx even if no skb
>
>     If dirty_tx isn't updated, then dma_unmap_single
>     will be called twice.
>
>     Signed-off-by: Troy Kisky <troy.kisky@boundarydevices.com>

Thanks!   This fixed the issue.  Have a

Tested-by: <holgerschurig@gmail.com>

WARNING: multiple messages have this Message-ID (diff)
From: Holger Schurig <holgerschurig@gmail.com>
To: Troy Kisky <troy.kisky@boundarydevices.com>
Cc: netdev@vger.kernel.org, davem@davemloft.net, fugang.duan@nxp.com,
	lznuaa@gmail.com, andrew@lunn.ch, stillcompiling@gmail.com,
	arnd@arndb.de, sergei.shtylyov@cogentembedded.com,
	gerg@uclinux.org, fabio.estevam@nxp.com,
	johannes@sipsolutions.net, l.stach@pengutronix.de,
	linux-arm-kernel@lists.infradead.org, tremyfr@gmail.com
Subject: Re: [PATCH net-next V3 00/16] net: fec: cleanup and fixes
Date: Fri, 15 Apr 2016 10:33:37 +0200	[thread overview]
Message-ID: <87bn5bp8ym.fsf@gmail.com> (raw)
In-Reply-To: <570FB998.2080900@boundarydevices.com> (Troy Kisky's message of "Thu, 14 Apr 2016 08:39:04 -0700")

> I think I've already fixed this, but I've only submitted once.
>
> commit 466cb4a2e5583d2e18470f30d5948edcf4b947f5
> Author: Troy Kisky <troy.kisky@boundarydevices.com>
> Date:   Wed Jan 20 12:52:10 2016 -0700
>
>     net: fec: update dirty_tx even if no skb
>
>     If dirty_tx isn't updated, then dma_unmap_single
>     will be called twice.
>
>     Signed-off-by: Troy Kisky <troy.kisky@boundarydevices.com>

Thanks!   This fixed the issue.  Have a

Tested-by: <holgerschurig@gmail.com>

  reply	other threads:[~2016-04-15  8:33 UTC|newest]

Thread overview: 72+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-04-06  2:25 [PATCH net-next V3 00/16] net: fec: cleanup and fixes Troy Kisky
2016-04-06  2:25 ` Troy Kisky
2016-04-06  2:25 ` [PATCH net-next V3 01/16] net: fec: only check queue 0 if RXF_0/TXF_0 interrupt is set Troy Kisky
2016-04-06  2:25   ` Troy Kisky
2016-04-06  2:25 ` [PATCH net-next V3 02/16] net: fec: remove unused interrupt FEC_ENET_TS_TIMER Troy Kisky
2016-04-06  2:25   ` Troy Kisky
2016-04-06  9:10   ` Fugang Duan
2016-04-06  9:10     ` Fugang Duan
2016-04-06  2:25 ` [PATCH net-next V3 03/16] net: fec: return IRQ_HANDLED if fec_ptp_check_pps_event handled it Troy Kisky
2016-04-06  2:25   ` Troy Kisky
2016-04-06  9:12   ` Fugang Duan
2016-04-06  9:12     ` Fugang Duan
2016-04-06  2:25 ` [PATCH net-next V3 04/16] net: fec: pass rxq/txq to fec_enet_rx/tx_queue instead of queue_id Troy Kisky
2016-04-06  2:25   ` Troy Kisky
2016-04-06  2:25 ` [PATCH net-next V3 05/16] net: fec: reduce interrupts Troy Kisky
2016-04-06  2:25   ` Troy Kisky
2016-04-06 10:06   ` Fugang Duan
2016-04-06 10:06     ` Fugang Duan
2016-04-06 21:20   ` David Miller
2016-04-06 21:20     ` David Miller
2016-04-07  0:42     ` Troy Kisky
2016-04-07  0:42       ` Troy Kisky
2016-04-07  3:57       ` David Miller
2016-04-07  3:57         ` David Miller
2016-04-07 15:50         ` Troy Kisky
2016-04-07 15:50           ` Troy Kisky
2016-04-06  2:25 ` [PATCH net-next V3 06/16] net: fec: split off napi routine with 3 queues Troy Kisky
2016-04-06  2:25   ` Troy Kisky
2016-04-06  2:25 ` [PATCH net-next V3 07/16] net: fec: don't clear all rx queue bits when just one is being checked Troy Kisky
2016-04-06  2:25   ` Troy Kisky
2016-04-06  2:25 ` [PATCH net-next V3 08/16] net: fec: set cbd_sc without relying on previous value Troy Kisky
2016-04-06  2:25   ` Troy Kisky
2016-04-06  9:51   ` Fugang Duan
2016-04-06  9:51     ` Fugang Duan
2016-04-06  2:25 ` [PATCH net-next V3 09/16] net: fec: eliminate calls to fec_enet_get_prevdesc Troy Kisky
2016-04-06  2:25   ` Troy Kisky
2016-04-06  2:25 ` [PATCH net-next V3 10/16] net: fec: move restart test for efficiency Troy Kisky
2016-04-06  2:25   ` Troy Kisky
2016-04-06  2:25 ` [PATCH net-next V3 11/16] net: fec: clear cbd_sc after transmission to help with debugging Troy Kisky
2016-04-06  2:25   ` Troy Kisky
2016-04-06  2:25 ` [PATCH net-next V3 12/16] net: fec: dump all tx queues in fec_dump Troy Kisky
2016-04-06  2:25   ` Troy Kisky
2016-04-06  2:25 ` [PATCH net-next V3 13/16] net: fec: detect tx int lost Troy Kisky
2016-04-06  2:25   ` Troy Kisky
2016-04-06  2:26 ` [PATCH net-next V3 14/16] net: fec: create subroutine reset_tx_queue Troy Kisky
2016-04-06  2:26   ` Troy Kisky
2016-04-06  2:26 ` [PATCH net-next V3 15/16] net: fec: call dma_unmap_single on mapped tx buffers at restart Troy Kisky
2016-04-06  2:26   ` Troy Kisky
2016-04-06  2:26 ` [PATCH net-next V3 16/16] net: fec: don't set cbd_bufaddr unless no mapping error Troy Kisky
2016-04-06  2:26   ` Troy Kisky
2016-04-06  8:51 ` [PATCH net-next V3 00/16] net: fec: cleanup and fixes Fugang Duan
2016-04-06  8:51   ` Fugang Duan
2016-04-06 16:43   ` Troy Kisky
2016-04-06 16:43     ` Troy Kisky
2016-04-06 16:47     ` Troy Kisky
2016-04-06 16:47       ` Troy Kisky
2016-04-07  1:23     ` Fugang Duan
2016-04-07  1:23       ` Fugang Duan
2016-04-06 21:20 ` David Miller
2016-04-06 21:20   ` David Miller
2016-04-07  1:09   ` Troy Kisky
2016-04-07  1:09     ` Troy Kisky
2016-04-07  3:58     ` David Miller
2016-04-07  3:58       ` David Miller
2016-04-07 15:35       ` Troy Kisky
2016-04-07 15:35         ` Troy Kisky
2016-04-14 10:13 ` Holger Schurig
2016-04-14 10:13   ` Holger Schurig
2016-04-14 15:39   ` Troy Kisky
2016-04-14 15:39     ` Troy Kisky
2016-04-15  8:33     ` Holger Schurig [this message]
2016-04-15  8:33       ` Holger Schurig

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87bn5bp8ym.fsf@gmail.com \
    --to=holgerschurig@gmail.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.