On Mon, Aug 30, 2010 at 09:48:57AM -0500, Larry Finger wrote:
> On 08/30/2010 09:12 AM, John W. Linville wrote:
> > On Sat, Aug 28, 2010 at 04:32:48PM -0500, Larry Finger wrote:
> >> On 08/28/2010 12:54 AM, Thadeu Lima de Souza Cascardo wrote:
> >>> The code for rtl8187 does not do per packet TX AGC. Resetting the per
> >>> packet TX AGC for rtl8187b appears to increase its overall TX power.
> >>> This allows the device to associate and a connection be established
> >>> using APs a little further away.
> >>> Tested-by: Thadeu Lima de Souza Cascardo
> >>> Signed-off-by: Thadeu Lima de Souza Cascardo
> >>> Cc: [email protected]
> >>> Cc: Larry Finger
> >>> Cc: Rogerio Luz Coelho
> >>> Cc: Herton Ronaldo Krzesinski
> >>> Cc: Hin-Tak Leung
> >>> ---
> >> I'm a little confused here. The subject says not to set the per packet
> >> while the submission text says that setting it helps. I don't have
access to any
> >> documents that describe this register, but I expect that clearing
> >> RTL818X_TX_AGC_CTL_PERPACKET_GAIN_SHIFT would disable that feature. As
> >> the code change seems to match the subject.
> >> I have not yet tested this code, but I do note that all the Realtek
> >> the RTL8187B set this bit.
> >> Until testing, I withhold judgment on the actual code change, but NACK
> >> inconsistency.
> > Perhaps the word "clearing" rather than "resetting" would be less
Thanks, John, for the clarification. Clearing is much better, really.
> I see the source of my confusion. I am in the habit of using lower case
> indicate the driver and upper case for the device. I would rewrite the
> code to say "The code for the RTL8187L does not do per packet TX AGC.
> the per packet TX AGC for RTL8187B ..."
I will use your rewriting. Thanks.
> I have now tested the patch. For my RTL8187B device, I get the following:
> Distance from AP Indicated rate Signal Measured TX rate
> 2m 18 Mb/s -17 dBm 10 Mb/s
> 10m(original) 11 Mb/s -48 dBm 2.6 Mb/s
> 10m(with patch) 11 Mb/s -43 dBm 4.1 Mb/s
> The measured TX rates were obtained using tcpperf. At a distance of 2 m,
> results were the same with and without the patch. At 10 m, the patch went
> through 2 exterior walls with drywall, insulation, and stucco. I'm not
> the increase from 2.6 to 4.1 Mb/s is reproducible. About all I can say is
> the change did not hurt the performance of the device, which is already
> If the OP can post numbers documenting the improvement, I have no
> this patch, even though every Realtek driver that I have seen turns "per
> TX AGC" on for the RTL8187B and turns it off for the RTL8187L.
Hum. Did you take a look at the version I've pointed out in the last
message, rtl8187B_linux_26.1056.1112.2009.release? It does not clear
PERPACKET_GAIN in the same function that it does for non-RTL8187B
devices. But it does in what should be the code for RTL8225Z2, in the
I did not measure the distances exactly here, while doing my tests. But
I could move the AP further away by about 5 or 6 meters. The AP
indicated a -84dB signal while receiving injected packages. It was a
rt73usb device. File transmission through scp was not working that
further away, but I could still get an improvement of about 3 to 4
meters of distance.
I am still going to do some more testing today in a customer's facility.
And I would like very much that Rogério, who has reported some problems
too, did some testing.