iwx: enable seqno offload
Needs ReviewPublic
Actions

Authored by adrian on Thu, Jun 5, 3:51 AM.

Details

Reviewers

thj
bz

Group Reviewers

wireless

Summary

Enable sequence number offload.

This should both enable the sequence number offloading and disable
the net80211 TX lock from being acquired/released/checked.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 64654
Build 61538: arc lint + arc unit

Event Timeline

adrian created this revision.Thu, Jun 5, 3:51 AM

Herald added a subscriber: imp. · View Herald TranscriptThu, Jun 5, 3:51 AM

adrian requested review of this revision.Thu, Jun 5, 3:51 AM

Harbormaster completed remote builds in B64654: Diff 156564.Thu, Jun 5, 3:51 AM

adrian added a parent revision: D50693: rtwn: enable seqno offload.Thu, Jun 5, 4:41 AM

adrian added a reviewer: wireless.

adrian added a project: wireless.

adrian added a reviewer: thj.

bz requested changes to this revision.Thu, Jun 5, 8:03 AM

bz added a subscriber: bz.

bz added inline comments.

sys/dev/iwx/if_iwx.c
5664	This is entirely done in firmware.

This revision now requires changes to proceed.Thu, Jun 5, 8:03 AM

adrian added inline comments.Thu, Jun 5, 2:34 PM

sys/dev/iwx/if_iwx.c
5664	This is entirely done in firmware. aha let me go and try that too then. If it works then I'll delete it and leave the comment there that seqno offload is done at that point.

Address bz's comment - yup, this works with seqno offload and
with it just not populated in the driver encap path. Neat!

Harbormaster completed remote builds in B64694: Diff 156658.Fri, Jun 6, 9:24 PM

adrian marked an inline comment as done.Fri, Jun 6, 9:29 PM

adrian added a child revision: D50747: run: enable seqno offload.Mon, Jun 9, 2:14 AM

As far as I can tell this is nop, but good ground work for future stuff.

Revision Contents
Changeset List

Path

Size

sys/

dev/

iwx/

if_iwx.c

8 lines

Diff 156564

View Options

iwx: enable seqno offloadNeeds ReviewPublicActions