[Intel-gfx] [PATCH i-g-t] benchmarks/wsim: Simulate and interpret .wsim
Chris Wilson
chris at chris-wilson.co.uk
Tue Jul 10 15:40:48 UTC 2018
Quoting Tvrtko Ursulin (2018-07-10 16:38:14)
>
> On 10/07/2018 14:47, Chris Wilson wrote:
> > Quoting Tvrtko Ursulin (2018-05-11 09:31:52)
> >>
> >> On 11/05/2018 08:11, Chris Wilson wrote:
> >>> A little tool I've been meaning to write for a while... Convert the
> >>> .wsim into their dag and find the longest chains and evaluate them on an
> >>> simulated machine.
> >>
> >> Very cool!
> >
> > Would you care to ack it in its current form, knowing that we will fix
> > it whenever we find a corner case interesting enough to study?
>
> I think just a little bit more polish is needed. I've just tried it out
> and polish I can see is:
>
> 1. -h / --help with just a brief comment on what the tool does and usage.
>
> 2. Lead stats displayed for each cmdline argument with parsed filename.
>
> 3. Replace result section names "Single client", "Simulated clients" and
> item names "total/ideal/packed" with a more descriptive text which would
> hopefully be somewhat self-explanatory what the numbers represent.
>
> 4. Report garbage input (and unknown wsim commands) instead of reporting
> some numbers for instance for ./sim_wsim README :)
>
> 5. #ifdef 0 graphviz ? add --graphviz cmddline option?
>
> 6. Oh.. and meson makefiles.. :)
But if it was upstream, you would just contribute those improvements
yourself.
So you are saying that this is just not useful?
-Chris
More information about the Intel-gfx
mailing list