Originally posted by computerquip
View Post
Announcement
Collapse
No announcement yet.
OpenGL Shader Cache Support For RadeonSI Is Making Progress
Collapse
X
-
I am completely confused by Mesa development process. As far as I understand, it is: post patches, get review, push into the main repo. But I look the mailing list sometimes, and in reality it doesn't work like this. E.g. the disk shader patches were posted 23 January, and aside of 5 preliminary patches didn't get any review. Looks like a bad sign, but the same was for big gallium-nine patchset (in December?), which nobody didn't even bother to review, but it was pushed nonetheless. And then there're patches that getting review, but nobody pushes, or patches like this one that even should go into stable, but didn't get "reviewed by".
- Likes 3
Comment
-
Originally posted by Hi-Angel View PostI am completely confused by Mesa development process. As far as I understand, it is: post patches, get review, push into the main repo. But I look the mailing list sometimes, and in reality it doesn't work like this. E.g. the disk shader patches were posted 23 January, and aside of 5 preliminary patches didn't get any review. Looks like a bad sign, but the same was for big gallium-nine patchset (in December?), which nobody didn't even bother to review, but it was pushed nonetheless. And then there're patches that getting review, but nobody pushes, or patches like this one that even should go into stable, but didn't get "reviewed by".
devs use irc a lot in addition to the mailing list.
Patches don't require reviews to be pushed if the author is trusted and the affected code is his area of expertise (Especially if that's an area other devs are not confident with). Patches that may be controversial, or touch different areas need more to be reviewed.
A patch may be reviewed, and get pushed later in a bigger package. Also a review doesn't mean it's going to be pushed, for example if the author decides to do with another approach for the patch.
- Likes 2
Comment
-
Originally posted by computerquip View PostI was going to make a comment on how this shouldn't have much performance gain, if any at all. Then I came to learn that some games compile shaders in the middle of their game
Last edited by pal666; 04 February 2017, 12:15 PM.
Comment
-
Originally posted by Hi-Angel View PostWell, the patchset I linked in the prev. comment says "Hardware agnostic on-disk shader cache", so yes, it should work for r600 as well.
- Likes 2
Comment
-
Originally posted by adler187 View Post
Well the on-disk shader cache was originally only hooked up for i965. Now it's getting hooked up for radeonsi. I was just wondering if anyone was going to bother with doing the same for r600.
Comment
Comment