Announcement

Collapse
No announcement yet.

Yet More AMDGPU DAL Patches This Week For Testing

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

  • Yet More AMDGPU DAL Patches This Week For Testing

    Phoronix: Yet More AMDGPU DAL Patches This Week For Testing

    It had been a few weeks since last seeing any new enablement patches for AMD's DAL display abstraction layer code, which is a big requirement for HDMI/DP audio, HDMI 2.0, potential FreeSync support, and also needed for next-generation GPUs. The lack of fresh DAL patches changed though this week when new patches were sent out and already another round of revising to this display code has now been mailed out for review...

    Phoronix, Linux Hardware Reviews, Linux hardware benchmarks, Linux server benchmarks, Linux benchmarking, Desktop Linux, Linux performance, Open Source graphics, Linux How To, Ubuntu benchmarks, Ubuntu hardware, Phoronix Test Suite

  • #2
    em, aren't these patches incremental? So thinks don't get better, just worse...

    Comment


    • #3
      The DAL code is down to 39 patches and yields 2744 insertions, 8390 deletions -- much better than originally when DAL was over 93k lines, which had lots of blank lines and comments but also too much abstractions that upstream DRM developers didn't appreciate.
      Trying to not be too snarky here but clearly the diffstat is only relative to the current DAL code so we're still talking about tens of thousands extra linues of code which should(?) be merged into Linus' tree.

      Comment


      • #4
        Originally posted by marvin42 View Post
        em, aren't these patches incremental? So thinks don't get better, just worse...
        Whoops, hadn't realized it and didn't look closely at the patches with not yet having my first cup of coffee for the day
        Michael Larabel
        https://www.michaellarabel.com/

        Comment


        • #5
          Originally posted by marvin42 View Post
          em, aren't these patches incremental? So thinks don't get better, just worse...
          Yes, clearly this is a not a stand-alone DAL pull request. But the diffstat itself is not too bad as they are deleting more lines than adding.

          Comment


          • #6
            Originally posted by schwarzman View Post

            Yes, clearly this is a not a stand-alone DAL pull request. But the diffstat itself is not too bad as they are deleting more lines than adding.
            right, 10k lines removed. Another 15-16 of such patch sets and this becomes pretty handy ;-)

            Comment


            • #7
              Progress is good! I am looking forward to seeing it land (as I am sure the AMD team are as well)

              Comment


              • #8
                Probably a bit of pressure to get work done at AMD considering they have a big launch happening very soon (Dec perhaps)

                Comment


                • #9
                  Originally posted by phoronix View Post
                  and also needed for next-generation GPUs
                  Next? Is the successor of GCN coming already?

                  Comment


                  • #10
                    According to Alex Deucher, looks like this code is nowhere near ready to be merged yet. They're uploading it to a mailing list just so it gets some more publicity, but generally it's merely the activity on their git tree. It's not up for review and not even an actual RFC. It's still a long way to go.

                    Comment

                    Working...
                    X