linux-kernel - RE: [PATCH] Add --buildids-blacklist option to perf archive command

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <MW4PR11MB82899DCA76DC72B7BE36FFA4957BA@MW4PR11MB8289.namprd11.prod.outlook.com>
Date: Wed, 25 Jun 2025 15:28:02 +0000
From: "Li, Tianyou" <tianyou.li@...el.com>
To: Namhyung Kim <namhyung@...nel.org>
CC: Peter Zijlstra <peterz@...radead.org>, Ingo Molnar <mingo@...hat.com>,
	Arnaldo Carvalho de Melo <acme@...nel.org>, Mark Rutland
	<mark.rutland@....com>, Alexander Shishkin
	<alexander.shishkin@...ux.intel.com>, Jiri Olsa <jolsa@...nel.org>, "Ian
 Rogers" <irogers@...gle.com>, "Hunter, Adrian" <adrian.hunter@...el.com>, Kan
 Liang <kan.liang@...ux.intel.com>, "Guo, Wangyang" <wangyang.guo@...el.com>,
	"linux-perf-users@...r.kernel.org" <linux-perf-users@...r.kernel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: RE: [PATCH] Add --buildids-blacklist option to perf archive command

Hi Namhyung,

Thanks for your comments, that make sense. I have sent a V2 patch named "[PATCH V2] tools/perf: Add --exclude-buildids option to perf archive command" for your review. 

Regards,
Tianyou

-----Original Message-----
From: Namhyung Kim <namhyung@...nel.org> 
Sent: Wednesday, June 25, 2025 2:50 AM
To: Li, Tianyou <tianyou.li@...el.com>
Cc: Peter Zijlstra <peterz@...radead.org>; Ingo Molnar <mingo@...hat.com>; Arnaldo Carvalho de Melo <acme@...nel.org>; Mark Rutland <mark.rutland@....com>; Alexander Shishkin <alexander.shishkin@...ux.intel.com>; Jiri Olsa <jolsa@...nel.org>; Ian Rogers <irogers@...gle.com>; Hunter, Adrian <adrian.hunter@...el.com>; Kan Liang <kan.liang@...ux.intel.com>; Guo, Wangyang <wangyang.guo@...el.com>; linux-perf-users@...r.kernel.org; linux-kernel@...r.kernel.org
Subject: Re: [PATCH] Add --buildids-blacklist option to perf archive command

Hello,

On Mon, Jun 09, 2025 at 11:55:42AM +0800, Tianyou Li wrote:
> When make a perf archive, it may contains the binaries that user did 
> not want to ship with, add --buildids-blacklist option to specify a 
> backlist file which contains the buildids need to be excluded. The blacklist file can be generated from command:
> 
>   perf buildid-list -i perf.data --with-hits | grep -v "^ " > 
> buildids_blacklist.txt
> 
> Then remove the lines from the buildids_blacklist.txt for buildids should be included.

Instead of blacklist, how about saying it like "--exclude-buildids"?

> 
> Signed-off-by: Tianyou Li <tianyou.li@...el.com>
> Reviewed-by: Wangyang Guo <wangyang.guo@...el.com>
> ---
>  tools/perf/perf-archive.sh | 35 ++++++++++++++++++++++++++++++-----
>  1 file changed, 30 insertions(+), 5 deletions(-)
> 
> diff --git a/tools/perf/perf-archive.sh b/tools/perf/perf-archive.sh 
> index 6ed7e52ab881..e5ff840ac6b4 100755
> --- a/tools/perf/perf-archive.sh
> +++ b/tools/perf/perf-archive.sh
> @@ -16,6 +16,13 @@ while [ $# -gt 0 ] ; do
>  	elif [ $1 == "--unpack" ]; then
>  		UNPACK=1
>  		shift
> +	elif [ $1 == "--buildids-blacklist" ]; then
> +		BUILDIDS_BLACKLIST="$2"
> +		if [ ! -e "$BUILDIDS_BLACKLIST" ]; then
> +			echo "Provided buildids blacklist file $BUILDIDS_BLACKLIST does not exist"
> +			exit 1
> +		fi
> +		shift 2
>  	else
>  		PERF_DATA=$1
>  		UNPACK_TAR=$1
> @@ -86,11 +93,29 @@ fi
>  
>  BUILDIDS=$(mktemp /tmp/perf-archive-buildids.XXXXXX)
>  
> -perf buildid-list -i $PERF_DATA --with-hits | grep -v "^ " > 
> $BUILDIDS -if [ ! -s $BUILDIDS ] ; then
> -	echo "perf archive: no build-ids found"
> -	rm $BUILDIDS || true
> -	exit 1
> +#
> +# BUILDIDS_BLACKLIST is an optional file that contains build-ids to 
> +be excluded from the # archive. It is a list of build-ids, one per line, without any leading or trailing spaces.
> +# If the file is empty, all build-ids will be included in the 
> +archive. To create a blacklist # file, you can use the following command:
> +# 	perf buildid-list -i perf.data --with-hits | grep -v "^ " > buildids_blacklist.txt
> +# You can edit the file to remove the lines that you want to keep in the archive, then:
> +# 	perf archive --buildids-blacklist buildids_blacklist.txt

Then you'll need to update the comment here.

Thanks,
Namhyung

> +#
> +if [ ! -s $BUILDIDS_BLACKLIST ]; then
> +	perf buildid-list -i $PERF_DATA --with-hits | grep -v "^ " > $BUILDIDS
> +	if [ ! -s $BUILDIDS ] ; then
> +		echo "perf archive: no build-ids found"
> +		rm $BUILDIDS || true
> +		exit 1
> +	fi
> +else
> +	perf buildid-list -i $PERF_DATA --with-hits | grep -v "^ " | grep -Fv -f $BUILDIDS_BLACKLIST > $BUILDIDS
> +	if [ ! -s $BUILDIDS ] ; then
> +		echo "perf archive: no build-ids found after applying blacklist"
> +		rm $BUILDIDS || true
> +		exit 1
> +	fi
>  fi
>  
>  MANIFEST=$(mktemp /tmp/perf-archive-manifest.XXXXXX)
> --
> 2.47.1
>