sctp: remove the typedef sctp_xmit_t
authorXin Long <lucien.xin@gmail.com>
Sat, 5 Aug 2017 11:59:57 +0000 (19:59 +0800)
committerDavid S. Miller <davem@davemloft.net>
Mon, 7 Aug 2017 04:33:42 +0000 (21:33 -0700)
This patch is to remove the typedef sctp_xmit_t, and
replace with enum sctp_xmit in the places where it's
using this typedef.

Signed-off-by: Xin Long <lucien.xin@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
include/net/sctp/constants.h
include/net/sctp/structs.h
net/sctp/output.c
net/sctp/outqueue.c

index 049868ea7ae9f828fe6d1aeaa1e3e8676481a5d9..311b2e3773e857574d14d46c038e82552d14f2f2 100644 (file)
@@ -312,12 +312,12 @@ enum { SCTP_MAX_GABS = 16 };
 /* These return values describe the success or failure of a number of
  * routines which form the lower interface to SCTP_outqueue.
  */
-typedef enum {
+enum sctp_xmit {
        SCTP_XMIT_OK,
        SCTP_XMIT_PMTU_FULL,
        SCTP_XMIT_RWND_FULL,
        SCTP_XMIT_DELAY,
-} sctp_xmit_t;
+};
 
 /* These are the commands for manipulating transports.  */
 enum sctp_transport_cmd {
index d098d1c4ed74d92e533d8a97afafd818b5eb4a55..6fab67e7f1e5f5887c16da55179c1c9b149a616f 100644 (file)
@@ -697,10 +697,11 @@ struct sctp_packet {
 void sctp_packet_init(struct sctp_packet *, struct sctp_transport *,
                      __u16 sport, __u16 dport);
 void sctp_packet_config(struct sctp_packet *, __u32 vtag, int);
-sctp_xmit_t sctp_packet_transmit_chunk(struct sctp_packet *,
-                                      struct sctp_chunk *, int, gfp_t);
-sctp_xmit_t sctp_packet_append_chunk(struct sctp_packet *,
-                                     struct sctp_chunk *);
+enum sctp_xmit sctp_packet_transmit_chunk(struct sctp_packet *packet,
+                                         struct sctp_chunk *chunk,
+                                         int one_packet, gfp_t gfp);
+enum sctp_xmit sctp_packet_append_chunk(struct sctp_packet *packet,
+                                       struct sctp_chunk *chunk);
 int sctp_packet_transmit(struct sctp_packet *, gfp_t);
 void sctp_packet_free(struct sctp_packet *);
 
index 9d8504985744f8153f985695e7123f48994d3494..4a865cd06d76cd5b2aa417de618da3203f7b53e4 100644 (file)
 #include <net/sctp/checksum.h>
 
 /* Forward declarations for private helpers. */
-static sctp_xmit_t __sctp_packet_append_chunk(struct sctp_packet *packet,
-                                             struct sctp_chunk *chunk);
-static sctp_xmit_t sctp_packet_can_append_data(struct sctp_packet *packet,
-                                          struct sctp_chunk *chunk);
+static enum sctp_xmit __sctp_packet_append_chunk(struct sctp_packet *packet,
+                                                struct sctp_chunk *chunk);
+static enum sctp_xmit sctp_packet_can_append_data(struct sctp_packet *packet,
+                                                 struct sctp_chunk *chunk);
 static void sctp_packet_append_data(struct sctp_packet *packet,
-                                          struct sctp_chunk *chunk);
-static sctp_xmit_t sctp_packet_will_fit(struct sctp_packet *packet,
-                                       struct sctp_chunk *chunk,
-                                       u16 chunk_len);
+                                   struct sctp_chunk *chunk);
+static enum sctp_xmit sctp_packet_will_fit(struct sctp_packet *packet,
+                                          struct sctp_chunk *chunk,
+                                          u16 chunk_len);
 
 static void sctp_packet_reset(struct sctp_packet *packet)
 {
@@ -181,11 +181,11 @@ void sctp_packet_free(struct sctp_packet *packet)
  * as it can fit in the packet, but any more data that does not fit in this
  * packet can be sent only after receiving the COOKIE_ACK.
  */
-sctp_xmit_t sctp_packet_transmit_chunk(struct sctp_packet *packet,
-                                      struct sctp_chunk *chunk,
-                                      int one_packet, gfp_t gfp)
+enum sctp_xmit sctp_packet_transmit_chunk(struct sctp_packet *packet,
+                                         struct sctp_chunk *chunk,
+                                         int one_packet, gfp_t gfp)
 {
-       sctp_xmit_t retval;
+       enum sctp_xmit retval;
 
        pr_debug("%s: packet:%p size:%zu chunk:%p size:%d\n", __func__,
                 packet, packet->size, chunk, chunk->skb ? chunk->skb->len : -1);
@@ -218,12 +218,12 @@ sctp_xmit_t sctp_packet_transmit_chunk(struct sctp_packet *packet,
 }
 
 /* Try to bundle an auth chunk into the packet. */
-static sctp_xmit_t sctp_packet_bundle_auth(struct sctp_packet *pkt,
-                                          struct sctp_chunk *chunk)
+static enum sctp_xmit sctp_packet_bundle_auth(struct sctp_packet *pkt,
+                                             struct sctp_chunk *chunk)
 {
        struct sctp_association *asoc = pkt->transport->asoc;
+       enum sctp_xmit retval = SCTP_XMIT_OK;
        struct sctp_chunk *auth;
-       sctp_xmit_t retval = SCTP_XMIT_OK;
 
        /* if we don't have an association, we can't do authentication */
        if (!asoc)
@@ -254,10 +254,10 @@ static sctp_xmit_t sctp_packet_bundle_auth(struct sctp_packet *pkt,
 }
 
 /* Try to bundle a SACK with the packet. */
-static sctp_xmit_t sctp_packet_bundle_sack(struct sctp_packet *pkt,
-                                          struct sctp_chunk *chunk)
+static enum sctp_xmit sctp_packet_bundle_sack(struct sctp_packet *pkt,
+                                             struct sctp_chunk *chunk)
 {
-       sctp_xmit_t retval = SCTP_XMIT_OK;
+       enum sctp_xmit retval = SCTP_XMIT_OK;
 
        /* If sending DATA and haven't aleady bundled a SACK, try to
         * bundle one in to the packet.
@@ -299,11 +299,11 @@ out:
 /* Append a chunk to the offered packet reporting back any inability to do
  * so.
  */
-static sctp_xmit_t __sctp_packet_append_chunk(struct sctp_packet *packet,
-                                             struct sctp_chunk *chunk)
+static enum sctp_xmit __sctp_packet_append_chunk(struct sctp_packet *packet,
+                                                struct sctp_chunk *chunk)
 {
-       sctp_xmit_t retval = SCTP_XMIT_OK;
        __u16 chunk_len = SCTP_PAD4(ntohs(chunk->chunk_hdr->length));
+       enum sctp_xmit retval = SCTP_XMIT_OK;
 
        /* Check to see if this chunk will fit into the packet */
        retval = sctp_packet_will_fit(packet, chunk, chunk_len);
@@ -353,10 +353,10 @@ finish:
 /* Append a chunk to the offered packet reporting back any inability to do
  * so.
  */
-sctp_xmit_t sctp_packet_append_chunk(struct sctp_packet *packet,
-                                    struct sctp_chunk *chunk)
+enum sctp_xmit sctp_packet_append_chunk(struct sctp_packet *packet,
+                                       struct sctp_chunk *chunk)
 {
-       sctp_xmit_t retval = SCTP_XMIT_OK;
+       enum sctp_xmit retval = SCTP_XMIT_OK;
 
        pr_debug("%s: packet:%p chunk:%p\n", __func__, packet, chunk);
 
@@ -653,8 +653,8 @@ out:
  ********************************************************************/
 
 /* This private function check to see if a chunk can be added */
-static sctp_xmit_t sctp_packet_can_append_data(struct sctp_packet *packet,
-                                          struct sctp_chunk *chunk)
+static enum sctp_xmit sctp_packet_can_append_data(struct sctp_packet *packet,
+                                                 struct sctp_chunk *chunk)
 {
        size_t datasize, rwnd, inflight, flight_size;
        struct sctp_transport *transport = packet->transport;
@@ -762,12 +762,12 @@ static void sctp_packet_append_data(struct sctp_packet *packet,
        sctp_chunk_assign_ssn(chunk);
 }
 
-static sctp_xmit_t sctp_packet_will_fit(struct sctp_packet *packet,
-                                       struct sctp_chunk *chunk,
-                                       u16 chunk_len)
+static enum sctp_xmit sctp_packet_will_fit(struct sctp_packet *packet,
+                                          struct sctp_chunk *chunk,
+                                          u16 chunk_len)
 {
+       enum sctp_xmit retval = SCTP_XMIT_OK;
        size_t psize, pmtu, maxsize;
-       sctp_xmit_t retval = SCTP_XMIT_OK;
 
        psize = packet->size;
        if (packet->transport->asoc)
index 08ee0ed9a0c69d46f34e959d54a141c0521fab2e..2966ff400755fe93e3658e09d3bb44b9d7d19d2e 100644 (file)
@@ -594,14 +594,14 @@ void sctp_retransmit(struct sctp_outq *q, struct sctp_transport *transport,
 static int sctp_outq_flush_rtx(struct sctp_outq *q, struct sctp_packet *pkt,
                               int rtx_timeout, int *start_timer)
 {
-       struct list_head *lqueue;
        struct sctp_transport *transport = pkt->transport;
-       sctp_xmit_t status;
        struct sctp_chunk *chunk, *chunk1;
-       int fast_rtx;
+       struct list_head *lqueue;
+       enum sctp_xmit status;
        int error = 0;
        int timer = 0;
        int done = 0;
+       int fast_rtx;
 
        lqueue = &q->retransmit;
        fast_rtx = q->fast_rtx;
@@ -781,7 +781,7 @@ static void sctp_outq_flush(struct sctp_outq *q, int rtx_timeout, gfp_t gfp)
        struct sctp_transport *transport = NULL;
        struct sctp_transport *new_transport;
        struct sctp_chunk *chunk, *tmp;
-       sctp_xmit_t status;
+       enum sctp_xmit status;
        int error = 0;
        int start_timer = 0;
        int one_packet = 0;