Re: [spice-gtk PATCH] Handle single headed monitors that have a non-zero x, y config offset

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hey,

On Mon, Jun 13, 2016 at 07:09:29PM +0200, Marc-André Lureau wrote:
> >> On Thu, Jun 18, 2015 at 07:14:13PM -0400, sstutsma@xxxxxxxxxx wrote:
> >> > From: Sandy Stutsman <sstutsma@xxxxxxxxxx>
> >> >
> >> > Each monitor on a Windows guest is represented as a separate, single-headed
> >> > device with its own framebuffer.  When there are multiple monitors, all
> >> > monitors but one will have a non-zero xy config position.  But even in
> >> > these cases the whole area (frame-buffer) of each monitor should be
> >> > updated.
> >> >
> >> > Addresses: https://bugzilla.redhat.com/show_bug.cgi?id=1202419


> 
> That's not compatible with virgl, which may use a monitor scanout
> different than +0+0 (in multimonitor for ex, but one monitor per
> display channel).
> 
> I could probably workaround the issue with an extra test such as if
> (monitors->len == 1 && !d->egl.enabled)... But that condition looks
> wrong to me anyway.
> 
> My understanding is that there is some confusing between monitor
> configuration position (the monitor config on main channel), and the
> scanout/primary position (the monitor config on display channel).

To be honest, I'm not sure about the difference between these... Is this
documented anywhere?
If you mean that the main channel monitor configuration message should
describe the overall monitor layout (including the (x, y) position) of
all monitors across all display channels, and then the display channel
monitor configuration message only describes the offset of the
monitor(s) within the scanout of the current display channel, then yes,
the agent changes corresponding to that patch were probably wrong.

Christophe

Attachment: signature.asc
Description: PGP signature

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel

[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]