diff options
| author | adrian <adrian@FreeBSD.org> | 2012-06-14 04:24:13 +0000 |
|---|---|---|
| committer | adrian <adrian@FreeBSD.org> | 2012-06-14 04:24:13 +0000 |
| commit | a65a9620af9a7fc5136edbcf7488e62299d44e0c (patch) | |
| tree | 5c1e0faa0f2e5e3aafc58e0c43da8fc87493fd63 | |
| parent | b639382febadbf28291aa6d978f0dd8a9bd846ee (diff) | |
| download | FreeBSD-src-a65a9620af9a7fc5136edbcf7488e62299d44e0c.zip FreeBSD-src-a65a9620af9a7fc5136edbcf7488e62299d44e0c.tar.gz | |
Shrink ath_buf a little more:
* Resize some types. In particular, bfs_seqno can be uint16_t for now.
Previous work would assign the unassigned seqno a value of -1, which
I obviously can't do here.
* Remove bfs_pktdur. It was in the original code but nothing so far uses
it.
This gets ath_buf down (on my i386 system) to 292 bytes from 300 bytes.
I'd rather it be much, much smaller.
| -rw-r--r-- | sys/dev/ath/if_athvar.h | 15 |
1 files changed, 8 insertions, 7 deletions
diff --git a/sys/dev/ath/if_athvar.h b/sys/dev/ath/if_athvar.h index d58c83f..2665c33 100644 --- a/sys/dev/ath/if_athvar.h +++ b/sys/dev/ath/if_athvar.h @@ -211,15 +211,16 @@ struct ath_buf { /* This state is kept to support software retries and aggregation */ struct { - int bfs_seqno; /* sequence number of this packet */ - int bfs_retries; /* retry count */ - uint16_t bfs_tid; /* packet TID (or TID_MAX for no QoS) */ - uint16_t bfs_pri; /* packet AC priority */ - struct ath_txq *bfs_txq; /* eventual dest hardware TXQ */ - uint16_t bfs_pktdur; /* packet duration (at current rate?) */ - uint16_t bfs_nframes; /* number of frames in aggregate */ + uint16_t bfs_seqno; /* sequence number of this packet */ uint16_t bfs_ndelim; /* number of delims for padding */ + uint8_t bfs_retries; /* retry count */ + uint8_t bfs_tid; /* packet TID (or TID_MAX for no QoS) */ + uint8_t bfs_nframes; /* number of frames in aggregate */ + uint8_t bfs_pri; /* packet AC priority */ + + struct ath_txq *bfs_txq; /* eventual dest hardware TXQ */ + u_int32_t bfs_aggr:1, /* part of aggregate? */ bfs_aggrburst:1, /* part of aggregate burst? */ bfs_isretried:1, /* retried frame? */ |
