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

sfxge: remove full_packet_size from sfxge_tso_state

Browse Source 
It makes sfxge_tso_state smaller and even makes tso_start_new_packet()
few bytes smaller. Data used to calculate packet size are used nearby,
so it should be no problems with cache etc.

Sponsored by:   Solarflare Communications, Inc.
Approved by:    gnn (mentor), glebius
This commit is contained in:
Andrew Rybchenko 2015-02-16 06:04:26 +00:00
parent 6004805208
commit 3f1e3499ed
1 changed files with 2 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
sys/dev/sfxge/sfxge_tx.c
View File

@ -792,8 +792,6 @@ struct sfxge_tso_state {
ssize_t nh_off; /* Offset of network header */
ssize_t tcph_off; /* Offset of TCP header */
unsigned header_len; /* Number of bytes of header */
int full_packet_size; /* Number of bytes to put in each outgoing
* segment */
};
static inline const struct ip *tso_iph(const struct sfxge_tso_state *tso)
@ -895,7 +893,6 @@ static void tso_start(struct sfxge_tso_state *tso, struct mbuf *mbuf)
}
tso->header_len = tso->tcph_off + 4 * tso_tcph(tso)->th_off;
tso->full_packet_size = tso->header_len + mbuf->m_pkthdr.tso_segsz;
tso->seqnum = ntohl(tso_tcph(tso)->th_seq);
@ -1015,7 +1012,8 @@ static int tso_start_new_packet(struct sfxge_txq *txq,
tso->seqnum += tso->mbuf->m_pkthdr.tso_segsz;
if (tso->out_len > tso->mbuf->m_pkthdr.tso_segsz) {
/* This packet will not finish the TSO burst. */
ip_length = tso->full_packet_size - tso->nh_off;
ip_length = tso->header_len - tso->nh_off +
tso->mbuf->m_pkthdr.tso_segsz;
tsoh_th->th_flags &= ~(TH_FIN | TH_PUSH);
} else {
/* This packet will be the last in the TSO burst. */