From owner-freebsd-wireless@FreeBSD.ORG Mon May 2 05:44:07 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 0BEB3106566C for ; Mon, 2 May 2011 05:44:07 +0000 (UTC) (envelope-from adrian.chadd@gmail.com) Received: from mail-vw0-f54.google.com (mail-vw0-f54.google.com [209.85.212.54]) by mx1.freebsd.org (Postfix) with ESMTP id BD4FD8FC08 for ; Mon, 2 May 2011 05:44:06 +0000 (UTC) Received: by vws18 with SMTP id 18so5327974vws.13 for ; Sun, 01 May 2011 22:44:05 -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=T56hnTkeD/83ha4YjIFLcu7WsRkua15G40m2LDa+3Po=; b=FWFWgomI/DWdDsyamf4e+0/Qi5Bo1Ulq7IhlW2p9QSHxN/AkLVcSHttr7bg1VYHT7F dYL5k18/bSdz04O229V1zSXUSZLf1RwVObzMHpl/xTk7i9Slyu3DffTkP1cLHOcC6o8/ Ckrqo3p5hPRkaT7rXQQ87FQuxP08rN2SDLHQc= 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=MijhlM7qY0xlSSzERnD5DG09Q6bnzW2cIJiXw0UhkGteyq4TgAJEMUOqLbqryhBbcQ Sj01mbsQhePdxv4pnBy7F51t9ctzDG5XLLhGxsNljlJSvZyzs/9krT0RhiB+R3L3dJe8 nqb7luH8gQENDQIshG4XboAy0jjuDOLHzaleI= MIME-Version: 1.0 Received: by 10.52.96.8 with SMTP id do8mr4222503vdb.142.1304315045769; Sun, 01 May 2011 22:44:05 -0700 (PDT) Sender: adrian.chadd@gmail.com Received: by 10.52.157.202 with HTTP; Sun, 1 May 2011 22:44:05 -0700 (PDT) Date: Mon, 2 May 2011 13:44:05 +0800 X-Google-Sender-Auth: TGfk2YlVClA_IbVu_4MJFiCzH7w Message-ID: From: Adrian Chadd To: freebsd-wireless@freebsd.org Content-Type: multipart/mixed; boundary=20cf307d064cfa710004a24482fa Subject: [net80211] patch: make aggressive mode a compile time option 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: Mon, 02 May 2011 05:44:07 -0000 --20cf307d064cfa710004a24482fa Content-Type: text/plain; charset=ISO-8859-1 Right now aggressive mode is enabled by default. This twiddles the WME parameters for the bulk data queue to allow bursting if there's no data on the higher priority queues. This patch disables it by default and adds a new compile time option that restores the previous behaviour. I'd like to suggest this be disabled by default as it differs from the "specification" 802.11e behaviour and I'm not yet entirely convinced it's working correctly. Specifically, I've seen hostap setups set the wme beacon field to the aggressive mode parameters but not update the wlan interface with them; making the AP wlan interface nconsistent with th erest of the stations. I'm open to making it on by default but still make it easy to disable as a build-time option. Thanks, Adrian --20cf307d064cfa710004a24482fa Content-Type: application/octet-stream; name="aggrmode-optional.diff" Content-Disposition: attachment; filename="aggrmode-optional.diff" Content-Transfer-Encoding: base64 X-Attachment-Id: f_gn6zhpi10 SW5kZXg6IHN5cy9jb25mL29wdGlvbnMKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gc3lzL2NvbmYvb3B0aW9ucwko cmV2aXNpb24gMjIxMzEyKQorKysgc3lzL2NvbmYvb3B0aW9ucwkod29ya2luZyBjb3B5KQpAQCAt ODM5LDYgKzgzOSw3IEBACiBJRUVFODAyMTFfU1VQUE9SVF9TVVBFUkcJb3B0X3dsYW4uaAogSUVF RTgwMjExX1NVUFBPUlRfVERNQQlvcHRfd2xhbi5oCiBJRUVFODAyMTFfQUxRCQlvcHRfd2xhbi5o CitJRUVFODAyMTFfU1VQUE9SVF9BR0dSTU9ERQlvcHRfd2xhbi5oCiAKICMgODAyLjExIFRETUEg c3VwcG9ydAogVERNQV9TTE9UTEVOX0RFRkFVTFQJb3B0X3RkbWEuaApJbmRleDogc3lzL25ldDgw MjExL2llZWU4MDIxMV9vdXRwdXQuYwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBzeXMvbmV0ODAyMTEvaWVlZTgw MjExX291dHB1dC5jCShyZXZpc2lvbiAyMjA5MTEpCisrKyBzeXMvbmV0ODAyMTEvaWVlZTgwMjEx X291dHB1dC5jCSh3b3JraW5nIGNvcHkpCkBAIC0yODIwLDYgKzI4MjAsNyBAQAogCWlmICh2YXAt Pml2X2ZsYWdzICYgSUVFRTgwMjExX0ZfV01FKSB7CiAJCXN0cnVjdCBpZWVlODAyMTFfd21lX3N0 YXRlICp3bWUgPSAmaWMtPmljX3dtZTsKIAorI2lmZGVmCUlFRUU4MDIxMV9TVVBQT1JUX0FHR1JN T0RFCiAJCS8qCiAJCSAqIENoZWNrIGZvciBhZ3Jlc3NpdmUgbW9kZSBjaGFuZ2UuICBXaGVuIHRo ZXJlIGlzCiAJCSAqIHNpZ25pZmljYW50IGhpZ2ggcHJpb3JpdHkgdHJhZmZpYyBpbiB0aGUgQlNT CkBAIC0yODUyLDYgKzI4NTMsOCBAQAogCQkJCXdtZS0+d21lX2hpcHJpX3RyYWZmaWMgPQogCQkJ CQl3bWUtPndtZV9oaXByaV9zd2l0Y2hfaHlzdGVyZXNpczsKIAkJfQorI2VuZGlmCS8qIElFRUU4 MDIxMV9TVVBQT1JUX0FHR1JNT0RFICovCisKIAkJaWYgKGlzc2V0KGJvLT5ib19mbGFncywgSUVF RTgwMjExX0JFQUNPTl9XTUUpKSB7CiAJCQkodm9pZCkgaWVlZTgwMjExX2FkZF93bWVfcGFyYW0o Ym8tPmJvX3dtZSwgd21lKTsKIAkJCWNscmJpdChiby0+Ym9fZmxhZ3MsIElFRUU4MDIxMV9CRUFD T05fV01FKTsKSW5kZXg6IHN5cy9uZXQ4MDIxMS9pZWVlODAyMTFfcHJvdG8uYwo9PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 Ci0tLSBzeXMvbmV0ODAyMTEvaWVlZTgwMjExX3Byb3RvLmMJKHJldmlzaW9uIDIyMDkxMSkKKysr IHN5cy9uZXQ4MDIxMS9pZWVlODAyMTFfcHJvdG8uYwkod29ya2luZyBjb3B5KQpAQCAtOTc0LDYg Kzk3NCw3IEBACiB2b2lkCiBpZWVlODAyMTFfd21lX3VwZGF0ZXBhcmFtc19sb2NrZWQoc3RydWN0 IGllZWU4MDIxMXZhcCAqdmFwKQogeworI2lmZGVmCUlFRUU4MDIxMV9TVVBQT1JUX0FHR1JNT0RF CiAJc3RhdGljIGNvbnN0IHBhcmFtVHlwZSBhZ2dyUGFyYW1bSUVFRTgwMjExX01PREVfTUFYXSA9 IHsKIAkgICAgW0lFRUU4MDIxMV9NT0RFX0FVVE9dCT0geyAyLCA0LCAxMCwgNjQsIDAgfSwKIAkg ICAgW0lFRUU4MDIxMV9NT0RFXzExQV0JPSB7IDIsIDQsIDEwLCA2NCwgMCB9LApAQCAtOTg4LDEw ICs5ODksMTQgQEAKIAkgICAgW0lFRUU4MDIxMV9NT0RFXzExTkFdCT0geyAyLCA0LCAxMCwgNjQs IDAgfSwJLyogWFhYY2hlY2sqLwogCSAgICBbSUVFRTgwMjExX01PREVfMTFOR10JPSB7IDIsIDQs IDEwLCA2NCwgMCB9LAkvKiBYWFhjaGVjayovCiAJfTsKKwlzdHJ1Y3Qgd21lUGFyYW1zICpic3Nw OworCisjZW5kaWYJLyogSUVFRTgwMjExX1NVUFBPUlRfQUdHUk1PREUgKi8KKwogCXN0cnVjdCBp ZWVlODAyMTFjb20gKmljID0gdmFwLT5pdl9pYzsKIAlzdHJ1Y3QgaWVlZTgwMjExX3dtZV9zdGF0 ZSAqd21lID0gJmljLT5pY193bWU7CiAJY29uc3Qgc3RydWN0IHdtZVBhcmFtcyAqd21lcDsKLQlz dHJ1Y3Qgd21lUGFyYW1zICpjaGFucCwgKmJzc3A7CisJc3RydWN0IHdtZVBhcmFtcyAqY2hhbnA7 CiAJZW51bSBpZWVlODAyMTFfcGh5bW9kZSBtb2RlOwogCWludCBpOwogCkBAIC0xMDI2LDYgKzEw MzEsNyBAQAogCWVsc2UKIAkJbW9kZSA9IElFRUU4MDIxMV9NT0RFX0FVVE87CiAKKyNpZmRlZglJ RUVFODAyMTFfU1VQUE9SVF9BR0dSTU9ERQogCS8qCiAJICogVGhpcyBpbXBsZW1lbnRzIGFncmVz c2l2ZSBtb2RlIGFzIGZvdW5kIGluIGNlcnRhaW4KIAkgKiB2ZW5kb3JzJyBBUCdzLiAgV2hlbiB0 aGVyZSBpcyBzaWduaWZpY2FudCBoaWdoCkBAIC0xMDgxLDYgKzEwODcsOCBAQAogCQkgICAgInVw ZGF0ZSAlcyAoY2hhbitic3MpIGxvZ2N3bWluICV1XG4iLAogCQkgICAgaWVlZTgwMjExX3dtZV9h Y25hbWVzW1dNRV9BQ19CRV0sIGNoYW5wLT53bWVwX2xvZ2N3bWluKTsKICAgICAJfQkKKyNlbmRp ZgkvKiBJRUVFODAyMTFfU1VQUE9SVF9BR0dSTU9ERSAqLworCiAJaWYgKHZhcC0+aXZfb3Btb2Rl ID09IElFRUU4MDIxMV9NX0hPU1RBUCkgewkvKiBYWFggaWJzcz8gKi8KIAkJLyoKIAkJICogQXJy YW5nZSBmb3IgYSBiZWFjb24gdXBkYXRlIGFuZCBidW1wIHRoZSBwYXJhbWV0ZXIK --20cf307d064cfa710004a24482fa--