From owner-freebsd-wireless@FreeBSD.ORG Wed May 18 06:02:58 2011 Return-Path: Delivered-To: freebsd-wireless@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 7A9FE1065672 for ; Wed, 18 May 2011 06:02:58 +0000 (UTC) (envelope-from adrian.chadd@gmail.com) Received: from mail-yw0-f54.google.com (mail-yw0-f54.google.com [209.85.213.54]) by mx1.freebsd.org (Postfix) with ESMTP id 3613F8FC14 for ; Wed, 18 May 2011 06:02:57 +0000 (UTC) Received: by ywf7 with SMTP id 7so545063ywf.13 for ; Tue, 17 May 2011 23:02:57 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:mime-version:sender:date:x-google-sender-auth :message-id:subject:from:to:content-type; bh=kguooRR3/uUwIVonxOGs2/Wh73A3es9vnZFWanlqWek=; b=lEabZOM4zjyjuLLO0eQQcfdix+gbfnVEwuLd/tNUpUffN0PdZSp17V4U/eJLcz4xMU hYHOJFKu6zNw5ODbNLJq6HWrxY8vWAop6oPa/+FtkAxWD+XQPu8BfRWQAx3tNcm2b9S2 tLCI4dd/9BU0FgFAhJUMZ0oWOYc8qt+Y3Nztg= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:sender:date:x-google-sender-auth:message-id:subject :from:to:content-type; b=tSlP3cNIt3x1pC5Ved0qziayNBRNVAEI61R5pvPmvTqZeN3PShZRipbxahc5qhCsxw 1pnHjYLdr+HZC/OlAv4/l5CwWf7phyVYrDKKLHO5u86VKQFih3UCfO/nei835499U4fe ntf0N2HthKd/W+06RvIqeV7ykIt5mA6I3bRns= MIME-Version: 1.0 Received: by 10.151.145.18 with SMTP id x18mr1162560ybn.133.1305698577314; Tue, 17 May 2011 23:02:57 -0700 (PDT) Sender: adrian.chadd@gmail.com Received: by 10.150.91.12 with HTTP; Tue, 17 May 2011 23:02:57 -0700 (PDT) Date: Wed, 18 May 2011 14:02:57 +0800 X-Google-Sender-Auth: uBX3J6DQAinxWr5D_wZ-zRUQQQc Message-ID: From: Adrian Chadd To: freebsd-wireless@freebsd.org Content-Type: multipart/mixed; boundary=00151750eae4e272dd04a386a3cf Subject: ath_rate_sample: only sample/try HT rates for HT nodes X-BeenThere: freebsd-wireless@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "Discussions of 802.11 stack, tools device driver development." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 18 May 2011 06:02:58 -0000 --00151750eae4e272dd04a386a3cf Content-Type: text/plain; charset=ISO-8859-1 Hi, The initial sample rate 11n code that I merged in from Rui didn't properly disable the non-HT rates when sampling and choosing a rate for a HT node. This meant that time was spent trying to sample non-11n rates. This patch disables this behaviour whilst leaving the non-11n rates in the 11na/11ng TX schedules. Since the management rate in an 11na network is a legacy rate (6mbit, not MCS0) then there will be instances of legacy TXing occuring. If the legacy rates are simply removed from the TX schedule, the sample rate module logs a complaint. I'd rather this not happen. This seems to work in legacy and non-legacy modes but I admit I haven't given it much testing yet in 11bg and 11a modes. I'd appreciate some testing and feedback. Thanks! Adrian --00151750eae4e272dd04a386a3cf Content-Type: application/octet-stream; name="sample-rate-11n.diff" Content-Disposition: attachment; filename="sample-rate-11n.diff" Content-Transfer-Encoding: base64 X-Attachment-Id: f_gntv7qay0 SW5kZXg6IHNhbXBsZS5jCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIHNhbXBsZS5jCShyZXZpc2lvbiAyMjE5NDcp CisrKyBzYW1wbGUuYwkod29ya2luZyBjb3B5KQpAQCAtMTYxLDkgKzE2MSwxMCBAQAogICogb3Ig LTEgaWYgYWxsIHRoZSBhdmVyYWdlX3R4X3RpbWVzIGFyZSAwLgogICovCiBzdGF0aWMgX19pbmxp bmUgaW50Ci1waWNrX2Jlc3RfcmF0ZShzdHJ1Y3Qgc2FtcGxlX25vZGUgKnNuLCBjb25zdCBIQUxf UkFURV9UQUJMRSAqcnQsCitwaWNrX2Jlc3RfcmF0ZShzdHJ1Y3QgYXRoX25vZGUgKmFuLCBjb25z dCBIQUxfUkFURV9UQUJMRSAqcnQsCiAgICAgaW50IHNpemVfYmluLCBpbnQgcmVxdWlyZV9hY2tl ZF9iZWZvcmUpCiB7CisJc3RydWN0IHNhbXBsZV9ub2RlICpzbiA9IEFUSF9OT0RFX1NBTVBMRShh bik7CiAgICAgICAgIGludCBiZXN0X3JhdGVfcml4LCBiZXN0X3JhdGVfdHQ7CiAJdWludDMyX3Qg bWFzazsKIAlpbnQgcml4LCB0dDsKQEAgLTE3NCw2ICsxNzUsMTIgQEAKIAkJaWYgKChtYXNrICYg MSkgPT0gMCkJCS8qIG5vdCBhIHN1cHBvcnRlZCByYXRlICovCiAJCQljb250aW51ZTsKIAorCQkv KiBEb24ndCBwaWNrIGEgbm9uLUhUIHJhdGUgZm9yIGEgSFQgbm9kZSAqLworCQlpZiAoKGFuLT5h bl9ub2RlLm5pX2ZsYWdzICYgSUVFRTgwMjExX05PREVfSFQpICYmCisJCSAgICAocnQtPmluZm9b cml4XS5waHkgIT0gSUVFRTgwMjExX1RfSFQpKSB7CisJCQljb250aW51ZTsKKwkJfQorCiAJCXR0 ID0gc24tPnN0YXRzW3NpemVfYmluXVtyaXhdLmF2ZXJhZ2VfdHhfdGltZTsKIAkJaWYgKHR0IDw9 IDAgfHwKIAkJICAgIChyZXF1aXJlX2Fja2VkX2JlZm9yZSAmJgpAQCAtMTk2LDExICsyMDMsMTIg QEAKICAqIFBpY2sgYSBnb29kICJyYW5kb20iIGJpdC1yYXRlIHRvIHNhbXBsZSBvdGhlciB0aGFu IHRoZSBjdXJyZW50IG9uZS4KICAqLwogc3RhdGljIF9faW5saW5lIGludAotcGlja19zYW1wbGVf cmF0ZShzdHJ1Y3Qgc2FtcGxlX3NvZnRjICpzc2MgLCBzdHJ1Y3Qgc2FtcGxlX25vZGUgKnNuLAor cGlja19zYW1wbGVfcmF0ZShzdHJ1Y3Qgc2FtcGxlX3NvZnRjICpzc2MgLCBzdHJ1Y3QgYXRoX25v ZGUgKmFuLAogICAgIGNvbnN0IEhBTF9SQVRFX1RBQkxFICpydCwgaW50IHNpemVfYmluKQogewog I2RlZmluZQlET1QxMVJBVEUoaXgpCShydC0+aW5mb1tpeF0uZG90MTFSYXRlICYgSUVFRTgwMjEx X1JBVEVfVkFMKQogI2RlZmluZQlNQ1MoaXgpCQkocnQtPmluZm9baXhdLmRvdDExUmF0ZSB8IElF RUU4MDIxMV9SQVRFX01DUykKKwlzdHJ1Y3Qgc2FtcGxlX25vZGUgKnNuID0gQVRIX05PREVfU0FN UExFKGFuKTsKIAlpbnQgY3VycmVudF9yaXgsIHJpeDsKIAl1bnNpZ25lZCBjdXJyZW50X3R0Owog CXVpbnQzMl90IG1hc2s7CkBAIC0yMDgsNiArMjE2LDcgQEAKIAljdXJyZW50X3JpeCA9IHNuLT5j dXJyZW50X3JpeFtzaXplX2Jpbl07CiAJaWYgKGN1cnJlbnRfcml4IDwgMCkgewogCQkvKiBubyBz dWNjZXNzZXMgeWV0LCBzZW5kIGF0IHRoZSBsb3dlc3QgYml0LXJhdGUgKi8KKwkJLyogWFhYIHNo b3VsZCByZXR1cm4gTUNTMCBpZiBIVCAqLwogCQlyZXR1cm4gMDsKIAl9CiAKQEAgLTIyMyw2ICsy MzIsMTMgQEAKIAkJCWNvbnRpbnVlOwogCQl9CiAKKwkJLyogaWYgdGhlIG5vZGUgaXMgSFQgYW5k IHRoZSByYXRlIGlzbid0IEhULCBkb24ndCBib3RoZXIgc2FtcGxlICovCisJCWlmICgoYW4tPmFu X25vZGUubmlfZmxhZ3MgJiBJRUVFODAyMTFfTk9ERV9IVCkgJiYKKwkJICAgIChydC0+aW5mb1ty aXhdLnBoeSAhPSBJRUVFODAyMTFfVF9IVCkpIHsKKwkJCW1hc2sgJj0gfigxPDxyaXgpOworCQkJ Z290byBuZXh0cmF0ZTsKKwkJfQorCiAJCS8qIHRoaXMgYml0LXJhdGUgaXMgYWx3YXlzIHdvcnNl IHRoYW4gdGhlIGN1cnJlbnQgb25lICovCiAJCWlmIChzbi0+c3RhdHNbc2l6ZV9iaW5dW3JpeF0u cGVyZmVjdF90eF90aW1lID4gY3VycmVudF90dCkgewogCQkJbWFzayAmPSB+KDE8PHJpeCk7CkBA IC0yMzYsMTEgKzI1MiwxMiBAQAogCQkJZ290byBuZXh0cmF0ZTsKIAkJfQogCi0JCS8qIGRvbid0 IHNhbXBsZSBtb3JlIHRoYW4gMiByYXRlcyBoaWdoZXIgZm9yIHJhdGVzID4gMTFNICovCi0JCWlm IChET1QxMVJBVEUocml4KSA+IDIqMTEgJiYgcml4ID4gY3VycmVudF9yaXggKyAyKSB7Ci0JCQlt YXNrICY9IH4oMTw8cml4KTsKLQkJCWdvdG8gbmV4dHJhdGU7Ci0JCX0KKwkJLyogZG9uJ3Qgc2Ft cGxlIG1vcmUgdGhhbiAyIHJhdGVzIGhpZ2hlciBmb3IgcmF0ZXMgPiAxMU0gZm9yIG5vbi1IVCBy YXRlcyAqLworCQlpZiAoISAoYW4tPmFuX25vZGUubmlfZmxhZ3MgJiBJRUVFODAyMTFfTk9ERV9I VCkpCisJCQlpZiAoRE9UMTFSQVRFKHJpeCkgPiAyKjExICYmIHJpeCA+IGN1cnJlbnRfcml4ICsg MikgeworCQkJCW1hc2sgJj0gfigxPDxyaXgpOworCQkJCWdvdG8gbmV4dHJhdGU7CisJCQl9CiAK IAkJc24tPmxhc3Rfc2FtcGxlX3JpeFtzaXplX2Jpbl0gPSByaXg7CiAJCXJldHVybiByaXg7CkBA IC0zMjcsNyArMzQ0LDcgQEAKIAkvKiBYWFggVE9ETzogdGhpcyBkb2Vzbid0IGtub3cgYWJvdXQg MTFnbiB2cyAxMWcgcHJvdGVjdGlvbjsgdGVhY2ggaXQgKi8KIAltcnIgPSBzYy0+c2NfbXJyZXRy eSAmJiAhKGljLT5pY19mbGFncyAmIElFRUU4MDIxMV9GX1VTRVBST1QpOwogCi0JYmVzdF9yaXgg PSBwaWNrX2Jlc3RfcmF0ZShzbiwgcnQsIHNpemVfYmluLCAhbXJyKTsKKwliZXN0X3JpeCA9IHBp Y2tfYmVzdF9yYXRlKGFuLCBydCwgc2l6ZV9iaW4sICFtcnIpOwogCWlmIChiZXN0X3JpeCA+PSAw KSB7CiAJCWF2ZXJhZ2VfdHhfdGltZSA9IHNuLT5zdGF0c1tzaXplX2Jpbl1bYmVzdF9yaXhdLmF2 ZXJhZ2VfdHhfdGltZTsKIAl9IGVsc2UgewpAQCAtMzM4LDcgKzM1NSw3IEBACiAJICogcmF0ZXMg dG8gc2FtcGxlX3JhdGUlIG9mIHRoZSB0b3RhbCB0cmFuc21pc3Npb24gdGltZS4KIAkgKi8KIAlp ZiAoc24tPnNhbXBsZV90dFtzaXplX2Jpbl0gPCBhdmVyYWdlX3R4X3RpbWUgKiAoc24tPnBhY2tl dHNfc2luY2Vfc2FtcGxlW3NpemVfYmluXSpzc2MtPnNhbXBsZV9yYXRlLzEwMCkpIHsKLQkJcml4 ID0gcGlja19zYW1wbGVfcmF0ZShzc2MsIHNuLCBydCwgc2l6ZV9iaW4pOworCQlyaXggPSBwaWNr X3NhbXBsZV9yYXRlKHNzYywgYW4sIHJ0LCBzaXplX2Jpbik7CiAJCUlFRUU4MDIxMV9OT1RFKGFu LT5hbl9ub2RlLm5pX3ZhcCwgSUVFRTgwMjExX01TR19SQVRFQ1RMLAogCQkgICAgICZhbi0+YW5f bm9kZSwgInNpemUgJXUgc2FtcGxlIHJhdGUgJWQgY3VycmVudCByYXRlICVkIiwKIAkJICAgICBi aW5fdG9fc2l6ZShzaXplX2JpbiksIFJBVEUocml4KSwK --00151750eae4e272dd04a386a3cf--