Hi, On Fri, 26 Sept 2025 at 15:45, Sebastian Wick <[email protected]> wrote: > So I'm going to argue that making the properties read-only or read-write is > useless. > > The only case where knowing the color pipeline of the previous user would be > useful is if you want to re-use the framebuffer of said user. Otherwise, the > color pipeline and the generated framebuffer have to somehow just match to > produce the desired output and that does not require any previous state, > making the legacy properties useless.
I don't think it's useless; if nothing else, drm_info is a thing and having it work is nice. > If we genuinely believe that this is something to be supported, then my > question is why the new color pipeline should not be able to accurate reflect > the state of the previous user, even if they used the legacy props? That's reasonable. My hunch was that it would be too much code in the kernel to essentially just do format reinterpretation on userspace's behalf. > The hardware was able to get into some state based on the legacy props, so it > will be able to get into the same state with the color pipeline props; it's > "just" a matter of exposing the right pipeline. > > If we are not able to accurate reflect the previous state with the pipeline > props, then use space will see inconsistent state between the legacy and > color pipeline props. Which state is the right one? We cannot know. The > previous user could have used either one. So having the legacy props does not > help because we don't know if we should use them or the pipeline state. > > So, I would argue that we should *remove* the legacy props if > DRM_CLIENT_CAP_POST_BLEND_COLOR_PIPELINE is set. If the handover is relevant > for a driver, they should ensure the legacy props state translates to the > correct color pipeline state. FWIW, the usecase I can see in mind would be doing a fade-style transition between the old and new clients. But I don't really care too strongly about it to be honest; I mostly care about having drm_info work because it's a super-useful tool. Cheers, Daniel
