Hi Stanislaw, On Tue, Mar 27, 2018 at 07:18:16PM +0200, Stanislaw Gruszka wrote: > On Tue, Mar 27, 2018 at 09:46:36AM +0200, Mathias Kresin wrote: > > > Could you test just RX AMPDU patches, i.e. > > > > > > rt2800_change_rx_ampdu_factor.patch > > > rt2800_change_rx_ampdu_density.patch > > > > > > I have somewhat positive results on RX performance on some devices > > > with those. Perhaps you could confirm that :-) > > > > This time I've done the test with HT20 only, to not annoy my neighbours. > > > > The test setup is the following: > > > > ath9k (STA) <=> (AP) o2 Box 6431 (RJ45) <=> desktop > > > > With the patches applied the bandwidth is somewhat around 10 MBit/sec lower. > > Even if I was able to reliable reproduce it, I'm not sure if it is within > > the measuring tolerance. > > Ok. I think when BBP/RF is correctly programmed patches will downgrade > performance. For not well optimized devices patches might > improve things , but this is not preferred solution. For 30+ client > scenario I suggest to just disable HT (11n) in OpenWRT config. > > Additionally since those errors do not happen frequently perhaps there > is possibility to create watchdog and reset the device when problems > start to happen. > > FWIW: I'm running wt3020 (mt7620 device) as my home router for 3 weeks now > and do not see any errors (I have 4 client devices). Performance measured > by iperf is not good though - about 40 Mbit/s . That's great news! Are you running OpenWrt or some self-baked vanilla Linux with some of the OpenWrt patchery on top? Also note that Tomislav Požega started porting more of the Rt6352 calibration routines, I'd be happy if you could review those as well so they can go upstream right away: https://github.com/openwrt/openwrt/pull/626 Just in case you are not aware of it, here is my staging tree with some experimental changes for MT7620 including the fixes/work-arounds you suggested as well as @psyborg55's patches: https://git.openwrt.org/?p=openwrt/staging/dangole.git;a=summary Cheers Daniel
