Re: [PATCH vdagent-linux] vdagentd: send monitors config to currently active agent only

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

 



Hi

----- Mensaje original -----
> > +    /* Send monitor config to currently active agent */
> > +    udscs_write(active_session_conn, VDAGENTD_MONITORS_CONFIG, 0,
> > 0,
> > +                (uint8_t *)mon_config, size);
> >
> >       /* Acknowledge reception of monitors config to spice server /
> >       client */
> >       reply.type  = VD_AGENT_MONITORS_CONFIG;
> >
> 
> Good catch! This may very well explain much if the issues we've been
> seeing. But this needs some
> more work, we need to also send an (potentially updated)
> monitor-config to the
> new active-agent when the active session changes, ie, we've 2 agents
> running for 2 sessions
> not sharing the same X-server, ie using user switching, then when we
> switch to the other user,
> the agent should be notified so that the resolution of the other
> X-server gets adjusted too.

I was thinking to rely on X events for that, if they exist. Also, I think we shouldn't wait for that extra work to apply the above fix, since error is much more likely to happen currently than with the multiple session agents situation (and that might be a recoverable situation).
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://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]