Subject: Re: [PATCH 03/11] iwlwifi: fix hidden ssid discovery in passive
	channels
From: Marcel Holtmann <holtmann@linux.intel.com>
To: Tomas Winkler <tomasw@gmail.com>
Cc: Zhu Yi <yi.zhu@intel.com>, linville@tuxdriver.com,
	linux-wireless@vger.kernel.org,
	Ron Rindjunsky <ron.rindjunsky@intel.com>,
	Cahill Ben <ben.m.cahill@intel.com>
In-Reply-To: <1ba2fa240808280340o216da98dwbf84541aa7bfd2e6@mail.gmail.com>
References: <1219915510-3647-1-git-send-email-yi.zhu@intel.com>
	 <1219915510-3647-2-git-send-email-yi.zhu@intel.com>
	 <1219915510-3647-3-git-send-email-yi.zhu@intel.com>
	 <1219915510-3647-4-git-send-email-yi.zhu@intel.com>
	 <1219926558.6064.11.camel@californication>
	 <1ba2fa240808280340o216da98dwbf84541aa7bfd2e6@mail.gmail.com>
Content-Type: text/plain
Date: Thu, 28 Aug 2008 14:49:31 +0200
Message-Id: <1219927771.6064.39.camel@californication> (sfid-20080828_124938_406733_00CA623B)
Mime-Version: 1.0
Sender: linux-wireless-owner@vger.kernel.org

Hi Thomas,

> >> This patch gives the HW the possibility to send direct probes in passive
> >> channels (as long as traffic was detected in that channel) if such ssid
> >> was requested, so hidden ssid can be found now in those channels as well.
> >
> > this also doesn't sound like a regression to me. Nice to have for sure,
> > but not acceptable according to the merge window rules.
> 
> If you cannot scan your AP you also not able to connect to it, drawing
> your card useless.
> 
> > Question here is was the driver supporting this before and we broke it
> > in the 2.6.27 phase or not.
> 
> Irrelevant, this is fixing show stopper bug.

it is relevant. So I read this commit message as, that now we can also
find hidden SSIDs in passive channels. Question that you should ask then
is if that was supported before. Yes or no. If yes, then it is a
regression, no then it is a feature. And regression fixes go in features
wait for the next merge window.

The term show stopper bug is irrelevant when it comes to merge window
rules.

Regards

Marcel