This is the mail archive of the
systemtap@sourceware.org
mailing list for the systemtap project.
Re: [PATCH 3/3] Add new case for stress test
- From: fche at redhat dot com (Frank Ch. Eigler)
- To: Zhou Wenjian <zhouwj-fnst at cn dot fujitsu dot com>
- Cc: <systemtap at sourceware dot org>
- Date: Tue, 15 Dec 2015 07:41:14 -0500
- Subject: Re: [PATCH 3/3] Add new case for stress test
- Authentication-results: sourceware.org; auth=none
- References: <1450164104-1508-1-git-send-email-zhouwj-fnst at cn dot fujitsu dot com> <1450164104-1508-3-git-send-email-zhouwj-fnst at cn dot fujitsu dot com>
zhouwj-fnst wrote:
> [...]
> +proc gen_load {} {
> + # if 'genload' from the ltp exists, use it to create a real load
> + set genload {/usr/local/ltp/testcases/bin/genload}
> + if [file executable $genload] {
> [...]
How about using the same logic used in lib/stap_run.exp looking for
/usr/bin/stress? Or $PATH searching? That /usr/local/ltp path is
just not a good fit for general installations.
- FChE