d/freebsd-dev
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

First pass of adding transmit parameters as an option for outbound

Browse Source 
802.11 mbufs.

The raw transmit path currently doesn't make it easy to queue
these frames:

* there's no node reference stored in the mbuf, like for the normal
  path, and
* the bpf supplied raw transmit parameters (rate, rts/cts, etc)
  are passed in as an argument, not as an mbuf tag.

In order to support driver queuing of these frames, we need to
be able to put the above into the mbuf before the driver gets it,
so the driver /can/ put it into a queue if needed.

Use an mbuf tag and for now just verbatim copy the bpf parameters
into it.  Later on it may grow to include more options but this
will do for now.

Why would you want to queue raw frames? Well, in the case of
iwn(4), we can't send the firmware frames to transmit before
we hear a beacon - the firmware will consider passive channels
as unavailable until it hears a beacon.  The firmware "passive"
channel state is cleared upon each RXON command, which is sent to
update association status.  So, when we attempt association and
authorisation, the RXON command causes the firmware to clear out
what it's already seen, and so we have to wait for a beacon before
we can transmit.

Before people get overly excited - this alone doesn't "fix" 5GHz
operation - it just makes it (more) possible.

The aim here is to convert all the drivers over to use a raw_xmit()
API that doesn't include the node and params - instead they'd get
those from the mbuf.  Then raw_xmit() becomes just a side-channel
version of the normal transmit path for management traffic.

MFC after:	2 weeks
Sponsored by:	Norse Corp, Inc.
This commit is contained in:
Adrian Chadd 2015-06-04 06:30:39 +00:00
parent 5d97054765
commit 9afc11b224
Notes: svn2git 2020-12-20 02:59:44 +00:00 
svn path=/head/; revision=283980
2 changed files with 45 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

34
sys/net80211/ieee80211_freebsd.c
View File

@ -474,6 +474,40 @@ ieee80211_add_callback(struct mbuf *m,
return 1;
}
int
ieee80211_add_xmit_params(struct mbuf *m,
const struct ieee80211_bpf_params *params)
{
struct m_tag *mtag;
struct ieee80211_tx_params *tx;
mtag = m_tag_alloc(MTAG_ABI_NET80211, NET80211_TAG_XMIT_PARAMS,
sizeof(struct ieee80211_tx_params), M_NOWAIT);
if (mtag == NULL)
return (0);
tx = (struct ieee80211_tx_params *)(mtag+1);
memcpy(&tx->params, params, sizeof(struct ieee80211_bpf_params));
m_tag_prepend(m, mtag);
return (1);
}
int
ieee80211_get_xmit_params(struct mbuf *m,
struct ieee80211_bpf_params *params)
{
struct m_tag *mtag;
struct ieee80211_tx_params *tx;
mtag = m_tag_locate(m, MTAG_ABI_NET80211, NET80211_TAG_XMIT_PARAMS,
NULL);
if (mtag == NULL)
return (-1);
tx = (struct ieee80211_tx_params *)(mtag + 1);
memcpy(params, &tx->params, sizeof(struct ieee80211_bpf_params));
return (0);
}
void
ieee80211_process_callback(struct ieee80211_node *ni,
struct mbuf *m, int status)

11
sys/net80211/ieee80211_freebsd.h
View File

@ -327,6 +327,9 @@ int ieee80211_add_callback(struct mbuf *m,
void (*func)(struct ieee80211_node *, void *, int), void *arg);
void ieee80211_process_callback(struct ieee80211_node *, struct mbuf *, int);
#define NET80211_TAG_XMIT_PARAMS 1
/* See below; this is after the bpf_params definition */
struct ieee80211com;
int ieee80211_parent_xmitpkt(struct ieee80211com *, struct mbuf *);
int ieee80211_vap_xmitpkt(struct ieee80211vap *, struct mbuf *);
@ -607,6 +610,14 @@ struct ieee80211_bpf_params {
uint8_t ibp_rate3; /* series 4 IEEE tx rate */
};
struct ieee80211_tx_params {
struct ieee80211_bpf_params params;
};
int ieee80211_add_xmit_params(struct mbuf *m,
const struct ieee80211_bpf_params *);
int ieee80211_get_xmit_params(struct mbuf *m,
struct ieee80211_bpf_params *);
/*
* Malloc API. Other BSD operating systems have slightly
* different malloc/free namings (eg DragonflyBSD.)