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;
Message-ID: <3ceecfc1-ab04-7d54-51bd-457a4708c0ac@gmail.com>
Date:   Thu, 20 Apr 2023 18:33:38 +0300
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.8.0
Subject: Re: RTL8188EU (LogiLink WL0151A) - Malformed packets
To:     Artem Makhutov <artem.makhutov@gmail.com>
Cc:     "linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>
References: <CALbLcfAXrHp4vsVGqU0nBdB_gFmKTKx0GcdrFJA7R_kodAy0Ew@mail.gmail.com>
 <bfe732f4-60dd-4d78-1259-5f9f6c11adbc@gmail.com>
 <CALbLcfB7cMctHC-Fv93z1WVCZkQ76uW2F_=P6rCW7ZSPXvmapw@mail.gmail.com>
 <2857f998-1b1a-e3c2-20fe-d03dadcf8f23@gmail.com>
 <CALbLcfCsyGzhvrqjSz37+7cLvc3rc4Y_kgspZV=kiPNs4TBE4w@mail.gmail.com>
 <CALbLcfBnF6njFMSfQk2TuOiZ3C3kHwqs4-c-MQGb72azzQ5sJA@mail.gmail.com>
 <38172c8e-d107-d96e-7292-09d0e3b061dd@gmail.com>
 <CALbLcfCHLU+zoFdTnTrJs22UUYwr2rETp5hqbv=NL352Qz-XgA@mail.gmail.com>
 <CALbLcfDZJmOyn-Gf24+9X-eHh3-3PY4_nP5soG3gHf_Vn85=EA@mail.gmail.com>
 <CALbLcfAiVgGi9iocEG4OfgET2XLDRq0dthbxDMP0vP3R6BBp_A@mail.gmail.com>
Content-Language: en-US
From:   Bitterblue Smith <rtl8821cerfe2@gmail.com>
In-Reply-To: <CALbLcfAiVgGi9iocEG4OfgET2XLDRq0dthbxDMP0vP3R6BBp_A@mail.gmail.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: bulk

On 20/04/2023 14:40, Artem Makhutov wrote:
> Hi,
> 
> I think that I have found the problem: It happens when rx_desc->rxht
> == 1 - then the urb is 4 bytes shorter...
> 

That's very interesting. So it has a problem with MCS rates

According to Johannes Berg, the 8 bytes at the end of the working ping
are the MIC, which mac80211 will strip. Of course mac80211 doesn't know
that rtl8xxxu sometimes receives 4 bytes fewer.

What kind of encryption does the network have? Is it possible to try
without any encryption?

Another thing to try is software crypto, using this tested patch and
the module parameter debug=0x8000:

diff --git a/rtl8xxxu_core.c b/rtl8xxxu_core.c
index cbf39ff..779f781 100644
--- a/rtl8xxxu_core.c
+++ b/rtl8xxxu_core.c
@@ -6651,6 +6651,9 @@ static int rtl8xxxu_set_key(struct ieee80211_hw *hw, enum set_key_cmd cmd,
 	dev_dbg(dev, "%s: cmd %02x, cipher %08x, index %i\n",
 		__func__, cmd, key->cipher, key->keyidx);
 
+	if (rtl8xxxu_debug & 0x8000)
+		return -EOPNOTSUPP; // use software crypto
+
 	if (vif->type != NL80211_IFTYPE_STATION)
 		return -EOPNOTSUPP;