All of lore.kernel.org
 help / color / mirror / Atom feed
From: Felix Fietkau <nbd@openwrt.org>
To: ath9k-devel@lists.ath9k.org
Subject: [ath9k-devel] [PATCHv2] ath9k_hw: Handle AR_INTR_SYNC_HOST1_FATAL on AR9003
Date: Tue, 02 Oct 2012 15:33:28 +0200	[thread overview]
Message-ID: <506AED28.4050903@openwrt.org> (raw)
In-Reply-To: <CAJ-VmontNVV4TyRO5kEM30T=OmfEN8YUciJSW=j_0HnbadF4JA@mail.gmail.com>

On 2012-10-02 3:13 PM, Adrian Chadd wrote:
> .. well, the rule here is "You shouldn't get PERR/FATAL interrupts."
> 
> Haven't I posted a summary of what those errors are?
> 
> Ok. So they're signals from the PCIe core (named host1_fatal and
> host1_perr. Helpfully.) Those errors occured during a DMA transfer.
> 
> So the question is why you're seeing PERR interrupts when creating an
> adhoc interface. That hints to me that something odd is going on..
> 
> I've seen these issues creep up when the NIC is in some way behaving
> very, very badly (lots of timeouts and sync errors with little to no
> traffic at all), which resulted in all kinds of odd and weird,
> unstable behaviour. After replacing the NIC with another NIC (in my
> case, an AR9280 -> AR9280 NIC :-) the errors went away and things
> continued swimmingly.
> 
> I'd have to go digging through the PCIe core source to figure out
> exactly what host1_peer and host1_fatal mean. I can if you'd like,
> it'll just take some time as I'm not familiar at all with the PCIe
> host interface.
According to the datasheet, AR_INTR_SYNC_HOST1_PERR is triggered by an
invalid register access, and AR_INTR_SYNC_HOST1_FATAL is triggered by
corrupt descriptors or other DMA issues.
Maybe you can get some information on the source of this PERR error if
you record the last register accesses outside of the irq context and
print them once this IRQ comes in.

- Felix

WARNING: multiple messages have this Message-ID (diff)
From: Felix Fietkau <nbd@openwrt.org>
To: Adrian Chadd <adrian@freebsd.org>
Cc: Sven Eckelmann <sven@narfation.org>,
	lindner_marek@yahoo.de, mcgrof@qca.qualcomm.com,
	ath9k-devel@lists.ath9k.org, linux-wireless@vger.kernel.org,
	linville@tuxdriver.com
Subject: Re: [ath9k-devel] [PATCHv2] ath9k_hw: Handle AR_INTR_SYNC_HOST1_FATAL on AR9003
Date: Tue, 02 Oct 2012 15:33:28 +0200	[thread overview]
Message-ID: <506AED28.4050903@openwrt.org> (raw)
In-Reply-To: <CAJ-VmontNVV4TyRO5kEM30T=OmfEN8YUciJSW=j_0HnbadF4JA@mail.gmail.com>

On 2012-10-02 3:13 PM, Adrian Chadd wrote:
> .. well, the rule here is "You shouldn't get PERR/FATAL interrupts."
> 
> Haven't I posted a summary of what those errors are?
> 
> Ok. So they're signals from the PCIe core (named host1_fatal and
> host1_perr. Helpfully.) Those errors occured during a DMA transfer.
> 
> So the question is why you're seeing PERR interrupts when creating an
> adhoc interface. That hints to me that something odd is going on..
> 
> I've seen these issues creep up when the NIC is in some way behaving
> very, very badly (lots of timeouts and sync errors with little to no
> traffic at all), which resulted in all kinds of odd and weird,
> unstable behaviour. After replacing the NIC with another NIC (in my
> case, an AR9280 -> AR9280 NIC :-) the errors went away and things
> continued swimmingly.
> 
> I'd have to go digging through the PCIe core source to figure out
> exactly what host1_peer and host1_fatal mean. I can if you'd like,
> it'll just take some time as I'm not familiar at all with the PCIe
> host interface.
According to the datasheet, AR_INTR_SYNC_HOST1_PERR is triggered by an
invalid register access, and AR_INTR_SYNC_HOST1_FATAL is triggered by
corrupt descriptors or other DMA issues.
Maybe you can get some information on the source of this PERR error if
you record the last register accesses outside of the irq context and
print them once this IRQ comes in.

- Felix


  reply	other threads:[~2012-10-02 13:33 UTC|newest]

Thread overview: 51+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-09-27 14:41 [ath9k-devel] [PATCH] ath9k_hw: Handle AR_INTR_SYNC_HOST1_(FATAL|PERR) on AR9003 Sven Eckelmann
2012-09-27 14:41 ` Sven Eckelmann
2012-10-02 10:33 ` [ath9k-devel] [PATCHv2] ath9k_hw: Handle AR_INTR_SYNC_HOST1_FATAL " Sven Eckelmann
2012-10-02 10:33   ` Sven Eckelmann
2012-10-02 13:13   ` [ath9k-devel] " Adrian Chadd
2012-10-02 13:13     ` Adrian Chadd
2012-10-02 13:33     ` Felix Fietkau [this message]
2012-10-02 13:33       ` [ath9k-devel] " Felix Fietkau
2012-10-02 13:35     ` Simon Wunderlich
2012-10-02 13:35       ` Simon Wunderlich
2012-10-02 14:06       ` [ath9k-devel] " Adrian Chadd
2012-10-02 14:06         ` Adrian Chadd
2012-10-02 15:02         ` [ath9k-devel] " Sven Eckelmann
2012-10-02 15:02           ` Sven Eckelmann
2012-10-02 15:20           ` [ath9k-devel] " Felix Fietkau
2012-10-02 15:20             ` Felix Fietkau
2012-10-03 14:51             ` Adrian Chadd
2012-10-03 14:51               ` Adrian Chadd
2012-10-05 11:08               ` Sven Eckelmann
2012-10-05 11:08                 ` Sven Eckelmann
2012-10-05 12:34                 ` Felix Fietkau
2012-10-05 12:34                   ` Felix Fietkau
2012-10-05 13:07                   ` Sven Eckelmann
2012-10-05 13:07                     ` Sven Eckelmann
2012-10-05 13:24                     ` Felix Fietkau
2012-10-05 13:24                       ` Felix Fietkau
2012-10-05 15:03                       ` Sven Eckelmann
2012-10-05 15:03                         ` Sven Eckelmann
2012-10-05 15:15                         ` Felix Fietkau
2012-10-05 15:15                           ` Felix Fietkau
2012-10-05 16:05                           ` Sven Eckelmann
2012-10-05 16:05                             ` Sven Eckelmann
2012-10-05 16:21                   ` Adrian Chadd
2012-10-05 16:21                     ` Adrian Chadd
2012-10-05 16:51                     ` Sven Eckelmann
2012-10-05 16:51                       ` Sven Eckelmann
2012-10-05 23:48                       ` Adrian Chadd
2012-10-05 23:48                         ` Adrian Chadd
2012-10-06  9:03                         ` Felix Fietkau
2012-10-06  9:03                           ` Felix Fietkau
2013-02-21 11:14                           ` Felix Liao
2013-02-21 11:14                             ` Felix Liao
2013-02-21 20:38                             ` Adrian Chadd
2013-02-22  1:19                               ` Felix Liao
2013-02-22  5:16                                 ` Adrian Chadd
2013-02-22  6:43                                   ` Felix Liao
2013-02-22  7:18                                     ` Adrian Chadd
2013-02-22  7:31                                       ` Felix Liao
2013-02-22  8:08                                       ` Felix Liao
2013-02-22  1:33                               ` Felix Liao
2013-02-22  2:22                               ` Felix Liao

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=506AED28.4050903@openwrt.org \
    --to=nbd@openwrt.org \
    --cc=ath9k-devel@lists.ath9k.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.