Message-ID: <51ED8283.3060901@odi.ch> (sfid-20130722_210600_636545_D73C2261)
Date: Mon, 22 Jul 2013 21:05:39 +0200
From: =?ISO-8859-1?Q?Ortwin_Gl=FCck?= <odi@odi.ch>
MIME-Version: 1.0
To: Stanislaw Gruszka <sgruszka@redhat.com>
CC: Arend van Spriel <arend@broadcom.com>,
	linux-kernel@vger.kernel.org, linux-wireless@vger.kernel.org
Subject: Re: [BUG] 3.10 regression: hang on suspend
References: <20130716065626.GA1587@redhat.com> <51E4F790.6090308@odi.ch> <51E51712.4020609@broadcom.com> <51E58FA5.4000804@odi.ch> <20130717122944.GA25894@redhat.com> <51E6E1AB.5010300@odi.ch> <20130718105550.GA4628@redhat.com> <51E83AB7.6020002@odi.ch> <20130719120842.GA1570@redhat.com> <51E9899E.3080307@odi.ch> <20130722112236.GA7364@redhat.com>
In-Reply-To: <20130722112236.GA7364@redhat.com>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Sender: linux-wireless-owner@vger.kernel.org

On 07/22/2013 01:22 PM, Stanislaw Gruszka wrote:
> We remove interface that we do not add in the driver. I think I found
> reason of that - I removed below code in bad commit:
>
>          list_for_each_entry(sdata, &local->interfaces, list) {
> [snip]
> -               switch (sdata->vif.type) {
> -               case NL80211_IFTYPE_AP_VLAN:
> -               case NL80211_IFTYPE_MONITOR:
> -                       /* skip these */
> -                       continue;

Oh yes, that makes a lot of sense! I have an extra monitoring interface 
configured. If I remove that before suspend, the crash does not occur.

And your patch does fix the problem. Very nice!

Thank you,

Ortwin