net: sctp: Inline the functions from command.c
sctp_init_cmd_seq() and sctp_next_cmd() are only called from one place. The call sequence for sctp_add_cmd_sf() is likely to be longer than the inlined code. With sctp_add_cmd_sf() inlined the compiler can optimise repeated calls. Signed-off-by: David Laight <david.laight@aculab.com> Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
parent
63ae88949b
commit
be1f4f48ce
|
@ -210,19 +210,38 @@ typedef struct {
|
|||
/* Initialize a block of memory as a command sequence.
|
||||
* Return 0 if the initialization fails.
|
||||
*/
|
||||
int sctp_init_cmd_seq(sctp_cmd_seq_t *seq);
|
||||
static inline int sctp_init_cmd_seq(sctp_cmd_seq_t *seq)
|
||||
{
|
||||
memset(seq, 0, sizeof(sctp_cmd_seq_t));
|
||||
return 1; /* We always succeed. */
|
||||
}
|
||||
|
||||
|
||||
/* Add a command to an sctp_cmd_seq_t.
|
||||
*
|
||||
* Use the SCTP_* constructors defined by SCTP_ARG_CONSTRUCTOR() above
|
||||
* to wrap data which goes in the obj argument.
|
||||
*/
|
||||
void sctp_add_cmd_sf(sctp_cmd_seq_t *seq, sctp_verb_t verb, sctp_arg_t obj);
|
||||
static inline void sctp_add_cmd_sf(sctp_cmd_seq_t *seq, sctp_verb_t verb,
|
||||
sctp_arg_t obj)
|
||||
{
|
||||
BUG_ON(seq->next_free_slot >= SCTP_MAX_NUM_COMMANDS);
|
||||
|
||||
seq->cmds[seq->next_free_slot].verb = verb;
|
||||
seq->cmds[seq->next_free_slot++].obj = obj;
|
||||
}
|
||||
|
||||
/* Return the next command structure in an sctp_cmd_seq.
|
||||
* Return NULL at the end of the sequence.
|
||||
*/
|
||||
sctp_cmd_t *sctp_next_cmd(sctp_cmd_seq_t *seq);
|
||||
static inline sctp_cmd_t *sctp_next_cmd(sctp_cmd_seq_t *seq)
|
||||
{
|
||||
sctp_cmd_t *retval = NULL;
|
||||
|
||||
if (seq->next_cmd < seq->next_free_slot)
|
||||
retval = &seq->cmds[seq->next_cmd++];
|
||||
|
||||
return retval;
|
||||
}
|
||||
|
||||
#endif /* __net_sctp_command_h__ */
|
||||
|
||||
|
|
|
@ -8,7 +8,7 @@ obj-$(CONFIG_NET_SCTPPROBE) += sctp_probe.o
|
|||
sctp-y := sm_statetable.o sm_statefuns.o sm_sideeffect.o \
|
||||
protocol.o endpointola.o associola.o \
|
||||
transport.o chunk.o sm_make_chunk.o ulpevent.o \
|
||||
inqueue.o outqueue.o ulpqueue.o command.o \
|
||||
inqueue.o outqueue.o ulpqueue.o \
|
||||
tsnmap.o bind_addr.o socket.o primitive.o \
|
||||
output.o input.o debug.o ssnmap.o auth.o
|
||||
|
||||
|
|
|
@ -1,68 +0,0 @@
|
|||
/* SCTP kernel implementation Copyright (C) 1999-2001
|
||||
* Cisco, Motorola, and IBM
|
||||
* Copyright 2001 La Monte H.P. Yarroll
|
||||
*
|
||||
* This file is part of the SCTP kernel implementation
|
||||
*
|
||||
* These functions manipulate sctp command sequences.
|
||||
*
|
||||
* This SCTP implementation is free software;
|
||||
* you can redistribute it and/or modify it under the terms of
|
||||
* the GNU General Public License as published by
|
||||
* the Free Software Foundation; either version 2, or (at your option)
|
||||
* any later version.
|
||||
*
|
||||
* This SCTP implementation is distributed in the hope that it
|
||||
* will be useful, but WITHOUT ANY WARRANTY; without even the implied
|
||||
* ************************
|
||||
* warranty of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
|
||||
* See the GNU General Public License for more details.
|
||||
*
|
||||
* You should have received a copy of the GNU General Public License
|
||||
* along with GNU CC; see the file COPYING. If not, see
|
||||
* <http://www.gnu.org/licenses/>.
|
||||
*
|
||||
* Please send any bug reports or fixes you make to the
|
||||
* email address(es):
|
||||
* lksctp developers <linux-sctp@vger.kernel.org>
|
||||
*
|
||||
* Written or modified by:
|
||||
* La Monte H.P. Yarroll <piggy@acm.org>
|
||||
* Karl Knutson <karl@athena.chicago.il.us>
|
||||
*/
|
||||
|
||||
#include <linux/types.h>
|
||||
#include <net/sctp/sctp.h>
|
||||
#include <net/sctp/sm.h>
|
||||
|
||||
/* Initialize a block of memory as a command sequence. */
|
||||
int sctp_init_cmd_seq(sctp_cmd_seq_t *seq)
|
||||
{
|
||||
memset(seq, 0, sizeof(sctp_cmd_seq_t));
|
||||
return 1; /* We always succeed. */
|
||||
}
|
||||
|
||||
/* Add a command to a sctp_cmd_seq_t.
|
||||
* Return 0 if the command sequence is full.
|
||||
*/
|
||||
void sctp_add_cmd_sf(sctp_cmd_seq_t *seq, sctp_verb_t verb, sctp_arg_t obj)
|
||||
{
|
||||
BUG_ON(seq->next_free_slot >= SCTP_MAX_NUM_COMMANDS);
|
||||
|
||||
seq->cmds[seq->next_free_slot].verb = verb;
|
||||
seq->cmds[seq->next_free_slot++].obj = obj;
|
||||
}
|
||||
|
||||
/* Return the next command structure in a sctp_cmd_seq.
|
||||
* Returns NULL at the end of the sequence.
|
||||
*/
|
||||
sctp_cmd_t *sctp_next_cmd(sctp_cmd_seq_t *seq)
|
||||
{
|
||||
sctp_cmd_t *retval = NULL;
|
||||
|
||||
if (seq->next_cmd < seq->next_free_slot)
|
||||
retval = &seq->cmds[seq->next_cmd++];
|
||||
|
||||
return retval;
|
||||
}
|
||||
|
Loading…
Reference in New Issue