From mboxrd@z Thu Jan  1 00:00:00 1970
Return-path: <linux-wireless-owner@vger.kernel.org>
Received: from mail-ie0-f176.google.com ([209.85.223.176]:45548 "EHLO
	mail-ie0-f176.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
	with ESMTP id S932465AbbAFTgO (ORCPT
	<rfc822;linux-wireless@vger.kernel.org>);
	Tue, 6 Jan 2015 14:36:14 -0500
Received: by mail-ie0-f176.google.com with SMTP id tr6so4599507ieb.7
        for <linux-wireless@vger.kernel.org>; Tue, 06 Jan 2015 11:36:13 -0800 (PST)
MIME-Version: 1.0
In-Reply-To: <1420570976.1966.34.camel@sipsolutions.net>
References: <1420366399-31535-1-git-send-email-eliad@wizery.com>
	<1420542274.1966.21.camel@sipsolutions.net>
	<1420553111.1966.32.camel@sipsolutions.net>
	<1420553688.1966.33.camel@sipsolutions.net>
	<CAB3XZEctcBuPG6f4ZrZFV5O3Z7+hi+ShFdwvFXdJovG2KeAufQ@mail.gmail.com>
	<1420570976.1966.34.camel@sipsolutions.net>
Date: Tue, 6 Jan 2015 21:36:07 +0200
Message-ID: <CAB3XZEdWsDsnVr_bJ3ScSYy6MEQ7VXvWw4VC36zheYN-B=yLrA@mail.gmail.com> (sfid-20150106_203621_346836_6D7912FE)
Subject: Re: [PATCH 1/3] mac80211: consider only relevant vifs for
 radar_required calculation
From: Eliad Peller <eliad@wizery.com>
To: Johannes Berg <johannes@sipsolutions.net>
Cc: "linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>
Content-Type: text/plain; charset=UTF-8
Sender: linux-wireless-owner@vger.kernel.org
List-ID: <linux-wireless.vger.kernel.org>

On Tue, Jan 6, 2015 at 9:02 PM, Johannes Berg <johannes@sipsolutions.net> wrote:
> On Tue, 2015-01-06 at 20:57 +0200, Eliad Peller wrote:
>> On Tue, Jan 6, 2015 at 4:14 PM, Johannes Berg <johannes@sipsolutions.net> wrote:
>> > On Tue, 2015-01-06 at 15:05 +0100, Johannes Berg wrote:
>> >> On Tue, 2015-01-06 at 12:04 +0100, Johannes Berg wrote:
>> >> > all three applied.
>> >>
>> >> Nope, dropped patch 2 again
>> >
>> > And dropped the other two also - with them the sequence of two tests I
>> > was looking at never completes.
>> >
>> err... sorry about that.
>> i'll look into it.
>
> I think there's probably two bugs - the test suite should wait for all
> the DFS stuff to finish.
>
> But the fact that it never completes now is strange, maybe somehow radar
> detection doesn't get turned off properly after your patches?
>
it's probably local->radar_detect_enabled doesn't get cleared.
patch (3) fixes a similar issue, but there seems to be another one.
(it seems that patch (1) exposed these issues, so i should probably
reorder the patches and make it the last one)

Eliad.