From owner-freebsd-wireless@FreeBSD.ORG Thu Aug 9 09:32:37 2012 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 CAEF8106566B for ; Thu, 9 Aug 2012 09:32:37 +0000 (UTC) (envelope-from moonlightakkiy@yahoo.ca) Received: from nm26-vm1.bullet.mail.ne1.yahoo.com (nm26-vm1.bullet.mail.ne1.yahoo.com [98.138.91.61]) by mx1.freebsd.org (Postfix) with SMTP id 793E18FC08 for ; Thu, 9 Aug 2012 09:32:37 +0000 (UTC) Received: from [98.138.90.57] by nm26.bullet.mail.ne1.yahoo.com with NNFMP; 09 Aug 2012 09:32:30 -0000 Received: from [98.138.226.129] by tm10.bullet.mail.ne1.yahoo.com with NNFMP; 09 Aug 2012 09:32:30 -0000 Received: from [127.0.0.1] by smtp216.mail.ne1.yahoo.com with NNFMP; 09 Aug 2012 09:32:30 -0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=yahoo.ca; s=s1024; t=1344504750; bh=K4D4pWh4qWrq2zSa5G6QqAILYnDFAN9Owr+rD5i7tM4=; h=X-Yahoo-Newman-Id:X-Yahoo-Newman-Property:X-YMail-OSG:X-Yahoo-SMTP:Received:Received:MIME-Version:Received:Received:Date:Message-ID:Subject:From:To:Cc:Content-Type; b=cQOebudV0XsalNZo90pujFRLEDnuPASo564eiCywuZ3U/FQmqF2xWIB5qRja1HyBtY2NWnob40PpmQNCEaKsRH5h4IFNOWAjcBd3a19hWwVtO41z5L1x9aVSb4ecD28FtZJWKL5KGRLiUOd0k1wBXAfufwuE+KMJ5VNn5hjD6a8= X-Yahoo-Newman-Id: 189205.5114.bm@smtp216.mail.ne1.yahoo.com X-Yahoo-Newman-Property: ymail-3 X-YMail-OSG: .rgazeUVM1kUaiZ1Rv18qFQuQvczbkEWPaKZBvV6UkHKB21 jjdaLswRi2Fg5KXRnPXNKlX0QrjP1oqLZjG7elYYioiaAlhBSKWdIGiFfwTw eZ247_iPel0GedrB5AkYxQAXdfUeWiYZMFffO8BjamjRzE5uncYz11m10VUJ 4pzyoKMzUAsxEO9sMWyt1kughnQC8PyMLnDsEieI9vPYnTwXqTX5qyC3Zv6O 8L06vwKo8dxDtJgVEsKNBpLfuEnAykLqWqlnbfGl8lyHzWlWmpvux3NnNLeX 0QlTf_oTF6ESE23wJEqBfDjGGE_cqrkJoFOzB8qZS_9LXXzzsSib4o5BXxfg DSN.CHvFjkKch3a43zZIZlr8Jrkv36z9lngCT5lLkS.e15hRAxpF9bSEtYGk PglQV00vq.Cb8q1CWYf8QCc14u4nQu_X14r9v7IPy0y5WX7NWTg-- X-Yahoo-SMTP: Xr6qjFWswBAEmd20sAvB4Q3keqXvXsIH9TjJ Received: from mail-vb0-f54.google.com (moonlightakkiy@209.85.212.54 with plain) by smtp216.mail.ne1.yahoo.com with SMTP; 09 Aug 2012 02:32:29 -0700 PDT Received: by vbmv11 with SMTP id v11so242524vbm.13 for ; Thu, 09 Aug 2012 02:32:29 -0700 (PDT) MIME-Version: 1.0 Received: by 10.220.116.6 with SMTP id k6mr2395647vcq.59.1344504749255; Thu, 09 Aug 2012 02:32:29 -0700 (PDT) Received: by 10.59.10.194 with HTTP; Thu, 9 Aug 2012 02:32:28 -0700 (PDT) Date: Thu, 9 Aug 2012 03:32:28 -0600 Message-ID: From: PseudoCylon To: Kim Culhan , Adrian Chadd Content-Type: multipart/mixed; boundary=f46d0438945ffa6b6504c6d1e749 Cc: freebsd-wireless@freebsd.org Subject: Re: (ANother) stall fixed, please update to HEAD 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: Thu, 09 Aug 2012 09:32:37 -0000 --f46d0438945ffa6b6504c6d1e749 Content-Type: text/plain; charset=ISO-8859-1 > ------------------------------ > > Message: 5 > Date: Tue, 7 Aug 2012 12:34:52 -0400 > From: Kim Culhan > Subject: Re: (ANother) stall fixed, please update to HEAD > To: Adrian Chadd > Cc: freebsd-wireless@freebsd.org > Message-ID: > > Content-Type: text/plain; charset=ISO-8859-1 > > >> Yup. Would you be able to work with PseudoCylon and test his >> ieee80211_iterate_nodes() patch? I'd like verification that it fixes >> it for you before I tidy it up and commit it to -HEAD. > > iter.patch and iter2.patch cannot be applied in that order > soo.. PseudoCylon could you please generate a diff > against -HEAD from your present local source? > No it won't. Only one of them need to apply. I guess I didn't explain well. iter.path only print outs debug message when the array overflowed (most unlikely it will). At this moment, this is what we need. iter2.patch does iter.patch + revert changes + abort iterating just for piece of mind. Probably this is unnecessary. The code need to be patched in the way the array won't overflow if it ever happens. (I leave it to committers what to commit.) The attached patch can be applied over iter.patch Sorry for the confusion. AK --f46d0438945ffa6b6504c6d1e749 Content-Type: application/octet-stream; name="addon.patch" Content-Disposition: attachment; filename="addon.patch" Content-Transfer-Encoding: base64 X-Attachment-Id: f_h5nmr6e50 ZGlmZiAtLWdpdCBhL2llZWU4MDIxMV9ub2RlLmMgYi9pZWVlODAyMTFfbm9kZS5jCmluZGV4IGE2 MDljYjEuLmExNWZjOTEgMTAwNjQ0Ci0tLSBhL2llZWU4MDIxMV9ub2RlLmMKKysrIGIvaWVlZTgw MjExX25vZGUuYwpAQCAtMjE2MCwxOCArMjE2MCwxOSBAQCBpZWVlODAyMTFfbm9kZV90aW1lb3V0 KHZvaWQgKmFyZykKICAqIE1heSBkaXJlY3RseSBiZSBjYWxsZWQgYW5kIGRvIGN1c3RvbWl6ZWQg aXRlcmF0ZSBmdW5jdGlvbnMuCiAgKiBPbmx5IHJlcXVpcmVtZW50IGlzIHRvIGRlY3JlbWVudCBl YWNoIG5vZGUncyByZWYgY291bnQuCiAgKi8KLXZvaWQKK2ludAogaWVlZTgwMjExX2l0ZXJhdGVf bnQoc3RydWN0IGllZWU4MDIxMV9ub2RlX3RhYmxlICpudCwKICAgICBzdHJ1Y3QgaWVlZTgwMjEx X25vZGUgKipuaV9hcnIsIHVpbnQxNl90IG1heF9haWQpCiB7CiAJc3RydWN0IGllZWU4MDIxMV9u b2RlICpuaTsKIAl1X2ludCBnZW47Ci0JaW50IGkgPSAwOworCWludCBpLCByZXQ7CiAKIAlJRUVF ODAyMTFfTk9ERV9JVEVSQVRFX0xPQ0sobnQpOwogCUlFRUU4MDIxMV9OT0RFX0xPQ0sobnQpOwog CiAJZ2VuID0gKytudC0+bnRfc2NhbmdlbjsKKwlpID0gcmV0ID0gMDsKIAogcmVzdGFydDoKIAlU QUlMUV9GT1JFQUNIKG5pLCAmbnQtPm50X25vZGUsIG5pX2xpc3QpIHsKQEAgLTIxNzksNiArMjE4 MCw3IEBAIHJlc3RhcnQ6CiAJCQljb250aW51ZTsKIAogCQlpZiAoaSA+PSBtYXhfYWlkKSB7CisJ CQlyZXQgPSBFMkJJRzsKIAkJCWlmX3ByaW50ZihudC0+bnRfaWMtPmljX2lmcCwKIAkJCSAgICAi Tm9kZSBhcnJheSBvdmVyZmxvdzogbWF4PSV1IiwgbWF4X2FpZCk7CiAJCQlicmVhazsKQEAgLTIx ODksOCArMjE5MSwyMSBAQCByZXN0YXJ0OgogCQlnb3RvIHJlc3RhcnQ7CiAJfQogCisJaWYgKHJl dCkgeworCQludC0+bnRfc2Nhbmdlbi0tOworCQlmb3IgKGkgPSAwOyBpIDwgbWF4X2FpZDsgaSsr KSB7CisJCQluaSA9ICoobmlfYXJyICsgaSk7CisJCQluaS0+bmlfc2Nhbmdlbi0tOworCQkJLyog bm9kZSBsb2NrIGlzIHJlY3Vyc2l2ZSAqLworCQkJaWVlZTgwMjExX2ZyZWVfbm9kZShuaSk7CisJ CX0KKwkJZnJlZShuaV9hcnIsIE1fODAyMTFfTk9ERSk7CisJfQorCiAJSUVFRTgwMjExX05PREVf VU5MT0NLKG50KTsKIAlJRUVFODAyMTFfTk9ERV9JVEVSQVRFX1VOTE9DSyhudCk7CisKKwlyZXR1 cm4gKHJldCk7CiB9CiAKIC8qCkBAIC0yMjE0LDcgKzIyMjksOCBAQCBpZWVlODAyMTFfaXRlcmF0 ZV9ub2RlcyhzdHJ1Y3QgaWVlZTgwMjExX25vZGVfdGFibGUgKm50LAogCWlmIChuaV9hcnIgPT0g TlVMTCkKIAkJcmV0dXJuOwogCi0JaWVlZTgwMjExX2l0ZXJhdGVfbnQobnQsIG5pX2FyciwgbWF4 X2FpZCk7CisJaWYgKCFpZWVlODAyMTFfaXRlcmF0ZV9udChudCwgbmlfYXJyLCBtYXhfYWlkKSkK KwkJcmV0dXJuOwogCiAJZm9yIChpID0gMDsgaSA8IG1heF9haWQ7IGkrKykgewogCQluaSA9ICoo bmlfYXJyICsgaSk7CmRpZmYgLS1naXQgYS9pZWVlODAyMTFfbm9kZS5oIGIvaWVlZTgwMjExX25v ZGUuaAppbmRleCAxZmJjODkyLi40OGVhZTJkIDEwMDY0NAotLS0gYS9pZWVlODAyMTFfbm9kZS5o CisrKyBiL2llZWU4MDIxMV9ub2RlLmgKQEAgLTQzOCw3ICs0MzgsNyBAQCBpbnQJaWVlZTgwMjEx X25vZGVfZGVsdWNhc3RrZXkoc3RydWN0IGllZWU4MDIxMV9ub2RlICopOwogdm9pZAlpZWVlODAy MTFfbm9kZV90aW1lb3V0KHZvaWQgKmFyZyk7CiAKIHR5cGVkZWYgdm9pZCBpZWVlODAyMTFfaXRl cl9mdW5jKHZvaWQgKiwgc3RydWN0IGllZWU4MDIxMV9ub2RlICopOwotdm9pZAlpZWVlODAyMTFf aXRlcmF0ZV9udChzdHJ1Y3QgaWVlZTgwMjExX25vZGVfdGFibGUgKiwKK2ludAlpZWVlODAyMTFf aXRlcmF0ZV9udChzdHJ1Y3QgaWVlZTgwMjExX25vZGVfdGFibGUgKiwKIAkJc3RydWN0IGllZWU4 MDIxMV9ub2RlICoqLCB1aW50MTZfdCk7CiB2b2lkCWllZWU4MDIxMV9pdGVyYXRlX25vZGVzKHN0 cnVjdCBpZWVlODAyMTFfbm9kZV90YWJsZSAqLAogCQlpZWVlODAyMTFfaXRlcl9mdW5jICosIHZv aWQgKik7Cg== --f46d0438945ffa6b6504c6d1e749--