Announcement
Collapse
No announcement yet.
AMDGPU DC Pull Request Submitted For Linux 4.15: Finally The New Display Stack
Collapse
X
-
Originally posted by L_A_G View PostI wonder what David is going to come up this time to stop DAL/DC from being merged? Not enough time to properly review the code? Too much redundant code because they're not cleaning away all the old AMD display code? Still too much abstraction going on? Code quality not up to standards?
The last one would be particularly eyebrow raising remembering the state in which he merged TinyDRM and (rightfully) found himself on the receiving end of one of Linus' rants where he called it "absolute pure shit that has never seen a compiler".
Your proof that I can't judge DAL code quality is that I merged tinydrm? Even though DAL code quality on any level is way worse than tinydrm.
so should I merge it or not?
Dave.
- Likes 9
Comment
-
Originally posted by airlied View Post
Lols, tinydrm had seen lots of compilers, Linus was just being an idiot. but I don't get your logic (possibly because the concept of logic it outside your grasp).
Your proof that I can't judge DAL code quality is that I merged tinydrm? Even though DAL code quality on any level is way worse than tinydrm.
so should I merge it or not?
Dave.
EDIT: What about putting it in staging? Is that a possibility?
Comment
-
-
It's really this problem right here.
Phoronix: Nouveau Developers Remain Blocked By NVIDIA From Advancing Open-Source Driver Longtime Nouveau contributors Martin Peres and Karol Herbst presented
That's not gonna get resolved until it gets upstream. I think AMD is trying really hard and they seem like they are doing the best they can right now. But these are really difficult problems and they don't have enough manpower to keep this out of the kernel.
Comment
-
Originally posted by airlied View Post
Lols, tinydrm had seen lots of compilers, Linus was just being an idiot. but I don't get your logic (possibly because the concept of logic it outside your grasp).
Your proof that I can't judge DAL code quality is that I merged tinydrm? Even though DAL code quality on any level is way worse than tinydrm.
so should I merge it or not?
Dave.
On the other hand: 1. I am looking forward for Vega, although I didn't bought one yet (waiting for custom designs). Especially for the MxGPU feature so I can have Linux as host and have a bunch of VMs for the gaming kids (and me). So finally having Linux as main system (will switch at work to Linux soon, too. Thank you Microsoft for Windows 10 :-) ).
2. Furthermore Linus rants are always fun to read.
3. And finally: as a long term AMD stock owner (bought 2007 with hope of a great new CPU), a long term AMD user (I am still waiting for the promised feature of combining the 690G onboard GPU with a discrete GPU and being able to switch when needed (like the laptops are doing)) and a soon-to-be Ryzen 7 owner, I vote for a merge (which makes point 1 double in value).
So it's 3:1 for a merge.
- Likes 4
Comment
-
I'm staying out of that mess... Anyone know if amdgpu.dc=1 is valid for the old DC stack (or if it will be ignored for the DC stack I have already in kernel 4.12 (from mBab). I have a Carrizo based notebook that will not work at all with out DC and want to be sure I can preset that in advance.
Comment
-
Originally posted by airlied View Post
Lols, tinydrm had seen lots of compilers, Linus was just being an idiot. but I don't get your logic (possibly because the concept of logic it outside your grasp).
Your proof that I can't judge DAL code quality is that I merged tinydrm? Even though DAL code quality on any level is way worse than tinydrm.
so should I merge it or not?
Dave.
If not - nah, it's need more work.
- Likes 1
Comment
Comment