On Saturday 31 January 2009 07:54:20 Larry Finger wrote:
> I changed the dma.c code to WARN_ON rather than BUG_ON

Can you send this patch upstream?

> b43 TX status reports:
> 
> index | cookie | seq | phy_stat | frame_count | rts_count |
> supp_reason | pm_indicated | intermediate | for_ampdu | acked
> ---
> 000 | 0x207E | 0x067E | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 001 | 0x2060 | 0x066E | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 002 | 0x2062 | 0x0670 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 003 | 0x2002 | 0x0680 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 004 | 0x2064 | 0x0670 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 005 | 0x2004 | 0x0681 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 006 | 0x2066 | 0x0672 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 007 | 0x2006 | 0x0682 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 008 | 0x2068 | 0x0672 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 009 | 0x2008 | 0x0683 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 010 | 0x206A | 0x0674 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 011 | 0x200A | 0x0684 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 012 | 0x206C | 0x0674 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 013 | 0x206E | 0x0676 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 014 | 0x200E | 0x0686 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 015 | 0x2070 | 0x0676 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 016 | 0x2072 | 0x0678 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 017 | 0x2012 | 0x0688 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 018 | 0x2074 | 0x0678 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 019 | 0x2076 | 0x067A | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 020 | 0x2016 | 0x068A | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 021 | 0x2078 | 0x067A | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 022 | 0x207A | 0x067C | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 023 | 0x201A | 0x068C | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 024 | 0x207C | 0x067C | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 025 | 0x201E | 0x068E | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 026 | 0x2000 | 0x067E | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 027 | 0x2020 | 0x068F | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 028 | 0x2002 | 0x0680 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 029 | 0x2022 | 0x0690 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 030 | 0x2004 | 0x0680 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 031 | 0x2006 | 0x0682 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 032 | 0x2026 | 0x0692 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 033 | 0x2008 | 0x0682 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 034 | 0x2028 | 0x0693 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 035 | 0x200A | 0x0684 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 036 | 0x202A | 0x0694 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 037 | 0x200C | 0x0684 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 038 | 0x202C | 0x0695 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 039 | 0x200E | 0x0686 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 040 | 0x202E | 0x0696 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 041 | 0x2010 | 0x0686 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 042 | 0x2030 | 0x0697 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 043 | 0x2012 | 0x0688 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 044 | 0x2034 | 0x0699 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 045 | 0x2016 | 0x068A | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 046 | 0x2036 | 0x069A | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 047 | 0x2018 | 0x068A | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 048 | 0x2038 | 0x069B | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 049 | 0x201A | 0x068C | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 050 | 0x201C | 0x068C | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 051 | 0x203C | 0x069D | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 052 | 0x201E | 0x068E | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 053 | 0x2020 | 0x068E | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 054 | 0x2040 | 0x069F | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 055 | 0x2022 | 0x0690 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 056 | 0x2042 | 0x06A0 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 057 | 0x2024 | 0x0690 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 058 | 0x2044 | 0x06A1 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 059 | 0x2026 | 0x0692 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 060 | 0x2046 | 0x06A2 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 061 | 0x2028 | 0x0692 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 062 | 0x2048 | 0x06A3 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 063 | 0x202A | 0x0694 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 064 | 0x204A | 0x06A4 | 0x00 | 0x5 | 0x0 | 0 | 0 | 0 | 0 | 1
> 065 | 0x202C | 0x0694 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 066 | 0x204C | 0x06A5 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 067 | 0x202E | 0x0696 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 068 | 0x204E | 0x06A6 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 069 | 0x2030 | 0x0696 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 070 | 0x2050 | 0x06A7 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 071 | 0x2032 | 0x0698 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 072 | 0x2052 | 0x06A8 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 073 | 0x2034 | 0x0698 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 074 | 0x2054 | 0x06A9 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 075 | 0x2036 | 0x069A | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 076 | 0x2056 | 0x06AA | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 077 | 0x2038 | 0x069A | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 078 | 0x2058 | 0x06AB | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 079 | 0x203A | 0x069C | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 080 | 0x205A | 0x06AC | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 081 | 0x203C | 0x069C | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 082 | 0x205C | 0x06AD | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 083 | 0x203E | 0x069E | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 084 | 0x205E | 0x06AE | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 085 | 0x2040 | 0x069E | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 086 | 0x2060 | 0x06AF | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 087 | 0x2042 | 0x06A0 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 088 | 0x2062 | 0x06B0 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 089 | 0x2044 | 0x06A0 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 090 | 0x2064 | 0x06B1 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 091 | 0x2046 | 0x06A2 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 092 | 0x2066 | 0x06B2 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 093 | 0x2048 | 0x06A2 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 094 | 0x2068 | 0x06B3 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 095 | 0x204A | 0x06A4 | 0x00 | 0x5 | 0x0 | 0 | 0 | 0 | 0 | 1
> 096 | 0x206A | 0x06B4 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 097 | 0x204C | 0x06A4 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 098 | 0x206C | 0x06B5 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1
> 099 | 0x204E | 0x06A6 | 0x00 | 0x1 | 0x0 | 0 | 0 | 0 | 0 | 1

Hmmm, I think the cookies should actually be more or less sequential, if
QoS and PIO are not used. That's rather strange. Can you try with original fw 
and
see what the cookies look like?

> Is there any way for me to me to freeze the interface when the first
> error occurs without inducing a kernel panic?

Introduce a flag that will drop all TX frames in b43. Just skb_free them.

-- 
Greetings, Michael.
_______________________________________________
Bcm43xx-dev mailing list
[email protected]
https://lists.berlios.de/mailman/listinfo/bcm43xx-dev

Reply via email to