Gitweb:     
http://git.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=df8f83fdd6369e1ba85f089fd6fe26bb2ddcb36f
Commit:     df8f83fdd6369e1ba85f089fd6fe26bb2ddcb36f
Parent:     2aaef4e47fef8a6c0bc7fc5d9d3eea4af290e04c
Author:     Gerrit Renker <[EMAIL PROTECTED]>
AuthorDate: Wed Dec 12 12:24:49 2007 -0200
Committer:  David S. Miller <[EMAIL PROTECTED]>
CommitDate: Mon Jan 28 14:57:15 2008 -0800

    [TFRC]: Put RX/TX initialisation into tfrc.c
    
    This separates RX/TX initialisation and puts all packet history / loss 
intervals
    initialisation into tfrc.c.
    The organisation is uniform: slab declaration -> {rx,tx}_init() -> 
{rx,tx}_exit()
    
    Signed-off-by: Gerrit Renker <[EMAIL PROTECTED]>
    Signed-off-by: Arnaldo Carvalho de Melo <[EMAIL PROTECTED]>
    Signed-off-by: David S. Miller <[EMAIL PROTECTED]>
---
 net/dccp/ccids/lib/packet_history.c |   68 ++++++++++++++++------------------
 net/dccp/ccids/lib/tfrc.c           |   31 ++++++++++++----
 2 files changed, 55 insertions(+), 44 deletions(-)

diff --git a/net/dccp/ccids/lib/packet_history.c 
b/net/dccp/ccids/lib/packet_history.c
index af44082..727b17d 100644
--- a/net/dccp/ccids/lib/packet_history.c
+++ b/net/dccp/ccids/lib/packet_history.c
@@ -57,6 +57,22 @@ struct tfrc_tx_hist_entry {
  */
 static struct kmem_cache *tfrc_tx_hist_slab;
 
+int __init tfrc_tx_packet_history_init(void)
+{
+       tfrc_tx_hist_slab = kmem_cache_create("tfrc_tx_hist",
+                                             sizeof(struct tfrc_tx_hist_entry),
+                                             0, SLAB_HWCACHE_ALIGN, NULL);
+       return tfrc_tx_hist_slab == NULL ? -ENOBUFS : 0;
+}
+
+void tfrc_tx_packet_history_exit(void)
+{
+       if (tfrc_tx_hist_slab != NULL) {
+               kmem_cache_destroy(tfrc_tx_hist_slab);
+               tfrc_tx_hist_slab = NULL;
+       }
+}
+
 static struct tfrc_tx_hist_entry *
        tfrc_tx_hist_find_entry(struct tfrc_tx_hist_entry *head, u64 seqno)
 {
@@ -119,6 +135,22 @@ EXPORT_SYMBOL_GPL(tfrc_tx_hist_rtt);
  */
 static struct kmem_cache *tfrc_rx_hist_slab;
 
+int __init tfrc_rx_packet_history_init(void)
+{
+       tfrc_rx_hist_slab = kmem_cache_create("tfrc_rxh_cache",
+                                             sizeof(struct tfrc_rx_hist_entry),
+                                             0, SLAB_HWCACHE_ALIGN, NULL);
+       return tfrc_rx_hist_slab == NULL ? -ENOBUFS : 0;
+}
+
+void tfrc_rx_packet_history_exit(void)
+{
+       if (tfrc_rx_hist_slab != NULL) {
+               kmem_cache_destroy(tfrc_rx_hist_slab);
+               tfrc_rx_hist_slab = NULL;
+       }
+}
+
 /**
  * tfrc_rx_hist_index - index to reach n-th entry after loss_start
  */
@@ -316,39 +348,3 @@ keep_ref_for_next_time:
        return sample;
 }
 EXPORT_SYMBOL_GPL(tfrc_rx_hist_sample_rtt);
-
-__init int packet_history_init(void)
-{
-       tfrc_tx_hist_slab = kmem_cache_create("tfrc_tx_hist",
-                                             sizeof(struct 
tfrc_tx_hist_entry), 0,
-                                             SLAB_HWCACHE_ALIGN, NULL);
-       if (tfrc_tx_hist_slab == NULL)
-               goto out_err;
-
-       tfrc_rx_hist_slab = kmem_cache_create("tfrc_rx_hist",
-                                             sizeof(struct 
tfrc_rx_hist_entry), 0,
-                                             SLAB_HWCACHE_ALIGN, NULL);
-       if (tfrc_rx_hist_slab == NULL)
-               goto out_free_tx;
-
-       return 0;
-
-out_free_tx:
-       kmem_cache_destroy(tfrc_tx_hist_slab);
-       tfrc_tx_hist_slab = NULL;
-out_err:
-       return -ENOBUFS;
-}
-
-void packet_history_exit(void)
-{
-       if (tfrc_tx_hist_slab != NULL) {
-               kmem_cache_destroy(tfrc_tx_hist_slab);
-               tfrc_tx_hist_slab = NULL;
-       }
-
-       if (tfrc_rx_hist_slab != NULL) {
-               kmem_cache_destroy(tfrc_rx_hist_slab);
-               tfrc_rx_hist_slab = NULL;
-       }
-}
diff --git a/net/dccp/ccids/lib/tfrc.c b/net/dccp/ccids/lib/tfrc.c
index 3a7a183..20763fa 100644
--- a/net/dccp/ccids/lib/tfrc.c
+++ b/net/dccp/ccids/lib/tfrc.c
@@ -14,27 +14,42 @@ module_param(tfrc_debug, bool, 0444);
 MODULE_PARM_DESC(tfrc_debug, "Enable debug messages");
 #endif
 
+extern int  tfrc_tx_packet_history_init(void);
+extern void tfrc_tx_packet_history_exit(void);
+extern int  tfrc_rx_packet_history_init(void);
+extern void tfrc_rx_packet_history_exit(void);
+
 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();
-       }
+       if (rc)
+               goto out;
+
+       rc = tfrc_tx_packet_history_init();
+       if (rc)
+               goto out_free_loss_intervals;
 
+       rc = tfrc_rx_packet_history_init();
+       if (rc)
+               goto out_free_tx_history;
+       return 0;
+
+out_free_tx_history:
+       tfrc_tx_packet_history_exit();
+out_free_loss_intervals:
+       dccp_li_exit();
+out:
        return rc;
 }
 
 static void __exit tfrc_module_exit(void)
 {
-       packet_history_exit();
+       tfrc_rx_packet_history_exit();
+       tfrc_tx_packet_history_exit();
        dccp_li_exit();
 }
 
-
To unsubscribe from this list: send the line "unsubscribe git-commits-head" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to