Received-SPF: pass (google.com: domain of linux-wireless-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Subject: Re: [PATCH v3] wifi: iwlwifi: add ethtool tx and rx stats.
To:     Simon Horman <simon.horman@corigine.com>
Cc:     linux-wireless@vger.kernel.org
References: <20230522233743.3058844-1-greearb@candelatech.com>
 <ZGyWQpFs9DDAlsx6@corigine.com>
From:   Ben Greear <greearb@candelatech.com>
Organization: Candela Technologies
Message-ID: <adf58b64-f1dc-82a0-ae37-4dbe8b53f3bc@candelatech.com>
Date:   Tue, 23 May 2023 07:50:22 -0700
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101
 Thunderbird/78.10.1
MIME-Version: 1.0
In-Reply-To: <ZGyWQpFs9DDAlsx6@corigine.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Language: en-MW
Content-Transfer-Encoding: 7bit
Precedence: bulk

On 5/23/23 3:32 AM, Simon Horman wrote:
> On Mon, May 22, 2023 at 04:37:43PM -0700, greearb@candelatech.com wrote:
>> From: Ben Greear <greearb@candelatech.com>
>>
>> Report tx/rx error codes, nss, mcs, bandwidth, mode, he types,
>> tx ampdu length histogram.
>>
>> To provide some clues about how transmit and receive path is
>> actually working.
>>
>> Signed-off-by: Ben Greear <greearb@candelatech.com>
>> ---
>>
>> v3:  Combine tx and rx ethtool patches into single patch.
>> Fix some stats calculations, especially the path that consumes
>> tx frames based on block-ack receipt.
>>
>> Comparing the stats with expected values now matches better, but
>> hard to verify everything exactly.
> 
> Hi Ben,
> 
> I am a little (lot) confused about what tree/branch this patch applies to.
> 

It is against upstream iwlwifi-backports driver.  I need to use that to use latest Intel
radio, and in general flow of patches seems to be iwlwif-backports -> kernel, so
I was hoping the patch had a chance that way.  I'm sure it could also be patched into
the kernel with a bit of work, and I'll be happy to do that work if that will make
patch acceptable.

Thanks,
Ben

-- 
Ben Greear <greearb@candelatech.com>
Candela Technologies Inc  http://www.candelatech.com