perf hist: Update use of pthread mutex

Switch to the use of mutex wrappers that provide better error checking.

Signed-off-by: Ian Rogers <[e-mail]>
Reviewed-by: Adrian Hunter <[e-mail]>
Acked-by: Namhyung Kim <[e-mail]>
Cc: Alexander Shishkin <[e-mail]>
Cc: Alexandre Truong <[e-mail]>
Cc: Alexey Bayduraev <[e-mail]>
Cc: Andi Kleen <[e-mail]>
Cc: Andres Freund <[e-mail]>
Cc: Andrii Nakryiko <[e-mail]>
Cc: André Almeida <[e-mail]>
Cc: Athira Jajeev <[e-mail]>
Cc: Christophe JAILLET <[e-mail]>
Cc: Colin Ian King <[e-mail]>
Cc: Dario Petrillo <[e-mail]>
Cc: Darren Hart <[e-mail]>
Cc: Dave Marchevsky <[e-mail]>
Cc: Davidlohr Bueso <[e-mail]>
Cc: Fangrui Song <[e-mail]>
Cc: Hewenliang <[e-mail]>
Cc: Ingo Molnar <[e-mail]>
Cc: James Clark <[e-mail]>
Cc: Jason Wang <[e-mail]>
Cc: Jiri Olsa <[e-mail]>
Cc: Kajol Jain <[e-mail]>
Cc: Kim Phillips <[e-mail]>
Cc: Leo Yan <[e-mail]>
Cc: Mark Rutland <[e-mail]>
Cc: Martin Liška <[e-mail]>
Cc: Masami Hiramatsu <[e-mail]>
Cc: Nathan Chancellor <[e-mail]>
Cc: Nick Desaulniers <[e-mail]>
Cc: Pavithra Gurushankar <[e-mail]>
Cc: Peter Zijlstra <[e-mail]>
Cc: Quentin Monnet <[e-mail]>
Cc: Ravi Bangoria <[e-mail]>
Cc: Remi Bernon <[e-mail]>
Cc: Riccardo Mancini <[e-mail]>
Cc: Song Liu <[e-mail]>
Cc: Stephane Eranian <[e-mail]>
Cc: Thomas Gleixner <[e-mail]>
Cc: Thomas Richter <[e-mail]>
Cc: Tom Rix <[e-mail]>
Cc: Weiguo Li <[e-mail]>
Cc: Wenyu Liu <[e-mail]>
Cc: William Cohen <[e-mail]>
Cc: Zechuan Chen <[e-mail]>
Cc: [e-mail]
Cc: [e-mail]
Cc: yaowenbin <[e-mail]>
Link: https://lore.kernel.org/r/[e-mail]
Signed-off-by: Arnaldo Carvalho de Melo <[e-mail]>

tools/perf/builtin-top.c

···
 		 * This function is now called with he->hists->lock held.
 		 * Release it before going to sleep.
 		 */
-		pthread_mutex_unlock(&he->hists->lock);
+		mutex_unlock(&he->hists->lock);
 
 		if (err == -ERANGE && !he->ms.map->erange_warned)
 			ui__warn_map_erange(he->ms.map, sym, ip);
···
 			sleep(1);
 		}
 
-		pthread_mutex_lock(&he->hists->lock);
+		mutex_lock(&he->hists->lock);
 	}
 }
 
···
 		else
 			iter.ops = &hist_iter_normal;
 
-		pthread_mutex_lock(&hists->lock);
+		mutex_lock(&hists->lock);
 
 		if (hist_entry_iter__add(&iter, &al, top->max_stack, top) < 0)
 			pr_err("Problem incrementing symbol period, skipping event\n");
 
-		pthread_mutex_unlock(&hists->lock);
+		mutex_unlock(&hists->lock);
 	}
 
 	addr_location__put(&al);

tools/perf/util/hist.c

···
 {
 	struct rb_root_cached *root;
 
-	pthread_mutex_lock(&hists->lock);
+	mutex_lock(&hists->lock);
 
 	root = hists->entries_in;
 	if (++hists->entries_in > &hists->entries_in_array[1])
 		hists->entries_in = &hists->entries_in_array[0];
 
-	pthread_mutex_unlock(&hists->lock);
+	mutex_unlock(&hists->lock);
 
 	return root;
 }
···
 	hists->entries_in = &hists->entries_in_array[0];
 	hists->entries_collapsed = RB_ROOT_CACHED;
 	hists->entries = RB_ROOT_CACHED;
-	pthread_mutex_init(&hists->lock, NULL);
+	mutex_init(&hists->lock);
 	hists->socket_filter = -1;
 	hists->hpp_list = hpp_list;
 	INIT_LIST_HEAD(&hists->hpp_formats);

tools/perf/util/hist.h

···
 
 #include <linux/rbtree.h>
 #include <linux/types.h>
-#include <pthread.h>
 #include "evsel.h"
 #include "color.h"
 #include "events_stats.h"
+#include "mutex.h"
 
 struct hist_entry;
 struct hist_entry_ops;
···
 	const struct dso	*dso_filter;
 	const char		*uid_filter_str;
 	const char		*symbol_filter_str;
-	pthread_mutex_t		lock;
+	struct mutex		lock;
 	struct hists_stats	stats;
 	u64			event_stream;
 	u16			col_len[HISTC_NR_COLS];