From patchwork Mon Jul 7 08:45:59 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jacob Erlbeck X-Patchwork-Id: 367450 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from ganesha.gnumonks.org (ganesha.gnumonks.org [IPv6:2001:780:45:1d:225:90ff:fe52:c662]) (using TLSv1 with cipher AES256-SHA (256/256 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 402D91400B8 for ; Mon, 7 Jul 2014 18:56:38 +1000 (EST) Received: from localhost ([127.0.0.1] helo=ganesha.gnumonks.org) by ganesha.gnumonks.org with esmtp (Exim 4.72) (envelope-from ) id 1X44ir-0003qi-Jx; Mon, 07 Jul 2014 10:56:29 +0200 Received: from mail.sysmocom.de ([144.76.43.93]) by ganesha.gnumonks.org with esmtp (Exim 4.72) (envelope-from ) id 1X44ZC-0002rv-W7 for openbsc@lists.osmocom.org; Mon, 07 Jul 2014 10:46:34 +0200 Received: from sysmocom-tmp.am93.sysmocom.de (91-65-194-134-dynip.superkabel.de [91.65.194.134]) by mail.sysmocom.de (Postfix) with ESMTPSA id E4057631AD; Mon, 7 Jul 2014 08:46:29 +0000 (UTC) From: Jacob Erlbeck To: openbsc@lists.osmocom.org Subject: [PATCH 1/3] gprs/test: Remove verbose parameter of gbprox_dump_* functions Date: Mon, 7 Jul 2014 10:45:59 +0200 Message-Id: <1404722761-25939-1-git-send-email-jerlbeck@sysmocom.de> X-Mailer: git-send-email 1.9.1 In-Reply-To: <1404386897-27883-1-git-send-email-jerlbeck@sysmocom.de> References: <1404386897-27883-1-git-send-email-jerlbeck@sysmocom.de> X-Spam-Score: 0.1 (/) X-Spam-Report: SpamASsassin versoin 3.3.1 on ganesha.gnumonks.org summary: Content analysis details: (0.1 points, 5.0 required) pts rule name description ---- ---------------------- -------------------------------------------------- 0.1 TW_GB BODY: Odd Letter Triples with GB Cc: Jacob Erlbeck X-BeenThere: openbsc@lists.osmocom.org X-Mailman-Version: 2.1.13 Precedence: list List-Id: Development of the OpenBSC GSM base station controller List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: openbsc-bounces@lists.osmocom.org Errors-To: openbsc-bounces@lists.osmocom.org This parameter is not used (the methods are always called with an argument of 1 in the third position). Thus the parameter is removed completely. Sponsored-by: On-Waves ehf --- openbsc/include/openbsc/gb_proxy.h | 4 +-- openbsc/src/gprs/gb_proxy.c | 10 ++----- openbsc/tests/gbproxy/gbproxy_test.c | 52 ++++++++++++++++++------------------ 3 files changed, 30 insertions(+), 36 deletions(-) diff --git a/openbsc/include/openbsc/gb_proxy.h b/openbsc/include/openbsc/gb_proxy.h index 4d189a6..5f1f3a7 100644 --- a/openbsc/include/openbsc/gb_proxy.h +++ b/openbsc/include/openbsc/gb_proxy.h @@ -37,7 +37,7 @@ int gbprox_signal(unsigned int subsys, unsigned int signal, /* Reset all persistent NS-VC's */ int gbprox_reset_persistent_nsvcs(struct gprs_ns_inst *nsi); -int gbprox_dump_global(FILE *stream, int indent, int verbose); -int gbprox_dump_peers(FILE *stream, int indent, int verbose); +int gbprox_dump_global(FILE *stream, int indent); +int gbprox_dump_peers(FILE *stream, int indent); void gbprox_reset(); #endif diff --git a/openbsc/src/gprs/gb_proxy.c b/openbsc/src/gprs/gb_proxy.c index 4799344..9de2ca9 100644 --- a/openbsc/src/gprs/gb_proxy.c +++ b/openbsc/src/gprs/gb_proxy.c @@ -806,7 +806,7 @@ int gbprox_signal(unsigned int subsys, unsigned int signal, return 0; } -int gbprox_dump_global(FILE *stream, int indent, int verbose) +int gbprox_dump_global(FILE *stream, int indent) { unsigned int i; const struct rate_ctr_group_desc *desc; @@ -816,9 +816,6 @@ int gbprox_dump_global(FILE *stream, int indent, int verbose) if (rc < 0) return rc; - if (!verbose) - return 0; - desc = get_global_ctrg()->desc; for (i = 0; i < desc->num_ctr; i++) { @@ -837,7 +834,7 @@ int gbprox_dump_global(FILE *stream, int indent, int verbose) return 0; } -int gbprox_dump_peers(FILE *stream, int indent, int verbose) +int gbprox_dump_peers(FILE *stream, int indent) { struct gbprox_peer *peer; struct gprs_ra_id raid; @@ -862,9 +859,6 @@ int gbprox_dump_peers(FILE *stream, int indent, int verbose) if (rc < 0) return rc; - if (!verbose) - continue; - desc = peer->ctrg->desc; for (i = 0; i < desc->num_ctr; i++) { diff --git a/openbsc/tests/gbproxy/gbproxy_test.c b/openbsc/tests/gbproxy/gbproxy_test.c index ea07a77..a8f4eaf 100644 --- a/openbsc/tests/gbproxy/gbproxy_test.c +++ b/openbsc/tests/gbproxy/gbproxy_test.c @@ -420,7 +420,7 @@ static void test_gbproxy() setup_ns(nsi, &bss_peer[0], 0x1001, 0x1000); setup_bssgp(nsi, &bss_peer[0], 0x1002); gprs_dump_nsi(nsi); - gbprox_dump_peers(stdout, 0, 1); + gbprox_dump_peers(stdout, 0); send_bssgp_reset_ack(nsi, &sgsn_peer, 0x1002); @@ -429,7 +429,7 @@ static void test_gbproxy() setup_ns(nsi, &bss_peer[1], 0x2001, 0x2000); setup_bssgp(nsi, &bss_peer[1], 0x2002); gprs_dump_nsi(nsi); - gbprox_dump_peers(stdout, 0, 1); + gbprox_dump_peers(stdout, 0); send_bssgp_reset_ack(nsi, &sgsn_peer, 0x2002); @@ -437,43 +437,43 @@ static void test_gbproxy() setup_ns(nsi, &bss_peer[2], 0x1001, 0x1000); gprs_dump_nsi(nsi); - gbprox_dump_peers(stdout, 0, 1); + gbprox_dump_peers(stdout, 0); printf("--- Move BSS 2 to former BSS 1 port ---\n\n"); setup_ns(nsi, &bss_peer[0], 0x2001, 0x2000); gprs_dump_nsi(nsi); - gbprox_dump_peers(stdout, 0, 1); + gbprox_dump_peers(stdout, 0); printf("--- Move BSS 1 to current BSS 2 port ---\n\n"); setup_ns(nsi, &bss_peer[0], 0x2001, 0x2000); gprs_dump_nsi(nsi); - gbprox_dump_peers(stdout, 0, 1); + gbprox_dump_peers(stdout, 0); printf("--- Move BSS 2 to new port ---\n\n"); setup_ns(nsi, &bss_peer[3], 0x2001, 0x2000); gprs_dump_nsi(nsi); - gbprox_dump_peers(stdout, 0, 1); + gbprox_dump_peers(stdout, 0); printf("--- Move BSS 2 to former BSS 1 port ---\n\n"); setup_ns(nsi, &bss_peer[2], 0x2001, 0x2000); gprs_dump_nsi(nsi); - gbprox_dump_peers(stdout, 0, 1); + gbprox_dump_peers(stdout, 0); printf("--- Move BSS 1 to original BSS 1 port ---\n\n"); setup_ns(nsi, &bss_peer[0], 0x1001, 0x1000); gprs_dump_nsi(nsi); - gbprox_dump_peers(stdout, 0, 1); + gbprox_dump_peers(stdout, 0); printf("--- Reset BSS 1 with a new BVCI ---\n\n"); setup_bssgp(nsi, &bss_peer[0], 0x1012); gprs_dump_nsi(nsi); - gbprox_dump_peers(stdout, 0, 1); + gbprox_dump_peers(stdout, 0); send_bssgp_reset_ack(nsi, &sgsn_peer, 0x1012); @@ -481,7 +481,7 @@ static void test_gbproxy() setup_bssgp(nsi, &bss_peer[0], 0x1002); gprs_dump_nsi(nsi); - gbprox_dump_peers(stdout, 0, 1); + gbprox_dump_peers(stdout, 0); send_bssgp_reset_ack(nsi, &sgsn_peer, 0x1002); @@ -489,7 +489,7 @@ static void test_gbproxy() setup_bssgp(nsi, &bss_peer[0], 0x1002); gprs_dump_nsi(nsi); - gbprox_dump_peers(stdout, 0, 1); + gbprox_dump_peers(stdout, 0); send_bssgp_reset_ack(nsi, &sgsn_peer, 0x1002); @@ -521,9 +521,9 @@ static void test_gbproxy() setup_bssgp(nsi, &bss_peer[2], 0x1002); gprs_dump_nsi(nsi); - gbprox_dump_peers(stdout, 0, 1); + gbprox_dump_peers(stdout, 0); - gbprox_dump_global(stdout, 0, 1); + gbprox_dump_global(stdout, 0); send_bssgp_reset_ack(nsi, &sgsn_peer, 0x1002); @@ -539,7 +539,7 @@ static void test_gbproxy() send_ns_unitdata(nsi, NULL, &sgsn_peer, 0x10ff, (uint8_t *)"", 0); - gbprox_dump_global(stdout, 0, 1); + gbprox_dump_global(stdout, 0); gprs_ns_destroy(nsi); nsi = NULL; @@ -585,13 +585,13 @@ static void test_gbproxy_ident_changes() setup_bssgp(nsi, &bss_peer[0], bvci[0]); send_bssgp_reset_ack(nsi, &sgsn_peer, bvci[0]); - gbprox_dump_peers(stdout, 0, 1); + gbprox_dump_peers(stdout, 0); printf("--- Setup BVCI 2 ---\n\n"); setup_bssgp(nsi, &bss_peer[0], bvci[1]); send_bssgp_reset_ack(nsi, &sgsn_peer, bvci[1]); - gbprox_dump_peers(stdout, 0, 1); + gbprox_dump_peers(stdout, 0); printf("--- Send message from BSS 1 to SGSN and back, BVCI 1 ---\n\n"); @@ -612,13 +612,13 @@ static void test_gbproxy_ident_changes() setup_bssgp(nsi, &bss_peer[0], bvci[0]); send_bssgp_reset_ack(nsi, &sgsn_peer, bvci[0]); - gbprox_dump_peers(stdout, 0, 1); + gbprox_dump_peers(stdout, 0); printf("--- Setup BVCI 3 ---\n\n"); setup_bssgp(nsi, &bss_peer[0], bvci[2]); send_bssgp_reset_ack(nsi, &sgsn_peer, bvci[2]); - gbprox_dump_peers(stdout, 0, 1); + gbprox_dump_peers(stdout, 0); printf("--- Send message from BSS 1 to SGSN and back, BVCI 1 ---\n\n"); @@ -629,9 +629,9 @@ static void test_gbproxy_ident_changes() " (should fail) ---\n\n"); send_ns_unitdata(nsi, NULL, &bss_peer[0], bvci[1], (uint8_t *)"", 0); - gbprox_dump_peers(stdout, 0, 1); + gbprox_dump_peers(stdout, 0); send_ns_unitdata(nsi, NULL, &sgsn_peer, bvci[1], (uint8_t *)"", 0); - gbprox_dump_peers(stdout, 0, 1); + gbprox_dump_peers(stdout, 0); printf("--- Send message from BSS 1 to SGSN and back, BVCI 3 ---\n\n"); @@ -647,13 +647,13 @@ static void test_gbproxy_ident_changes() setup_bssgp(nsi, &bss_peer[0], bvci[0]); send_bssgp_reset_ack(nsi, &sgsn_peer, bvci[0]); - gbprox_dump_peers(stdout, 0, 1); + gbprox_dump_peers(stdout, 0); printf("--- Setup BVCI 4 ---\n\n"); setup_bssgp(nsi, &bss_peer[0], bvci[3]); send_bssgp_reset_ack(nsi, &sgsn_peer, bvci[3]); - gbprox_dump_peers(stdout, 0, 1); + gbprox_dump_peers(stdout, 0); printf("--- Send message from BSS 1 to SGSN and back, BVCI 1 ---\n\n"); @@ -664,9 +664,9 @@ static void test_gbproxy_ident_changes() " (should fail) ---\n\n"); send_ns_unitdata(nsi, NULL, &bss_peer[0], bvci[1], (uint8_t *)"", 0); - gbprox_dump_peers(stdout, 0, 1); + gbprox_dump_peers(stdout, 0); send_ns_unitdata(nsi, NULL, &sgsn_peer, bvci[1], (uint8_t *)"", 0); - gbprox_dump_peers(stdout, 0, 1); + gbprox_dump_peers(stdout, 0); printf("--- Send message from BSS 1 to SGSN and back, BVCI 3 ---\n\n"); @@ -678,8 +678,8 @@ static void test_gbproxy_ident_changes() send_ns_unitdata(nsi, NULL, &bss_peer[0], bvci[3], (uint8_t *)"", 0); send_ns_unitdata(nsi, NULL, &sgsn_peer, bvci[3], (uint8_t *)"", 0); - gbprox_dump_global(stdout, 0, 1); - gbprox_dump_peers(stdout, 0, 1); + gbprox_dump_global(stdout, 0); + gbprox_dump_peers(stdout, 0); gprs_ns_destroy(nsi); nsi = NULL;