On Tue, 2008-08-05 at 11:33 -0400, Clem Taylor wrote:
> On Tue, Aug 5, 2008 at 5:42 AM, Ondrej Zajicek <santiago <at> crfreenet.org> wrote:
> > https://madwifi.org/ticket/705
> > https://madwifi.org/attachment/ticket/705/ani_sensitivity_fix_v2.diff
> 
> This patch seemed to make a huge difference in some configurations. In
> one case I was getting 2.5MB/s without the patch and 3.5MB/s with. In
> others it didn't have any effect. It seems like a good patch for a
> 0.9.4.1 release.

The problems with the patch are:

1) It has never been in subversion on any branch.  This means that it
has received too little testing so far.

2) It's too opaque.  It calls ath_hal_setcapability() with undocumented
arguments 3 and 1.

3) We would need testing results in all modes (master, ad-hoc, managed).

4) We would probably need some evidence that the patch makes HAL change
some registers that e.g the Windows driver would change in the same way.
It would be nice to know what the original value is before we put 1
there.

It looks like I'm too limited in time to make a release today or
tomorrow, so we could apply thins patch for others to test until the end
of August.  But first I would want to make sure that the release indeed
cannot be made today.  Otherwise, the patch would be a good candidate
for 0.9.4.2.

-- 
Regards,
Pavel Roskin

-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/