Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 02ad0702 authored by Davidlohr Bueso's avatar Davidlohr Bueso Committed by Arnaldo Carvalho de Melo
Browse files

perf lock: Remove dead code 



No need for break statements after goto jumps.

Signed-off-by: default avatarDavidlohr Bueso <davidlohr@hp.com>
Cc: Aswin Chandramouleeswaran <aswin@hp.com>
Cc: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Hitoshi Mitake <mitake@dcl.info.waseda.ac.jp>
Cc: Ingo Molnar <mingo@kernel.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Link: http://lkml.kernel.org/r/1378693159-8747-2-git-send-email-davidlohr@hp.com


Signed-off-by: default avatarArnaldo Carvalho de Melo <acme@redhat.com>
parent 918512b4

tools/perf/builtin-lock.c

+0 −6
Original line number Diff line number Diff line
@@ -446,7 +446,6 @@ broken: 
		list_del(&seq->list);

		free(seq);

		goto end;

		break;

	default:

		BUG_ON("Unknown state of lock sequence found!\n");

		break;
@@ -508,8 +507,6 @@ static int report_lock_acquired_event(struct perf_evsel *evsel, 
		list_del(&seq->list);

		free(seq);

		goto end;

		break;



	default:

		BUG_ON("Unknown state of lock sequence found!\n");

		break;
@@ -564,7 +561,6 @@ static int report_lock_contended_event(struct perf_evsel *evsel, 
		list_del(&seq->list);

		free(seq);

		goto end;

		break;

	default:

		BUG_ON("Unknown state of lock sequence found!\n");

		break;
@@ -606,7 +602,6 @@ static int report_lock_release_event(struct perf_evsel *evsel, 
	switch (seq->state) {

	case SEQ_STATE_UNINITIALIZED:

		goto end;

		break;

	case SEQ_STATE_ACQUIRED:

		break;

	case SEQ_STATE_READ_ACQUIRED:
@@ -624,7 +619,6 @@ static int report_lock_release_event(struct perf_evsel *evsel, 
		ls->discard = 1;

		bad_hist[BROKEN_RELEASE]++;

		goto free_seq;

		break;

	default:

		BUG_ON("Unknown state of lock sequence found!\n");

		break;