<div dir="ltr">Hi Doug,<div><br></div><div>Thanks for the revision, yes the patch looks fine to me feel free to have Reed commit.</div><div><br></div><div> - Daniel</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">
On Thu, Jul 25, 2013 at 4:23 PM, Doug Gilmore <span dir="ltr"><<a href="mailto:Doug.Gilmore@imgtec.com" target="_blank">Doug.Gilmore@imgtec.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On Thu, 2013-07-25 at 14:47 -0700, Daniel Dunbar wrote:<br>
> Ok, this seems like a pretty custom situation, I'd rather that have<br>
> minimal impact on the LNT code paths. It doesn't generally make any<br>
> sense to compile "timeit-target" for the host.<br>
><br>
><br>
> Could you instead implement this as a patch to instead cause "timeit"<br>
> to be used instead of "timeit-target". This should be as simple as<br>
> adding a new Makefile variable that will be used in Makefile.programs<br>
> to cause RUNSAFELY to use timeit instead of timeit-target. Then you<br>
> can use LNT's existing --make-param option to pass that in.<br>
><br>
><br>
>  - Daniel<br>
</div>That's great -- my patch needed cleanup anyway and your approach<br>
finesses that issue.<br>
<br>
Patch attached.<br>
<br>
Thanks!<br>
<span class="HOEnZb"><font color="#888888"><br>
Doug<br>
</font></span><div class="HOEnZb"><div class="h5">><br>
><br>
> On Wed, Jul 24, 2013 at 11:37 AM, Doug Gilmore<br>
> <<a href="mailto:Doug.Gilmore@imgtec.com">Doug.Gilmore@imgtec.com</a>> wrote:<br>
>         On Tue, 2013-07-23 at 17:17 -0700, an Reed Kotler wrote:<br>
>         ><br>
>         > In this case we are not using lnt under Qemu user mode for<br>
>         benchmarking;<br>
>         > just as a way to run test-suite to test whether the code is<br>
>         correct.<br>
>         ><br>
>         > Qemu user mode emulates target instructions, but when it<br>
>         gets a Unix<br>
>         > Kernel trap, it uses the host to emulate those.<br>
>         ><br>
>         > For example, file I/O.<br>
>         ><br>
>         > It is possible to run target timeit under qemu and let it<br>
>         launch the app<br>
>         > or a wrapper.<br>
>         > (But it is more limited as to what can be done here under<br>
>         qemu vs under<br>
>         > the host OS directly).<br>
>         ><br>
>         > For time functions, it is also going to use the host to<br>
>         emulate those.<br>
>         ><br>
>         > So whether timeit is running under qemu or directly on the<br>
>         host, the<br>
>         > answers regarding time will be the same.<br>
>         ><br>
>         > But running timeit under qemu will be much slower as far as<br>
>         elapsed time<br>
>         > than running it on the host directly.<br>
>         ><br>
>         > We would also need to add some new mechanism to Lnt or the<br>
>         makefiles to<br>
>         > also wrap timeit.<br>
><br>
>         Good point Reed -- thanks!<br>
>         ><br>
>         > Reed<br>
>         ><br>
>         ><br>
>         > On 07/23/2013 02:19 PM, Daniel Dunbar wrote:<br>
>         > > Wouldn't it be a more accurate simulation to run<br>
>         timeit-target under<br>
>         > > the emulator as well? Or is that too much to ask?<br>
>         > ><br>
>         > >  - Daniel<br>
><br>
>         Hi Daniel,<br>
><br>
>         I agree with Reed's discussion of the issues.  We are mainly<br>
>         concerned with<br>
>         the correctness when running under QEMU, though the the timing<br>
>         data<br>
>         might be useful at a very course grain level.<br>
><br>
>         Doug<br>
>         > ><br>
>         > ><br>
>         > > On Mon, Jul 22, 2013 at 6:47 PM, Reed Kotler <rkotler at<br>
>         <a href="http://mips.com" target="_blank">mips.com</a><br>
>         > > <mailto:<a href="mailto:rkotler">rkotler</a> at <a href="http://mips.com" target="_blank">mips.com</a>>> wrote:<br>
>         > > ...<br>
><br>
><br>
><br>
><br>
<br>
</div></div></blockquote></div><br></div>