On 06/28/2013 03:06 AM, Hans de Goede wrote:
Before this patch-set virt-viewer was calling spice_session_has_channel_type(
session, SPICE_CHANNEL_USBREDIR) from the session-initialized signal handler,

So as soon as the display channel gets added to the session, the check was
done. This causes the check to return FALSE for usbredir capable vms if
the usbredir channel(s) get added to the session after the display channed.

This patch refactors things to not depend on channel creation order.

Signed-off-by: Hans de Goede <[email protected]>
---
  src/virt-viewer-app.c           | 18 +++++++++++++-----
  src/virt-viewer-session-spice.c | 28 +++++++++++++++-------------
  src/virt-viewer-session.c       | 39 ++++++++++++++++++++++++++++++++-------
  src/virt-viewer-session.h       |  5 +++--
  4 files changed, 63 insertions(+), 27 deletions(-)


This patch fixed my issue.
And I noticed that you committed b85ca792 in spice-gtk which makes spice_channel_string_to_type public. This works well too.
>>> SpiceClientGLib.Channel.string_to_type('usbredir')
9
Thanks for both. It's time to send my v3 patch for virt-manager.

Guannan

_______________________________________________
virt-tools-list mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/virt-tools-list

Reply via email to