Page MenuHomeFreeBSD

Prepare for network stack as a module.
ClosedPublic

Authored by stevek on Jun 24 2015, 9:20 PM.
Tags
None
Referenced Files
Unknown Object (File)
Feb 28 2024, 4:36 AM
Unknown Object (File)
Feb 8 2024, 2:22 AM
Unknown Object (File)
Jan 12 2024, 8:15 AM
Unknown Object (File)
Jan 8 2024, 10:04 PM
Unknown Object (File)
Jan 8 2024, 10:04 PM
Unknown Object (File)
Jan 8 2024, 10:04 PM
Unknown Object (File)
Jan 8 2024, 10:03 PM
Unknown Object (File)
Jan 8 2024, 9:11 PM

Details

Summary
  • Move cr_canseeinpcb to sys/netinet/in_prot.c in order to separate the INET and INET6-specific code from the rest of the prot code (It is only used by the network stack, so it makes sense for it to live with the other network stack code.)
  • Move cr_canseeinpcb prototype from sys/systm.h to netinet/in_systm.h
  • Rename cr_seeotheruids to cr_canseeotheruids and cr_seeothergids to cr_canseeothergids, make them non-static, and add prototypes (so they can be see/called by in_prot.c functions.)
  • Remove sw_csum variable from ip6_forward in ip6_forward.c, as it is an unused variable.

Obtained from: Juniper Networks, Inc.

Diff Detail

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

Event Timeline

stevek retitled this revision from to Prepare for network stack as a module..
stevek updated this object.
stevek edited the test plan for this revision. (Show Details)
stevek set the repository for this revision to rS FreeBSD src repository - subversion.
stevek changed the visibility from "Public (No Login Required)" to "All Users".Jun 24 2015, 9:22 PM
gnn added a reviewer: bz.
gnn added a reviewer: gnn.
gnn added a subscriber: gnn.

This looks fine but I'm hoping that bz@ (now added) will take a look as well.

This revision is now accepted and ready to land.Nov 12 2015, 2:31 AM
stevek added subscribers: sjg, jtl.
bz requested changes to this revision.Jun 8 2016, 8:19 PM
bz edited edge metadata.

To me this sounds like a convolution of a couple of changes.

Can we get the moves of the functions, then rename them for consistency maybe, and then do the iocgroup change. That'll help to understand things and move forward more easily.

This revision now requires changes to proceed.Jun 8 2016, 8:19 PM

I assume you mean splitting the prot changes from the ioctl group bits?
Would you prefer two (or three) new Phabricator reviews for that or use this one for one of the reviews?

stevek updated this object.
stevek edited the test plan for this revision. (Show Details)
stevek edited edge metadata.
stevek changed the visibility from "All Users" to "Public (No Login Required)".

Trimmed down to first smaller piece.

jtl added a reviewer: jtl.
gnn edited edge metadata.

Accepted. Go for it.

Best,
George

This revision was automatically updated to reflect the committed changes.