On 2012-06-15 7:58 PM, Rajkumar Manoharan wrote: > On Fri, Jun 15, 2012 at 03:25:23PM +0200, Felix Fietkau wrote: >> It was found to be buggy on a variety of chipsets from AR913x to AR928x. >> The new version (which was introduced along with AR93xx support) is more >> reliable in preventing connectivity dropouts and also fixes MIB interrupt >> storm issues. >> >> Signed-off-by: Felix Fietkau <nbd@xxxxxxxxxxx> > > I dont think it is good idea to completely removing old ANI for all AR9002 > family chips. No one have ever tested the new ANI with all ar9002 chips. May > be there could be some regressions with old ANI. As we discussed in irc did > you get any feedback from openwrt users with partial revert patch? So it is > not viable to wipe out old ani support for older chips. Several OpenWrt users have tested this on various AR9001 and AR9002 devices and have reported a very significant increase in stability in a variety of environments. Sure, this needs a bit more testing, but the old code is known broken in many ways, yet at the same time a bit too convoluted for proper bug analysis. I think removing the old code is our best option for having a chance to again be able to fully understand the code and its various side effects. Leaving the old code in place just makes it more likely for regressions to show up when the new code is improved (as has already happened on several occasions). - Felix -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
