diff options
author | Christian Lamparter <chunkeey@googlemail.com> | 2010-05-01 18:18:18 +0200 |
---|---|---|
committer | John W. Linville <linville@tuxdriver.com> | 2010-05-07 14:55:47 -0400 |
commit | f3926b49b7122f66f8f2a1da4ae3275b112ab3e7 (patch) | |
tree | d4c595cc976d5b5e76077a67afc3e754f06c1c62 /fs/anon_inodes.c | |
parent | adfba3c7c026a6a5560d2a43fefc9b198cb74462 (diff) | |
download | talos-op-linux-f3926b49b7122f66f8f2a1da4ae3275b112ab3e7.tar.gz talos-op-linux-f3926b49b7122f66f8f2a1da4ae3275b112ab3e7.zip |
ar9170usb: remove deprecated aggregation code
This patch removes the incomplete AMPDU implementation in ar9170usb.
The code in question is:
* too big and complex (more than 550 SLOC.)
This is enough to qualify for a new separate code file!
* unbalanced quantity & quality
over-engineered areas like:
* xmit scheduling and queuing frames for multiple HT peers
* redundant frame sorting
are confronted by gaping holes:
* accurate transmission feedback
* firmware error-handling and device reset
* HT rate control algorithm
* error-prone
Since its inclusion, hardly anything was done to fix
any of the outlined flaws from the initial commit message.
=> This also indicates poor maintainability.
* relies heavily on several spinlocks.
As a result of this shortcomings, the code is slow and does not
even support the most basic 11n requirement: HT station mode.
Therefore, I request to purge my heap of **** from the kernel:
"ar9170: implement transmit aggregation".
The next item on the agenda is: (re-)start from scratch with
an adequate design to accommodate the special requirements
and features of the available frameworks and tools.
Signed-off-by: Christian Lamparter <chunkeey@googlemail.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'fs/anon_inodes.c')
0 files changed, 0 insertions, 0 deletions