[PATCH v3 7/7] [wip] virtio-gpu: add page flip support
Gerd Hoffmann
kraxel at redhat.com
Tue May 31 07:34:45 UTC 2016
Hi,
> > Right now the virtual outputs are linked to drm_crtc. To apply any
> > changes I need to lookup the crtc to figure which virtual output should
> > be updated.
> > So, setting active_only should make sure I have a valid crtc pointer on
> > plane updates, right? It probably also skips the disable + enable crtc
> > steps on commit? What happens when outputs are disabled?
> Nah, I just misunderstood your patch. If it's all about finding the
> corresponding crtc, then you're all good.
Yes, it's all about finding the crtc.
> I thought there was some
> other reason (like the virtual hw getting upset about certain things).
virtio wouldn't be upset.
It's a pointless exercise though to first disable the output, just to
re-enable it the next moment with the new page-flipped framebuffer. So
I guess I should look at the active_only thing nevertheless.
> btw can you pls drop an ack or r-b
> onto my virtio conversion? I already added your tested-by.
Grr, mail is not in my dri-devel folder. Guess that is the <censored>
"avoid-duplicates" mailman option at work.
Feel free to just add the r-b too. Or I'll send it for the next version
of the series.
cheers,
Gerd
More information about the dri-devel
mailing list