Commit c40616c5 authored by Gerrit Renker's avatar Gerrit Renker Committed by David S. Miller

[TFRC]: Provide central source file and debug facility

This patch changes the tfrc_lib module in the following manner:

 (1) a dedicated tfrc source file to call the packet history &
     loss interval init/exit functions.
 (2) a dedicated tfrc_pr_debug macro with toggle switch `tfrc_debug'.

Commiter note: renamed tfrc_module.c to tfrc.c, and made CONFIG_IP_DCCP_CCID3
select IP_DCCP_TFRC_LIB.
Signed-off-by: default avatarGerrit Renker <gerrit@erg.abdn.ac.uk>
Signed-off-by: default avatarIan McDonald <ian.mcdonald@jandi.co.nz>
Signed-off-by: default avatarArnaldo Carvalho de Melo <acme@redhat.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent f8b33fdf
...@@ -38,6 +38,7 @@ config IP_DCCP_CCID2_DEBUG ...@@ -38,6 +38,7 @@ config IP_DCCP_CCID2_DEBUG
config IP_DCCP_CCID3 config IP_DCCP_CCID3
tristate "CCID3 (TCP-Friendly) (EXPERIMENTAL)" tristate "CCID3 (TCP-Friendly) (EXPERIMENTAL)"
def_tristate IP_DCCP def_tristate IP_DCCP
select IP_DCCP_TFRC_LIB
---help--- ---help---
CCID 3 denotes TCP-Friendly Rate Control (TFRC), an equation-based CCID 3 denotes TCP-Friendly Rate Control (TFRC), an equation-based
rate-controlled congestion control mechanism. TFRC is designed to rate-controlled congestion control mechanism. TFRC is designed to
...@@ -63,10 +64,6 @@ config IP_DCCP_CCID3 ...@@ -63,10 +64,6 @@ config IP_DCCP_CCID3
If in doubt, say M. If in doubt, say M.
config IP_DCCP_TFRC_LIB
depends on IP_DCCP_CCID3
def_tristate IP_DCCP_CCID3
config IP_DCCP_CCID3_DEBUG config IP_DCCP_CCID3_DEBUG
bool "CCID3 debugging messages" bool "CCID3 debugging messages"
depends on IP_DCCP_CCID3 depends on IP_DCCP_CCID3
...@@ -110,5 +107,13 @@ config IP_DCCP_CCID3_RTO ...@@ -110,5 +107,13 @@ config IP_DCCP_CCID3_RTO
is serious network congestion: experimenting with larger values should is serious network congestion: experimenting with larger values should
therefore not be performed on WANs. therefore not be performed on WANs.
config IP_DCCP_TFRC_LIB
tristate
default n
config IP_DCCP_TFRC_DEBUG
bool
depends on IP_DCCP_TFRC_LIB
default y if IP_DCCP_CCID3_DEBUG
endmenu endmenu
obj-$(CONFIG_IP_DCCP_TFRC_LIB) += dccp_tfrc_lib.o obj-$(CONFIG_IP_DCCP_TFRC_LIB) += dccp_tfrc_lib.o
dccp_tfrc_lib-y := loss_interval.o packet_history.o tfrc_equation.o dccp_tfrc_lib-y := tfrc.o tfrc_equation.o packet_history.o loss_interval.o
...@@ -35,7 +35,6 @@ ...@@ -35,7 +35,6 @@
* Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA. * Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
*/ */
#include <linux/module.h>
#include <linux/string.h> #include <linux/string.h>
#include "packet_history.h" #include "packet_history.h"
...@@ -277,39 +276,19 @@ void dccp_rx_hist_purge(struct dccp_rx_hist *hist, struct list_head *list) ...@@ -277,39 +276,19 @@ void dccp_rx_hist_purge(struct dccp_rx_hist *hist, struct list_head *list)
EXPORT_SYMBOL_GPL(dccp_rx_hist_purge); EXPORT_SYMBOL_GPL(dccp_rx_hist_purge);
extern int __init dccp_li_init(void); __init int packet_history_init(void)
extern void dccp_li_exit(void);
static __init int packet_history_init(void)
{ {
if (dccp_li_init() != 0)
goto out;
tfrc_tx_hist = kmem_cache_create("tfrc_tx_hist", tfrc_tx_hist = kmem_cache_create("tfrc_tx_hist",
sizeof(struct tfrc_tx_hist_entry), 0, sizeof(struct tfrc_tx_hist_entry), 0,
SLAB_HWCACHE_ALIGN, NULL); SLAB_HWCACHE_ALIGN, NULL);
if (tfrc_tx_hist == NULL)
goto out_li_exit;
return 0; return tfrc_tx_hist == NULL ? -ENOBUFS : 0;
out_li_exit:
dccp_li_exit();
out:
return -ENOBUFS;
} }
module_init(packet_history_init);
static __exit void packet_history_exit(void) void packet_history_exit(void)
{ {
if (tfrc_tx_hist != NULL) { if (tfrc_tx_hist != NULL) {
kmem_cache_destroy(tfrc_tx_hist); kmem_cache_destroy(tfrc_tx_hist);
tfrc_tx_hist = NULL; tfrc_tx_hist = NULL;
} }
dccp_li_exit();
} }
module_exit(packet_history_exit);
MODULE_AUTHOR("Ian McDonald <ian.mcdonald@jandi.co.nz>, "
"Arnaldo Carvalho de Melo <acme@ghostprotocols.net>");
MODULE_DESCRIPTION("DCCP TFRC library");
MODULE_LICENSE("GPL");
...@@ -39,8 +39,7 @@ ...@@ -39,8 +39,7 @@
#include <linux/ktime.h> #include <linux/ktime.h>
#include <linux/list.h> #include <linux/list.h>
#include <linux/slab.h> #include <linux/slab.h>
#include "tfrc.h"
#include "../../dccp.h"
/* Number of later packets received before one is considered lost */ /* Number of later packets received before one is considered lost */
#define TFRC_RECV_NUM_LATE_LOSS 3 #define TFRC_RECV_NUM_LATE_LOSS 3
......
/*
* TFRC: main module holding the pieces of the TFRC library together
*
* Copyright (c) 2007 The University of Aberdeen, Scotland, UK
* Copyright (c) 2007 Arnaldo Carvalho de Melo <acme@redhat.com>
*/
#include <linux/module.h>
#include <linux/moduleparam.h>
#include "tfrc.h"
#ifdef CONFIG_IP_DCCP_TFRC_DEBUG
int tfrc_debug;
module_param(tfrc_debug, bool, 0444);
MODULE_PARM_DESC(tfrc_debug, "Enable debug messages");
#endif
extern int dccp_li_init(void);
extern void dccp_li_exit(void);
extern int packet_history_init(void);
extern void packet_history_exit(void);
static int __init tfrc_module_init(void)
{
int rc = dccp_li_init();
if (rc == 0) {
rc = packet_history_init();
if (rc != 0)
dccp_li_exit();
}
return rc;
}
static void __exit tfrc_module_exit(void)
{
packet_history_exit();
dccp_li_exit();
}
module_init(tfrc_module_init);
module_exit(tfrc_module_exit);
MODULE_AUTHOR("Gerrit Renker <gerrit@erg.abdn.ac.uk>, "
"Ian McDonald <ian.mcdonald@jandi.co.nz>, "
"Arnaldo Carvalho de Melo <acme@redhat.com>");
MODULE_DESCRIPTION("DCCP TFRC library");
MODULE_LICENSE("GPL");
...@@ -3,8 +3,9 @@ ...@@ -3,8 +3,9 @@
/* /*
* net/dccp/ccids/lib/tfrc.h * net/dccp/ccids/lib/tfrc.h
* *
* Copyright (c) 2005 The University of Waikato, Hamilton, New Zealand. * Copyright (c) 2007 The University of Aberdeen, Scotland, UK
* Copyright (c) 2005 Ian McDonald <ian.mcdonald@jandi.co.nz> * Copyright (c) 2005-6 The University of Waikato, Hamilton, New Zealand.
* Copyright (c) 2005-6 Ian McDonald <ian.mcdonald@jandi.co.nz>
* Copyright (c) 2005 Arnaldo Carvalho de Melo <acme@conectiva.com.br> * Copyright (c) 2005 Arnaldo Carvalho de Melo <acme@conectiva.com.br>
* Copyright (c) 2003 Nils-Erik Mattsson, Joacim Haggmark, Magnus Erixzon * Copyright (c) 2003 Nils-Erik Mattsson, Joacim Haggmark, Magnus Erixzon
* *
...@@ -15,6 +16,14 @@ ...@@ -15,6 +16,14 @@
*/ */
#include <linux/types.h> #include <linux/types.h>
#include <asm/div64.h> #include <asm/div64.h>
#include "../../dccp.h"
#ifdef CONFIG_IP_DCCP_TFRC_DEBUG
extern int tfrc_debug;
#define tfrc_pr_debug(format, a...) DCCP_PR_DEBUG(tfrc_debug, format, ##a)
#else
#define tfrc_pr_debug(format, a...)
#endif
/* integer-arithmetic divisions of type (a * 1000000)/b */ /* integer-arithmetic divisions of type (a * 1000000)/b */
static inline u64 scaled_div(u64 a, u32 b) static inline u64 scaled_div(u64 a, u32 b)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment