rx: Remove unneeded rxi_ReceiveDataPacket params 02/13602/9
authorAndrew Deason <adeason@sinenomine.net>
Sat, 8 Dec 2018 23:16:43 +0000 (17:16 -0600)
committerBenjamin Kaduk <kaduk@mit.edu>
Wed, 30 Dec 2020 05:58:06 +0000 (00:58 -0500)
rxi_SplitJumboPacket doesn't use its 'host', 'port', or 'first'
arguments, and rxi_ReceiveDataPacket only uses its 'host' and 'port'
arguments to pass to rxi_SplitJumboPacket.

Remove these unused parameters from both functions. While we're
changing rxi_SplitJumboPacket anyway, move the declaration for
rxi_SplitJumboPacket to rx_internal.h, so it's no longer in a public
header.

This commit should have no visible impact; it is just code
reorganization.

Change-Id: I16a7f613957d8cd2d415f65fa083e11d8a13edc8
Reviewed-on: https://gerrit.openafs.org/13602
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Cheyenne Wills <cwills@sinenomine.net>
Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>

src/rx/rx.c
src/rx/rx_internal.h
src/rx/rx_packet.c
src/rx/rx_prototypes.h

index 2296014..417e919 100644 (file)
@@ -131,8 +131,7 @@ static struct rx_connection
 static struct rx_packet
        *rxi_ReceiveDataPacket(struct rx_call *call, struct rx_packet *np,
                               int istack, osi_socket socket,
-                              afs_uint32 host, u_short port, int *tnop,
-                              struct rx_call **newcallp);
+                              int *tnop, struct rx_call **newcallp);
 static struct rx_packet
        *rxi_ReceiveAckPacket(struct rx_call *call, struct rx_packet *np,
                              int istack, int *a_invalid);
@@ -3613,8 +3612,7 @@ rxi_ReceivePacket(struct rx_packet *np, osi_socket socket,
        if (type == RX_CLIENT_CONNECTION && !opr_queue_IsEmpty(&call->tq))
            rxi_AckAllInTransmitQueue(call);
 
-       np = rxi_ReceiveDataPacket(call, np, 1, socket, host, port, tnop,
-                                  newcallp);
+       np = rxi_ReceiveDataPacket(call, np, 1, socket, tnop, newcallp);
        break;
     case RX_PACKET_TYPE_ACK:
        /* Respond immediately to ack packets requesting acknowledgement
@@ -3896,8 +3894,7 @@ TryAttach(struct rx_call *acall, osi_socket socket,
 static struct rx_packet *
 rxi_ReceiveDataPacket(struct rx_call *call,
                      struct rx_packet *np, int istack,
-                     osi_socket socket, afs_uint32 host, u_short port,
-                     int *tnop, struct rx_call **newcallp)
+                     osi_socket socket, int *tnop, struct rx_call **newcallp)
 {
     int ackNeeded = 0;         /* 0 means no, otherwise ack_reason */
     int newPackets = 0;
@@ -3956,7 +3953,7 @@ rxi_ReceiveDataPacket(struct rx_call *call,
        /* The RX_JUMBO_PACKET is set in all but the last packet in each
         * AFS 3.5 jumbogram. */
        if (flags & RX_JUMBO_PACKET) {
-           tnp = rxi_SplitJumboPacket(np, host, port, isFirst);
+           tnp = rxi_SplitJumboPacket(np);
        } else {
            tnp = NULL;
        }
index b420a23..afef972 100644 (file)
@@ -75,3 +75,4 @@ extern int rxi_SendIovecs(struct rx_connection *conn, struct iovec *iov,
                          int iovcnt, size_t length, int istack);
 extern void rxi_SendRaw(struct rx_call *call, struct rx_connection *conn,
                        int type, char *data, int bytes, int istack);
+extern struct rx_packet *rxi_SplitJumboPacket(struct rx_packet *p);
index 6cf997b..a8ea7bc 100644 (file)
@@ -1544,8 +1544,7 @@ rxi_ReadPacket(osi_socket socket, struct rx_packet *p, afs_uint32 * host,
  * last two pad bytes. */
 
 struct rx_packet *
-rxi_SplitJumboPacket(struct rx_packet *p, afs_uint32 host, short port,
-                    int first)
+rxi_SplitJumboPacket(struct rx_packet *p)
 {
     struct rx_packet *np;
     struct rx_jumboHeader *jp;
index 4e5c0d0..e1c0a04 100644 (file)
@@ -402,9 +402,6 @@ extern struct rx_packet *rxi_AllocSendPacket(struct rx_call *call,
                                             int want);
 extern int rxi_ReadPacket(osi_socket socket, struct rx_packet *p,
                          afs_uint32 * host, u_short * port);
-extern struct rx_packet *rxi_SplitJumboPacket(struct rx_packet *p,
-                                             afs_uint32 host, short port,
-                                             int first);
 #ifndef KERNEL
 extern int osi_NetSend(osi_socket socket, void *addr, struct iovec *dvec,
                       int nvecs, int length, int istack);