[ovs-dev] [next2 5/7] ofproto: Remove unused coverage counters.

Ethan Jackson ethan at nicira.com
Wed May 11 18:35:12 UTC 2011


Looks Good.

I didn't check that you didn't miss any.
Ethan

On Tue, Apr 26, 2011 at 13:51, Ben Pfaff <blp at nicira.com> wrote:
> These were mostly moved into ofproto-dpif.c, but the definitions weren't
> deleted along with them.
> ---
>  ofproto/ofproto.c |    9 ---------
>  1 files changed, 0 insertions(+), 9 deletions(-)
>
> diff --git a/ofproto/ofproto.c b/ofproto/ofproto.c
> index 5a36ca6..028a50a 100644
> --- a/ofproto/ofproto.c
> +++ b/ofproto/ofproto.c
> @@ -49,24 +49,15 @@
>
>  VLOG_DEFINE_THIS_MODULE(ofproto);
>
> -COVERAGE_DEFINE(odp_overflow);
>  COVERAGE_DEFINE(ofproto_agg_request);
> -COVERAGE_DEFINE(ofproto_costly_flags);
> -COVERAGE_DEFINE(ofproto_ctlr_action);
>  COVERAGE_DEFINE(ofproto_error);
> -COVERAGE_DEFINE(ofproto_expiration);
> -COVERAGE_DEFINE(ofproto_expired);
>  COVERAGE_DEFINE(ofproto_flows_req);
>  COVERAGE_DEFINE(ofproto_flush);
> -COVERAGE_DEFINE(ofproto_invalidated);
>  COVERAGE_DEFINE(ofproto_no_packet_in);
> -COVERAGE_DEFINE(ofproto_ofp2odp);
> -COVERAGE_DEFINE(ofproto_packet_in);
>  COVERAGE_DEFINE(ofproto_packet_out);
>  COVERAGE_DEFINE(ofproto_queue_req);
>  COVERAGE_DEFINE(ofproto_recv_openflow);
>  COVERAGE_DEFINE(ofproto_reinit_ports);
> -COVERAGE_DEFINE(ofproto_unexpected_rule);
>  COVERAGE_DEFINE(ofproto_uninstallable);
>  COVERAGE_DEFINE(ofproto_update_port);
>
> --
> 1.7.4.4
>
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev
>



More information about the dev mailing list