Page MenuHomeFreeBSD

[iwm] Get rid of SYNC_RESP_STRUCT and SYNC_RESP_PTR macros.
ClosedPublic

Authored by ivadasz on Sep 18 2016, 11:59 AM.
Tags
None
Referenced Files
F91700276: D7939.diff
Wed, Aug 21, 4:38 AM
Unknown Object (File)
Sun, Aug 18, 7:03 AM
Unknown Object (File)
Mon, Jul 29, 11:37 AM
Unknown Object (File)
Jul 3 2024, 12:10 PM
Unknown Object (File)
Jul 3 2024, 2:36 AM
Unknown Object (File)
Jun 29 2024, 1:36 PM
Unknown Object (File)
Jun 29 2024, 1:26 PM
Unknown Object (File)
Jun 4 2024, 4:54 PM
Subscribers

Details

Summary

SYNC_RESP_STRUCT and SYNC_RESP_PTR originate from the OpenBSD version of
iwm, and they weren't serving any real purpose in the FreeBSD port.

We just do a single bus_dmamap_sync for syncing the complete received frame,
instead of explicitly bus_dmamap_sync-ing subranges of the frame like in
the OpenBSD iwm code.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

ivadasz retitled this revision from to [iwm] Get rid of SYNC_RESP_STRUCT and SYNC_RESP_PTR macros..
ivadasz updated this object.
ivadasz added reviewers: adrian, cognet.

hm, so we're already correctly sync'ing the payload as well as the descriptor?

In (line numbers from right side of the diff) line 5034 the descriptors are synced, and in line 5049 the payload is synced. All the SYNC_RESP_STRUCT macro usages were just repeating the syncing of the payload.

adrian edited edge metadata.
This revision is now accepted and ready to land.Nov 1 2016, 10:24 PM
This revision was automatically updated to reflect the committed changes.