Radv: Reduce CPU overhead of draw command recording Code | Hacker News Reader

Radv: Reduce CPU overhead of draw command recording Code

supergoodcode.com

52 pointspantalaimon10 comments

The pasta absurdity obscures the author's attempt at communication.

Agreed, looks interesting enough that I should care, but having a hard time to...

This post is really difficult to read due to the weird metaphor, which is unfortunate because it seems otherwise interesting.

There are two competing GPU drivers. One of them was winning a benchmark. Now the other one, which Mike works on, is winning a benchmark. Mostly through not doing work you don't need to do, and ALWAYS_INLINE (inline heuristics in most compilers are really simple and usually pretty awful).

Not sure where the pasta methaphor came from, but I generally like the author's style, and the performance gains are intriguing. I'd be curious what the actual impact is on real-world Vulkan/zink test-cases?

Apart from all the childish spaghetti talk, etc. i am wondering if all the removal of code and inlining of functions has some negative side effects (on maintainability, stability, bugs).

The style of the post makes me think this is some child relentlessly removing structure and security checks...

edit: at least the function was not inlined in code but with some compiler flag.

> The style of the post makes me think this is some child relentlessly removing structure and security checks...

Mike is far from a child and his contributions to Mesa speak for themselves. It’s simply his style of writing for these blog posts.

I see. Good to know. Still, that's the impression for outsiders.

200% outsider, and not my impression at all, I largely don't see how "quirky" has to translate to "childish". I could follow the post just well, but maybe my mind was infected by _why's general style.

Please, allow me to have one more round of chunky bacon.

> bugs

Unsurprisingly it did: https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/18...

But that's what code review and testing is for :).

Reply on news.ycombinator.com