Re: [PATCH] wifi: cfg80211: Correctly handle Medium Synchronization Delay

[Lingbo Kong <quic_lingbok@quicinc.com>]

On 2024/10/31 18:47, Johannes Berg wrote:
> On Thu, 2024-10-31 at 10:25 +0100, Benjamin Berg wrote:
>>
>> Right, which does seem to imply that a non-NSTR AP can very well set a
>> Medium Delay Synchronization.
> 
> Yes, we also pass it to the firmware in iwlwifi, so definitely.
> 
>>> Regarding the statement “Could it be that this is intended to be used by
>>> NSTR APs only,” I did not find any explicit indication in the spec that
>>> the multi-link probe response carrying the medium sync delay subfield is
>>> solely applicable to NSTR APs.
>>>
>>> Additionally, do you have any suggestions on how to address this issue?🙂
>>
>> Not really. I don't grasp the intention of the specification here. But
>> if an AP may include the field in an ML Probe Response, then we
>> obviously cannot rely on the value to be consistent.
>>
>> Seems to me that the option to remove the check is still the best way
>> forward right now.
> 
> Agree, let's just remove it - Lingbo do you want to send a patch?
> 
> We should probably compare "extended MLD capabilities and ops" though,
> haha.
> 
> johannes

hi, johannes,

I completely agree with your perspective. I will resend a revised patch 
that removes this validity check.🙂

/lingbok