diff options
author | James Clark <[email protected]> | 2024-05-07 15:12:06 +0100 |
---|---|---|
committer | Arnaldo Carvalho de Melo <[email protected]> | 2024-05-09 18:48:19 -0300 |
commit | fd81f52e311f11f3eba842439948f989ffa8ccc2 (patch) | |
tree | 32a9e637f721ab3a1d311152b7a3838d4255b963 /net/unix/sysctl_net_unix.c | |
parent | 9fe410a7ef483a9aca08bf620d8ddfd35ac99bc7 (diff) |
perf maps: Re-use __maps__free_maps_by_name()
maps__merge_in() hard codes the steps to free the maps_by_name list. It
seems to not map__put() each element before freeing, and it sets
maps_by_name_sorted to true after freeing, which may be harmless but
is inconsistent with maps__init() and other functions.
maps__maps_by_name_addr() is also quite hard to read because we already
have maps__maps_by_name() and maps__maps_by_address(), but the function
is only used in that place so delete it.
Reviewed-by: Ian Rogers <[email protected]>
Signed-off-by: James Clark <[email protected]>
Cc: Adrian Hunter <[email protected]>
Cc: Alexander Shishkin <[email protected]>
Cc: Athira Rajeev <[email protected]>
Cc: Ingo Molnar <[email protected]>
Cc: Jiri Olsa <[email protected]>
Cc: Mark Rutland <[email protected]>
Cc: Namhyung Kim <[email protected]>
Cc: Peter Zijlstra <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Arnaldo Carvalho de Melo <[email protected]>
Diffstat (limited to 'net/unix/sysctl_net_unix.c')
0 files changed, 0 insertions, 0 deletions