Announcement

Collapse
No announcement yet.

R600 Gallium3D Patch Boosts Unigine By ~30%

Collapse
X
  • Filter
  • Time
  • Show
Clear All
new posts

  • R600 Gallium3D Patch Boosts Unigine By ~30%

    Phoronix: R600 Gallium3D Patch Boosts Unigine By ~30%

    A rather simple patch by Vadim Girlin has led to a reported significant performance improvement within the R600 Gallium3D graphics driver...

    http://www.phoronix.com/vr.php?view=MTMwNDY

  • #2
    And right after:

    Please ignore this, there are some issues with this patch undetected by
    piglit.

    I suspect it's something similar to the hw bug discussed here recently:

    http://www.mail-archive.com/mesa-dev.../msg33361.html

    Comment


    • #3
      He's posted a v2 patch now.

      Comment


      • #4
        I am curious, how many spots like this, there are in r600g at the moment.

        Comment


        • #5
          Originally posted by archibald View Post
          He's posted a v2 patch now.
          Link to patch:
          http://lists.freedesktop.org/archive...ry/034728.html

          No a nice one-liner anymore tho.

          Comment


          • #6
            Well, AMD could host farm of GPU's of various generations/configs. With some automated tools that would allow previlaged devs to submit patches to test performance/piglit regresions.

            QA would also had easier time (if dev's could detect bad code earlier).


            Great idea, so why its not already done?

            Comment


            • #7
              Originally posted by przemoli View Post
              Well, AMD could host farm of GPU's of various generations/configs. With some automated tools that would allow previlaged devs to submit patches to test performance/piglit regresions.

              QA would also had easier time (if dev's could detect bad code earlier).


              Great idea, so why its not already done?
              Or Stop developing the Driver in the Kernel Tree. For an user its much more easy to update an out of tree kernel module. To tell an user he has to recompile his kernel is is a mess.

              Comment


              • #8
                Originally posted by Xake View Post
                No a nice one-liner anymore tho.
                Yes, now it is an even nicer patch, as it in total removes 24 lines from the driver

                Thanks for the link.

                Comment


                • #9
                  Originally posted by przemoli View Post
                  Well, AMD could host farm of GPU's of various generations/configs. With some automated tools that would allow previlaged devs to submit patches to test performance/piglit regresions.
                  From what I recall, they don't have such resources to spare. Though who knows, maybe the situation has changed.

                  Originally posted by Nille View Post
                  Or Stop developing the Driver in the Kernel Tree. For an user its much more easy to update an out of tree kernel module. To tell an user he has to recompile his kernel is is a mess.
                  Not for Gentoo users it isn't!

                  Comment


                  • #10
                    Originally posted by GreatEmerald View Post
                    Not for Gentoo users it isn't!
                    Sorry, i missed that gentoo is the center of the world

                    *scnr*

                    Comment


                    • #11
                      It was a valid response to a blanket statement....

                      Comment


                      • #12
                        Originally posted by duby229 View Post
                        It was a valid response to a blanket statement....
                        do you miss the *scnr*?

                        Comment


                        • #13
                          Originally posted by przemoli View Post
                          Well, AMD could host farm of GPU's of various generations/configs. With some automated tools that would allow previlaged devs to submit patches to test performance/piglit regresions.

                          QA would also had easier time (if dev's could detect bad code earlier).


                          Great idea, so why its not already done?
                          What automated tools would you suggest? There weren't any piglit tests which triggered the issue. Unfortunately, automated testing is often a challenge for GPU development. For best results you generally need physical access to the hardware.

                          Comment


                          • #14
                            Originally posted by Nille View Post
                            Or Stop developing the Driver in the Kernel Tree. For an user its much more easy to update an out of tree kernel module. To tell an user he has to recompile his kernel is is a mess.
                            And for developers it is much more of a mess to maintain an out of kernel module, which will also result in much less out of the box support from the driver, which is much of the point of R600g. If you want a module that is released outside of the kernel use one of the proprietary binary blobs like Catalyst - and then see how well that works out.

                            Comment


                            • #15
                              Originally posted by Hamish Wilson View Post
                              And for developers it is much more of a mess to maintain an out of kernel module, which will also result in much less out of the box support from the driver, which is much of the point of R600g. If you want a module that is released outside of the kernel use one of the proprietary binary blobs like Catalyst - and then see how well that works out.
                              And the main reason for that mess is that Linux kernel don't provide stable kernel ABI and API. If kernel interfaces are stable you can develop drivers with just kernel headers available.

                              Comment

                              Working...
                              X