From owner-freebsd-current@FreeBSD.ORG Mon Nov 11 15:04:05 2013 Return-Path: Delivered-To: freebsd-current@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTP id 96D3D3EC for ; Mon, 11 Nov 2013 15:04:05 +0000 (UTC) (envelope-from lars.engels@0x20.net) Received: from mail.0x20.net (mail.0x20.net [IPv6:2001:aa8:fffb:1::3]) (using TLSv1 with cipher ADH-CAMELLIA256-SHA (256/256 bits)) (No client certificate requested) by mx1.freebsd.org (Postfix) with ESMTPS id 591952DA7 for ; Mon, 11 Nov 2013 15:04:05 +0000 (UTC) Received: from 0x20.net (0x20.net [217.69.76.212]) (Authenticated sender: lala) by mail.0x20.net (Postfix) with ESMTPA id A16056A6004 for ; Mon, 11 Nov 2013 16:03:59 +0100 (CET) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Date: Mon, 11 Nov 2013 16:03:59 +0100 From: Lars Engels To: freebsd-current@freebsd.org Subject: Re: iwn(4) hangs after r257133 In-Reply-To: <20131111081857.GA1809@mole.fafoe.narf.at> References: <20131110121737.GA1834@mole.fafoe.narf.at> <20131110163340.GA1778@mole.fafoe.narf.at> <20131110193421.GB1778@mole.fafoe.narf.at> <20131111081857.GA1809@mole.fafoe.narf.at> Message-ID: <751d83f294765dbf0c213e18e8eb721b@mail.0x20.net> X-Sender: lars.engels@0x20.net User-Agent: Roundcube Webmail/0.7 X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 11 Nov 2013 15:04:05 -0000 Am 11.11.2013 09:18, schrieb Stefan Farfeleder: > On Sun, Nov 10, 2013 at 02:01:12PM -0800, Adrian Chadd wrote: >> Yes. So one of the.. unfortunately broken things in iwn is the ampdu >> tx >> code doesn't do retransmits. So if amrr picks a rate that fails to >> transmit >> everything, the driver doesn't retransmit them. It frees them. >> >> Now when amrr is enabled the hardware will retry at lower rates until >> something succeeds. But it still doesn't retransmit things. I believe >> it >> should be. >> >> So yes its more broken with Mrr disabled. But enabling mrr doesn't fix >> it. >> It just makes it less broken. Someone needs to implement ampdu >> retransmit. >> >> The latest iwn code in head at least tells the rate selection code >> that a >> total ampdu tx failure occured. This BTW is one of the hangs I fixed - >> if >> you hit a point where you never successfully transmitted an ampdu at a >> rate >> the rate would never be decreased. >> >> Now to be clear. I won't be in implementing ampdu retransmit. I'll >> maybe >> fix last multi rate retry once the new hardware support is in. I would >> really appreciate help here with these. Everyone with iwn hardware >> will >> appreciate it :) > > In the mean time, wouldn't it make sense to disable ampdu tx in iwn > then? Or to disallow the combination Mrr + ampdu? Yes, please. I'm also encountering this.