Message-ID: <1523087213.4488.15.camel@intel.com> (sfid-20180407_094710_559437_9D4B8799)
Subject: Re: iwlwifi: FW error in SYNC CMD TIME_QUOTA_CMD
From: Luciano Coelho <luciano.coelho@intel.com>
To: Daniel Borkmann <daniel@iogearbox.net>
Cc: emmanuel.grumbach@intel.com, linux-wireless@vger.kernel.org,
        linuxwifi@intel.com, johannes.berg@intel.com
Date: Sat, 07 Apr 2018 10:46:53 +0300
In-Reply-To: <1523084065.4488.11.camel@intel.com>
References: <799ecceb-8b2a-bf8c-b408-374a02280306@iogearbox.net>
         <1523084065.4488.11.camel@intel.com>
Content-Type: text/plain; charset="UTF-8"
Mime-Version: 1.0
Sender: linux-wireless-owner@vger.kernel.org

On Sat, 2018-04-07 at 09:54 +0300, Luciano Coelho wrote:
> On Thu, 2018-04-05 at 16:55 +0200, Daniel Borkmann wrote:
> > Hey guys,
> 
> Hi Daniel,
> 
> 
> > after upgrading to 4608f064532c ("Merge
> > git://git.kernel.org/pub/scm/linux/kernel/git/davem/sparc-next")
> > my kernel log gets constantly flooded with the below message and
> > wireless stops
> > working ... just in case you haven't had a report on this yet.
> > Downgrading back
> > to older v4.16-rcX kernel gets everything back to normal working
> > state; haven't
> > had time for bisection yet, but hopefully the below would provide
> > some clue.
> > 
> 
> [...]
> 
> > [  513.429773] iwlwifi 0000:04:00.0: Failed to send quota: -5
> > [  513.429775] iwlwifi 0000:04:00.0: Failed to update quotas after
> > restart (-5)
> > [  514.163758] iwlwifi 0000:04:00.0: Microcode SW error
> > detected.  Restarting 0x82000000.
> > [  514.163987] iwlwifi 0000:04:00.0: Start IWL Error Log Dump:
> > [  514.163988] iwlwifi 0000:04:00.0: Status: 0x00000100, count: 6
> > [  514.163989] iwlwifi 0000:04:00.0: Loaded firmware version:
> > 27.455470.0
> > [  514.163991] iwlwifi 0000:04:00.0: 0x00000038 | BAD_COMMAND
> 
> [...]
> 
> I did add support for new FW APIs, so maybe some check for the FW
> version is missing somewhere.
> 
> Which NIC are you using? I can see that you're using an older version
> of the firmware (27.455470.0).  For 7265D and 3168 we already have
> version 29.  For 8000C and 8260 we have vesion 36.  You can find all
> the latest versions in our fork of linux-firmware.git[1].  And soon
> the
> main linux-firmware.git[2] will also include the latest versions,
> since
> I sent a pull request yesteday ago.
> 
> Meanwhile, I'll investigate this problem.  It shouldn't happen even
> if
> you're using an older firmware.  Maybe I'll have to stop supporting
> version 27 in newer kernels, let's see.
> 
> [1] https://git.kernel.org/pub/scm/linux/kernel/git/iwlwifi/linux-fir
> mware.git/
> [2] https://git.kernel.org/pub/scm/linux/kernel/git/firmware/linux-fi
> rmware.git

I investigated this a bit further and the issue is with the firmware
version you have.  It's not even the latest version of the 27 ucode. 
For 27 we are in version 541033 and you're using a much older version,
455470.

I tested both versions with Linus' latest and I could reproduce it with
the version you have but not with 541033.  So please upgrade.

Additionally, I would recommend that you upgrade to the latest version
of the firmware, because we have a bunch of fixes and improvements that
are supported by the newest driver.

--
Cheers,
Luca.