[ltt-dev] [PATCH] Clear up ambiguity with benchmark instructions.

[ltt-dev] [PATCH] Clear up ambiguity with benchmark instructions.

[Matthew Khouzam]

The instructions for the benchmark are a bit ambiguous. The program name 
is run and the instruction is run.

This patch changes the lines to "launch ./run". This way scripts calling 
the executable will not have to be patched also.

[ltt-dev] [PATCH] Clear up ambiguity with benchmark instructions.

[Matthew Khouzam]

Thank you Douglas. I feel excessively smart right now.

On 10-09-03 12:44 PM, Douglas Santos wrote:
> I guess you forgot to send the patch.
>
> Quoting Matthew Khouzam<matthew.khouzam at ericsson.com>:
>
>    
>> The instructions for the benchmark are a bit ambiguous. The program name
>> is run and the instruction is run.
>>
>> This patch changes the lines to "launch ./run". This way scripts calling
>> the executable will not have to be patched also.
>>
>> _______________________________________________
>> ltt-dev mailing list
>> ltt-dev at lists.casi.polymtl.ca
>> http://lists.casi.polymtl.ca/cgi-bin/mailman/listinfo/ltt-dev
>>
>>      
>
>
>    
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Author-Matthew-Khouzam-matthew.khouzam-polymtl.ca.patch
Type: text/x-patch
Size: 927 bytes
Desc: not available
URL: <http://lists.casi.polymtl.ca/pipermail/lttng-dev/attachments/20100903/0c340aae/attachment-0002.bin>

[ltt-dev] [PATCH] Clear up ambiguity with benchmark instructions.

[Matthew Khouzam]

Thank you Douglas. I feel excessively smart right now.

On 10-09-03 12:44 PM, Douglas Santos wrote:
> I guess you forgot to send the patch.
>
> Quoting Matthew Khouzam<matthew.khouzam at ericsson.com>:
>
>    
>> The instructions for the benchmark are a bit ambiguous. The program name
>> is run and the instruction is run.
>>
>> This patch changes the lines to "launch ./run". This way scripts calling
>> the executable will not have to be patched also.
>>
>> _______________________________________________
>> ltt-dev mailing list
>> ltt-dev at lists.casi.polymtl.ca
>> http://lists.casi.polymtl.ca/cgi-bin/mailman/listinfo/ltt-dev
>>
>>      
>
>
>    
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Author-Matthew-Khouzam-matthew.khouzam-polymtl.ca.patch
Type: text/x-patch
Size: 927 bytes
Desc: not available
URL: <http://lists.casi.polymtl.ca/pipermail/lttng-dev/attachments/20100903/0c340aae/attachment-0003.bin>

[ltt-dev] [PATCH] Clear up ambiguity with benchmark instructions.

[Matthew Khouzam]

Thank you Douglas. I feel excessively smart right now.

On 10-09-03 12:44 PM, Douglas Santos wrote:
> I guess you forgot to send the patch.
>
> Quoting Matthew Khouzam<matthew.khouzam at ericsson.com>:
>
>    
>> The instructions for the benchmark are a bit ambiguous. The program name
>> is run and the instruction is run.
>>
>> This patch changes the lines to "launch ./run". This way scripts calling
>> the executable will not have to be patched also.
>>
>> _______________________________________________
>> ltt-dev mailing list
>> ltt-dev at lists.casi.polymtl.ca
>> http://lists.casi.polymtl.ca/cgi-bin/mailman/listinfo/ltt-dev
>>
>>      
>
>
>    
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Author-Matthew-Khouzam-matthew.khouzam-polymtl.ca.patch
Type: text/x-patch
Size: 927 bytes
Desc: not available
URL: <http://lists.casi.polymtl.ca/pipermail/ltt-dev/attachments/20100903/0c340aae/attachment.bin>

[ltt-dev] [UST PATCH] Clear up ambiguity with benchmark instructions.

[Mathieu Desnoyers]

Please specify that this patch is for UST in your next post. A subject
line starting with [UST PATCH] would be appropriate, or at least specify
this in the changelog.

Thanks,

Mathieu

* Matthew Khouzam (matthew.khouzam at ericsson.com) wrote:
> The instructions for the benchmark are a bit ambiguous. The program name  
> is run and the instruction is run.
>
> This patch changes the lines to "launch ./run". This way scripts calling  
> the executable will not have to be patched also.
>
> _______________________________________________
> ltt-dev mailing list
> ltt-dev at lists.casi.polymtl.ca
> http://lists.casi.polymtl.ca/cgi-bin/mailman/listinfo/ltt-dev
>

-- 
Mathieu Desnoyers
Operating System Efficiency R&D Consultant
EfficiOS Inc.
http://www.efficios.com

[ltt-dev] [PATCH] Clear up ambiguity with benchmark instructions.

[Mathieu Desnoyers]

* Matthew Khouzam (matthew.khouzam at ericsson.com) wrote:
> Thank you Douglas. I feel excessively smart right now.
>
> On 10-09-03 12:44 PM, Douglas Santos wrote:
>> I guess you forgot to send the patch.
>>
>> Quoting Matthew Khouzam<matthew.khouzam at ericsson.com>:
>>
>>    
>>> The instructions for the benchmark are a bit ambiguous. The program name
>>> is run and the instruction is run.
>>>
>>> This patch changes the lines to "launch ./run". This way scripts calling
>>> the executable will not have to be patched also.
>>>
>>> _______________________________________________
>>> ltt-dev mailing list
>>> ltt-dev at lists.casi.polymtl.ca
>>> http://lists.casi.polymtl.ca/cgi-bin/mailman/listinfo/ltt-dev
>>>
>>>      
>>
>>
>>    

> From 34444ce22297e160bd0e2a80ba645e8f39a7507f Mon Sep 17 00:00:00 2001
> From: root <matthew.khouzam@polymtl.ca>
> Date: Fri, 3 Sep 2010 11:09:29 -0400
> Subject: [PATCH] Author: Matthew Khouzam <matthew.khouzam at polymtl.ca>
>  Comments: Several people were confused with the instruction "- run ./run"  for the UST benchmark. Changed run to launch
> 
> ---
>  tests/benchmark/README |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/tests/benchmark/README b/tests/benchmark/README
> index 8a9321d..435de86 100644
> --- a/tests/benchmark/README
> +++ b/tests/benchmark/README
> @@ -2,8 +2,8 @@ UST Benchmark
>  
>  INSTALLATION INSTRUCTIONS:
>  
> -  - run ./install  to install and compile UST/benchmark
> -  - run ./run  to perform the benchmark
> +  - launch ./install  to install and compile UST/benchmark
> +  - launch ./run  to perform the benchmark

Please change to the following and resend the patch:

INSTALLATION INSTRUCTIONS:

Compile and install UST/benchmark with:

% ./install

Launch the benchark with:

% ./run


Here I assume that it is correct to execute these as simple user.
Otherwise, if root access is needed, then change % for #.

Thanks,

Mathieu


>  
>  
>  The trace files will be automatically saved at $HOME/.usttraces
> -- 
> 1.7.0.4
> 

> _______________________________________________
> ltt-dev mailing list
> ltt-dev at lists.casi.polymtl.ca
> http://lists.casi.polymtl.ca/cgi-bin/mailman/listinfo/ltt-dev


-- 
Mathieu Desnoyers
Operating System Efficiency R&D Consultant
EfficiOS Inc.
http://www.efficios.com