From mboxrd@z Thu Jan 1 00:00:00 1970 From: Arnaldo Carvalho de Melo Subject: Re: [PATCH] Fixes hang in zstd compression test by changing the source of random data. Date: Thu, 22 Aug 2019 18:24:07 -0300 Message-ID: <20190822212407.GJ3929@kernel.org> References: <3d8cc701-df4e-f949-1715-5118b530e990@arm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <3d8cc701-df4e-f949-1715-5118b530e990@arm.com> Sender: linux-kernel-owner@vger.kernel.org To: James Clark Cc: "peterz@infradead.org" , "mingo@redhat.com" , "alexander.shishkin@linux.intel.com" , "jolsa@redhat.com" , "namhyung@kernel.org" , "alexey.budankov@linux.intel.com" , "linux-perf-users@vger.kernel.org" , nd , "linux-kernel@vger.kernel.org" , Jeremy Linton List-Id: linux-perf-users.vger.kernel.org Em Thu, Aug 22, 2019 at 01:55:15PM +0000, James Clark escreveu: > Running 'perf test' with zstd compression linked will hang at the test > 'Zstd perf.data compression/decompression' because /dev/random blocks > reads until there is enough entropy. This means that the test will > appear to never complete unless the mouse is continually moved while > running it. message came mangled, had to do it by hand and then hook up your header so as to get the correct date, attribution, etc, please check Documentation/process/email-clients.rst, - Arnaldo > Signed-off-by: James Clark > --- > tools/perf/tests/shell/record+zstd_comp_decomp.sh | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/tools/perf/tests/shell/record+zstd_comp_decomp.sh b/tools/perf/tests/shell/record+zstd_comp_decomp.sh > index 899604d1..63a91ec 100755 > --- a/tools/perf/tests/shell/record+zstd_comp_decomp.sh > +++ b/tools/perf/tests/shell/record+zstd_comp_decomp.sh > @@ -13,7 +13,7 @@ skip_if_no_z_record() { > collect_z_record() { > echo "Collecting compressed record file:" > $perf_tool record -o $trace_file -g -z -F 5000 -- \ > - dd count=500 if=/dev/random of=/dev/null > + dd count=500 if=/dev/urandom of=/dev/null > } > > check_compressed_stats() { > -- > 2.7.4 > -- - Arnaldo