Received-SPF: pass (google.com: domain of linux-wireless-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Mon, 09 Aug 2021 21:40:38 +0200
From:   David Heidelberg <david@ixit.cz>
Subject: Re: [PATCH v2] ath10k: fix wmi mgmt tx queue full due to race
 condition
To:     Brian Norris <briannorris@chromium.org>
Cc:     Miaoqing Pan <miaoqing@codeaurora.org>,
        Kalle Valo <kvalo@codeaurora.org>,
        linux-wireless <linux-wireless@vger.kernel.org>
Message-Id: <QZ7LXQ.RQD9UODPTG672@ixit.cz>
In-Reply-To: <CA+ASDXOHFijzJeRFEFy6okgNYNZKdc6HZtzZ9-6mdkYEfzL-2g@mail.gmail.com>
References: <3PGKXQ.OUNHZGPTI7091@ixit.cz>
        <CA+ASDXOHFijzJeRFEFy6okgNYNZKdc6HZtzZ9-6mdkYEfzL-2g@mail.gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii; format=flowed
Precedence: bulk

Thank you,

you're right, I messed up Reply-to. I'll follow the Option 2.

Thank you
Best regards
David Heidelberg

On Mon, Aug 9 2021 at 12:01:45 -0700, Brian Norris 
<briannorris@chromium.org> wrote:
> (NB: I think your Reply-To header was wrong, so I've chosen to modify
> that in hopes of actually reaching you.)
> 
> On Mon, Aug 9, 2021 at 2:52 AM David Heidelberg <david@ixit.cz> wrote:
>>  since I noticed this issue is very common (at least for me and some
>>  others) on 4.14 kernels [1] [2] would you think that backporting 
>> this
>>  patch into stable would make sense? I assume that at some point it
>>  could help some OpenWRT/LEDE and other devices (since for Turris 
>> it'll
>>  be most likely backported anyway).
>> 
>>  Thank you for the working on this!
>>  David
>> 
>>  [1]
>>  
>> https://forum.turris.cz/t/5-2-4-patch-wifi-fails-after-while-wmi-mgmt-tx-queue-is-full/15510
>>  [2] https://forum.turris.cz/t/unstable-wifi-on-mox-b/11065/
> 
> Seems reasonable to me. The right way to submit such a request is
> documented here:
> 
> https://www.kernel.org/doc/html/latest/process/stable-kernel-rules.html
> 
> Because this wasn't identified as a -stable candidate when first
> submitted, you'll need either Option 2 or Option 3.
> 
> Regards,
> Brian