* [ath9k-devel] ath9k_htc test release: please test
2012-11-28 23:50 [ath9k-devel] ath9k_htc test release: please test Adrian Chadd
@ 2012-11-30 2:14 ` Stefan Lippers-Hollmann
2012-11-30 4:46 ` vikranth reddy
` (2 subsequent siblings)
3 siblings, 0 replies; 15+ messages in thread
From: Stefan Lippers-Hollmann @ 2012-11-30 2:14 UTC (permalink / raw)
To: ath9k-devel
Hi
On Thursday 29 November 2012, Adrian Chadd wrote:
> Hi all,
>
> I've done an updated build of the ath9k_htc firmware images. I don't
> currently have a HTC setup working so this is just a recompilation of
> the current firmware.
>
> http://dev.qca.qualcomm.com/~adrian/htc/20121128/
>
> Please let me know if it works or doesn't work for you.
[?]
Seems to work for me with ar9271/ TP-Link TL-WN721N, I don't notice
anything strange so far. Feel free to ask me for any specific tests.
Bus 001 Device 003: ID 0cf3:9271 Atheros Communications, Inc. AR9271 802.11n
61a01f78317eff6675f59f6a111846dd */lib/firmware/htc_9271.fw
[ 68.843124] usb 1-2: new high-speed USB device number 3 using ehci_hcd
[ 68.977959] usb 1-2: New USB device found, idVendor=0cf3, idProduct=9271
[ 68.977977] usb 1-2: New USB device strings: Mfr=16, Product=32, SerialNumber=48
[ 68.977989] usb 1-2: Product: USB2.0 WLAN
[ 68.978000] usb 1-2: Manufacturer: ATHEROS
[ 68.978070] usb 1-2: SerialNumber: 12345
[ 69.013788] usb 1-2: ath9k_htc: Firmware htc_9271.fw requested
[ 69.013873] usbcore: registered new interface driver ath9k_htc
[ 69.308128] usb 1-2: ath9k_htc: Transferred FW: htc_9271.fw, size: 51272
[ 69.543716] ath9k_htc 1-2:1.0: ath9k_htc: HTC initialized with 33 credits
[ 69.753110] ath9k_htc 1-2:1.0: ath9k_htc: FW Version: 1.3
[ 69.753123] ath: EEPROM regdomain: 0x809c
[ 69.753130] ath: EEPROM indicates we should expect a country code
[ 69.753138] ath: doing EEPROM country->regdmn map search
[ 69.753145] ath: country maps to regdmn code: 0x52
[ 69.753153] ath: Country alpha2 being used: CN
[ 69.753160] ath: Regpair used: 0x52
[ 69.755825] ieee80211 phy1: Atheros AR9271 Rev:1
[ 69.756598] cfg80211: Calling CRDA for country: CN
[ 69.762598] Registered led device: ath9k_htc-phy1
[ 69.784992] cfg80211: Current regulatory domain intersected:
[ 69.785376] cfg80211: (start_freq - end_freq @ bandwidth), (max_antenna_gain, max_eirp)
[ 69.785387] cfg80211: (2402000 KHz - 2482000 KHz @ 40000 KHz), (N/A, 2000 mBm)
[ 72.936100] cfg80211: Calling CRDA to update world regulatory domain
[ 72.956353] cfg80211: World regulatory domain updated:
[ 72.956361] cfg80211: (start_freq - end_freq @ bandwidth), (max_antenna_gain, max_eirp)
[ 72.956368] cfg80211: (2402000 KHz - 2472000 KHz @ 40000 KHz), (300 mBi, 2000 mBm)
[ 72.956373] cfg80211: (2457000 KHz - 2482000 KHz @ 20000 KHz), (300 mBi, 2000 mBm)
[ 72.956378] cfg80211: (2474000 KHz - 2494000 KHz @ 20000 KHz), (300 mBi, 2000 mBm)
[ 72.956383] cfg80211: (5170000 KHz - 5250000 KHz @ 40000 KHz), (300 mBi, 2000 mBm)
[ 72.956389] cfg80211: (5735000 KHz - 5835000 KHz @ 40000 KHz), (300 mBi, 2000 mBm)
[ 73.708543] IPv6: ADDRCONF(NETDEV_UP): wlan1: link is not ready
[ 74.906125] wlan1: authenticate with <MAC redacted>
[ 75.000387] wlan1: send auth to <MAC redacted> (try 1/3)
[ 75.002913] wlan1: authenticated
[ 75.052045] wlan1: associate with <MAC redacted> (try 1/3)
[ 75.055530] wlan1: RX AssocResp from <MAC redacted> (capab=0x411 status=0 aid=1)
[ 75.059506] wlan1: associated
[ 75.059586] IPv6: ADDRCONF(NETDEV_CHANGE): wlan1: link becomes ready
[ 75.059739] cfg80211: Calling CRDA for country: DE
[ 75.077662] ath: regdomain 0x8114 updated by CountryIE
[ 75.077673] ath: EEPROM regdomain: 0x8114
[ 75.077678] ath: EEPROM indicates we should expect a country code
[ 75.077683] ath: doing EEPROM country->regdmn map search
[ 75.077688] ath: country maps to regdmn code: 0x37
[ 75.077693] ath: Country alpha2 being used: DE
[ 75.077698] ath: Regpair used: 0x37
[ 75.077704] cfg80211: Regulatory domain changed to country: DE
[ 75.077710] cfg80211: (start_freq - end_freq @ bandwidth), (max_antenna_gain, max_eirp)
[ 75.077718] cfg80211: (2400000 KHz - 2483500 KHz @ 40000 KHz), (N/A, 2000 mBm)
[ 75.077726] cfg80211: (5150000 KHz - 5250000 KHz @ 40000 KHz), (N/A, 2000 mBm)
[ 75.077732] cfg80211: (5250000 KHz - 5350000 KHz @ 40000 KHz), (N/A, 2000 mBm)
[ 75.077739] cfg80211: (5470000 KHz - 5725000 KHz @ 40000 KHz), (N/A, 2698 mBm)
The access point is a TP-Link TL-WDR4300 (AR9344+AR9340) running a
recent OpenWrt build/ ath9k.
By the way, do you happen to know an ar7010+ar9280 dualband/ 2-stream
device that's a little easier to get hold of than the Netgear WNDA3200
(UK-only) or the Sony UWA-BR100?
Regards
Stefan Lippers-Hollmann
^ permalink raw reply [flat|nested] 15+ messages in thread* [ath9k-devel] ath9k_htc test release: please test
2012-11-28 23:50 [ath9k-devel] ath9k_htc test release: please test Adrian Chadd
2012-11-30 2:14 ` Stefan Lippers-Hollmann
@ 2012-11-30 4:46 ` vikranth reddy
2012-11-30 8:59 ` Adrian Chadd
2012-11-30 17:50 ` Oleksij Rempel
2012-12-01 14:31 ` Christian Lamparter
3 siblings, 1 reply; 15+ messages in thread
From: vikranth reddy @ 2012-11-30 4:46 UTC (permalink / raw)
To: ath9k-devel
Hi,
I have tested it using Sony BR-100 (AR7010 + AR9280). PFA the time diff
logs for the new firmware.(htc_7010.fw)
I added prints for time diff in
file:- ~/drivers/net/wireless/ath/ath9k/htc_drv_txrx.c
function:- ath9k_rx_tasklet
position:- just before ieee80211_rx function.
*Code Snippet:*
void ath9k_rx_tasklet(unsigned long data)
{
struct ath9k_htc_priv *priv = (struct ath9k_htc_priv *)data;
struct ath9k_htc_rxbuf *rxbuf = NULL, *tmp_buf = NULL;
struct ieee80211_rx_status rx_status;
struct sk_buff *skb;
unsigned long flags;
struct ieee80211_hdr *hdr;
/* Added to measure Time-Difference between successive WiFi Packets */
static u64 prev1,now1;
long long diff1 = 0;
static u32 counter1 = 0;
/****************************************/
do {
spin_lock_irqsave(&priv->rx.rxbuflock, flags);
list_for_each_entry(tmp_buf, &priv->rx.rxbuf, list) {
if (tmp_buf->in_process) {
rxbuf = tmp_buf;
break;
}
}
if (rxbuf == NULL) {
spin_unlock_irqrestore(&priv->rx.rxbuflock, flags);
break;
}
if (!rxbuf->skb)
goto requeue;
if (!ath9k_rx_prepare(priv, rxbuf, &rx_status)) {
dev_kfree_skb_any(rxbuf->skb);
goto requeue;
}
memcpy(IEEE80211_SKB_RXCB(rxbuf->skb), &rx_status,
sizeof(struct ieee80211_rx_status));
skb = rxbuf->skb;
hdr = (struct ieee80211_hdr *) skb->data;
if (ieee80211_is_beacon(hdr->frame_control) && priv->ps_enabled)
ieee80211_queue_work(priv->hw, &priv->ps_work);
spin_unlock_irqrestore(&priv->rx.rxbuflock, flags);
/* Added to measure Time-Difference between successive WiFi Packets */
now1 = be64_to_cpu(rxbuf->rxstatus.rs_tstamp);
diff1 = now1 - prev1;
if(diff1 < 0) {
printk("#######now = %llu prev = %llu diff = %lld counter =
%u\n",now1,prev1,diff1,counter1++);
}
prev1 = now1;
/**********************************************************************/
ieee80211_rx(priv->hw, skb);
spin_lock_irqsave(&priv->rx.rxbuflock, flags);
...................
...........
}
Nov 29 10 36 29 [ 103.724123] #######now = 2683579500078 prev =
2684354679445 diff = -775179367
Nov 29 10 36 49 [ 122.787519] #######now = 64609 prev = 2683598441314
diff = -2683598376705
Nov 29 10 36 49 [ 122.947963] #######now = 2683598748946 prev =
2684354687871 diff = -755938925
Nov 29 10 36 50 [ 123.766144] #######now = 2683599568206 prev =
2684354697547 diff = -755129341
Nov 29 10 36 50 [ 124.072922] #######now = 2683599875386 prev =
2684354601000 diff = -754725614
Nov 29 10 36 50 [ 124.481997] #######now = 2683600284967 prev =
2684354598998 diff = -754314031
Nov 29 10 36 51 [ 124.890184] #######now = 2683600693668 prev =
2684354605871 diff = -753912203
Nov 29 10 36 51 [ 125.298751] #######now = 51576 prev = 2683601031091
diff = -2683600979515
Still I see the negative time-differences between successive WiFi-packets,
when it is stayed connected to single AP. In the log, *now *is the TSF of
current packet and *prev *is previous WiFi Packet's timestamp. All the
variables are 64-bit.
But the difference between new firmware and old-firmware is, I see less
number of negative differences comparatively.
Thanks
Vikranth
On Thu, Nov 29, 2012 at 5:20 AM, Adrian Chadd <adrian@freebsd.org> wrote:
> Hi all,
>
> I've done an updated build of the ath9k_htc firmware images. I don't
> currently have a HTC setup working so this is just a recompilation of
> the current firmware.
>
> http://dev.qca.qualcomm.com/~adrian/htc/20121128/
>
> Please let me know if it works or doesn't work for you.
>
> The specific two changes from Sujith's last release:
>
> * Fix the TSF extension logic to correctly extend TSF from 32 bit to 64
> bit;
> * Update the embedded runtime OS code to the latest from Tensilica, in
> preparation for open sourcing it;
>
> I don't plan on fixing any bugs that already exist in the latest
> ath9k_htc drop in the firmware git repository; I just want to check
> that the build hasn't broken any existing behaviour (ie, "made things
> worse.")
>
> Thanks,
>
>
> Adrian
> _______________________________________________
> ath9k-devel mailing list
> ath9k-devel at lists.ath9k.org
> https://lists.ath9k.org/mailman/listinfo/ath9k-devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.ath9k.org/pipermail/ath9k-devel/attachments/20121130/e0b3402e/attachment-0001.htm
-------------- next part --------------
Nov 29 10 36 29 [ 103.724123] #######now = 2683579500078 prev = 2684354679445 diff = -775179367
Nov 29 10 36 49 [ 122.787519] #######now = 64609 prev = 2683598441314 diff = -2683598376705
Nov 29 10 36 49 [ 122.947963] #######now = 2683598748946 prev = 2684354687871 diff = -755938925
Nov 29 10 36 50 [ 123.766144] #######now = 2683599568206 prev = 2684354697547 diff = -755129341
Nov 29 10 36 50 [ 124.072922] #######now = 2683599875386 prev = 2684354601000 diff = -754725614
Nov 29 10 36 50 [ 124.481997] #######now = 2683600284967 prev = 2684354598998 diff = -754314031
Nov 29 10 36 51 [ 124.890184] #######now = 2683600693668 prev = 2684354605871 diff = -753912203
Nov 29 10 36 51 [ 125.298751] #######now = 51576 prev = 2683601031091 diff = -2683600979515
Nov 29 10 36 51 [ 125.402879] #######now = 2683601207126 prev = 2684354714137 diff = -753507011
Nov 29 10 36 52 [ 125.709112] #######now = 2683601513747 prev = 2684354616508 diff = -753102761
Nov 29 10 36 52 [ 126.118192] #######now = 2683601923367 prev = 2684354621369 diff = -752698002
Nov 29 10 36 52 [ 126.525913] #######now = 2683602331626 prev = 2684354622966 diff = -752291340
Nov 29 10 36 53 [ 126.934895] #######now = 66060 prev = 2683602538321 diff = -2683602472261
Nov 29 10 36 53 [ 126.937001] #######now = 2683602743237 prev = 2684354627766 diff = -751884529
Nov 29 10 36 53 [ 127.344169] #######now = 43804 prev = 2683602947705 diff = -2683602903901
Nov 29 10 36 53 [ 127.446028] #######now = 2683603252869 prev = 2684354605530 diff = -751352661
Nov 29 10 36 54 [ 127.855970] #######now = 119256 prev = 2683603357457 diff = -2683603238201
Nov 29 10 36 54 [ 127.957972] #######now = 2683603765423 prev = 2684354681002 diff = -750915579
Nov 29 10 36 54 [ 128.367178] #######now = 2683604175173 prev = 2684354653927 diff = -750478754
Nov 29 10 36 54 [ 128.577010] #######now = 66304 prev = 2683604279313 diff = -2683604213009
Nov 29 10 36 55 [ 128.778373] #######now = 2683604586967 prev = 2684354728542 diff = -750141575
Nov 29 10 36 55 [ 129.186205] #######now = 2683604995418 prev = 2684354630673 diff = -749635255
Nov 29 10 36 55 [ 129.697537] #######now = 2683605507346 prev = 2684354637989 diff = -749130643
Nov 29 10 36 56 [ 130.208891] #######now = 2683606019328 prev = 2684354646094 diff = -748626766
Nov 29 10 36 57 [ 130.720212] #######now = 2683606531366 prev = 2684354654961 diff = -748123595
Nov 29 10 36 57 [ 131.231544] #######now = 2683607043367 prev = 2684354662088 diff = -747618721
Nov 29 10 36 58 [ 131.742895] #######now = 2683607555326 prev = 2684354670467 diff = -747115141
Nov 29 10 36 58 [ 132.181622] #######now = 46914 prev = 2683607760127 diff = -2683607713213
Nov 29 10 36 58 [ 132.254231] #######now = 2683608067328 prev = 2684354677961 diff = -746610633
Nov 29 10 36 59 [ 132.764206] #######now = 127595 prev = 2683608272129 diff = -2683608144534
Nov 29 10 36 59 [ 132.866188] #######now = 2683608680070 prev = 2684354689635 diff = -746009565
Nov 29 10 36 59 [ 133.275498] #######now = 2683609090045 prev = 2684354694477 diff = -745604432
Nov 29 10 36 59 [ 133.685979] #######now = 2683609500968 prev = 2684354600330 diff = -745099362
Nov 29 10 37 00 [ 134.197341] #######now = 2683610012948 prev = 2684354607825 diff = -744594877
Nov 29 10 37 01 [ 134.708585] #######now = 2683610524968 prev = 2684354619956 diff = -744094988
Nov 29 10 37 01 [ 135.220045] #######now = 2683611037154 prev = 2684354696474 diff = -743659320
Nov 29 10 37 01 [ 135.629090] #######now = 2683611446586 prev = 2684354669672 diff = -743223086
Nov 29 10 37 02 [ 136.038231] #######now = 2683611856410 prev = 2684354643750 diff = -742787340
Nov 29 10 37 02 [ 136.447144] #######now = 2683612265768 prev = 2684354617356 diff = -742351588
Nov 29 10 37 03 [ 137.265206] #######now = 1691147881440 prev = 1692217253220 diff = -1069371780
Nov 29 10 37 29 [ 162.728097] #######now = 62373 prev = 1691173277442 diff = -1691173215069
Nov 29 10 37 29 [ 162.732520] #######now = 1691173381622 prev = 1692217178674 diff = -1043797052
Nov 29 10 37 29 [ 162.932563] #######now = 1691173581973 prev = 1692217188997 diff = -1043607024
Nov 29 10 37 29 [ 163.239439] #######now = 78127 prev = 1691173788448 diff = -1691173710321
Nov 29 10 37 29 [ 163.341701] #######now = 1691173991572 prev = 1692217194407 diff = -1043202835
Nov 29 10 37 29 [ 163.648393] #######now = 82713 prev = 1691174196464 diff = -1691174113751
Nov 29 10 37 30 [ 163.750758] #######now = 1691174401169 prev = 1692217199033 diff = -1042797864
Nov 29 10 37 30 [ 164.057483] #######now = 85560 prev = 1691174505659 diff = -1691174420099
Nov 29 10 37 30 [ 164.159756] #######now = 1691174810764 prev = 1692217201840 diff = -1042391076
Nov 29 10 37 30 [ 164.466578] #######now = 89902 prev = 1691175015563 diff = -1691174925661
Nov 29 10 37 30 [ 164.468544] #######now = 1691175120010 prev = 1692217206202 diff = -1042086192
Nov 29 10 37 31 [ 164.875697] #######now = 91895 prev = 1691175418903 diff = -1691175327008
Nov 29 10 37 31 [ 164.977923] #######now = 1691175629958 prev = 1692217208236 diff = -1041578278
Nov 29 10 37 31 [ 165.284777] #######now = 93248 prev = 1691175828502 diff = -1691175735254
Nov 29 10 37 31 [ 165.387056] #######now = 1691176039555 prev = 1692217209589 diff = -1041170034
Nov 29 10 37 32 [ 165.693735] #######now = 95669 prev = 1691176143632 diff = -1691176047963
Nov 29 10 37 32 [ 165.796120] #######now = 1691176449153 prev = 1692217212009 diff = -1040762856
Nov 29 10 37 32 [ 166.102830] #######now = 96565 prev = 1691176553229 diff = -1691176456664
Nov 29 10 37 32 [ 166.205087] #######now = 1691176858749 prev = 1692217212866 diff = -1040354117
Nov 29 10 37 32 [ 166.511926] #######now = 98406 prev = 1691176962806 diff = -1691176864400
Nov 29 10 37 32 [ 166.614196] #######now = 1691177268347 prev = 1692217214727 diff = -1039946380
Nov 29 10 37 33 [ 166.921012] #######now = 100886 prev = 1691177373175 diff = -1691177272289
Nov 29 10 37 33 [ 167.023289] #######now = 1691177677943 prev = 1692217217227 diff = -1039539284
Nov 29 10 37 33 [ 167.329981] #######now = 77474 prev = 1691177782039 diff = -1691177704565
Nov 29 10 37 33 [ 167.432350] #######now = 1691178087540 prev = 1692217193755 diff = -1039106215
Nov 29 10 37 34 [ 167.739042] #######now = 51197 prev = 1691178192136 diff = -1691178140939
Nov 29 10 37 34 [ 167.841354] #######now = 1691178497138 prev = 1692217167517 diff = -1038670379
Nov 29 10 37 34 [ 168.250395] #######now = 128441 prev = 1691178601562 diff = -1691178473121
Nov 29 10 37 34 [ 168.352676] #######now = 1691179009133 prev = 1692217244760 diff = -1038235627
Nov 29 10 37 35 [ 168.659523] #######now = 101184 prev = 1691179113190 diff = -1691179012006
Nov 29 10 37 35 [ 168.761745] #######now = 1691179418730 prev = 1692217217465 diff = -1037798735
Nov 29 10 37 35 [ 169.170861] #######now = 109837 prev = 1691179522846 diff = -1691179413009
Nov 29 10 37 35 [ 169.273048] #######now = 1691179930730 prev = 1692217226177 diff = -1037295447
Nov 29 10 37 36 [ 169.682202] #######now = 116905 prev = 1691180034844 diff = -1691179917939
Nov 29 10 37 36 [ 169.784415] #######now = 1691180442724 prev = 1692217233225 diff = -1036790501
Nov 29 10 37 36 [ 170.193540] #######now = 125588 prev = 1691180546938 diff = -1691180421350
Nov 29 10 37 36 [ 170.295746] #######now = 1691180954719 prev = 1692217241928 diff = -1036287209
Nov 29 10 37 37 [ 170.704881] #######now = 133538 prev = 1691181058815 diff = -1691180925277
Nov 29 10 37 37 [ 170.807101] #######now = 1691181466715 prev = 1692217249878 diff = -1035783163
Nov 29 10 37 37 [ 171.113852] #######now = 38938 prev = 1691181468828 diff = -1691181429890
Nov 29 10 37 37 [ 171.216227] #######now = 1691181876314 prev = 1692217155259 diff = -1035278945
Nov 29 10 37 37 [ 171.625196] #######now = 47042 prev = 1691181980368 diff = -1691181933326
Nov 29 10 37 38 [ 171.727565] #######now = 1691182388308 prev = 1692217163383 diff = -1034775075
Nov 29 10 37 38 [ 172.136614] #######now = 56031 prev = 1691182492506 diff = -1691182436475
Nov 29 10 37 38 [ 172.238831] #######now = 1691182900306 prev = 1692217172371 diff = -1034272065
Nov 29 10 37 39 [ 172.647956] #######now = 64043 prev = 1691183004401 diff = -1691182940358
Nov 29 10 37 39 [ 172.649853] #######now = 1691183311916 prev = 1692217180343 diff = -1033868427
Nov 29 10 37 39 [ 173.159299] #######now = 71170 prev = 1691183516724 diff = -1691183445554
Nov 29 10 37 39 [ 173.261526] #######now = 1691183924297 prev = 1692217187471 diff = -1033263174
Nov 29 10 37 40 [ 173.670627] #######now = 79152 prev = 1691184028372 diff = -1691183949220
Nov 29 10 37 40 [ 173.772862] #######now = 1691184436293 prev = 1692217195472 diff = -1032759179
Nov 29 10 37 40 [ 174.181969] #######now = 87533 prev = 1691184540766 diff = -1691184453233
Nov 29 10 37 40 [ 174.284196] #######now = 1691184948290 prev = 1692217203832 diff = -1032255542
Nov 29 10 37 41 [ 174.693306] #######now = 99141 prev = 1691185052386 diff = -1691184953245
Nov 29 10 37 41 [ 174.695191] #######now = 1691185359881 prev = 1692217215482 diff = -1031855601
Nov 29 10 37 41 [ 175.102370] #######now = 72739 prev = 1691185564362 diff = -1691185491623
Nov 29 10 37 41 [ 175.204667] #######now = 1691185869883 prev = 1692217189060 diff = -1031319177
Nov 29 10 37 41 [ 175.511391] #######now = 46825 prev = 1691185974181 diff = -1691185927356
Nov 29 10 37 41 [ 175.613708] #######now = 1691186279480 prev = 1692217163106 diff = -1030883626
Nov 29 10 37 42 [ 176.022679] #######now = 123706 prev = 1691186383595 diff = -1691186259889
Nov 29 10 37 42 [ 176.125183] #######now = 1691186791700 prev = 1692217240027 diff = -1030448327
Nov 29 10 37 42 [ 176.432048] #######now = 96505 prev = 1691186895551 diff = -1691186799046
Nov 29 10 37 42 [ 176.536125] #######now = 1691187203098 prev = 1692217314997 diff = -1030111899
Nov 29 10 37 43 [ 177.048757] #######now = 2683652920304 prev = 2684354622051 diff = -701701747
Nov 29 10 38 29 [ 222.645074] #######now = 67522 prev = 2683698484870 diff = -2683698417348
Nov 29 10 38 29 [ 222.656606] #######now = 2683698587570 prev = 2684354638805 diff = -656051235
Nov 29 10 38 29 [ 222.760299] #######now = 2683698691450 prev = 2684354646295 diff = -655954845
Nov 29 10 38 29 [ 223.169372] #######now = 2683699101017 prev = 2684354651713 diff = -655550696
Nov 29 10 38 29 [ 223.521693] #######now = 38390 prev = 2683699408246 diff = -2683699369856
Nov 29 10 38 30 [ 223.578406] #######now = 2683699510587 prev = 2684354653503 diff = -655142916
Nov 29 10 38 30 [ 223.936505] #######now = 37791 prev = 2683699810214 diff = -2683699772423
Nov 29 10 38 30 [ 223.987437] #######now = 2683699920126 prev = 2684354647261 diff = -654727135
Nov 29 10 38 30 [ 224.358962] #######now = 48751 prev = 2683700217950 diff = -2683700169199
Nov 29 10 38 30 [ 224.396528] #######now = 2683700329747 prev = 2684354644862 diff = -654315115
Nov 29 10 38 31 [ 224.757642] #######now = 42748 prev = 2683700635268 diff = -2683700592520
Nov 29 10 38 31 [ 224.805705] #######now = 2683700739387 prev = 2684354649174 diff = -653909787
Nov 29 10 38 31 [ 225.171112] #######now = 49009 prev = 2683701039022 diff = -2683700990013
Nov 29 10 38 31 [ 225.214657] #######now = 2683701148967 prev = 2684354651014 diff = -653502047
Nov 29 10 38 32 [ 225.567660] #######now = 37952 prev = 2683701354078 diff = -2683701316126
Nov 29 10 38 32 [ 225.623744] #######now = 2683701558590 prev = 2684354652512 diff = -653093922
Nov 29 10 38 32 [ 226.024013] #######now = 87143 prev = 2683701763368 diff = -2683701676225
Nov 29 10 38 32 [ 226.032847] #######now = 2683701968166 prev = 2684354654329 diff = -652686163
Nov 29 10 38 32 [ 226.388713] #######now = 45488 prev = 2683702172987 diff = -2683702127499
Nov 29 10 38 32 [ 226.441764] #######now = 2683702377727 prev = 2684354657086 diff = -652279359
Nov 29 10 38 33 [ 226.827426] #######now = 79753 prev = 2683702582588 diff = -2683702502835
Nov 29 10 38 33 [ 226.850979] #######now = 2683702787347 prev = 2684354661672 diff = -651874325
Nov 29 10 38 33 [ 227.220603] #######now = 38680 prev = 2683702992278 diff = -2683702953598
Nov 29 10 38 33 [ 227.260052] #######now = 2683703196987 prev = 2684354636428 diff = -651439441
Nov 29 10 38 34 [ 227.658517] #######now = 40515 prev = 2683703401767 diff = -2683703361252
Nov 29 10 38 34 [ 227.669238] #######now = 2683703606644 prev = 2684354609636 diff = -651002992
Nov 29 10 38 34 [ 228.098492] #######now = 45679 prev = 2683703811446 diff = -2683703765767
Nov 29 10 38 34 [ 228.180384] #######now = 2683704118549 prev = 2684354686147 diff = -650567598
Nov 29 10 38 34 [ 228.454890] #######now = 62564 prev = 2683704323386 diff = -2683704260822
Nov 29 10 38 34 [ 228.525806] #######now = 40664 prev = 84587 diff = -43923
Nov 29 10 38 35 [ 228.589511] #######now = 2683704528126 prev = 2684354662731 diff = -650134605
Nov 29 10 38 35 [ 229.033831] #######now = 46110 prev = 2683704732928 diff = -2683704686818
Nov 29 10 38 35 [ 229.100876] #######now = 2683705040132 prev = 2684354671572 diff = -649631440
Nov 29 10 38 35 [ 229.553279] #######now = 61237 prev = 2683705245110 diff = -2683705183873
Nov 29 10 38 36 [ 229.612684] #######now = 2683705552734 prev = 2684354679099 diff = -649126365
Nov 29 10 38 36 [ 230.049570] #######now = 54215 prev = 2683705756928 diff = -2683705702713
Nov 29 10 38 36 [ 230.123905] #######now = 2683706064574 prev = 2684354686820 diff = -648622246
Nov 29 10 38 36 [ 230.549348] #######now = 50711 prev = 2683706268987 diff = -2683706218276
Nov 29 10 38 37 [ 230.634881] #######now = 2683706576146 prev = 2684354694718 diff = -648118572
Nov 29 10 38 37 [ 231.035908] #######now = 39075 prev = 2683706678528 diff = -2683706639453
Nov 29 10 38 37 [ 231.044023] #######now = 2683706985835 prev = 2684354605535 diff = -647619700
Nov 29 10 38 37 [ 231.546377] #######now = 46171 prev = 2683707190567 diff = -2683707144396
Nov 29 10 38 37 [ 231.555308] #######now = 2683707497768 prev = 2684354613467 diff = -647115699
Nov 29 10 38 38 [ 232.043097] #######now = 38544 prev = 2683707702568 diff = -2683707664024
Nov 29 10 38 38 [ 232.067121] #######now = 2683708010192 prev = 2684354620850 diff = -646610658
Nov 29 10 38 38 [ 232.551491] #######now = 44134 prev = 2683708214810 diff = -2683708170676
Nov 29 10 38 39 [ 232.579283] #######now = 2683708523063 prev = 2684354629182 diff = -646106119
Nov 29 10 38 39 [ 233.062457] #######now = 51528 prev = 2683708737211 diff = -2683708685683
Nov 29 10 38 39 [ 233.089998] #######now = 2683709034572 prev = 2684354637304 diff = -645602732
Nov 29 10 38 40 [ 233.557255] #######now = 42182 prev = 2683709238526 diff = -2683709196344
Nov 29 10 38 40 [ 233.600554] #######now = 2683709545786 prev = 2684354643929 diff = -645098143
Nov 29 10 38 40 [ 234.061606] #######now = 43679 prev = 2683709750526 diff = -2683709706847
Nov 29 10 38 40 [ 234.111915] #######now = 2683710057747 prev = 2684354652430 diff = -644594683
Nov 29 10 38 41 [ 234.567255] #######now = 51039 prev = 2683710262568 diff = -2683710211529
Nov 29 10 38 41 [ 234.623262] #######now = 2683710569747 prev = 2684354665445 diff = -644095698
Nov 29 10 38 41 [ 235.008750] #######now = 56451 prev = 2683710774546 diff = -2683710718095
Nov 29 10 38 41 [ 235.032331] #######now = 2683710979368 prev = 2684354638401 diff = -643659033
Nov 29 10 38 41 [ 235.441301] #######now = 2683711388929 prev = 2684354610626 diff = -643221697
Nov 29 10 49 29 [ 882.590355] #######now = 39368 prev = 2684359175487 diff = -2684359136119
Nov 29 10 49 30 [ 883.004198] #######now = 41105 prev = 2684359639453 diff = -2684359598348
Nov 29 10 49 30 [ 883.411291] #######now = 40341 prev = 2684360143478 diff = -2684360103137
Nov 29 10 49 31 [ 883.816886] #######now = 38864 prev = 2684360542791 diff = -2684360503927
Nov 29 10 49 31 [ 884.233038] #######now = 43294 prev = 2684360959698 diff = -2684360916404
Nov 29 10 49 32 [ 884.634946] #######now = 38248 prev = 2684361229014 diff = -2684361190766
Nov 29 10 49 32 [ 885.044041] #######now = 39908 prev = 2684361633105 diff = -2684361593197
Nov 29 10 49 32 [ 885.449088] #######now = 38246 prev = 2684362124882 diff = -2684362086636
Nov 29 10 49 33 [ 885.862354] #######now = 39277 prev = 2684362457930 diff = -2684362418653
Nov 29 10 49 33 [ 886.329452] #######now = 72506 prev = 2684362876970 diff = -2684362804464
Nov 29 10 49 34 [ 886.776875] #######now = 2684354644362 prev = 2684363373915 diff = -8729553
Nov 29 10 49 34 [ 887.169774] #######now = 41937 prev = 2684363783519 diff = -2684363741582
Nov 29 10 49 34 [ 887.519897] #######now = 52601 prev = 2684364204023 diff = -2684364151422
Nov 29 10 49 34 [ 887.603832] #######now = 40313 prev = 65602 diff = -25289
Nov 29 10 49 35 [ 888.106181] #######now = 2684354599663 prev = 2684364602724 diff = -10003061
Nov 29 10 49 35 [ 888.617587] #######now = 2684354608523 prev = 2684365115033 diff = -10506510
Nov 29 10 49 36 [ 889.114124] #######now = 41700 prev = 2684365666776 diff = -2684365625076
Nov 29 10 49 36 [ 889.617343] #######now = 40992 prev = 2684366138737 diff = -2684366097745
Nov 29 10 49 37 [ 890.119939] #######now = 39012 prev = 2684366650803 diff = -2684366611791
Nov 29 10 49 38 [ 890.621286] #######now = 38290 prev = 2684367162748 diff = -2684367124458
Nov 29 10 49 39 [ 891.628599] #######now = 39142 prev = 2684368186906 diff = -2684368147764
Nov 29 10 49 39 [ 892.128824] #######now = 40846 prev = 2684368727756 diff = -2684368686910
Nov 29 10 49 40 [ 892.631054] #######now = 39345 prev = 2684369211049 diff = -2684369171704
Nov 29 10 49 40 [ 893.154699] #######now = 60123 prev = 2684369722761 diff = -2684369662638
Nov 29 10 49 41 [ 893.645177] #######now = 46290 prev = 2684370235001 diff = -2684370188711
Nov 29 10 49 41 [ 894.074169] #######now = 38945 prev = 2684370644371 diff = -2684370605426
Nov 29 10 49 41 [ 894.508728] #######now = 39003 prev = 2684371156373 diff = -2684371117370
Nov 29 10 49 42 [ 894.950959] #######now = 45884 prev = 2684371565974 diff = -2684371520090
Nov 29 10 49 42 [ 895.381594] #######now = 40157 prev = 2684371975579 diff = -2684371935422
Nov 29 10 49 43 [ 895.813911] #######now = 37792 prev = 2684372481669 diff = -2684372443877
Nov 29 10 51 29 [ 1001.520285] #######now = 2684354608920 prev = 2684478398565 diff = -123789645
Nov 29 10 51 29 [ 1001.622321] #######now = 2684354616934 prev = 2684478485757 diff = -123868823
Nov 29 10 51 29 [ 1002.031433] #######now = 2684354618589 prev = 2684478894799 diff = -124276210
Nov 29 10 51 29 [ 1002.418732] #######now = 37972 prev = 2684479284869 diff = -2684479246897
Nov 29 10 51 30 [ 1002.836127] #######now = 48226 prev = 2684479613139 diff = -2684479564913
Nov 29 10 51 30 [ 1003.258586] #######now = 2684354622737 prev = 2684480095475 diff = -125472738
Nov 29 10 51 31 [ 1003.667679] #######now = 2684354624547 prev = 2684480505079 diff = -125880532
Nov 29 10 51 31 [ 1004.076780] #######now = 2684354622516 prev = 2684480914683 diff = -126292167
Nov 29 10 51 32 [ 1004.467573] #######now = 45247 prev = 2684481324282 diff = -2684481279035
Nov 29 10 51 32 [ 1004.881108] #######now = 52270 prev = 2684481646833 diff = -2684481594563
Nov 29 10 51 32 [ 1005.303918] #######now = 2684354627702 prev = 2684482056392 diff = -127428690
Nov 29 10 51 33 [ 1005.702919] #######now = 60694 prev = 2684482520673 diff = -2684482459979
^ permalink raw reply [flat|nested] 15+ messages in thread* [ath9k-devel] ath9k_htc test release: please test
2012-11-30 4:46 ` vikranth reddy
@ 2012-11-30 8:59 ` Adrian Chadd
2012-11-30 14:57 ` vikranth reddy
0 siblings, 1 reply; 15+ messages in thread
From: Adrian Chadd @ 2012-11-30 8:59 UTC (permalink / raw)
To: ath9k-devel
.. be64_to_cpu() ? That field is not already converted for you?
Adrian
On 29 November 2012 20:46, vikranth reddy
<vikranth.reddydeveloper@gmail.com> wrote:
> Hi,
>
> I have tested it using Sony BR-100 (AR7010 + AR9280). PFA the time diff logs
> for the new firmware.(htc_7010.fw)
>
> I added prints for time diff in
> file:- ~/drivers/net/wireless/ath/ath9k/htc_drv_txrx.c
> function:- ath9k_rx_tasklet
> position:- just before ieee80211_rx function.
>
> Code Snippet:
> void ath9k_rx_tasklet(unsigned long data)
> {
> struct ath9k_htc_priv *priv = (struct ath9k_htc_priv *)data;
> struct ath9k_htc_rxbuf *rxbuf = NULL, *tmp_buf = NULL;
> struct ieee80211_rx_status rx_status;
> struct sk_buff *skb;
> unsigned long flags;
> struct ieee80211_hdr *hdr;
> /* Added to measure Time-Difference between successive WiFi Packets */
> static u64 prev1,now1;
> long long diff1 = 0;
> static u32 counter1 = 0;
> /****************************************/
>
> do {
> spin_lock_irqsave(&priv->rx.rxbuflock, flags);
> list_for_each_entry(tmp_buf, &priv->rx.rxbuf, list) {
> if (tmp_buf->in_process) {
> rxbuf = tmp_buf;
> break;
> }
> }
>
> if (rxbuf == NULL) {
> spin_unlock_irqrestore(&priv->rx.rxbuflock, flags);
> break;
> }
>
> if (!rxbuf->skb)
> goto requeue;
>
> if (!ath9k_rx_prepare(priv, rxbuf, &rx_status)) {
> dev_kfree_skb_any(rxbuf->skb);
> goto requeue;
> }
>
> memcpy(IEEE80211_SKB_RXCB(rxbuf->skb), &rx_status,
> sizeof(struct ieee80211_rx_status));
> skb = rxbuf->skb;
> hdr = (struct ieee80211_hdr *) skb->data;
>
> if (ieee80211_is_beacon(hdr->frame_control) && priv->ps_enabled)
> ieee80211_queue_work(priv->hw, &priv->ps_work);
>
> spin_unlock_irqrestore(&priv->rx.rxbuflock, flags);
> /* Added to measure Time-Difference between successive WiFi Packets */
> now1 = be64_to_cpu(rxbuf->rxstatus.rs_tstamp);
> diff1 = now1 - prev1;
> if(diff1 < 0) {
> printk("#######now = %llu prev = %llu diff = %lld counter =
> %u\n",now1,prev1,diff1,counter1++);
> }
> prev1 = now1;
> /**********************************************************************/
> ieee80211_rx(priv->hw, skb);
>
> spin_lock_irqsave(&priv->rx.rxbuflock, flags);
>
>
>
> ...................
> ...........
>
>
> }
>
> Nov 29 10 36 29 [ 103.724123] #######now = 2683579500078 prev =
> 2684354679445 diff = -775179367
> Nov 29 10 36 49 [ 122.787519] #######now = 64609 prev = 2683598441314
> diff = -2683598376705
> Nov 29 10 36 49 [ 122.947963] #######now = 2683598748946 prev =
> 2684354687871 diff = -755938925
> Nov 29 10 36 50 [ 123.766144] #######now = 2683599568206 prev =
> 2684354697547 diff = -755129341
> Nov 29 10 36 50 [ 124.072922] #######now = 2683599875386 prev =
> 2684354601000 diff = -754725614
> Nov 29 10 36 50 [ 124.481997] #######now = 2683600284967 prev =
> 2684354598998 diff = -754314031
> Nov 29 10 36 51 [ 124.890184] #######now = 2683600693668 prev =
> 2684354605871 diff = -753912203
> Nov 29 10 36 51 [ 125.298751] #######now = 51576 prev = 2683601031091
> diff = -2683600979515
>
> Still I see the negative time-differences between successive WiFi-packets,
> when it is stayed connected to single AP. In the log, now is the TSF of
> current packet and prev is previous WiFi Packet's timestamp. All the
> variables are 64-bit.
>
>
> But the difference between new firmware and old-firmware is, I see less
> number of negative differences comparatively.
>
> Thanks
> Vikranth
> On Thu, Nov 29, 2012 at 5:20 AM, Adrian Chadd <adrian@freebsd.org> wrote:
>>
>> Hi all,
>>
>> I've done an updated build of the ath9k_htc firmware images. I don't
>> currently have a HTC setup working so this is just a recompilation of
>> the current firmware.
>>
>> http://dev.qca.qualcomm.com/~adrian/htc/20121128/
>>
>> Please let me know if it works or doesn't work for you.
>>
>> The specific two changes from Sujith's last release:
>>
>> * Fix the TSF extension logic to correctly extend TSF from 32 bit to 64
>> bit;
>> * Update the embedded runtime OS code to the latest from Tensilica, in
>> preparation for open sourcing it;
>>
>> I don't plan on fixing any bugs that already exist in the latest
>> ath9k_htc drop in the firmware git repository; I just want to check
>> that the build hasn't broken any existing behaviour (ie, "made things
>> worse.")
>>
>> Thanks,
>>
>>
>> Adrian
>> _______________________________________________
>> ath9k-devel mailing list
>> ath9k-devel at lists.ath9k.org
>> https://lists.ath9k.org/mailman/listinfo/ath9k-devel
>
>
^ permalink raw reply [flat|nested] 15+ messages in thread* [ath9k-devel] ath9k_htc test release: please test
2012-11-30 8:59 ` Adrian Chadd
@ 2012-11-30 14:57 ` vikranth reddy
2012-11-30 17:43 ` Adrian Chadd
0 siblings, 1 reply; 15+ messages in thread
From: vikranth reddy @ 2012-11-30 14:57 UTC (permalink / raw)
To: ath9k-devel
Hi Adrian,
No its not already converted.
See the below code-snippet in ath9k_rx_prepare
rx_status->mactime = be64_to_cpu(rxbuf->rxstatus.rs_tstamp);
rx_status->band = hw->conf.channel->band;
rx_status->freq = hw->conf.channel->center_freq;
rx_status->signal = rxbuf->rxstatus.rs_rssi + ATH_DEFAULT_NOISE_FLOOR;
rx_status->antenna = rxbuf->rxstatus.rs_antenna;
rx_status->flag |= RX_FLAG_MACTIME_MPDU;
Which calculates the mactime, using be64_to_cpu.
Thanks
Vikranth
On Fri, Nov 30, 2012 at 2:29 PM, Adrian Chadd <adrian@freebsd.org> wrote:
> .. be64_to_cpu() ? That field is not already converted for you?
>
>
>
> Adrian
>
>
> On 29 November 2012 20:46, vikranth reddy
> <vikranth.reddydeveloper@gmail.com> wrote:
> > Hi,
> >
> > I have tested it using Sony BR-100 (AR7010 + AR9280). PFA the time diff
> logs
> > for the new firmware.(htc_7010.fw)
> >
> > I added prints for time diff in
> > file:- ~/drivers/net/wireless/ath/ath9k/htc_drv_txrx.c
> > function:- ath9k_rx_tasklet
> > position:- just before ieee80211_rx function.
> >
> > Code Snippet:
> > void ath9k_rx_tasklet(unsigned long data)
> > {
> > struct ath9k_htc_priv *priv = (struct ath9k_htc_priv *)data;
> > struct ath9k_htc_rxbuf *rxbuf = NULL, *tmp_buf = NULL;
> > struct ieee80211_rx_status rx_status;
> > struct sk_buff *skb;
> > unsigned long flags;
> > struct ieee80211_hdr *hdr;
> > /* Added to measure Time-Difference between successive WiFi Packets */
> > static u64 prev1,now1;
> > long long diff1 = 0;
> > static u32 counter1 = 0;
> > /****************************************/
> >
> > do {
> > spin_lock_irqsave(&priv->rx.rxbuflock, flags);
> > list_for_each_entry(tmp_buf, &priv->rx.rxbuf, list) {
> > if (tmp_buf->in_process) {
> > rxbuf = tmp_buf;
> > break;
> > }
> > }
> >
> > if (rxbuf == NULL) {
> > spin_unlock_irqrestore(&priv->rx.rxbuflock, flags);
> > break;
> > }
> >
> > if (!rxbuf->skb)
> > goto requeue;
> >
> > if (!ath9k_rx_prepare(priv, rxbuf, &rx_status)) {
> > dev_kfree_skb_any(rxbuf->skb);
> > goto requeue;
> > }
> >
> > memcpy(IEEE80211_SKB_RXCB(rxbuf->skb), &rx_status,
> > sizeof(struct ieee80211_rx_status));
> > skb = rxbuf->skb;
> > hdr = (struct ieee80211_hdr *) skb->data;
> >
> > if (ieee80211_is_beacon(hdr->frame_control) && priv->ps_enabled)
> > ieee80211_queue_work(priv->hw, &priv->ps_work);
> >
> > spin_unlock_irqrestore(&priv->rx.rxbuflock, flags);
> > /* Added to measure Time-Difference between successive WiFi Packets */
> > now1 = be64_to_cpu(rxbuf->rxstatus.rs_tstamp);
> > diff1 = now1 - prev1;
> > if(diff1 < 0) {
> > printk("#######now = %llu prev = %llu diff = %lld counter =
> > %u\n",now1,prev1,diff1,counter1++);
> > }
> > prev1 = now1;
> > /**********************************************************************/
> > ieee80211_rx(priv->hw, skb);
> >
> > spin_lock_irqsave(&priv->rx.rxbuflock, flags);
> >
> >
> >
> > ...................
> > ...........
> >
> >
> > }
> >
> > Nov 29 10 36 29 [ 103.724123] #######now = 2683579500078 prev =
> > 2684354679445 diff = -775179367
> > Nov 29 10 36 49 [ 122.787519] #######now = 64609 prev = 2683598441314
> > diff = -2683598376705
> > Nov 29 10 36 49 [ 122.947963] #######now = 2683598748946 prev =
> > 2684354687871 diff = -755938925
> > Nov 29 10 36 50 [ 123.766144] #######now = 2683599568206 prev =
> > 2684354697547 diff = -755129341
> > Nov 29 10 36 50 [ 124.072922] #######now = 2683599875386 prev =
> > 2684354601000 diff = -754725614
> > Nov 29 10 36 50 [ 124.481997] #######now = 2683600284967 prev =
> > 2684354598998 diff = -754314031
> > Nov 29 10 36 51 [ 124.890184] #######now = 2683600693668 prev =
> > 2684354605871 diff = -753912203
> > Nov 29 10 36 51 [ 125.298751] #######now = 51576 prev = 2683601031091
> > diff = -2683600979515
> >
> > Still I see the negative time-differences between successive
> WiFi-packets,
> > when it is stayed connected to single AP. In the log, now is the TSF of
> > current packet and prev is previous WiFi Packet's timestamp. All the
> > variables are 64-bit.
> >
> >
> > But the difference between new firmware and old-firmware is, I see less
> > number of negative differences comparatively.
> >
> > Thanks
> > Vikranth
> > On Thu, Nov 29, 2012 at 5:20 AM, Adrian Chadd <adrian@freebsd.org>
> wrote:
> >>
> >> Hi all,
> >>
> >> I've done an updated build of the ath9k_htc firmware images. I don't
> >> currently have a HTC setup working so this is just a recompilation of
> >> the current firmware.
> >>
> >> http://dev.qca.qualcomm.com/~adrian/htc/20121128/
> >>
> >> Please let me know if it works or doesn't work for you.
> >>
> >> The specific two changes from Sujith's last release:
> >>
> >> * Fix the TSF extension logic to correctly extend TSF from 32 bit to 64
> >> bit;
> >> * Update the embedded runtime OS code to the latest from Tensilica, in
> >> preparation for open sourcing it;
> >>
> >> I don't plan on fixing any bugs that already exist in the latest
> >> ath9k_htc drop in the firmware git repository; I just want to check
> >> that the build hasn't broken any existing behaviour (ie, "made things
> >> worse.")
> >>
> >> Thanks,
> >>
> >>
> >> Adrian
> >> _______________________________________________
> >> ath9k-devel mailing list
> >> ath9k-devel at lists.ath9k.org
> >> https://lists.ath9k.org/mailman/listinfo/ath9k-devel
> >
> >
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.ath9k.org/pipermail/ath9k-devel/attachments/20121130/08677afb/attachment.htm
^ permalink raw reply [flat|nested] 15+ messages in thread* [ath9k-devel] ath9k_htc test release: please test
2012-11-30 14:57 ` vikranth reddy
@ 2012-11-30 17:43 ` Adrian Chadd
2012-12-04 6:06 ` vikranth reddy
0 siblings, 1 reply; 15+ messages in thread
From: Adrian Chadd @ 2012-11-30 17:43 UTC (permalink / raw)
To: ath9k-devel
As I said, the cozybit guys reported that the TSF increment is
behaving correctly. Maybe it's only behaving correctly for beacons
here, hm.
Also, look:
> Nov 29 10 36 49 [ 122.787519] #######now = 64609 prev = 2683598441314 diff = -2683598376705
> Nov 29 10 36 49 [ 122.947963] #######now = 2683598748946 prev = 2684354687871 diff = -755938925
Your now/prev values are all messed up, the second prev should been
64609, right?
In any case, unless it's really obvious to fix, I'm just going to
leave this as-is as it's the same behaviour as previous firmware.
(Which is my aim here..)
Adrian
^ permalink raw reply [flat|nested] 15+ messages in thread
* [ath9k-devel] ath9k_htc test release: please test
2012-11-30 17:43 ` Adrian Chadd
@ 2012-12-04 6:06 ` vikranth reddy
2013-10-16 10:19 ` vikranth reddy
0 siblings, 1 reply; 15+ messages in thread
From: vikranth reddy @ 2012-12-04 6:06 UTC (permalink / raw)
To: ath9k-devel
Hi Adrian,
Sorry for delayed reply!
Here the debug-logs are added only for the cases where there is negative
time-difference, not for success case. Logs below, that you pointed out,
are not from consecutive packets.
> Nov 29 10 36 49 [ 122.787519] #######now = 64609 prev = 2683598441314
diff = -2683598376705
> Nov 29 10 36 49 [ 122.947963] #######now = 2683598748946 prev =
2684354687871 diff = -755938925
All the guys here, who reported as successfully working, have tested with
AR9271 not with AR7010+AR9280. Not sure if there is any difference between
two firmwares (htc_7010.fw and htc_9271.fw). ?
Like you mentioned we are measuring time-difference between two consecutive
packets, irrespective of packet-type. That might have an issue still.
Hope to see a fix for this issue in next official release
Thanks for your support on this issue.
Thanks
Vikranth
On Fri, Nov 30, 2012 at 11:13 PM, Adrian Chadd <adrian@freebsd.org> wrote:
> As I said, the cozybit guys reported that the TSF increment is
> behaving correctly. Maybe it's only behaving correctly for beacons
> here, hm.
>
> Also, look:
>
> > Nov 29 10 36 49 [ 122.787519] #######now = 64609 prev =
> 2683598441314 diff = -2683598376705
> > Nov 29 10 36 49 [ 122.947963] #######now = 2683598748946 prev =
> 2684354687871 diff = -755938925
>
> Your now/prev values are all messed up, the second prev should been
> 64609, right?
>
> In any case, unless it's really obvious to fix, I'm just going to
> leave this as-is as it's the same behaviour as previous firmware.
> (Which is my aim here..)
>
>
> Adrian
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.ath9k.org/pipermail/ath9k-devel/attachments/20121204/7c96778a/attachment.htm
^ permalink raw reply [flat|nested] 15+ messages in thread
* [ath9k-devel] ath9k_htc test release: please test
2012-12-04 6:06 ` vikranth reddy
@ 2013-10-16 10:19 ` vikranth reddy
0 siblings, 0 replies; 15+ messages in thread
From: vikranth reddy @ 2013-10-16 10:19 UTC (permalink / raw)
To: ath9k-devel
Hi Adrian, List,
We are still finding the same negative timestamp difference (between
consecutive packets) issue with some of packets, with latest ath9_htc
open-source firmware version.
Adrian,
Did you add this fix into open-source version of ath9k_htc
firmware already? or still its not updated in it.
Thanks
Vikranth
On Tue, Dec 4, 2012 at 11:36 AM, vikranth reddy <
vikranth.reddydeveloper@gmail.com> wrote:
> Hi Adrian,
>
> Sorry for delayed reply!
>
> Here the debug-logs are added only for the cases where there is negative
> time-difference, not for success case. Logs below, that you pointed out,
> are not from consecutive packets.
>
>
> > Nov 29 10 36 49 [ 122.787519] #######now = 64609 prev =
> 2683598441314 diff = -2683598376705
> > Nov 29 10 36 49 [ 122.947963] #######now = 2683598748946 prev =
> 2684354687871 diff = -755938925
>
> All the guys here, who reported as successfully working, have tested with
> AR9271 not with AR7010+AR9280. Not sure if there is any difference between
> two firmwares (htc_7010.fw and htc_9271.fw). ?
>
> Like you mentioned we are measuring time-difference between two
> consecutive packets, irrespective of packet-type. That might have an issue
> still.
>
> Hope to see a fix for this issue in next official release
>
> Thanks for your support on this issue.
>
> Thanks
> Vikranth
>
> On Fri, Nov 30, 2012 at 11:13 PM, Adrian Chadd <adrian@freebsd.org> wrote:
>
>> As I said, the cozybit guys reported that the TSF increment is
>> behaving correctly. Maybe it's only behaving correctly for beacons
>> here, hm.
>>
>> Also, look:
>>
>> > Nov 29 10 36 49 [ 122.787519] #######now = 64609 prev =
>> 2683598441314 diff = -2683598376705
>> > Nov 29 10 36 49 [ 122.947963] #######now = 2683598748946 prev =
>> 2684354687871 diff = -755938925
>>
>> Your now/prev values are all messed up, the second prev should been
>> 64609, right?
>>
>> In any case, unless it's really obvious to fix, I'm just going to
>> leave this as-is as it's the same behaviour as previous firmware.
>> (Which is my aim here..)
>>
>>
>> Adrian
>>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.ath9k.org/pipermail/ath9k-devel/attachments/20131016/77fc0870/attachment.htm
^ permalink raw reply [flat|nested] 15+ messages in thread
* [ath9k-devel] ath9k_htc test release: please test
2012-11-28 23:50 [ath9k-devel] ath9k_htc test release: please test Adrian Chadd
2012-11-30 2:14 ` Stefan Lippers-Hollmann
2012-11-30 4:46 ` vikranth reddy
@ 2012-11-30 17:50 ` Oleksij Rempel
2012-12-01 14:31 ` Christian Lamparter
3 siblings, 0 replies; 15+ messages in thread
From: Oleksij Rempel @ 2012-11-30 17:50 UTC (permalink / raw)
To: ath9k-devel
Am 29.11.2012 00:50, schrieb Adrian Chadd:
> Hi all,
>
> I've done an updated build of the ath9k_htc firmware images. I don't
> currently have a HTC setup working so this is just a recompilation of
> the current firmware.
>
> http://dev.qca.qualcomm.com/~adrian/htc/20121128/
>
> Please let me know if it works or doesn't work for you.
>
> The specific two changes from Sujith's last release:
>
> * Fix the TSF extension logic to correctly extend TSF from 32 bit to 64 bit;
> * Update the embedded runtime OS code to the latest from Tensilica, in
> preparation for open sourcing it;
>
> I don't plan on fixing any bugs that already exist in the latest
> ath9k_htc drop in the firmware git repository; I just want to check
> that the build hasn't broken any existing behaviour (ie, "made things
> worse.")
>
> Thanks,
>
>
Works without regressions on ALFA AWUS036NHA (Atheros AR9271 Rev:1)
--
Regards,
Oleksij
^ permalink raw reply [flat|nested] 15+ messages in thread
* [ath9k-devel] ath9k_htc test release: please test
2012-11-28 23:50 [ath9k-devel] ath9k_htc test release: please test Adrian Chadd
` (2 preceding siblings ...)
2012-11-30 17:50 ` Oleksij Rempel
@ 2012-12-01 14:31 ` Christian Lamparter
2012-12-03 22:32 ` Adrian Chadd
3 siblings, 1 reply; 15+ messages in thread
From: Christian Lamparter @ 2012-12-01 14:31 UTC (permalink / raw)
To: ath9k-devel
On Thursday 29 November 2012 00:50:59 Adrian Chadd wrote:
> Hi all,
>
> I've done an updated build of the ath9k_htc firmware images. I don't
> currently have a HTC setup working so this is just a recompilation of
> the current firmware.
>
> http://dev.qca.qualcomm.com/~adrian/htc/20121128/
>
> Please let me know if it works or doesn't work for you.
Thanks! It seems to work/hold.
Can you please bump the version
old:
[ 26.106713] ath9k_htc 1-10:1.0: ath9k_htc: FW Version: 1.3
new:
[16308.270828] ath9k_htc 1-10:1.0: ath9k_htc: FW Version: 1.3
> The specific two changes from Sujith's last release:
> * Update the embedded runtime OS code to the latest from Tensilica, in
> preparation for open sourcing it;
Hey, I like to hear more about that. :)
Regards,
Chr
^ permalink raw reply [flat|nested] 15+ messages in thread* [ath9k-devel] ath9k_htc test release: please test
2012-12-01 14:31 ` Christian Lamparter
@ 2012-12-03 22:32 ` Adrian Chadd
0 siblings, 0 replies; 15+ messages in thread
From: Adrian Chadd @ 2012-12-03 22:32 UTC (permalink / raw)
To: ath9k-devel
On 1 December 2012 06:31, Christian Lamparter <chunkeey@googlemail.com> wrote:
>> Please let me know if it works or doesn't work for you.
>
> Thanks! It seems to work/hold.
Thanks!
> Can you please bump the version
>
> old:
> [ 26.106713] ath9k_htc 1-10:1.0: ath9k_htc: FW Version: 1.3
> new:
> [16308.270828] ath9k_htc 1-10:1.0: ath9k_htc: FW Version: 1.3
I'll bump the firmware version when it's time to do a release. I'll
see how difficult it'll be to have a development/snapshot and a
release version number scheme too.
Adrian
^ permalink raw reply [flat|nested] 15+ messages in thread
* [ath9k-devel] ath9k_htc test release: please test
2013-10-16 15:11 Oleksij Rempel
@ 2013-10-16 19:40 ` Adrian Chadd
2013-10-17 13:31 ` vikranth reddy
1 sibling, 0 replies; 15+ messages in thread
From: Adrian Chadd @ 2013-10-16 19:40 UTC (permalink / raw)
To: ath9k-devel
Hi,
Is this for aggregate frames?
If it is, I _think_ aggregate frames only have a valid timestamp in the
_last_ subframe.
-adrian
On 16 October 2013 08:11, Oleksij Rempel <linux@rempel-privat.de> wrote:
> Hi Vikranth,
>
> i can't find detailed description of this problem.
> Can you please attach patches you use for debugging and information about
> your testing setup, hw, usb controller, etc. Please describe symptom of
> this
> problem, for example connection lost etc.
>
> Am 16.10.2013 12:19, schrieb vikranth reddy:
>
> > Hi Adrian, List,
> >
> > We are still finding the same negative timestamp difference (between
> > consecutive packets) issue with some of packets, with latest ath9_htc
> > open-source firmware version.
> >
> > Adrian,
> >
> > Did you add this fix into open-source version of ath9k_htc
> > firmware already? or still its not updated in it.
> >
> >
> > Thanks
> > Vikranth
> >
> >
> >
> >
> > On Tue, Dec 4, 2012 at 11:36 AM, vikranth reddy
> > <vikranth.reddydeveloper@gmail.com
> > <mailto:vikranth.reddydeveloper@gmail.com>> wrote:
> >
> > Hi Adrian,
> >
> > Sorry for delayed reply!
> >
> > Here the debug-logs are added only for the cases where there is
> > negative time-difference, not for success case. Logs below, that you
> > pointed out, are not from consecutive packets.
> >
> >
> > > Nov 29 10 36 49 [ 122.787519] #######now = 64609 prev =
> > 2683598441314 diff = -2683598376705
> > > Nov 29 10 36 49 [ 122.947963] #######now = 2683598748946 prev
> > = 2684354687871 diff = -755938925
> >
> > All the guys here, who reported as successfully working, have tested
> > with AR9271 not with AR7010+AR9280. Not sure if there is any
> > difference between two firmwares (htc_7010.fw and htc_9271.fw). ?
> >
> > Like you mentioned we are measuring time-difference between two
> > consecutive packets, irrespective of packet-type. That might have an
> > issue still.
> >
> > Hope to see a fix for this issue in next official release
> >
> > Thanks for your support on this issue.
> >
> > Thanks
> > Vikranth
> >
> > On Fri, Nov 30, 2012 at 11:13 PM, Adrian Chadd <adrian@freebsd.org
> > <mailto:adrian@freebsd.org>> wrote:
> >
> > As I said, the cozybit guys reported that the TSF increment is
> > behaving correctly. Maybe it's only behaving correctly for beacons
> > here, hm.
> >
> > Also, look:
> >
> > > Nov 29 10 36 49 [ 122.787519] #######now = 64609 prev =
> > 2683598441314 diff = -2683598376705
> > > Nov 29 10 36 49 [ 122.947963] #######now = 2683598748946
> > prev = 2684354687871 diff = -755938925
> >
> > Your now/prev values are all messed up, the second prev should been
> > 64609, right?
> >
> > In any case, unless it's really obvious to fix, I'm just going to
> > leave this as-is as it's the same behaviour as previous firmware.
> > (Which is my aim here..)
> >
> >
> > Adrian
> >
> >
> >
> >
> >
> > _______________________________________________
> > ath9k-devel mailing list
> > ath9k-devel at lists.ath9k.org
> > https://lists.ath9k.org/mailman/listinfo/ath9k-devel
> >
>
>
> --
> Regards,
> Oleksij
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.ath9k.org/pipermail/ath9k-devel/attachments/20131016/8c8cb32c/attachment.htm
^ permalink raw reply [flat|nested] 15+ messages in thread
* [ath9k-devel] ath9k_htc test release: please test
2013-10-16 15:11 Oleksij Rempel
2013-10-16 19:40 ` Adrian Chadd
@ 2013-10-17 13:31 ` vikranth reddy
2013-10-20 17:16 ` Oleksij Rempel
1 sibling, 1 reply; 15+ messages in thread
From: vikranth reddy @ 2013-10-17 13:31 UTC (permalink / raw)
To: ath9k-devel
Hi Oleksij, Adrian
Thanks for Reply.
We are using Sony-UWA BR100 Adapter with AR7010+AR9280 chipset. We are
using compat-wireless-3.6.8 in x86 machine with Intel Core i5.
In monitor mode, we see for some packets the TSF difference is negative for
consecutive captured packets, which confuses our analysis software in
application.
Last year Nov, Adrian had given a test-release with a fix for this issue, I
was wondering whether this fix is updated in latest ath9k_htc firmware or
not.
Thanks
Vikranth
On Wed, Oct 16, 2013 at 8:41 PM, Oleksij Rempel <linux@rempel-privat.de>wrote:
> Hi Vikranth,
>
> i can't find detailed description of this problem.
> Can you please attach patches you use for debugging and information about
> your testing setup, hw, usb controller, etc. Please describe symptom of
> this
> problem, for example connection lost etc.
>
> Am 16.10.2013 12:19, schrieb vikranth reddy:
>
> > Hi Adrian, List,
> >
> > We are still finding the same negative timestamp difference (between
> > consecutive packets) issue with some of packets, with latest ath9_htc
> > open-source firmware version.
> >
> > Adrian,
> >
> > Did you add this fix into open-source version of ath9k_htc
> > firmware already? or still its not updated in it.
> >
> >
> > Thanks
> > Vikranth
> >
> >
> >
> >
> > On Tue, Dec 4, 2012 at 11:36 AM, vikranth reddy
> > <vikranth.reddydeveloper@gmail.com
> > <mailto:vikranth.reddydeveloper@gmail.com>> wrote:
> >
> > Hi Adrian,
> >
> > Sorry for delayed reply!
> >
> > Here the debug-logs are added only for the cases where there is
> > negative time-difference, not for success case. Logs below, that you
> > pointed out, are not from consecutive packets.
> >
> >
> > > Nov 29 10 36 49 [ 122.787519] #######now = 64609 prev =
> > 2683598441314 diff = -2683598376705
> > > Nov 29 10 36 49 [ 122.947963] #######now = 2683598748946 prev
> > = 2684354687871 diff = -755938925
> >
> > All the guys here, who reported as successfully working, have tested
> > with AR9271 not with AR7010+AR9280. Not sure if there is any
> > difference between two firmwares (htc_7010.fw and htc_9271.fw). ?
> >
> > Like you mentioned we are measuring time-difference between two
> > consecutive packets, irrespective of packet-type. That might have an
> > issue still.
> >
> > Hope to see a fix for this issue in next official release
> >
> > Thanks for your support on this issue.
> >
> > Thanks
> > Vikranth
> >
> > On Fri, Nov 30, 2012 at 11:13 PM, Adrian Chadd <adrian@freebsd.org
> > <mailto:adrian@freebsd.org>> wrote:
> >
> > As I said, the cozybit guys reported that the TSF increment is
> > behaving correctly. Maybe it's only behaving correctly for beacons
> > here, hm.
> >
> > Also, look:
> >
> > > Nov 29 10 36 49 [ 122.787519] #######now = 64609 prev =
> > 2683598441314 diff = -2683598376705
> > > Nov 29 10 36 49 [ 122.947963] #######now = 2683598748946
> > prev = 2684354687871 diff = -755938925
> >
> > Your now/prev values are all messed up, the second prev should been
> > 64609, right?
> >
> > In any case, unless it's really obvious to fix, I'm just going to
> > leave this as-is as it's the same behaviour as previous firmware.
> > (Which is my aim here..)
> >
> >
> > Adrian
> >
> >
> >
> >
> >
> > _______________________________________________
> > ath9k-devel mailing list
> > ath9k-devel at lists.ath9k.org
> > https://lists.ath9k.org/mailman/listinfo/ath9k-devel
> >
>
>
> --
> Regards,
> Oleksij
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.ath9k.org/pipermail/ath9k-devel/attachments/20131017/fc157245/attachment.htm
^ permalink raw reply [flat|nested] 15+ messages in thread
* [ath9k-devel] ath9k_htc test release: please test
2013-10-17 13:31 ` vikranth reddy
@ 2013-10-20 17:16 ` Oleksij Rempel
0 siblings, 0 replies; 15+ messages in thread
From: Oleksij Rempel @ 2013-10-20 17:16 UTC (permalink / raw)
To: ath9k-devel
Hi Vikranth,
i was not able to reproduce your issue. For some time you have sended
some kernel debug messages. Plase send kernel patch you used to get this
result. Describe host configuration and kind of monitored traffik needed
to reproduce it.
Am 17.10.2013 15:31, schrieb vikranth reddy:
> Hi Oleksij, Adrian
>
> Thanks for Reply.
>
> We are using Sony-UWA BR100 Adapter with AR7010+AR9280 chipset. We are
> using compat-wireless-3.6.8 in x86 machine with Intel Core i5.
>
> In monitor mode, we see for some packets the TSF difference is negative
> for consecutive captured packets, which confuses our analysis software
> in application.
>
> Last year Nov, Adrian had given a test-release with a fix for this
> issue, I was wondering whether this fix is updated in latest ath9k_htc
> firmware or not.
>
> Thanks
> Vikranth
>
>
> On Wed, Oct 16, 2013 at 8:41 PM, Oleksij Rempel <linux@rempel-privat.de
> <mailto:linux@rempel-privat.de>> wrote:
>
> Hi Vikranth,
>
> i can't find detailed description of this problem.
> Can you please attach patches you use for debugging and information
> about
> your testing setup, hw, usb controller, etc. Please describe
> symptom of this
> problem, for example connection lost etc.
>
> Am 16.10.2013 12:19, schrieb vikranth reddy:
>
> > Hi Adrian, List,
> >
> > We are still finding the same negative timestamp difference (between
> > consecutive packets) issue with some of packets, with latest ath9_htc
> > open-source firmware version.
> >
> > Adrian,
> >
> > Did you add this fix into open-source version of ath9k_htc
> > firmware already? or still its not updated in it.
> >
> >
> > Thanks
> > Vikranth
> >
> >
> >
> >
> > On Tue, Dec 4, 2012 at 11:36 AM, vikranth reddy
> > <vikranth.reddydeveloper@gmail.com
> <mailto:vikranth.reddydeveloper@gmail.com>
> > <mailto:vikranth.reddydeveloper@gmail.com
> <mailto:vikranth.reddydeveloper@gmail.com>>> wrote:
> >
> > Hi Adrian,
> >
> > Sorry for delayed reply!
> >
> > Here the debug-logs are added only for the cases where there is
> > negative time-difference, not for success case. Logs below, that you
> > pointed out, are not from consecutive packets.
> >
> >
> > > Nov 29 10 36 49 [ 122.787519] #######now = 64609 prev =
> > 2683598441314 diff = -2683598376705
> > > Nov 29 10 36 49 [ 122.947963] #######now = 2683598748946 prev
> > = 2684354687871 diff = -755938925
> >
> > All the guys here, who reported as successfully working, have tested
> > with AR9271 not with AR7010+AR9280. Not sure if there is any
> > difference between two firmwares (htc_7010.fw and htc_9271.fw). ?
> >
> > Like you mentioned we are measuring time-difference between two
> > consecutive packets, irrespective of packet-type. That might have an
> > issue still.
> >
> > Hope to see a fix for this issue in next official release
> >
> > Thanks for your support on this issue.
> >
> > Thanks
> > Vikranth
> >
> > On Fri, Nov 30, 2012 at 11:13 PM, Adrian Chadd <adrian@freebsd.org
> <mailto:adrian@freebsd.org>
> > <mailto:adrian at freebsd.org <mailto:adrian@freebsd.org>>> wrote:
> >
> > As I said, the cozybit guys reported that the TSF increment is
> > behaving correctly. Maybe it's only behaving correctly for beacons
> > here, hm.
> >
> > Also, look:
> >
> > > Nov 29 10 36 49 [ 122.787519] #######now = 64609 prev =
> > 2683598441314 diff = -2683598376705
> > > Nov 29 10 36 49 [ 122.947963] #######now = 2683598748946
> > prev = 2684354687871 diff = -755938925
> >
> > Your now/prev values are all messed up, the second prev should been
> > 64609, right?
> >
> > In any case, unless it's really obvious to fix, I'm just going to
> > leave this as-is as it's the same behaviour as previous firmware.
> > (Which is my aim here..)
> >
> >
> > Adrian
--
Regards,
Oleksij
^ permalink raw reply [flat|nested] 15+ messages in thread