Differential D44491

inpcb: fully retire inp_ppcb pointer
ClosedPublic
Actions

Authored by glebius on Mar 24 2024, 5:35 PM.

Tags

None

Referenced Files

	F151283993: D44491.id136342.diff
	Tue, Apr 7, 7:58 AM

	Unknown Object (File)
	Sun, Apr 5, 5:09 PM

	Unknown Object (File)
	Fri, Apr 3, 9:50 PM

	Unknown Object (File)
	Mon, Mar 23, 6:20 AM

	Unknown Object (File)
	Fri, Mar 20, 2:15 PM

	Unknown Object (File)
	Fri, Mar 20, 2:08 PM

	Unknown Object (File)
	Mar 7 2026, 9:48 PM

	Unknown Object (File)
	Mar 1 2026, 11:18 PM

Subscribers

Details

Reviewers

Group Reviewers

network
transport

Commits

rG1a8d176432e7: inpcb: fully retire inp_ppcb pointer

Summary

Before a protocol specific control block started to embed inpcb in self
(see 0aa120d52f3c, e68b3792440c, 483fe96511ec) this pointer used to point
at it.

Retain kf_sock_inpcb field in the struct kinfo_file in <sys/user.h>. The
exp-run detected a minimal use of the field in ports:

sysutils/lsof - patched
net-mgmt/etdata - patch submitted upstream
emulators/qemu-user-static - upstream master branch seems not using the field anymore

We can keep the field around for some time, but eventually it may be reused
for something else.

PR: 277659 (exp-run)

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

glebius created this revision.Mar 24 2024, 5:35 PM

Herald added 1 blocking reviewer(s): transport. · View Herald TranscriptMar 24 2024, 5:35 PM

Herald added subscribers: olce, melifaro, imp. · View Herald Transcript

glebius requested review of this revision.Mar 24 2024, 5:35 PM

Harbormaster completed remote builds in B56766: Diff 136169.Mar 24 2024, 5:35 PM

tuexen accepted this revision as: transport, tuexen.Mar 24 2024, 6:17 PM

This revision is now accepted and ready to land.Mar 24 2024, 6:17 PM

Closed by commit rG1a8d176432e7: inpcb: fully retire inp_ppcb pointer (authored by glebius). · Explain WhyMar 29 2024, 7:20 PM

This revision was automatically updated to reflect the committed changes.

glebius added a commit: rG1a8d176432e7: inpcb: fully retire inp_ppcb pointer.

Revision Contents
Changeset List

Path

Size

lib/

libprocstat/

1 line

29 lines

libprocstat_compat.c

2 lines

sys/

kern/

8 lines

netinet/

9 lines

9 lines

7 lines

sys/

2 lines

usr.bin/

fstat/

15 lines

systat/

28 lines

Diff 136342

lib/libprocstat/libprocstat.h

Loading...

lib/libprocstat/libprocstat.c

Loading...

lib/libprocstat/libprocstat_compat.c

Loading...

sys/kern/sys_socket.c

Loading...

sys/netinet/in_pcb.h

Loading...

sys/netinet/in_pcb.c

Loading...

sys/netinet/tcp_subr.c

Loading...

sys/sys/user.h

Loading...

usr.bin/fstat/fstat.c

Loading...

usr.bin/systat/netstat.c

Loading...