Linux wireless drivers development
 help / color / mirror / Atom feed
From: Dan Williams <dcbw@redhat.com>
To: Sebastian Siewior <ipw+2200@ml.breakpoint.cc>
Cc: "Chatre, Reinette" <reinette.chatre@intel.com>,
	"Zhu, Yi" <yi.zhu@intel.com>,
	James Ketrenos <jketreno@linux.intel.com>,
	linux-wireless@vger.kernel.org,
	ipw2100-devel@lists.sourceforge.net
Subject: Re: ipw2200 stalls on high load
Date: Mon, 04 Feb 2008 18:24:10 -0500	[thread overview]
Message-ID: <1202167450.5190.2.camel@localhost.localdomain> (raw)
In-Reply-To: <20080204224502.GA22717@Chamillionaire.breakpoint.cc>

On Mon, 2008-02-04 at 23:45 +0100, Sebastian Siewior wrote:
> * Chatre, Reinette | 2008-02-04 10:23:49 [-0800]:
> 
> >On Monday, February 04, 2008 4:37 AM, Dan Williams  wrote:
> >
> >> Something like the following?  Turns out the rxq->processed
> >> isn't really
> >> used that much, and 3945/4965 don't use that field at all (but use
> >> ->read exclusively instead).  And since it appears that the replenish
> >> function is simpler in the 2200, it doesn't need to be split like
> >> 3945/4965.  I haven't been able to stress my 2200 enough to trigger
> >> the new codepath though. 
> >
> >Thank you very much! 
> Yes, thanks for the patch.
> 
> >Sebastian, does this change work for you?
> No, it doesn't. I get the following:

Could you put some debugging information into ipw_rx() to print out the
values of r and i right before the while (i != r) loop, and inside the
if (fill_rx) block later down what count and i are?

Also, what's the procedure to reproduce this again?  I couldn't get that
bit to trigger but I wasn't really sure what to do to stress the 2200
that far, otherwise I could have tested the patch more before posting.

Thanks,
Dan

> | ipw2200: Firmware error detected.  Restarting.
> | ipw2200: Start IPW Error Log Dump:
> | ipw2200: Status: 0x800000E0, Config: 00000347
> | ipw2200: NMI_INTERRUPT 493005888 0x000003b4  0x00000000  0x00000250  0x0000f420  0x00000000
> | ipw2200: DMA_STATUS 493005892 0x00027980  0x00027740  0x01540002  0x00000000  0x00000000
> | ipw2200: DMA_STATUS 493005895 0x00028400  0x00028670  0x00540001  0x00000000  0x00000001
> | ipw2200: DMA_STATUS 493005899 0x00028000  0x00028190  0x00540001  0x00000000  0x00000002
> | ipw2200: DMA_STATUS 493005903 0x00400000  0x00408000  0x00408000  0x00000087  0x00000003
> | ipw2200: 492475810	0x00000008	50
> | ipw2200: 492475836	0x0000003c	264
> | ipw2200: 492475841	0x0002a9c0	74
> | ipw2200: 492475846	0x00000042	208
> | ipw2200: 492477710	0x00000008	32
> | ipw2200: 492477738	0x00000008	50
> | ipw2200: 492477790	0x0000003c	264
> | ipw2200: 492477796	0x0002a930	74
> | ipw2200: 492477800	0x00000042	208
> | ipw2200: 492479989	0x00000008	32
> | ipw2200: 492480017	0x00000008	50
> | ipw2200: 492480043	0x0000003c	264
> | ipw2200: 492480048	0x0002a990	74
> | ipw2200: 492480053	0x00000042	208
> | ipw2200: 492481989	0x00000008	32
> | ipw2200: 492482017	0x00000008	50
> | ipw2200: 492482051	0x0000003c	264
> | ipw2200: 492482056	0x0002a970	74
> | ipw2200: 492482061	0x00000042	208
> | ipw2200: 492484133	0x00000008	32
> | ipw2200: 492484161	0x00000008	50
> | ipw2200: 492484189	0x0000003c	264
> | ipw2200: 492484194	0x0002a880	74
> | ipw2200: 492484199	0x00000042	208
> | ipw2200: 492498961	0x00000001	198
> | ipw2200: 492499005	0x0002a8e0	67
> | ipw2200: 492499025	0x0000026c	61
> | ipw2200: 492507155	0x00000389	140
> | ipw2200: 492507158	0x00000061	139
> | ipw2200: 492507161	0x00000392	140
> | ipw2200: 492507169	0x00000001	136
> | ipw2200: 492507173	0x0000029c	138
> | ipw2200: 492507177	0x000002ca	138
> | ipw2200: 492507180	0x00000177	84
> | ipw2200: 492507185	0x00000005	81
> | ipw2200: 492507188	0x00000003	82
> | ipw2200: 492507191	0x00000006	83
> | ipw2200: 492507196	0x0000039f	140
> | ipw2200: 492507199	0x00000006	139
> | ipw2200: 492507202	0x000003ad	139
> | ipw2200: 492509617	0x00000001	32
> | ipw2200: 492509620	0x0000023f	179
> | ipw2200: 492509624	0x00000633	140
> | ipw2200: 492509627	0x00000640	140
> | ipw2200: 492509631	0x00000177	84
> | ipw2200: 492509635	0x00000006	81
> | ipw2200: 492509638	0x00000004	82
> | ipw2200: 492509641	0x00000007	83
> | ipw2200: 492509645	0x0000054d	183
> | ipw2200: 492509651	0x00000009	184
> | ipw2200: 492509654	0x00000455	189
> | ipw2200: 492509657	0x00000000	189
> | ipw2200: 492509661	0x00000007	184
> | ipw2200: 492509664	0x00000004	183
> | ipw2200: 492509669	0x0000042b	191
> | ipw2200: 492448433	0x0000003d	264
> | ipw2200: 492448438	0x0002a960	74
> | ipw2200: 492448547	0x000000b1	200
> | ipw2200: 492450315	0x00000008	32
> | ipw2200: 492450343	0x00000008	50
> | ipw2200: 492450369	0x0000003d	264
> | ipw2200: 492450374	0x0002a9e0	74
> | ipw2200: 492450483	0x000000b1	200
> | ipw2200: 492452305	0x00000008	32
> | ipw2200: 492452333	0x00000008	50
> | ipw2200: 492452359	0x0000003d	264
> | ipw2200: 492452364	0x0002a9a0	74
> | ipw2200: 492452473	0x000000b1	200
> | ipw2200: 492454503	0x00000008	32
> | ipw2200: 492454531	0x00000008	50
> | ipw2200: 492454557	0x0000003d	264
> | ipw2200: 492454562	0x0002a960	74
> | ipw2200: 492454671	0x000000b1	200
> | ipw2200: 492456782	0x00000008	32
> | ipw2200: 492456810	0x00000008	50
> | ipw2200: 492456836	0x0000003d	264
> | ipw2200: 492456841	0x0002a9e0	74
> | ipw2200: 492456950	0x000000b1	200
> | ipw2200: 492458746	0x00000008	32
> | ipw2200: 492458774	0x00000008	50
> | ipw2200: 492458800	0x0000003d	264
> | ipw2200: 492458805	0x0002a9a0	74
> | ipw2200: 492458914	0x000000b1	200
> | ipw2200: 492459161	0x00000001	198
> | ipw2200: 492459201	0x0002a8e0	67
> | ipw2200: 492459221	0x0000026c	61
> | ipw2200: 492459341	0x00000008	32
> | ipw2200: 492459352	0x0000000b	35
> | ipw2200: 492459356	0x0000000b	24
> | ipw2200: 492459365	0x00000172	25
> | ipw2200: 492459369	0x0002a8e0	98
> | ipw2200: 492461603	0x00000008	32
> | ipw2200: 492461631	0x00000008	50
> | ipw2200: 492461657	0x0000003c	264
> | ipw2200: 492461662	0x0002a990	74
> | ipw2200: 492461771	0x000000b1	200
> | ipw2200: 492463197	0x00000008	32
> | ipw2200: 492463225	0x00000008	50
> | ipw2200: 492463251	0x0000003c	264
> | ipw2200: 492463256	0x0002a9c0	74
> | ipw2200: 492463365	0x000000b1	200
> | ipw2200: 492465224	0x00000008	32
> | ipw2200: 492465252	0x00000008	50
> | ipw2200: 492465290	0x0000003c	264
> | ipw2200: 492465296	0x0002a930	74
> | ipw2200: 492465404	0x000000b1	200
> | ipw2200: 492467359	0x00000008	32
> | ipw2200: 492467387	0x00000008	50
> | ipw2200: 492467413	0x0000003c	264
> | ipw2200: 492467418	0x0002a990	74
> | ipw2200: 492467527	0x000000b1	200
> | ipw2200: 492469413	0x00000008	32
> | ipw2200: 492469441	0x00000008	50
> | ipw2200: 492469467	0x0000003c	264
> | ipw2200: 492469472	0x0002a9c0	74
> | ipw2200: 492469581	0x000000b1	200
> | ipw2200: 492471611	0x00000008	32
> | ipw2200: 492471639	0x00000008	50
> | ipw2200: 492471665	0x0000003c	264
> | ipw2200: 492471670	0x0002a930	74
> | ipw2200: 492471779	0x000000b1	200
> | ipw2200: 492473854	0x00000008	32
> | ipw2200: 492473882	0x00000008	50
> | ipw2200: 492473908	0x0000003c	264
> | ipw2200: 492473913	0x0002a990	74
> | ipw2200: 492474022	0x000000b1	200
> | ipw2200: 492474164	0x00000042	208
> | ipw2200: 492475782	0x00000008	32
> | ipw2200: U ipw_load initial device response after 10ms
> | ipw2200: U ipw_stop_master stop master 0ms
> | ipw2200: U ipw_load_ucode Microcode OK, rev. 53594 (0xd15a) dev. 3 (0x3) of 11/22/04 20:27
> | ipw2200: U ipw_load device response after 50ms
> 
> I can provide you a full log if you want.
> >Reinette
> >
> 
> Sebastian


  reply	other threads:[~2008-02-04 23:24 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-01-26 13:29 ipw2200 stalls on high load Sebastian Siewior
2008-01-28 18:40 ` Chatre, Reinette
2008-01-30 22:57   ` Sebastian Siewior
2008-02-01 22:29     ` Chatre, Reinette
2008-02-04 12:37       ` Dan Williams
2008-02-04 18:23         ` Chatre, Reinette
2008-02-04 22:45           ` Sebastian Siewior
2008-02-04 23:24             ` Dan Williams [this message]
2008-02-05  8:35               ` Sebastian Siewior
2008-02-05 15:09                 ` Dan Williams
2008-02-05 16:50                   ` [Ipw2100-devel] " Cahill, Ben M
2008-02-05 23:53                   ` Sebastian Siewior
2008-02-06  4:56                     ` Dan Williams
2008-02-08  7:50                       ` Sebastian Siewior
2008-01-28 18:53 ` [Ipw2100-devel] " Cahill, Ben M
2008-01-28 22:54   ` Sebastian Siewior
2008-01-30 22:48   ` Sebastian Siewior

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=1202167450.5190.2.camel@localhost.localdomain \
    --to=dcbw@redhat.com \
    --cc=ipw+2200@ml.breakpoint.cc \
    --cc=ipw2100-devel@lists.sourceforge.net \
    --cc=jketreno@linux.intel.com \
    --cc=linux-wireless@vger.kernel.org \
    --cc=reinette.chatre@intel.com \
    --cc=yi.zhu@intel.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox