All of lore.kernel.org
 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 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.