On Wed, 21 Feb 2024 at 21:05, Lucas De Marchi lucas.demarchi@intel.com wrote:
this has a potential to cause conflicts with upcoming work, so I think it's better to apply this through drm-xe-next. Let me know if you agree.
I disagree. Violently.
For this to be fixed, we need to have the printf format checking enabled.
And we can't enable it until all the problems have been fixed.
Which means that we should *not* have to wait for [N] different trees to fix their issues separately.
This should get fixed in the Kunit tree, so that the Kunit tree can just send a pull request to me to enable format checking for the KUnit tests, together with all the fixes. Trying to spread those fixes out to different git branches will only result in pain and pointless dependencies between different trees.
Honestly, the reason I noticed the problem in the first place was that the drm tree had a separate bug, that had been apparently noted in linux-next, and *despite* that it made it into a pull request to me and caused new build failures in rc5.
So as things are, I am not IN THE LEAST interested in some kind of "let us fix this in the drm tree separately" garbage. We're not making things worse by trying to fix this in different trees.
We're fixing this in the Kunit tree, and I do not want to get *more* problems from the drm side. I've had enough.
Linus