Page MenuHomeFreeBSD
Differential D26334

cxgbe: Apply more checks for INET and INET6
ClosedPublic
Actions

Authored by np on Sep 5 2020, 12:18 AM.
Tags
None
Referenced Files
F108746864: D26334.id89646.diff
Mon, Jan 27, 4:52 PM
F108661959: D26334.diff
Mon, Jan 27, 3:53 AM
Unknown Object (File)
Mon, Jan 13, 4:29 PM
Unknown Object (File)
Mon, Jan 13, 9:32 AM
Unknown Object (File)
Dec 25 2024, 11:42 PM
Unknown Object (File)
Dec 20 2024, 10:11 PM
Unknown Object (File)
Nov 27 2024, 1:45 PM
Unknown Object (File)
Nov 12 2024, 8:11 PM
View All 68 Files
Subscribers
ae
imp

Details

Reviewers
bz
Commits
rGfed21efcfce4: cxgbe(4): Fix build warnings with NOINET kernels.
rGffbb373c5a95: cxgbe(4): Fix build warnings with NOINET kernels.
Summary

Currently a NOINET LINT kernels complains about 5 unsued static
functions. Properly hide them behind appropriate #if defined checks
for INET or INET6 to make the kernel compile without warnings.

Reviwed by: ...

Diff Detail

Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 33408
Build 30706: arc lint + arc unit

Event Timeline

bz created this revision.Sep 5 2020, 12:18 AM
Herald added a subscriber: ae. · View Herald TranscriptSep 5 2020, 12:18 AM
bz requested review of this revision.Sep 5 2020, 12:18 AM
Harbormaster completed remote builds in B33408: Diff 76681.Sep 5 2020, 12:18 AM
np accepted this revision.Oct 19 2020, 5:27 PM
This revision is now accepted and ready to land.Oct 19 2020, 5:27 PM
bz added a comment.Oct 19 2020, 6:30 PM

It seems by now there is also needs_inner_tcp_csum() ?
Can you possibly incorporate this in your next driver update and do it yourself @np ?

np commandeered this revision.Oct 19 2020, 6:39 PM
np edited reviewers, added: bz; removed: np.

okay, I'll take care of this.

This revision now requires review to proceed.Oct 19 2020, 6:39 PM
bz added a comment.Oct 19 2020, 7:54 PM
In D26334#598980, @np wrote:

okay, I'll take care of this.

Great, thanks! :)

This revision was not accepted when it landed; it landed in state Needs Review.May 22 2021, 3:50 AM
Closed by commit rGffbb373c5a95: cxgbe(4): Fix build warnings with NOINET kernels. (authored by np). · Explain Why
This revision was automatically updated to reflect the committed changes.
np added a commit: rGffbb373c5a95: cxgbe(4): Fix build warnings with NOINET kernels..
np added a commit: rGfed21efcfce4: cxgbe(4): Fix build warnings with NOINET kernels..Oct 20 2021, 2:11 PM
Herald added a subscriber: imp. · View Herald TranscriptOct 20 2021, 2:11 PM

Revision Contents
Changeset List

PathSize
sys/
dev/
cxgbe/
12 lines

Diff 76681

View Options

sys/dev/cxgbe/t4_sge.c

Loading...