All of lore.kernel.org
 help / color / mirror / Atom feed
From: Marcin Slusarz <marcin.slusarz-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
To: nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org,
	Martin Peres
	<martin.peres-Iz16wY1oaNPLSKGbIzaifA@public.gmane.org>
Cc: Ben Skeggs <bskeggs-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
Subject: "enable ctxprog xfer only when we need it to save power" introduces big performance regression
Date: Sat, 29 Oct 2011 19:08:01 +0200	[thread overview]
Message-ID: <20111029170801.GA3219@joi.lan> (raw)

Hi

I've bisected pretty big performance regression (nv92):

$ git bisect good
b2737681d5442f05ab6419e05468c3d2511a5ced is the first bad commit
commit b2737681d5442f05ab6419e05468c3d2511a5ced
Author: Martin Peres <martin.peres-Iz16wY1oaNPLSKGbIzaifA@public.gmane.org>
Date:   Sat Jul 30 23:08:45 2011 +0200

    drm/nv50/gr: enable ctxprog xfer only when we need it to save power

    This patch adds instructions to ctxprog and by doing, impacts context
    switching performance.  My testcase showed a 1% performance cost using
    glxgears that is a context-switch bound application.

    Please test and report bugs/performance/power/other.

    Many thanks to Maxim Levitsky for his dedicated work on lowering power
    consumption with nouveau.

    More patches are coming thanks to his work:

    https://bugs.freedesktop.org/show_bug.cgi?id=37922

    Signed-off-by: Martin Peres <martin.peres-Iz16wY1oaNPLSKGbIzaifA@public.gmane.org>
    Signed-off-by: Ben Skeggs <bskeggs-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>

:040000 040000 3fcd1388cf04c71cb7a11f29f7be6033c948f119 418a527ee5970eabaa0eb6f8cb3dc96b4da523ea M      drivers

The numbers are (FPS):
nexuiz:            61 ->   49 (-19%)
xonotic:           59 ->   37 (-37%)
openarena:        163 ->  127 (-22%)
world of padman:  242 ->  181 (-25%)
warsow:            91 ->   61 (-32%)
glxgears:        1198 -> 1107 (- 7%)

Now, what can we do about it?

Marcin

             reply	other threads:[~2011-10-29 17:08 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-10-29 17:08 Marcin Slusarz [this message]
     [not found] ` <20111029170801.GA3219-OI9uyE9O0yo@public.gmane.org>
2011-10-29 17:29   ` "enable ctxprog xfer only when we need it to save power" introduces big performance regression Marcin Slusarz
     [not found]     ` <20111029172923.GA3110-OI9uyE9O0yo@public.gmane.org>
2011-10-29 18:00       ` Maarten Maathuis
     [not found]         ` <CAGZ4FETt6QzMkxrMQwJsszjuQq0wdt85ujRZbxkdurkduaHh2A-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2011-10-30  1:15           ` Ben Skeggs
2011-10-29 19:09       ` Marcin Slusarz
     [not found]         ` <20111029190938.GA3105-OI9uyE9O0yo@public.gmane.org>
2011-10-30  0:25           ` Maxim Levitsky
2011-10-30 10:33             ` Marcin Slusarz
     [not found]               ` <20111030103349.GA3157-OI9uyE9O0yo@public.gmane.org>
2011-10-30 10:37                 ` Marcin Kościelnicki
2011-11-09 22:10           ` Marcin Slusarz
     [not found]             ` <20111109221009.GD3402-OI9uyE9O0yo@public.gmane.org>
2011-11-10  7:10               ` Martin Peres
     [not found]                 ` <4EBB78F4.2020902-Iz16wY1oaNPLSKGbIzaifA@public.gmane.org>
2011-12-28 21:39                   ` Marcin Slusarz
     [not found]                     ` <20111228213902.GA4275-OI9uyE9O0yo@public.gmane.org>
2011-12-28 23:58                       ` Martin Peres
     [not found]                         ` <4EFBAD2B.8080207-Iz16wY1oaNPLSKGbIzaifA@public.gmane.org>
2011-12-29  1:38                           ` Marcin Slusarz
     [not found]                             ` <20111229013850.GA4076-OI9uyE9O0yo@public.gmane.org>
2012-01-13 21:26                               ` Martin Peres
     [not found]                                 ` <4F10A18E.1030908-GANU6spQydw@public.gmane.org>
2012-01-17 20:55                                   ` Lucas Stach
2012-01-17 21:17                                     ` Martin Peres

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20111029170801.GA3219@joi.lan \
    --to=marcin.slusarz-re5jqeeqqe8avxtiumwx3w@public.gmane.org \
    --cc=bskeggs-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \
    --cc=martin.peres-Iz16wY1oaNPLSKGbIzaifA@public.gmane.org \
    --cc=nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.