Re: [Mesa-dev] Testing patches 9987 and 9988

[⚛]

On Wed, Jul 20, 2016 at 3:10 PM, Marek Olšák  wrote:
> Did you test this?
> https://cgit.freedesktop.org/~mareko/mesa/commit/?h=si-mid-ib-gfx-fence=a993d93a16da86ef1ead4a55aed969752ad3965a

I compiled your git branch and tested some games.

Bioshock Infinite benchmark on Ultra settings goes from 27.72 to 34.44
FPS. Gameplay on Ultra settings is 20-50 FPS.

Other Steam games I tested are unaffected, or show a slight
improvement which is within error bounds of measurement. I didn't
notice any performance regressions.
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] Testing patches 9987 and 9988

[Marek Olšák]

On Wed, Jul 20, 2016 at 1:35 PM, ⚛ <0xe2.0x9a.0...@gmail.com> wrote:
> Results for Bioshock Infinite on R9 390:
>
>  TIMEAVG   MIN   MAX
> 2016-05-25.15-51-53: 139.06, 9.81, 1.72, 58.78
> 2016-06-14.01-13-33: 100.91, 16.12, 3.06, 60.06
> 2016-07-07.16-44-55: 95.88, 18.45, 3.74, 68.32
> 2016-07-20.11-55-04: 85.97, 27.20, 4.54, 68.63 (git-0b626d7)
> 2016-07-20.12-17-38: 84.93, 27.72, 4.54, 71.60 (git-3c78d89)
>
> This roughly shows how performance improved over time.
>
> Mesa git-0b626d7 is just before http://patchwork.freedesktop.org/series/9987
>
> Mesa git-3c78d89 is after 9987 has been merged.
>
> The biggest performance change happened between 2016-07-07 and
> 2016-07-20, but I do not have enough data to tell exactly when.

Did you test this?
https://cgit.freedesktop.org/~mareko/mesa/commit/?h=si-mid-ib-gfx-fence=a993d93a16da86ef1ead4a55aed969752ad3965a

Marek
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] Testing patches 9987 and 9988

[Martin Peres]

On 20/07/16 14:35, ⚛ wrote:

Results for Bioshock Infinite on R9 390:

 TIMEAVG   MIN   MAX
2016-05-25.15-51-53: 139.06, 9.81, 1.72, 58.78
2016-06-14.01-13-33: 100.91, 16.12, 3.06, 60.06
2016-07-07.16-44-55: 95.88, 18.45, 3.74, 68.32
2016-07-20.11-55-04: 85.97, 27.20, 4.54, 68.63 (git-0b626d7)
2016-07-20.12-17-38: 84.93, 27.72, 4.54, 71.60 (git-3c78d89)

This roughly shows how performance improved over time.

Mesa git-0b626d7 is just before http://patchwork.freedesktop.org/series/9987

Mesa git-3c78d89 is after 9987 has been merged.

The biggest performance change happened between 2016-07-07 and
2016-07-20, but I do not have enough data to tell exactly when.


Ezbench can bisect these improvements for you. Join me on #ezbench if 
you need help!


Martin
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] Testing patches 9987 and 9988

[⚛]

Results for Bioshock Infinite on R9 390:

 TIMEAVG   MIN   MAX
2016-05-25.15-51-53: 139.06, 9.81, 1.72, 58.78
2016-06-14.01-13-33: 100.91, 16.12, 3.06, 60.06
2016-07-07.16-44-55: 95.88, 18.45, 3.74, 68.32
2016-07-20.11-55-04: 85.97, 27.20, 4.54, 68.63 (git-0b626d7)
2016-07-20.12-17-38: 84.93, 27.72, 4.54, 71.60 (git-3c78d89)

This roughly shows how performance improved over time.

Mesa git-0b626d7 is just before http://patchwork.freedesktop.org/series/9987

Mesa git-3c78d89 is after 9987 has been merged.

The biggest performance change happened between 2016-07-07 and
2016-07-20, but I do not have enough data to tell exactly when.

---

st_validate_state() is definitely affecting performance in games as
measured by "perf stat", so I am looking forward to your rewrite.

On Tue, Jul 19, 2016 at 11:54 PM, Marek Olšák  wrote:
> The first one just landed.
>
> The second one will take some time I guess.
>
> This branch contains both, but it won't be merged in this form:
> https://cgit.freedesktop.org/~mareko/mesa/log/?h=si-mid-ib-gfx-fence
>
> Marek
>
> On Tue, Jul 19, 2016 at 3:53 PM, ⚛ <0xe2.0x9a.0...@gmail.com> wrote:
>> Hello
>>
>> I would like to test http://patchwork.freedesktop.org/series/9987/ and
>> http://patchwork.freedesktop.org/series/9988/ but the mbox patches
>> aren't compatible with mesa-git.
>>
>> Would it be possible to update 9987 and 9988 to match mesa-git?
>>
>> Do 9987 and 9988 assume additional public patches that need to be
>> applied prior to them?
>>
>> Thanks
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] Testing patches 9987 and 9988

[Marek Olšák]

The first one just landed.

The second one will take some time I guess.

This branch contains both, but it won't be merged in this form:
https://cgit.freedesktop.org/~mareko/mesa/log/?h=si-mid-ib-gfx-fence

Marek

On Tue, Jul 19, 2016 at 3:53 PM, ⚛ <0xe2.0x9a.0...@gmail.com> wrote:
> Hello
>
> I would like to test http://patchwork.freedesktop.org/series/9987/ and
> http://patchwork.freedesktop.org/series/9988/ but the mbox patches
> aren't compatible with mesa-git.
>
> Would it be possible to update 9987 and 9988 to match mesa-git?
>
> Do 9987 and 9988 assume additional public patches that need to be
> applied prior to them?
>
> Thanks
> ___
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] Testing patches 9987 and 9988

[Mike Lothian]

Hi

You're best replying directly to the posts on the mailing list for these.
Most folk won't know the their patch series by their patchwork ID

I think Marek posted a branch with his patches applied, it might be easier
to test that, I'm sure he'll rebase his patches after review

Cheers

Mike

On Tue, 19 Jul 2016, 2:54 p.m. ⚛, <0xe2.0x9a.0...@gmail.com> wrote:

> Hello
>
> I would like to test http://patchwork.freedesktop.org/series/9987/ and
> http://patchwork.freedesktop.org/series/9988/ but the mbox patches
> aren't compatible with mesa-git.
>
> Would it be possible to update 9987 and 9988 to match mesa-git?
>
> Do 9987 and 9988 assume additional public patches that need to be
> applied prior to them?
>
> Thanks
> ___
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

[Mesa-dev] Testing patches 9987 and 9988

[⚛]

Hello

I would like to test http://patchwork.freedesktop.org/series/9987/ and
http://patchwork.freedesktop.org/series/9988/ but the mbox patches
aren't compatible with mesa-git.

Would it be possible to update 9987 and 9988 to match mesa-git?

Do 9987 and 9988 assume additional public patches that need to be
applied prior to them?

Thanks
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev