git: 027fda80fe8b - main - inpcb: remove unused KPIs to manipulate inpcbs

From: Gleb Smirnoff <glebius_at_FreeBSD.org>
Date: Mon, 18 Mar 2024 15:52:59 UTC
The branch main has been updated by glebius:

URL: https://cgit.FreeBSD.org/src/commit/?id=027fda80fe8b8f7f76c9e1799753601674954901

commit 027fda80fe8b8f7f76c9e1799753601674954901
Author:     Gleb Smirnoff <glebius@FreeBSD.org>
AuthorDate: 2024-03-18 15:49:39 +0000
Commit:     Gleb Smirnoff <glebius@FreeBSD.org>
CommitDate: 2024-03-18 15:49:39 +0000

    inpcb: remove unused KPIs to manipulate inpcbs
    
    These KPIs were added in 9d29c635daa69 and through 15 years had zero use.
    They slightly remind what IfAPI does for struct ifnet.  But IfAPI does
    that for the sake of large collection of NIC drivers not being aware of
    struct ifnet.  For the inpcb it is unclear what could be a large
    collection of externally written kernel modules that need extensively use
    inpcb and not be aware of its internals at the same time. This isolation
    of a structure knowledge requires a lot of work, and just throwing in a
    few KPIs isn't helpful.
    
    Reviewed by:            kib, bz, markj
    Differential Revision:  https://reviews.freebsd.org/D44310
---
 sys/netinet/in_pcb.c  | 22 ----------------------
 sys/netinet/in_pcb.h  |  4 ----
 sys/netinet/toecore.h |  1 +
 3 files changed, 1 insertion(+), 26 deletions(-)

diff --git a/sys/netinet/in_pcb.c b/sys/netinet/in_pcb.c
index 981654577deb..16a3707e7c89 100644
--- a/sys/netinet/in_pcb.c
+++ b/sys/netinet/in_pcb.c
@@ -2835,28 +2835,6 @@ inp_inpcbtosocket(struct inpcb *inp)
 	return (inp->inp_socket);
 }
 
-struct tcpcb *
-inp_inpcbtotcpcb(struct inpcb *inp)
-{
-
-	INP_WLOCK_ASSERT(inp);
-	return ((struct tcpcb *)inp->inp_ppcb);
-}
-
-int
-inp_ip_tos_get(const struct inpcb *inp)
-{
-
-	return (inp->inp_ip_tos);
-}
-
-void
-inp_ip_tos_set(struct inpcb *inp, int val)
-{
-
-	inp->inp_ip_tos = val;
-}
-
 void
 inp_4tuple_get(struct inpcb *inp, uint32_t *laddr, uint16_t *lp,
     uint32_t *faddr, uint16_t *fp)
diff --git a/sys/netinet/in_pcb.h b/sys/netinet/in_pcb.h
index 3d633741dc27..7e9de02b0c7c 100644
--- a/sys/netinet/in_pcb.h
+++ b/sys/netinet/in_pcb.h
@@ -479,12 +479,8 @@ void inp_unlock_assert(struct inpcb *);
 
 void	inp_apply_all(struct inpcbinfo *, void (*func)(struct inpcb *, void *),
 	    void *arg);
-int 	inp_ip_tos_get(const struct inpcb *inp);
-void 	inp_ip_tos_set(struct inpcb *inp, int val);
 struct socket *
 	inp_inpcbtosocket(struct inpcb *inp);
-struct tcpcb *
-	inp_inpcbtotcpcb(struct inpcb *inp);
 void 	inp_4tuple_get(struct inpcb *inp, uint32_t *laddr, uint16_t *lp,
 		uint32_t *faddr, uint16_t *fp);
 
diff --git a/sys/netinet/toecore.h b/sys/netinet/toecore.h
index a8e5afd6b50a..612c2fe1caf5 100644
--- a/sys/netinet/toecore.h
+++ b/sys/netinet/toecore.h
@@ -36,6 +36,7 @@
 #include <netinet/tcp.h>
 #include <sys/_eventhandler.h>
 
+struct tcpcb;
 struct tcpopt;
 struct tcphdr;
 struct in_conninfo;