Re: [Acx100-devel] [PATCH 00/20] another batch

[Oliver W. <oli...@ol...>]

Hi Jim,

On 05/15/2012 08:54 PM, Jim Cromie wrote:

> More suspiciously, Im getting tail errors:

Regarding this one: It was striking me as well. But I think that is just
a difference in log-levels. Before, this was a "log(L_BUFT,..." and in
normal log-config not shown, now it's a pr_warn.

So probably that's no real issue and just the logging needs to be adjusted.

merge.c (HEAD)
---
> 		if ((Ctl_8 & DESC_CTL_ACXDONE_HOSTOWN)
> 			!= DESC_CTL_ACXDONE_HOSTOWN) {
> 			/* maybe remove if wrapper */
> 			if (unlikely(!num_cleaned))
> 				pr_warn("clean_txdesc: tail isn't free. "
> 					"finger=%d, tail=%d, head=%d\n",
> 					finger,	adev->tx.tail, adev->tx_head);
---

previous pci.c (0301ec35)
---
> 		if ((Ctl_8 & DESC_CTL_ACXDONE_HOSTOWN) != DESC_CTL_ACXDONE_HOSTOWN) {
> 			//if (unlikely(!num_cleaned)) { /* maybe remove completely */
> 			log(L_BUFT, "clean_txdesc: tail isn't free. "
> 				"finger=%d, tail=%d, head=%d\n", finger, adev->tx_tail,
> 					adev->tx_head);
---

previous mem.c (0301ec35)
---
> 		if ((Ctl_8 & DESC_CTL_ACXDONE_HOSTOWN) != DESC_CTL_ACXDONE_HOSTOWN) {
> 			//if (unlikely(!num_cleaned)) { /* maybe remove completely */
> 			log(L_BUFT, "clean_txdesc: tail isn't free. "
> 				"finger=%d, tail=%d, head=%d\n", finger, adev->tx_tail,
> 					adev->tx_head);
---

Best Regards, Oliver

> 
> acx.acx_tx_clean_txdesc: clean_txdesc: tail isn't free. finger=2, tail=2, head=9
> acx.acx_tx_clean_txdesc: clean_txdesc: tail isn't free. finger=11,
> tail=11, head=1
> acx.acx_tx_clean_txdesc: clean_txdesc: tail isn't free. finger=10,
> tail=10, head=2
> acx.acx_tx_clean_txdesc: clean_txdesc: tail isn't free. finger=15,
> tail=15, head=4
> acx.acx_tx_clean_txdesc: clean_txdesc: tail isn't free. finger=14,
> tail=14, head=5
> acx.acx_tx_clean_txdesc: clean_txdesc: tail isn't free. finger=0, tail=0, head=9
> acx.acx_tx_clean_txdesc: clean_txdesc: tail isn't free. finger=7,
> tail=7, head=14
> acx.acx_tx_clean_txdesc: clean_txdesc: tail isn't free. finger=0,
> tail=0, head=12
> acx.acx_tx_clean_txdesc: clean_txdesc: tail isn't free. finger=12,
> tail=12, head=1
> acx.acx_tx_clean_txdesc: clean_txdesc: tail isn't free. finger=7, tail=7, head=8
> acx.acx_tx_clean_txdesc: clean_txdesc: tail isn't free. finger=8,
> tail=8, head=13
> 
> 
> In all cases (I think), finger==tail, which suggests an off-by-one error,
> probably due to merge, but I havent looked yet.
> 
> Iperf testing works, as does scp of a large tarball, checksums match,
> so its affecting thruput at worst.
> 
> 
> there are a bunch of retries though:
> 
> root@voyage:~# iw dev wlan1 station dump
> Failed to read classid file: Object not found
> Station 00:14:d1:e8:65:0a (on wlan1)
> 	inactive time:	14740 ms
> 	rx bytes:	61152680
> 	rx packets:	276428
> 	tx bytes:	72244666
> 	tx packets:	65471
> 	tx retries:	18447
> 	tx failed:	2
> 	tx bitrate:	54.0 MBit/s
> 
> 
> root@voyage:~# more /dbg/acx_mac80211/phy10/info
> acx.acx_dbgfs_open: opening filename=info fmode=35 fidx=0 adev=c0bd2fc0
> acx driver version:		v0.6.0-glarry-512-19-g0fde744 (git: larry-512-19
> -g0fde744)
> Wireless extension version:	22
> chip name:			TNETW1130 (0x03010101)
> radio type:			0x16
> form factor:			0x01
> EEPROM version:			0x05
> firmware version:		Rev 1.2.1.34 (0x01020134)
> 
> larry-512 is the (mis-dated) tag I applied to Larrys 15 on 509
> 
> 
> Im attaching a test script Ive been using to bang on the driver
> Dot it into your shell, and go from there.
> 
> root@voyage:~# . wifi-probe.rc
> wifi-help for info
> root@voyage:~# wifi-help
> 
> provides functions to probe, test wifi drivers using iw-tools
>    wifi-probe:	run the probes/tests
>    mon-up:	create monitor ifaces, do tcpdump captures on them
>    mon-down:	close those monitor interfaces
>    wifi-save:	save the files to to driver-version dir
>    wifi-net:	run some connectivity tests (ifdowns eth0 wlan0)
> vars: # set to change behavior
>    devs:	wlan0 wlan1
>    drvname:	acx
>    apmac:	00:14:d1:e8:65:0a		# not used now
>    chan:	11
>    macs:	00:18:DE:06:B2:0C 00:30:BD:4D:E2:47 00:18:4D:24:B3:FA
>    iperfsvrs:	192.168.10.127 192.168.10.128	# iperf server addresses
>    iperfargs:	-t30 -i5
>    others:	eth0 wlan0		# ifdown these in wifi-ent
> 
> 
> Please suggests mods, enhancements useful to you.
> 2 Ive just thought of:
> - snapshot of /dbg/acx-mac80211/phyX
> - snapshots of /dbg/ieee80211/phyX
> 
> Maybe commit this to the tree ?
> 
> ------------------------------------------------------------------------------
> Live Security Virtual Conference
> Exclusive live event will cover all the ways today's security and 
> threat landscape has changed and how IT managers can respond. Discussions 
> will include endpoint security, mobile security and the latest in malware 
> threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
> _______________________________________________
> Acx100-devel mailing list
> Acx...@li...
> https://lists.sourceforge.net/lists/listinfo/acx100-devel
>