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
F97554863: D26334.diff
Mon, Sep 30, 1:23 AM
Unknown Object (File)
Fri, Sep 27, 3:51 AM
Unknown Object (File)
Fri, Sep 27, 3:50 AM
Unknown Object (File)
Fri, Sep 27, 3:46 AM
Unknown Object (File)
Fri, Sep 27, 3:20 AM
Unknown Object (File)
Thu, Sep 26, 10:16 PM
Unknown Object (File)
Thu, Sep 26, 2:03 AM
Unknown Object (File)
Sat, Sep 21, 2:28 AM
View All 53 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

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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/
14 lines

Diff 89646

View Options

sys/dev/cxgbe/t4_sge.c

Loading...