sfrench/cifs-2.6.git
16 years agodrivers/net/skfp: Remove dead code referencing pci_find_device()
Jeff Garzik [Thu, 2 Aug 2007 20:31:25 +0000 (16:31 -0400)]
drivers/net/skfp: Remove dead code referencing pci_find_device()

Signed-off-by: Jeff Garzik <jeff@garzik.org>
16 years agoIOC3: Switch hw checksumming to ethtool configurable.
Ralf Baechle [Wed, 25 Jul 2007 11:31:57 +0000 (12:31 +0100)]
IOC3: Switch hw checksumming to ethtool configurable.

Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
16 years agodrivers/net/cxgb3: removed several unneeded zero initilization
Denis Cheng [Wed, 18 Jul 2007 07:24:49 +0000 (15:24 +0800)]
drivers/net/cxgb3: removed several unneeded zero initilization

Cc: linux-bugs@chelsio.com
Signed-off-by: Denis Cheng <crquan@gmail.com>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
16 years agodev->priv to netdev_priv(dev), drivers/net/tokenring/
Yoann Padioleau [Mon, 23 Jul 2007 13:18:21 +0000 (15:18 +0200)]
dev->priv to netdev_priv(dev), drivers/net/tokenring/

Replacing accesses to dev->priv to netdev_priv(dev). The replacment
is safe when netdev_priv is used to access a private structure that is
right next to the net_device structure in memory.
Cf http://groups.google.com/group/comp.os.linux.development.system/browse_thread/thread/de19321bcd94dbb8/0d74a4adcd6177bd
This is the case when the net_device structure was allocated with
a call to alloc_netdev or one of its derivative.

Here is an excerpt of the semantic patch that performs the transformation

@ rule1 @
type T;
struct net_device *dev;
@@

 dev =
(
        alloc_netdev
|
        alloc_etherdev
|
        alloc_trdev
)
   (sizeof(T), ...)

@ rule1bis @
struct net_device *dev;
expression E;
@@
 dev->priv = E

@ rule2 depends on rule1 && !rule1bis  @
struct net_device *dev;
type rule1.T;
@@

- (T*) dev->priv
+ netdev_priv(dev)

PS: I have performed the same transformation on the whole kernel
and it affects around 70 files, most of them in drivers/net/.
Should I split my patch for each subnet directories ? (wireless/, wan/, etc)

Thanks to Thomas Surrel for helping me refining my semantic patch.

Signed-off-by: Yoann Padioleau <padator@wanadoo.fr>
 3c359.c       |   58 +++++++++++++++++++++++++++++-----------------------------
 ibmtr.c       |   38 +++++++++++++++++++-------------------
 lanstreamer.c |   32 ++++++++++++++++----------------
 madgemc.c     |    4 ++--
 olympic.c     |   36 ++++++++++++++++++------------------
 tmspci.c      |    4 ++--
 6 files changed, 86 insertions(+), 86 deletions(-)
Signed-off-by: Jeff Garzik <jeff@garzik.org>
16 years ago[netdrvr] skfp: remove a bunch of dead code
Jeff Garzik [Tue, 24 Jul 2007 05:30:36 +0000 (01:30 -0400)]
[netdrvr] skfp: remove a bunch of dead code

The driver has not compiled in anything except PCI support for many
years (see drivers/net/skfp/Makefile).  This driver is also unmaintained
for many years, so arguments for keeping the cross-OS, cross-bus (ISA,
EISA, MCA) code do not exist.

Signed-off-by: Jeff Garzik <jeff@garzik.org>
16 years ago[PATCH] libertas: bump driver version
Dan Williams [Fri, 3 Aug 2007 13:43:03 +0000 (09:43 -0400)]
[PATCH] libertas: bump driver version

Signed-off-by: Dan Williams <dcbw@redhat.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: fix sparse-reported problems
Dan Williams [Fri, 3 Aug 2007 13:40:55 +0000 (09:40 -0400)]
[PATCH] libertas: fix sparse-reported problems

A few fields being converted to the wrong sized type, and a few missed
endian conversions.

Signed-off-by: Dan Williams <dcbw@redhat.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: better descriptions for association errors
Dan Williams [Thu, 2 Aug 2007 17:20:29 +0000 (13:20 -0400)]
[PATCH] libertas: better descriptions for association errors

Describe the association response status code the firmware
returns, based on mail to libertas-dev from Ronak.

Signed-off-by: Dan Williams <dcbw@redhat.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: ignore spurious mesh autostart events
Dan Williams [Thu, 2 Aug 2007 17:20:12 +0000 (13:20 -0400)]
[PATCH] libertas: ignore spurious mesh autostart events

Don't trust the firmware to always send them at the right time,
ignore them when the driver thinks mesh autostart is disabled.

Signed-off-by: Dan Williams <dcbw@redhat.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: fix misspelling in debug message
Dan Williams [Thu, 2 Aug 2007 17:19:56 +0000 (13:19 -0400)]
[PATCH] libertas: fix misspelling in debug message

Signed-off-by: Dan Williams <dcbw@redhat.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: pass boot2 version to firmware
Luis Carlos Cobo [Thu, 2 Aug 2007 17:19:24 +0000 (13:19 -0400)]
[PATCH] libertas: pass boot2 version to firmware

Boot2 version used to be hardcoded in the uploaded firmware,
this patch preserves the boot2 version before uploading firmware
and sends it to the firmware again on resume.

Signed-off-by: Luis Carlos Cobo <luisca@cozybit.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: push WEXT scan requests to a work queue
Dan Williams [Thu, 2 Aug 2007 17:19:04 +0000 (13:19 -0400)]
[PATCH] libertas: push WEXT scan requests to a work queue

Push WEXT scan requests to a workqueue and have each partial scan queue
the next part, then only report results when the complete scan has finished.
Full scans don't go through the work queue.

Signed-off-by: Dan Williams <dcbw@redhat.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] drivers/net/wireless/libertas/cmd.c: fix adapter->driver_lock dereference
Eugene Teo [Thu, 2 Aug 2007 17:18:07 +0000 (13:18 -0400)]
[PATCH] drivers/net/wireless/libertas/cmd.c: fix adapter->driver_lock dereference

adapter is NULL if cmdnode is not.

Signed-off-by: Eugene Teo <eugeneteo@kernel.sg>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: fix assignment of WEP key type
Dan Williams [Thu, 2 Aug 2007 17:17:41 +0000 (13:17 -0400)]
[PATCH] libertas: fix assignment of WEP key type

keytype is a u8

Signed-off-by: Dan Williams <dcbw@redhat.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: monitor mode support for OLPC firmware
Luis Carlos Cobo [Thu, 2 Aug 2007 17:16:55 +0000 (13:16 -0400)]
[PATCH] libertas: monitor mode support for OLPC firmware

Driver support for the monitor mode support that will be available in the next
OLPC 'bleeding edge' Marvell firmware release (most likely, 5.110.16.p2).

To activate monitor mode,

echo mode > /sys/class/net/{ethX,mshX}/device/libertas_rtap

where mode is the hex mask that specifies which frames to sniff (in short, 0x1
for data, 0x2 for all management but beacons, 0x4 for beacons). Any non zero
mode will activate the monitor mode, inhibiting transmission in ethX and mshX
interfaces and routing all the incoming traffic to a new rtapX interface that
will output the packets in 802.11+radiotap headers format.

Signed-off-by: Luis Carlos Cobo <luisca@cozybit.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: fix a few wext abuses...
Jean Tourrilhes [Thu, 2 Aug 2007 17:16:30 +0000 (13:16 -0400)]
[PATCH] libertas: fix a few wext abuses...

o SIOCGIWNAME is not designed to return the version number of the driver.
On the other hand, you are free to abuse SIOCGIWNICKN for that purpose.

o Don't attempt to fix the WE19/WE20 transition in the driver, because
your fixes are bogus, and redundant with the code in the kernel (you may
endup with +2, you can't read 32 char ESSID...).

o In SIOCSIWTXPOW, if you specified in iwrange that you want dBm, you
should only get dBm, which allow to reduce code bloat.

Signed-off-by: Jean Tourrilhes <jt@hpl.hp.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: keep mesh autostart enabled while asleep
Luis Carlos Cobo [Thu, 2 Aug 2007 17:16:02 +0000 (13:16 -0400)]
[PATCH] libertas: keep mesh autostart enabled while asleep

After loading the firmware, mesh autostart will be disabled. After that, the
user will still be able to enable or disable it at will. On suspend, it will be
always activated and later on resume it will go back to the state it had before
going to sleep.

Signed-off-by: Luis Carlos Cobo <luisca@cozybit.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: revert CAPINFO_MASK to its original value
Luis Carlos Cobo [Thu, 2 Aug 2007 17:15:40 +0000 (13:15 -0400)]
[PATCH] libertas: revert CAPINFO_MASK to its original value

CAPINFO_MASK changed on commits 981f187b and a091095b. Reverting to the original
value.  Also move CAPINFO_MASK into the sole user, join.c.  CAPINFO_MASK
should be in host CPU byte order; capability is converted to device
byte order elsewhere.

This fixes OLPC ticket #2161

Signed-off-by: Luis Carlos Cobo <luisca@cozybit.com>
Acked-by: Dan Williams <dcbw@redhat.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: push mesh beacon bit to userspace in scan results
Dan Williams [Thu, 2 Aug 2007 17:14:56 +0000 (13:14 -0400)]
[PATCH] libertas: push mesh beacon bit to userspace in scan results

Signed-off-by: Dan Williams <dcbw@redhat.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: send association events on adhoc reassociation
Dan Williams [Thu, 2 Aug 2007 17:14:29 +0000 (13:14 -0400)]
[PATCH] libertas: send association events on adhoc reassociation

Send association event to userspace when reassociating to the same
ad-hoc network, because it's still an association.

Signed-off-by: Dan Williams <dcbw@redhat.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: fix two debug statements in cmdresp.c
Holger Schurig [Thu, 2 Aug 2007 17:14:07 +0000 (13:14 -0400)]
[PATCH] libertas: fix two debug statements in cmdresp.c

Purely cosmetic: this moves an lbs_deb_enter() to the proper place
and changes an erraneous lbs_deb_enter_args() into lbs_deb_leave_args()

Signed-off-by: Holger Schurig <hs4233@mail.mn-solutions.de>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: remove bss_descriptor->timestamp
Holger Schurig [Thu, 2 Aug 2007 17:13:00 +0000 (13:13 -0400)]
[PATCH] libertas: remove bss_descriptor->timestamp

Noone used this variable.

Signed-off-by: Holger Schurig <hs4233@mail.mn-solutions.de>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: remove bss_descriptior->networktsf
Holger Schurig [Thu, 2 Aug 2007 17:12:45 +0000 (13:12 -0400)]
[PATCH] libertas: remove bss_descriptior->networktsf

This value was parsed out, but then nowhere used ... except in
some debugfs output. I can't imagine anyone wanting to use this
value for anything real (as no other driver exports it), so
bye-bye.

Along this, made the columns of
/sys/kernel/debug/libertas_wireless/*/getscantable align again.

Signed-off-by: Holger Schurig <hs4233@mail.mn-solutions.de>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: remove adapter->scantype
Holger Schurig [Thu, 2 Aug 2007 17:12:27 +0000 (13:12 -0400)]
[PATCH] libertas: remove adapter->scantype

scantype was initialized with CMD_SCAN_TYPE_ACTIVE, but there is no code
that would ever change it, so we can use that variable directly.

Signed-off-by: Holger Schurig <hs4233@mail.mn-solutions.de>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: remove adapter->scanmode
Holger Schurig [Thu, 2 Aug 2007 17:12:12 +0000 (13:12 -0400)]
[PATCH] libertas: remove adapter->scanmode

scanmode was initialized with CMD_BSS_TYPE_ANY, but there is no code
that ever can store another value there, so it can go away.

Signed-off-by: Holger Schurig <hs4233@mail.mn-solutions.de>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: remove adapter->beaconperiod
Holger Schurig [Thu, 2 Aug 2007 17:11:38 +0000 (13:11 -0400)]
[PATCH] libertas: remove adapter->beaconperiod

beaconperiod was initialized with MRVDRV_BEACON_INTERVAL, but there is
no code that would ever change it's value. We can use the define directly.

Signed-off-by: Holger Schurig <hs4233@mail.mn-solutions.de>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: remove adapter->adhoc_grate_enabled
Holger Schurig [Thu, 2 Aug 2007 17:11:19 +0000 (13:11 -0400)]
[PATCH] libertas: remove adapter->adhoc_grate_enabled

The variable was initialized with 0 (false). There is no code that would
ever change it, so we can use the false-patch directly.

Signed-off-by: Holger Schurig <hs4233@mail.mn-solutions.de>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: remove adapter->listeninterval
Holger Schurig [Thu, 2 Aug 2007 17:10:59 +0000 (13:10 -0400)]
[PATCH] libertas: remove adapter->listeninterval

listeninterval was initialized with MRVDRV_DEFAULT_LISTEN_INTERVAL, but
there exists that would ever change it. So we can use this define directly.

Signed-off-by: Holger Schurig <hs4233@mail.mn-solutions.de>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: remove adapter->regiontableindex
Holger Schurig [Thu, 2 Aug 2007 17:10:41 +0000 (13:10 -0400)]
[PATCH] libertas: remove adapter->regiontableindex

The value was computed, but then never used.

Signed-off-by: Holger Schurig <hs4233@mail.mn-solutions.de>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: remove adapter->atimwindow
Holger Schurig [Thu, 2 Aug 2007 17:10:05 +0000 (13:10 -0400)]
[PATCH] libertas: remove adapter->atimwindow

This varaible was initialized with 0 but there is no code that would ever
change it's value. So it can go away.

Signed-off-by: Holger Schurig <hs4233@mail.mn-solutions.de>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: remove adapter->multipledtim
Holger Schurig [Thu, 2 Aug 2007 17:09:49 +0000 (13:09 -0400)]
[PATCH] libertas: remove adapter->multipledtim

multipledtim was initialized with MRVDRV_DEFAULT_MULTIPLE_DTIM and then
kept at that value, so we could use that define directly.

Signed-off-by: Holger Schurig <hs4233@mail.mn-solutions.de>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: remove adapter->locallisteninterval
Holger Schurig [Thu, 2 Aug 2007 17:09:34 +0000 (13:09 -0400)]
[PATCH] libertas: remove adapter->locallisteninterval

locallisteninterval was initialized with 0, but there is no code that
changes it, rendering it rather useless.

Signed-off-by: Holger Schurig <hs4233@mail.mn-solutions.de>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: remove adapter->nullpktinterval
Holger Schurig [Thu, 2 Aug 2007 17:09:15 +0000 (13:09 -0400)]
[PATCH] libertas: remove adapter->nullpktinterval

No code ever initialized this variable, so it was 0 because of kzalloc().
But no other code changes it, making it rather useless.

Signed-off-by: Holger Schurig <hs4233@mail.mn-solutions.de>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: remove adapter->{data,bcn}_avg_factor
Holger Schurig [Thu, 2 Aug 2007 17:08:24 +0000 (13:08 -0400)]
[PATCH] libertas: remove adapter->{data,bcn}_avg_factor

Those two variables were initialized with some default values, but there
is no code that would ever change them. So we could use as well the defaults
directly.

Signed-off-by: Holger Schurig <hs4233@mail.mn-solutions.de>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: remove adapter->rxpd_rate
Holger Schurig [Thu, 2 Aug 2007 17:07:56 +0000 (13:07 -0400)]
[PATCH] libertas: remove adapter->rxpd_rate

No code uses the contents of this variable, so it can go.

Signed-off-by: Holger Schurig <hs4233@mail.mn-solutions.de>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: remove adapter->txrate
Holger Schurig [Thu, 2 Aug 2007 17:07:15 +0000 (13:07 -0400)]
[PATCH] libertas: remove adapter->txrate

The value of txrate was only set by a CMD_802_11_TX_RATE_QUERY command,
but there was no code in the driver that ever issued this command.

Signed-off-by: Holger Schurig <hs4233@mail.mn-solutions.de>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: fix WEXT quality reporting
Dan Williams [Thu, 2 Aug 2007 17:06:54 +0000 (13:06 -0400)]
[PATCH] libertas: fix WEXT quality reporting

Found by Ronak and others at Marvell.

Signed-off-by: Dan Williams <dcbw@redhat.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: remove adapter->pkttxctrl
Holger Schurig [Thu, 2 Aug 2007 17:06:11 +0000 (13:06 -0400)]
[PATCH] libertas: remove adapter->pkttxctrl

The variable was initialized to 0 and nowhere else changed, so basically
the per-packet TX control wasn't used.

Signed-off-by: Holger Schurig <hs4233@mail.mn-solutions.de>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: remove adapter->scanprobes
Holger Schurig [Thu, 2 Aug 2007 17:05:53 +0000 (13:05 -0400)]
[PATCH] libertas: remove adapter->scanprobes

The variable was initialized to 0 and nowhere else to anything
different.

Signed-off-by: Holger Schurig <hs4233@mail.mn-solutions.de>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: remove adapter->prescan
Holger Schurig [Thu, 2 Aug 2007 17:05:32 +0000 (13:05 -0400)]
[PATCH] libertas: remove adapter->prescan

The value 1 was assigned to it and there was nowhere any code
that would have changed that to 0.

Signed-off-by: Holger Schurig <hs4233@mail.mn-solutions.de>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: remove adapter->{rx,tx}antenna
Holger Schurig [Thu, 2 Aug 2007 17:05:08 +0000 (13:05 -0400)]
[PATCH] libertas: remove adapter->{rx,tx}antenna

There was nowhere any code that used the values of those
variables.

This patch also removes two static functions that are now unused.

Signed-off-by: Holger Schurig <hs4233@mail.mn-solutions.de>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: remove unused adapter->prev_XXXX variables
Holger Schurig [Thu, 2 Aug 2007 17:04:28 +0000 (13:04 -0400)]
[PATCH] libertas: remove unused adapter->prev_XXXX variables

There were just used in some debug output, but nowhere else.

Signed-off-by: Holger Schurig <hs4233@mail.mn-solutions.de>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] add support for Marvell 8385 CF cards
Holger Schurig [Thu, 4 Oct 2007 00:25:41 +0000 (17:25 -0700)]
[PATCH] add support for Marvell 8385 CF cards

This patch adds support for Marvell based 8385 compact flash cards.

Signed-off-by: Holger Schurig <hs4233@mail.mn-solutions.de>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
16 years ago[PATCH] libertas: Avoid MESH_AUTOSTARTED spam on console
Luis Carlos Cobo [Thu, 2 Aug 2007 15:55:22 +0000 (11:55 -0400)]
[PATCH] libertas: Avoid MESH_AUTOSTARTED spam on console

Signed-off-by: Luis Carlos Cobo <luisca@cozybit.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: use LBS_DEB_HOST for host-to-card communications
Holger Schurig [Thu, 2 Aug 2007 15:54:31 +0000 (11:54 -0400)]
[PATCH] libertas: use LBS_DEB_HOST for host-to-card communications

... and LBS_DEB_CMD for command execution. Also tidies misc
comments to give a consistent output.

Signed-off-by: Holger Schurig <hs4233@mail.mn-solutions.de>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: use LBS_DEB_HOST for host-to-card communications
Holger Schurig [Thu, 2 Aug 2007 15:54:10 +0000 (11:54 -0400)]
[PATCH] libertas: use LBS_DEB_HOST for host-to-card communications

... and LBS_DEB_CMD for command execution. Also tidies misc
comments to give a consistent output.

Signed-off-by: Holger Schurig <hs4233@mail.mn-solutions.de>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: remove a hundred CMD_RET_xxx definitions
Holger Schurig [Thu, 2 Aug 2007 15:53:36 +0000 (11:53 -0400)]
[PATCH] libertas: remove a hundred CMD_RET_xxx definitions

types.h contains the same amount of CMD_RET_xxx and CMD_xxx definitions.
They contains the same info: the firmware command opcode and, when the
firmware sends back a result, the command opcode ORed with 0x8000.

Having the same data twice in the source code is redundant and can lead to
errors (e.g. if you update or delete only one instance). This patch removed
all CMD_RET_xxx definitions and introduces a simple CMD_RET() macro.

Signed-off-by: Holger Schurig <hs4233@mail.mn-solutions.de>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: make the hex dumper nicer
Holger Schurig [Thu, 2 Aug 2007 15:53:06 +0000 (11:53 -0400)]
[PATCH] libertas: make the hex dumper nicer

Currently, when you define LBS_DEB_HEX, you get every hex dump in the
whole driver, e.g. for LBS_DEB_CMD, LBS_DEB_RX, LBS_DEB_TX etc. This
patch makes sure that you only get the hexdump that you're interested in.

Renamed lbs_dbg_hex() into lbs_deb_hex(), like the other lbs_deb_XXX()
macros.

Made lbs_deb_hex() issue a line feed (and a new prompt) after 16 bytes.

As lbs_deb_hex() now prints the ":" after the prompt by itself, removed
the misc colons in the various *.c files.

lbs_deb_XXX() now print the debug category as well.

As lbs_deb_XXX() --- and especially lbs_deb_11d() --- now print the
category, I removed various "11D:" prefixes in 11d.c as well.

Signed-off-by: Holger Schurig <hs4233@mail.mn-solutions.de>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] Support for mesh autostart deactivation through sysfs
Luis Carlos Cobo [Thu, 2 Aug 2007 15:52:29 +0000 (11:52 -0400)]
[PATCH] Support for mesh autostart deactivation through sysfs

echo 0 > /sys/class/net/mshX/autostart_enabled

This is supported from Marvell firmware version 5.110.16.p0 (to be released).

Signed-off-by: Luis Carlos Cobo <luisca@cozybit.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: remove debugmode
Holger Schurig [Thu, 2 Aug 2007 15:51:18 +0000 (11:51 -0400)]
[PATCH] libertas: remove debugmode

There is nowhere any place that set's this variable.

Signed-off-by: Holger Schurig <hs4233@mail.mn-solutions.de>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: tune hardware info output
Holger Schurig [Thu, 2 Aug 2007 15:50:35 +0000 (11:50 -0400)]
[PATCH] libertas: tune hardware info output

This changes the output of hardware related info from:

libertas: GET_HW_SPEC: FWReleaseVersion- 5.0.11.p0
libertas: GET_HW_SPEC: Permanent addr-  0:16:41:72:f6:a8
libertas: GET_HW_SPEC: hwifversion=0x2  version=0x213

to:

libertas: GET_HW_SPEC: FWReleaseVersion: 5.0.11.p0
libertas: GET_HW_SPEC: Permanent addr: 00:16:41:72:f6:a8
libertas: GET_HW_SPEC: hwifversion: 0x2, version: 0x213

Signed-off-by: Holger Schurig <hs4233@mail.mn-solutions.de>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: access mesh_dev more carefully
Holger Schurig [Thu, 2 Aug 2007 15:50:12 +0000 (11:50 -0400)]
[PATCH] libertas: access mesh_dev more carefully

The CF/SDIO firmware doesn't support Mesh, so priv->mesh_dev is
NULL there. Protect all accesses.

Signed-off-by: Holger Schurig <hs4233@mail.mn-solutions.de>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: uppercase some #defines
Holger Schurig [Thu, 2 Aug 2007 15:49:45 +0000 (11:49 -0400)]
[PATCH] libertas: uppercase some #defines

Usually constants defined by #define are in ALL_UPPERCASE. This patch
fixes this.

I also shuffled the bits around so that they match the bit positions in the
host-interrupt-state register of the CF/SDIO card :-)

Signed-off-by: Holger Schurig <hs4233@mail.mn-solutions.de>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: make more functions static & remove unused functions
Holger Schurig [Thu, 2 Aug 2007 15:49:06 +0000 (11:49 -0400)]
[PATCH] libertas: make more functions static & remove unused functions

Some functions where declared in header files, but used only once. They are
now static functions.

After doing this, I found out that some functions weren't used at all. I
removed this dead code.

Signed-off-by: Holger Schurig <hs4233@mail.mn-solutions.de>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: fix one more sparse warning
Holger Schurig [Thu, 2 Aug 2007 15:45:30 +0000 (11:45 -0400)]
[PATCH] libertas: fix one more sparse warning

adhoc_rates_b is only used locally, so make it static

Signed-off-by: Holger Schurig <hs4233@mail.mn-solutions.de>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: remove fw.c
Holger Schurig [Thu, 2 Aug 2007 15:45:12 +0000 (11:45 -0400)]
[PATCH] libertas: remove fw.c

Firmware download is quite different for different hardware. The SDIO and CF
cards have two flat files that need to be downloaded, whereas the USB driver
needs only one file, but with an internal structure.

The code that handles this (USB only) structured file is currently in fw.c.
This patch moves this code into if_usb.c. The remaining functions in fw.c
have not much to do with firmware, they are various card- and network-stack
initialisation functions. I've moved them into main.c.

Signed-off-by: Holger Schurig <hs4233@mail.mn-solutions.de>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: simplify and clean up data rate handling
Dan Williams [Thu, 2 Aug 2007 15:40:45 +0000 (11:40 -0400)]
[PATCH] libertas: simplify and clean up data rate handling

Remove unused/duplicated fields and consolidate static data rate arrays,
for example the libertas_supported_rates[] and datarates[] arrays in
the bss_descriptor structure, and the libertas_supported_rates field
in the wlan_adapter structure.

Introduce libertas_fw_index_to_data_rate and libertas_data_rate_to_fw_index
functions and use them everywhere firmware requires a rate index rather
than a rate array.

The firmware requires the 4 basic rates to have the MSB set, but most
other stuff doesn't, like WEXT and mesh ioctls.  Therefore, only set the MSB
on basic rates when pushing rate arrays to firmware instead of doing a ton
of (rate & 0x7f) everywhere.

Signed-off-by: Dan Williams <dcbw@redhat.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: wlan_ -> libertas_ function prefix renames for main.c
Dan Williams [Thu, 2 Aug 2007 15:39:19 +0000 (11:39 -0400)]
[PATCH] libertas: wlan_ -> libertas_ function prefix renames for main.c

Signed-off-by: Dan Williams <dcbw@redhat.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: move generic firmware reset command to common code
Dan Williams [Thu, 2 Aug 2007 15:36:22 +0000 (11:36 -0400)]
[PATCH] libertas: move generic firmware reset command to common code

It's not USB specific, so move it out of the USB interface code.

Signed-off-by: Dan Williams <dcbw@redhat.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: fix more mixed-case abuse
Dan Williams [Thu, 2 Aug 2007 15:35:46 +0000 (11:35 -0400)]
[PATCH] libertas: fix more mixed-case abuse

Mistakently introduced by a previous patch to upper-case all command
constants.

Signed-off-by: Dan Williams <dcbw@redhat.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: bump version to 322.p1
Dan Williams [Thu, 2 Aug 2007 15:34:47 +0000 (11:34 -0400)]
[PATCH] libertas: bump version to 322.p1

Signed-off-by: Dan Williams <dcbw@redhat.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: new mesh control knobs
Dan Williams [Thu, 2 Aug 2007 15:34:24 +0000 (11:34 -0400)]
[PATCH] libertas: new mesh control knobs

Support for new mesh control knobs on firmware 5.220.11.p4:

Signed-off-by: Luis Carlos Cobo <luisca@cozybit.com>
Signed-off-by: Dan Williams <dcbw@redhat.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: remove thread.h and make kthread usage clearer
Dan Williams [Thu, 2 Aug 2007 15:32:25 +0000 (11:32 -0400)]
[PATCH] libertas: remove thread.h and make kthread usage clearer

Remove the thread.h abstractions and opencode kthread stuff
to make it clearer.

Signed-off-by: Dan Williams <dcbw@redhat.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: fix debug build breakage due to field rename
Dan Williams [Thu, 2 Aug 2007 15:31:49 +0000 (11:31 -0400)]
[PATCH] libertas: fix debug build breakage due to field rename

Missed when fixing mixed-case structure field names.

Signed-off-by: Dan Williams <dcbw@redhat.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: re-uppercase command defines and other constants
Dan Williams [Thu, 2 Aug 2007 15:31:18 +0000 (11:31 -0400)]
[PATCH] libertas: re-uppercase command defines and other constants

For readability.

Signed-off-by: Dan Williams <dcbw@redhat.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: fix mixed-case abuse in cmd_ds_802_11_ad_hoc_start
Dan Williams [Thu, 2 Aug 2007 15:18:40 +0000 (11:18 -0400)]
[PATCH] libertas: fix mixed-case abuse in cmd_ds_802_11_ad_hoc_start

Signed-off-by: Dan Williams <dcbw@redhat.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: fix mixed-case abuse in cmd_ds_802_11_ad_hoc_result
Dan Williams [Thu, 2 Aug 2007 15:18:23 +0000 (11:18 -0400)]
[PATCH] libertas: fix mixed-case abuse in cmd_ds_802_11_ad_hoc_result

Signed-off-by: Dan Williams <dcbw@redhat.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: fix mixed-case abuse in cmd_ds_802_11_scan
Dan Williams [Thu, 2 Aug 2007 15:16:07 +0000 (11:16 -0400)]
[PATCH] libertas: fix mixed-case abuse in cmd_ds_802_11_scan

Signed-off-by: Dan Williams <dcbw@redhat.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: remove if_bootcmd.c
Dan Williams [Thu, 2 Aug 2007 15:14:49 +0000 (11:14 -0400)]
[PATCH] libertas: remove if_bootcmd.c

Move the only function in it to if_usb.c, which was its
only user anyway.

Signed-off-by: Dan Williams <dcbw@redhat.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: specific mesh scan for mshX interface
Luis Carlos Cobo [Thu, 2 Aug 2007 14:51:27 +0000 (10:51 -0400)]
[PATCH] libertas: specific mesh scan for mshX interface

With this patch, scanning with mshX interface will only return mesh networks. To
differentiate them, a specific mesh IE in beacons/probe responses is used. This
IE has been introduced in firmware release 5.110.14. Note:

Even though there can be at most a single mesh per channel, this scan might
return several networks in the same channel.

If all nodes in a mesh network are associated to an AP, they won't produce
beacons/probe responses, thus the network will not be listed. This will be fixed
in future firmware releases.

Scan on ethX interface is not filtered, so it will list both mesh and non-mesh
networks.

Signed-off-by: Luis Carlos Cobo <luisca@cozybit.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: clean up 802.11 IE post-scan handling
Dan Williams [Thu, 2 Aug 2007 14:48:02 +0000 (10:48 -0400)]
[PATCH] libertas: clean up 802.11 IE post-scan handling

Remove struct IE_WPA and just use direct checking of the IE
bytes like ipw.  Remove WLAN_802_11_VARIABLE_IEs because
it's unused.

Kill ieeetypes_elementid enum and just use MFIE_* from
ieee80211.h.  Also use struct ieee80211_info_element for
scan buffer processing to simplify pointer usage.

Signed-off-by: Dan Williams <dcbw@redhat.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: clean up indentation in libertas_association_worker
Dan Williams [Thu, 2 Aug 2007 14:47:17 +0000 (10:47 -0400)]
[PATCH] libertas: clean up indentation in libertas_association_worker

Signed-off-by: Dan Williams <dcbw@redhat.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: rename WLAN_802_11_KEY to enc_key and clean up usage
Dan Williams [Thu, 2 Aug 2007 14:45:55 +0000 (10:45 -0400)]
[PATCH] libertas: rename WLAN_802_11_KEY to enc_key and clean up usage

It doesn't touch hardware and therefore doesn't need endian notations
either.

Signed-off-by: Dan Williams <dcbw@redhat.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] libertas: kill ieeetypes_capinfo bitfield, use ieee80211.h types
Dan Williams [Thu, 2 Aug 2007 14:43:44 +0000 (10:43 -0400)]
[PATCH] libertas: kill ieeetypes_capinfo bitfield, use ieee80211.h types

Use standard BSS capability field constants from ieee80211.h.

Signed-off-by: Dan Williams <dcbw@redhat.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] dev->priv to netdev_priv(dev), for drivers/net/wireless
Yoann Padioleau [Fri, 3 Aug 2007 17:37:16 +0000 (19:37 +0200)]
[PATCH] dev->priv to netdev_priv(dev), for drivers/net/wireless

Replacing accesses to dev->priv to netdev_priv(dev). The replacment
is safe when netdev_priv is used to access a private structure that is
right next to the net_device structure in memory. Cf
http://groups.google.com/group/comp.os.linux.development.system/browse_thread/thread/de19321bcd94dbb8/0d74a4adcd6177bd
This is the case when the net_device structure was allocated with
a call to alloc_netdev or one of its derivative.

Signed-off-by: Yoann Padioleau <padator@wanadoo.fr>
Cc: mcgrof@gmail.com
Cc: linux-wireless@vger.kernel.org
Cc: akpm@linux-foundation.org
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] drivers/net/wireless/prism54/oid_mgt.c: kmalloc + memset conversion to kzalloc
Mariusz Kozlowski [Tue, 31 Jul 2007 21:34:50 +0000 (23:34 +0200)]
[PATCH] drivers/net/wireless/prism54/oid_mgt.c: kmalloc + memset conversion to kzalloc

Signed-off-by: Mariusz Kozlowski <m.kozlowski@tuxland.pl>
Acked-by: Luis R. Rodriguez <mcgrof@gmail.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] Use mutex instead of semaphore in the Host AP driver
Matthias Kaehlcke [Mon, 30 Jul 2007 05:40:04 +0000 (07:40 +0200)]
[PATCH] Use mutex instead of semaphore in the Host AP driver

The Host AP driver uses a semaphore as mutex. Use the mutex API
instead of the (binary) semaphore.

Signed-off-by: Matthias Kaehlcke <matthias.kaehlcke@gmail.com>
Acked-by: Satyam Sharma <satyam@infradead.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] remove gratuitous space in airo module description
Bill Nottingham [Fri, 27 Jul 2007 23:43:17 +0000 (19:43 -0400)]
[PATCH] remove gratuitous space in airo module description

Currently the modinfo looks like:

description:    Support for Cisco/Aironet 802.11 wireless ethernet                    cards.  Direct support for ISA/PCI/MPI cards and support               for PCMCIA when used with airo_cs.

Arguably, it should be cut at the end of the first sentence.
This at least makes it somewhat more legible.

Signed-off-by: Bill Nottingham <notting@redhat.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] Kconfig: remove references of pcmcia-cs
Faidon Liambotis [Sun, 22 Jul 2007 13:16:59 +0000 (16:16 +0300)]
[PATCH] Kconfig: remove references of pcmcia-cs

pcmcia-cs/cardmgr is deprecated and mentioning it in the help text is
misleading.

Signed-off-by: Faidon Liambotis <paravoid@debian.org>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] Kconfig: order options
Faidon Liambotis [Sun, 22 Jul 2007 13:16:36 +0000 (16:16 +0300)]
[PATCH] Kconfig: order options

Reorder the Atmel options so that the menu appears saner.
Before:
  < >   Hermes chipset 802.11b support (Orinoco/Prism2/Symbol)
  <*>   Atmel at76c50x chipset  802.11b support
  < >     Atmel at76c506 PCI cards (NEW)
  < > Cisco/Aironet 34X/35X/4500/4800 PCMCIA cards
  < > Atmel at76c502/at76c504 PCMCIA cards (NEW)

After:
  < >   Hermes chipset 802.11b support (Orinoco/Prism2/Symbol)
  <*>   Atmel at76c50x chipset  802.11b support
  < >     Atmel at76c506 PCI cards (NEW)
  < >     Atmel at76c502/at76c504 PCMCIA cards (NEW)
  < >   Cisco/Aironet 34X/35X/4500/4800 PCMCIA cards

Signed-off-by: Faidon Liambotis <paravoid@debian.org>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] zd1211rw: monitor all packets
Ulrich Kunitz [Sat, 21 Jul 2007 21:42:13 +0000 (22:42 +0100)]
[PATCH] zd1211rw: monitor all packets

While in monitor mode the zd1211rw received only a limited
set of packets. This patch forwards now all packets the device
receives. Notify that while monitoring no FCS checks are done; so
strange packets might appear in the network sniffer of your
choice.

ATTENTION: Support for multiple interfaces on a single ZD1211
device is currently broken. So this code works only on the first
interface.

Here is an example to put the device in monitor mode.

iwconfig wlan0 mode monitor
ifconfig wlan0 up
iwconfig wlan0 channel 10

[dsd@gentoo.org: backport to mainline]
Signed-off-by: Ulrich Kunitz <kune@deine-taler.de>
Signed-off-by: Daniel Drake <dsd@gentoo.org>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[PATCH] Add adm8211 802.11b wireless driver
Michael Wu [Fri, 31 Aug 2007 05:15:25 +0000 (01:15 -0400)]
[PATCH] Add adm8211 802.11b wireless driver

This patch adds a mac80211 wireless driver for ADMtek ADM8211 based
wireless cards.

Signed-off-by: Michael Wu <flamingice@sourmilk.net>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
16 years ago[MAC80211]: remove/change some comments about Michael MIC hardware offload
Johannes Berg [Mon, 10 Sep 2007 12:15:25 +0000 (14:15 +0200)]
[MAC80211]: remove/change some comments about Michael MIC hardware offload

There are a few TODO comments in the mac80211 sources regarding
hardware offload for Michael MIC verification. Those items are,
however, better handled in the driver instead of the stack, if
any device requires such hand-holding.

Signed-off-by: Johannes Berg <johannes@sipsolutions.net>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
16 years ago[MAC80211]: PS mode fix
Tomas Winkler [Sun, 2 Sep 2007 19:58:21 +0000 (22:58 +0300)]
[MAC80211]: PS mode fix

tx.mode must be set also for buffered frames. It is used in the tx hanlders

Signed-off-by: Tomas Winkler <tomas.winkler@intel.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
16 years ago[MAC80211]: use internal network device stats
Stephen Hemminger [Fri, 24 Aug 2007 18:29:34 +0000 (11:29 -0700)]
[MAC80211]: use internal network device stats

Stats are now available for device usage inside network_device

Signed-off-by: Stephen Hemminger <shemminger@linux-foundation.org>
Acked-by: Johannes Berg <johannes@sipsolutions.net>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
16 years ago[MAC80211]: get STA after tx radiotap snipped
warmcat [Fri, 14 Sep 2007 15:10:25 +0000 (11:10 -0400)]
[MAC80211]: get STA after tx radiotap snipped

Johannes Berg noticed that in __ieee80211_tx_prepare() we try to get the
STA from addr1 of the ieee80211 header when the radiotap header is actually
still at the front of the packet.  This patch defers doing that until the
radiotap header is gone.

Signed-off-by: Andy Green <andy@warmcat.com>
Signed-off-by: Johannes Berg <johannes@sipsolutions.net>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
16 years ago[MAC80211]: ignore key index on pairwise key (WEP only)
Volker Braun [Fri, 14 Sep 2007 15:10:25 +0000 (11:10 -0400)]
[MAC80211]: ignore key index on pairwise key (WEP only)

Work-around for broken APs that use a non-zero key index for WEP
pairwise keys. With this patch, WEP encryption only is exempt from
providing a zero key index.

Signed-off-by: Volker Braun <volker.braun@physik.hu-berlin.de>
Signed-off-by: Johannes Berg <johannes@sipsolutions.net>
Acked-by: Michael Wu <flamingice@sourmilk.net>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
16 years ago[MAC80211]: remove TKIP mixing for hw accel again
Johannes Berg [Fri, 14 Sep 2007 15:10:25 +0000 (11:10 -0400)]
[MAC80211]: remove TKIP mixing for hw accel again

The TKIP mixing code was added for the benefit of Intel's ipw3945
chipset but that code ended up not using it. We have previously
identified many problems with this code and it crystallized that
library functions for mixing are likely to handle this in much
more generality and might allow b43 to take advantage of hardware
acceleration for TKIP.

Due to these reasons, remove the TKIP mixing for hardware
accelerated crypto operations.

Signed-off-by: Johannes Berg <johannes@sipsolutions.net>
Acked-by: Michael Buesch <mb@bu3sch.de>
Acked-by: Michael Wu <flamingice@sourmilk.net>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
16 years ago[MAC80211]: remove HW_KEY_IDX_INVALID
Johannes Berg [Fri, 14 Sep 2007 15:10:25 +0000 (11:10 -0400)]
[MAC80211]: remove HW_KEY_IDX_INVALID

This patch makes the mac80211/driver interface rely only on the
IEEE80211_TXCTL_DO_NOT_ENCRYPT flag to signal to the driver whether
a frame should be encrypted or not, since mac80211 internally no
longer relies on HW_KEY_IDX_INVALID either this removes it, changes
the key index to be a u8 in all places and makes the full range of
the value available to drivers.

Signed-off-by: Johannes Berg <johannes@sipsolutions.net>
Acked-by: Michael Wu <flamingice@sourmilk.net>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
16 years ago[MAC80211]: some more documentation
Johannes Berg [Fri, 14 Sep 2007 15:10:25 +0000 (11:10 -0400)]
[MAC80211]: some more documentation

This patch formats some documentation in mac80211.h into kerneldoc
and also adds some more explanations for hardware crypto.

Signed-off-by: Johannes Berg <johannes@sipsolutions.net>
Acked-by: Michael Wu <flamingice@sourmilk.net>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
16 years ago[MAC80211]: remove set_key_idx callback
Johannes Berg [Fri, 14 Sep 2007 15:10:25 +0000 (11:10 -0400)]
[MAC80211]: remove set_key_idx callback

No existing drivers use this callback, hence there's no telling
how it might be used. In fact, it is unlikely to be of much use
as-is because the default key index isn't something that the
driver can do much with without knowing which interface it was
for etc. And if it needs the key index for the transmitted frame,
it can get it by keeping a reference to the key_conf structure
and looking it up by hw_key_idx.

Signed-off-by: Johannes Berg <johannes@sipsolutions.net>
Acked-by: Michael Wu <flamingice@sourmilk.net>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
16 years ago[MAC80211]: rework hardware crypto flags
Johannes Berg [Fri, 14 Sep 2007 15:10:25 +0000 (11:10 -0400)]
[MAC80211]: rework hardware crypto flags

This patch reworks the various hardware crypto related
flags to make them more local, i.e. put them with each
key or each packet instead of into the hw struct.

Signed-off-by: Johannes Berg <johannes@sipsolutions.net>
Acked-by: Michael Wu <flamingice@sourmilk.net>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
16 years ago[MAC80211]: remove turbo modes
Johannes Berg [Fri, 14 Sep 2007 15:10:25 +0000 (11:10 -0400)]
[MAC80211]: remove turbo modes

This patch removes all mention of the atheros turbo modes that
can't possibly work properly anyway since in some places we don't
check for them when we should.

I have no idea what the iwlwifi drivers were doing with these but
it can't possibly have been correct.

Cc: Zhu Yi <yi.zhu@intel.com>
Signed-off-by: Johannes Berg <johannes@sipsolutions.net>
Acked-by: Michael Wu <flamingice@sourmilk.net>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
16 years ago[MAC80211]: fix race conditions with keys
Johannes Berg [Fri, 14 Sep 2007 15:10:24 +0000 (11:10 -0400)]
[MAC80211]: fix race conditions with keys

During receive processing, we select the key long before using it and
because there's no locking it is possible that we kfree() the key
after having selected it but before using it for crypto operations.
Obviously, this is bad.

Secondly, during transmit processing, there are two possible races: We
have a similar race between select_key() and using it for encryption,
but we also have a race here between select_key() and hardware
encryption (both when a key is removed.)

This patch solves these issues by using RCU: when a key is to be freed,
we first remove the pointer from the appropriate places (sdata->keys,
sdata->default_key, sta->key) using rcu_assign_pointer() and then
synchronize_rcu(). Then, we can safely kfree() the key and remove it
from the hardware. There's a window here where the hardware may still
be using it for decryption, but we can't work around that without having
two hardware callbacks, one to disable the key for RX and one to disable
it for TX; but the worst thing that will happen is that we receive a
packet decrypted that we don't find a key for any more and then drop it.

When we add a key, we first need to upload it to the hardware and then,
using rcu_assign_pointer() again, link it into our structures.

In the code using keys (TX/RX paths) we use rcu_dereference() to get the
key and enclose the whole tx/rx section in a rcu_read_lock() ...
rcu_read_unlock() block. Because we've uploaded the key to hardware
before linking it into internal structures, we can guarantee that it is
valid once get to into tx().

One possible race condition remains, however: when we have hardware
acceleration enabled and the driver shuts down the queues, we end up
queueing the frame. If now somebody removes the key, the key will be
removed from hwaccel and then then driver will be asked to encrypt the
frame with a key index that has been removed. Hence, drivers will need
to be aware that the hw_key_index they are passed might not be under
all circumstances. Most drivers will, however, simply ignore that
condition and encrypt the frame with the selected key anyway, this
only results in a frame being encrypted with a wrong key or dropped
(rightfully) because the key was not valid. There isn't much we can
do about it unless we want to walk the pending frame queue every time
a key is removed and remove all frames that used it.

This race condition, however, will most likely be solved once we add
multiqueue support to mac80211 because then frames will be queued
further up the stack instead of after being processed.

Signed-off-by: Johannes Berg <johannes@sipsolutions.net>
Acked-by: Michael Wu <flamingice@sourmilk.net>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
16 years ago[MAC80211]: don't send invalid QoS frames
Johannes Berg [Fri, 14 Sep 2007 15:10:24 +0000 (11:10 -0400)]
[MAC80211]: don't send invalid QoS frames

Kalle Valo noticed that QoS frames are sent with an invalid QoS control
field; this is because we increase the header length but neither
initialise the space nor actually have enough space in the header
structure for the QoS control field.

This patch fixes it by treating the QoS field specially and appending it
explicitly, initialising it to zero.

Signed-off-by: Johannes Berg <johannes@sipsolutions.net>
Acked-by: Michael Wu <flamingice@sourmilk.net>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
16 years ago[MAC80211]: remove spy wext ioctls
Johannes Berg [Fri, 14 Sep 2007 15:10:24 +0000 (11:10 -0400)]
[MAC80211]: remove spy wext ioctls

mac80211 never calls wireless_spy_update so these aren't
useful.

Signed-off-by: Johannes Berg <johannes@sipsolutions.net>
Acked-by: Michael Wu <flamingice@sourmilk.net>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
16 years ago[IPV4]: Convert rt_check_expire() from softirq processing to workqueue.
Eric Dumazet [Sat, 15 Sep 2007 17:55:54 +0000 (10:55 -0700)]
[IPV4]: Convert rt_check_expire() from softirq processing to workqueue.

On loaded/big hosts, rt_check_expire() if of litle use, because it
generally breaks out of its main loop because of a jiffies change.

It can take a long time (read : timer invocations) to actually
scan the whole hash table, freeing unused entries.

Converting it to use a workqueue instead of softirq is a nice
move because we can allow rt_check_expire() to do the scan
it is supposed to do, without hogging the CPU.

This has an impact on the average number of entries in cache,
reducing ram usage. Cache is more responsive to parameter
changes (/proc/sys/net/ipv4/route/gc_timeout and
/proc/sys/net/ipv4/route/gc_interval)

Note: Maybe the default value of gc_interval (60 seconds)
is too high, since this means we actually need 5 (300/60)
invocations to scan the whole table.

Signed-off-by: Eric Dumazet <dada1@cosmosbay.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
16 years ago[RFKILL]: Add rfkill documentation
Ivo van Doorn [Thu, 13 Sep 2007 07:22:55 +0000 (09:22 +0200)]
[RFKILL]: Add rfkill documentation

Add a documentation file which contains
a short description about rfkill with some
notes about drivers and the userspace interface.

Changes since v1 and v2:
 - Spellchecking

Signed-off-by: Ivo van Doorn <IvDoorn@gmail.com>
Acked-by: Dmitry Torokhov <dtor@mail.ru>
Acked-by: Randy Dunlap <randy.dunlap@oracle.com>
16 years ago[RFKILL]: Add support for ultrawideband
Ivo van Doorn [Thu, 13 Sep 2007 07:21:31 +0000 (09:21 +0200)]
[RFKILL]: Add support for ultrawideband

This patch will add support for UWB keys to rfkill,
support for this has been requested by Inaky.

Signed-off-by: Ivo van Doorn <IvDoorn@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
16 years ago[RFKILL]: Remove IRDA
Ivo van Doorn [Thu, 13 Sep 2007 07:20:42 +0000 (09:20 +0200)]
[RFKILL]: Remove IRDA

As Dmitry pointed out earlier, rfkill-input.c
doesn't support irda because there are no users
and we shouldn't add unrequired KEY_ defines.

However, RFKILL_TYPE_IRDA was defined in the
rfkill.h header file and would confuse people
about whether it is implemented or not.

This patch removes IRDA support completely,
so it can be added whenever a driver wants the
feature.

Signed-off-by: Ivo van Doorn <IvDoorn@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>