Re: [Spice-devel] guest tools for Win 7 64 bit

2013-01-08 Thread flooding Controlled
Do you resolve it now?

Thanks.


2012/7/6 Fernando Ferrer 

> Hi there;
>
>  I'm a avid user who had a few Windows 32 bits guest VM's installed; I
> recently installed a Win 7 64 bit guest VM and I'm not able to install the
> graphic adapter or the driver for the sound device. I get the famous code
> 52 "can't verified signature" error. Could you advise me how to proceed? is
> there another driver for 64 bit guests?
>
>  Thank you.
>
> Fernando.
>
> ___
> Spice-devel mailing list
> Spice-devel@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/spice-devel
>
>
___
Spice-devel mailing list
Spice-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/spice-devel


Re: [Spice-devel] [PATCH spice-gtk V5 0/3] simply implement of file transfer

2013-01-08 Thread Dunrong Huang
Marc-André: ping? Are you happy with these patches, can they be applied?

On Sat, Jan 5, 2013 at 6:59 PM, Dunrong Huang  wrote:
> These are the fifth version of patches which implement file transfer from
> client to guest.
>
> V4 -> V5:
>* Only update first patch
>* Imporve error report
>* Add spice_main_file_copy_finish() function.
>* Some code cleanup. (e.g. file_xfer_task_free())
>* Limit spice_main_file_copy() to a single file.
>
> V3 -> V4:
>* Address Marc-André's comments
>* s/spice_main_file_xfer/spice_main_file_copy
>* Use a new algorithm to send data, instead of g_idle_add().
>* Use gio async API to open, read, query files
>
> V2 -> V3:
>* s/drag-and-drop/file-xfer
>* Use key-value text as data included in start message
>
> V1 -> V2:
>* Address Marc-André and hans's comments, most of the code are rewritten.
>* Move all file operation to the channel level
>* Use glib API for file operations
>* spice protocol related file transfer has been extended for the future's 
> work
>* Only export a single API spice_main_file_xfer()
>
> Dunrong Huang (3):
>   file-xfer: handling various transfer messages in main channel
>   file-xfer: handle "drag-data-received" signal
>   file-xfer: disable file-xfer when agent is not connected
>
>  gtk/channel-main.c  | 515 
> 
>  gtk/channel-main.h  |  12 ++
>  gtk/map-file|   2 +
>  gtk/spice-glib-sym-file |   2 +
>  gtk/spice-widget.c  |  46 +
>  5 files changed, 577 insertions(+)
>
> --
> 1.8.0
>



-- 
Best Regards,

Dunrong Huang
___
Spice-devel mailing list
Spice-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/spice-devel


Re: [Spice-devel] [PATCH spice-html5] Add spice_auto.html for automated console display

2013-01-08 Thread Jeremy White
On 01/08/2013 03:44 AM, Daniel P. Berrange wrote:
> From: "Daniel P. Berrange" 
> 
> Add a spice_auto.html which removes the form inputs and obtains
> connection details from query parameters. The optional 'token'
> parameter is an opaque string passed to the websockets server
> to allow it to auto-determine the correct console. Otherwise
> the host/port/password/etc can be provided explicitly.
> 
> This is sufficient to integrate SPICE HTML5 with OpenStack

Couldn't this all be done as an optional feature of spice.html?

That is, auto detect the inbound parameters and switch the behavior of
the page?  It seems a shame to have two largely duplicate entry pages.

Cheers,

Jeremy
___
Spice-devel mailing list
Spice-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/spice-devel


[Spice-devel] [spice-server PATCH 2/2] red_worker.c: clearing the stream vis_region, after it has been detached

2013-01-08 Thread Yonit Halperin
The stream vis_region should be cleared after the stream region was sent
to the client losslessly. Otherwise, we might send redundant stream upgrades
if we process more drawables that are dependent on the stream region.
---
 server/red_worker.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/server/red_worker.c b/server/red_worker.c
index 5e00cb6..085e3e6 100644
--- a/server/red_worker.c
+++ b/server/red_worker.c
@@ -2646,7 +2646,7 @@ static inline void 
red_display_detach_stream_gracefully(DisplayChannelClient *dc
 spice_debug("stream %d: upgrade by linked drawable. sized %d, box 
==>",
 stream_id, stream->current->sized_stream != NULL);
 rect_debug(&stream->current->red_drawable->bbox);
-return;
+goto clear_vis_region;
 }
 spice_debug("stream %d: upgrade by drawable. sized %d, box ==>",
 stream_id, stream->current->sized_stream != NULL);
@@ -2680,7 +2680,8 @@ static inline void 
red_display_detach_stream_gracefully(DisplayChannelClient *dc
 }
 red_add_surface_area_image(dcc, 0, &upgrade_area, NULL, FALSE);
 }
-
+clear_vis_region:
+region_clear(&agent->vis_region);
 }
 
 static inline void red_detach_stream_gracefully(RedWorker *worker, Stream 
*stream,
-- 
1.8.0.1

___
Spice-devel mailing list
Spice-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/spice-devel


[Spice-devel] [spice-server PATCH 1/2] red_worker.c: insert a drawable to its position in the current tree before calling red_detach_streams_behind

2013-01-08 Thread Yonit Halperin
resolves: rhbz#891326

Starting from commit 81fe00b08ad4f, red_detach_streams_behind can
trigger modifications in the current tree (by update_area calls). Thus,
after calling red_detach_streams_behind it is not safe to access tree
entries that were calculated before the call.
This patch inserts the drawable to the tree before the call to
red_detach_streams_behind. This change also requires making sure
that rendering operations that can be triggered by
red_detach_streams_behind will not include this drawable (which is now part of 
the tree).
---
 server/red_worker.c | 55 -
 1 file changed, 42 insertions(+), 13 deletions(-)

diff --git a/server/red_worker.c b/server/red_worker.c
index 1a9c375..5e00cb6 100644
--- a/server/red_worker.c
+++ b/server/red_worker.c
@@ -1022,6 +1022,8 @@ static void red_current_flush(RedWorker *worker, int 
surface_id);
 #else
 static void red_draw_drawable(RedWorker *worker, Drawable *item);
 static void red_update_area(RedWorker *worker, const SpiceRect *area, int 
surface_id);
+static void red_update_area_till(RedWorker *worker, const SpiceRect *area, int 
surface_id,
+ Drawable *last);
 #endif
 static void red_release_cursor(RedWorker *worker, CursorItem *cursor);
 static inline void release_drawable(RedWorker *worker, Drawable *item);
@@ -2615,7 +2617,9 @@ static int 
red_display_drawable_is_in_pipe(DisplayChannelClient *dcc, Drawable *
  * after red_display_detach_stream_gracefully is called for all the display 
channel clients,
  * red_detach_stream should be called. See comment (1).
  */
-static inline void red_display_detach_stream_gracefully(DisplayChannelClient 
*dcc, Stream *stream)
+static inline void red_display_detach_stream_gracefully(DisplayChannelClient 
*dcc,
+Stream *stream,
+Drawable 
*update_area_limit)
 {
 int stream_id = get_stream_id(dcc->common.worker, stream);
 StreamAgent *agent = &dcc->stream_agents[stream_id];
@@ -2669,27 +2673,41 @@ static inline void 
red_display_detach_stream_gracefully(DisplayChannelClient *dc
 spice_debug("stream %d: upgrade by screenshot. has current %d. box 
==>",
 stream_id, stream->current != NULL);
 rect_debug(&upgrade_area);
-red_update_area(dcc->common.worker, &upgrade_area, 0);
+if (update_area_limit) {
+red_update_area_till(dcc->common.worker, &upgrade_area, 0, 
update_area_limit);
+} else {
+red_update_area(dcc->common.worker, &upgrade_area, 0);
+}
 red_add_surface_area_image(dcc, 0, &upgrade_area, NULL, FALSE);
 }
 
 }
 
-static inline void red_detach_stream_gracefully(RedWorker *worker, Stream 
*stream)
+static inline void red_detach_stream_gracefully(RedWorker *worker, Stream 
*stream,
+Drawable *update_area_limit)
 {
 RingItem *item;
 DisplayChannelClient *dcc;
 
 WORKER_FOREACH_DCC(worker, item, dcc) {
-red_display_detach_stream_gracefully(dcc, stream);
+red_display_detach_stream_gracefully(dcc, stream, update_area_limit);
 }
 if (stream->current) {
 red_detach_stream(worker, stream, TRUE);
 }
 }
 
-// region should be a primary surface region
-static void red_detach_streams_behind(RedWorker *worker, QRegion *region)
+/*
+ * region  : a primary surface region. Streams that intersects with the given
+ *   region will be detached.
+ * drawable: If detaching the stream is triggered by the addition of a new 
drawable
+ *   that is dependent on the given region, and the drawable is 
already a part
+ *   of the "current tree", the drawable parameter should be set with
+ *   this drawable, otherwise, it should be NULL. Then, if detaching 
the stream
+ *   involves sending an upgrade image to the client, this drawable 
won't be rendered
+ *   (see red_display_detach_stream_gracefully).
+ */
+static void red_detach_streams_behind(RedWorker *worker, QRegion *region, 
Drawable *drawable)
 {
 Ring *ring = &worker->streams;
 RingItem *item = ring_get_head(ring);
@@ -2706,7 +2724,7 @@ static void red_detach_streams_behind(RedWorker *worker, 
QRegion *region)
 StreamAgent *agent = &dcc->stream_agents[get_stream_id(worker, 
stream)];
 
 if (region_intersects(&agent->vis_region, region)) {
-red_display_detach_stream_gracefully(dcc, stream);
+red_display_detach_stream_gracefully(dcc, stream, drawable);
 detach_stream = 1;
 spice_debug("stream %d", get_stream_id(worker, stream));
 }
@@ -2798,7 +2816,7 @@ static inline void red_handle_streams_timout(RedWorker 
*worker)
 Stream *stream = SPICE_CONTAINEROF(item, Stream, link);
 item = ring_next(ring, item);
 if (no

Re: [Spice-devel] Fedora 17 Guests for Desktop Virtualization

2013-01-08 Thread Nahum Shalman
Below I've pasted a stripped down version of a kickstart file I used to 
get Fedora 17 working as a spice VM.

(original is at http://www.shalman.org/spice/fedora17.ks)

It gave me a working KDE VM.

A plain install of Fedora 17 seemed to freeze for me, but I haven't done 
any digging into the differences between that plain install and the 
stuff in this kickstart file (modified from an older version for Fedora 16).


Kevin, perhaps you could give this kickstart file (or the full original) 
a try and see if that helps (the install will still prompt for a few 
details)?


As an aside, it might be interesting for there to be a yum group that 
installs all the right packages for a VM that uses spice so that someday 
one could run "yum groupinstall spicevm" or put "@spicevm" in a 
kickstart file and have everything "Just Work"...


-Nahum

--

install
url --url=http://mirrors.us.kernel.org/fedora/releases/17/Fedora/x86_64/os/
reboot
lang en_US.UTF-8
keyboard us
network --onboot yes --device eth0 --bootproto dhcp --noipv6
authconfig --enableshadow --passalgo=sha512
selinux --disabled
firewall --disabled

repo --name="Fedora 17" 
--baseurl=http://mirrors.us.kernel.org/fedora/releases/17/Fedora/x86_64/os/
repo --name="Fedora 17 updates" 
--baseurl=http://mirrors.us.kernel.org/fedora/updates/17/x86_64/


%packages
@Base
@Fonts
@Core
@online-docs
xorg-x11-drv-qxl
xorg-x11-drv-evdev
spice-vdagent
kdm
konsole
kde-settings-pulseaudio
kdemultimedia-kmix
kwin-gles
xorg-x11-xauth
xorg-x11-xinit
xorg-x11-utils
xorg-x11-server-Xorg
xorg-x11-server-utils
xorg-x11-fonts-*
xorg-x11-font-utils
firstboot
smolt-firstboot
less
vim
openssh-clients
openssh-server
pulseaudio
phonon-backend-gstreamer
libusb*
usbutils*
usbredir*
%end
___
Spice-devel mailing list
Spice-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/spice-devel


[Spice-devel] Announcing spice-vdagent 0.12.1

2013-01-08 Thread Hans de Goede

Hi All,

I'm happy to announce a new bugfix release
of the spice Linux agent in the stable spice-0.12.x series.

Changes in spice-vdagent-0.12.1:

* Various bugfixes for multi-monitor and arbritary resolution support
* Requires libXrandr >= 1.3, Note 0.12.0 also required this, but did not
  check for it. For older distributions use 0.10.1

You can download source tarbals here:
http://spice-space.org/download.html

Regards,

Hans
___
Spice-devel mailing list
Spice-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/spice-devel


Re: [Spice-devel] Questions about graphic driver

2013-01-08 Thread LazySid
I think I got the answer by myself. In the Sources file of mydisplay project, I 
write this:
TARGETLIBS=$(DDK_LIB_PATH)\ntoskrnl.lib
but it should be:
TARGETLIBS=$(DDK_LIB_PATH)\ntstrsafe.lib

The DrvEnableDriver() in mydisplay.dll is successfully called now.

Hope this information could help some newbies.

Regards,

LazySid

At 2013-01-08 11:56:49,LazySid  wrote:
>I have my graphic driver installed on windows xp sp3. After reboot, I got a 
>popup dialog box said "Cann't use current graphic driver, it is designed for 
>the old version, it is not compatible with current windows version". In 
>windbg, I can see FindAdapter(), Initialize() and GetChildDescriptor() in 
>miniport.sys are called, and only these three functions are called.
>
>When I change InstalledDisplayDrivers to cirrus in inf file, the popup dialog 
>box doesn't show, and StartIO() in miniport.sys also called.
>
>There seems to be some differences between cirrus.dll and mydisplay.dll.
>
>How does the OS recognise mydisplay.dll version?
>
>Thanks
>
>Regards
>
>LazySid
>___
>Spice-devel mailing list
>Spice-devel@lists.freedesktop.org
>http://lists.freedesktop.org/mailman/listinfo/spice-devel
___
Spice-devel mailing list
Spice-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/spice-devel


Re: [Spice-devel] [Users] Fedora 17 Guests for Desktop Virtualization

2013-01-08 Thread David Jaša
Hi Kevin,

Itamar Heim píše v Ne 06. 01. 2013 v 17:25 +0200:
> On 01/06/2013 04:52 PM, Kevin Daly wrote:
> > I am trying to set up Fedora 17 guests and I've come across a problem
> > with Spice.. It works fine with a Gnome 3 sesson, but if I try et
> > Cinnamon or KDE4, the bottom panel is unresponsive.. You cannot click on
> > anything in the panel, cannot configure it etc etc...
> >
> > With KDE4, I have configured a panel at the top, it works fine.. but if
> > I try move it to the bottom it's dead.
> >
> > Any insights would be helpful.

1) does KDE Plasma Failsafe session work fine?

2) in Gnome 3, does bottom notification panel work?

David

> >
> > I am running vdagent and vdagentd on the Fedora guest.
> >
> > --
> > 
> > Kevin Daly
> >
> >
> > ___
> > Users mailing list
> > us...@ovirt.org
> > http://lists.ovirt.org/mailman/listinfo/users
> >
> 
> adding spice-devel
> 
> ___
> Users mailing list
> us...@ovirt.org
> http://lists.ovirt.org/mailman/listinfo/users

-- 

David Jaša, RHCE

SPICE QE based in Brno
GPG Key: 22C33E24 
Fingerprint: 513A 060B D1B4 2A72 7F0D 0278 B125 CD00 22C3 3E24



___
Spice-devel mailing list
Spice-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/spice-devel


Re: [Spice-devel] [PATCH vdagent 2/2] vdagentd: Fix building warning

2013-01-08 Thread Hans de Goede

Hi,

Thanks, applied.

Regards,

Hans

On 12/05/2012 09:01 AM, Dunrong Huang wrote:

Move some variable declarations to right place.

Signed-off-by: Dunrong Huang 
---
  src/vdagentd.c | 12 
  1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/src/vdagentd.c b/src/vdagentd.c
index 7006d80..a51ac4c 100644
--- a/src/vdagentd.c
+++ b/src/vdagentd.c
@@ -510,10 +510,9 @@ void release_clipboards(void)

  void update_active_session_connection(void)
  {
-struct udscs_connection *new_conn = NULL;
-int n;
-
  #ifdef HAVE_SESSION_INFO
+int n;
+struct udscs_connection *new_conn = NULL;
  if (!active_session)
  active_session = session_info_get_active_session(session_info);

@@ -541,7 +540,9 @@ void update_active_session_connection(void)

  void agent_connect(struct udscs_connection *conn)
  {
+#ifdef HAVE_SESSION_INFO
  uint32_t pid;
+#endif
  struct agent_data *agent_data;

  agent_data = calloc(1, sizeof(*agent_data));
@@ -707,7 +708,10 @@ void daemonize(void)
  void main_loop(void)
  {
  fd_set readfds, writefds;
-int n, nfds, ck_fd = 0;
+int n, nfds;
+#ifdef HAVE_SESSION_INFO
+int ck_fd = 0;
+#endif

  while (!quit) {
  FD_ZERO(&readfds);


___
Spice-devel mailing list
Spice-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/spice-devel


Re: [Spice-devel] [PATCH vdagent 1/2] vdagent-x11-randr: Fix building warning

2013-01-08 Thread Hans de Goede

Hi,

Thanks, applied minus the pointer casts. I'll write a better fix for that 
myself.

Regards,

Hans


On 12/05/2012 09:01 AM, Dunrong Huang wrote:

This patch can be safely applied since it just deletes unused variables.

Signed-off-by: Dunrong Huang 
---
  src/vdagent-x11-randr.c | 8 +---
  1 file changed, 1 insertion(+), 7 deletions(-)

diff --git a/src/vdagent-x11-randr.c b/src/vdagent-x11-randr.c
index 2c1819a..e171e7a 100644
--- a/src/vdagent-x11-randr.c
+++ b/src/vdagent-x11-randr.c
@@ -204,9 +204,6 @@ static void delete_mode(struct vdagent_x11 *x11, int 
output_index, const char* n
  int m;
  XRRModeInfo *mode;
  XRROutputInfo *output_info;
-XRRCrtcInfo *crtc_info;
-RRCrtc crtc;
-int current_mode = -1;

  output_info = x11->randr.outputs[output_index];
  if (output_info->ncrtc != 1) {
@@ -214,9 +211,6 @@ static void delete_mode(struct vdagent_x11 *x11, int 
output_index, const char* n
 "failed to delete mode", output_info->ncrtc);
  return;
  }
-crtc_info = crtc_from_id(x11, output_info->crtcs[0]);
-current_mode = crtc_info->mode;
-crtc = output_info->crtc;
  for (m = 0 ; m < x11->randr.res->nmode; ++m) {
  mode = &x11->randr.res->modes[m];
  if (strcmp(mode->name, name) == 0)
@@ -689,7 +683,7 @@ void vdagent_x11_set_monitor_config(struct vdagent_x11 *x11,
  if (!xrandr_add_and_set(x11, i, x, y, width, height) &&
  mon_config->num_of_monitors == 1) {
  set_screen_to_best_size(x11, width, height,
-&primary_w, &primary_h);
+(int *)&primary_w, (int *)&primary_h);
  goto update;
  }
  }


___
Spice-devel mailing list
Spice-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/spice-devel


[Spice-devel] [PATCH spice-html5] Add spice_auto.html for automated console display

2013-01-08 Thread Daniel P. Berrange
From: "Daniel P. Berrange" 

Add a spice_auto.html which removes the form inputs and obtains
connection details from query parameters. The optional 'token'
parameter is an opaque string passed to the websockets server
to allow it to auto-determine the correct console. Otherwise
the host/port/password/etc can be provided explicitly.

This is sufficient to integrate SPICE HTML5 with OpenStack

Signed-off-by: Daniel P. Berrange 
---
 spice_auto.html | 153 
 1 file changed, 153 insertions(+)
 create mode 100644 spice_auto.html

diff --git a/spice_auto.html b/spice_auto.html
new file mode 100644
index 000..a27b52d
--- /dev/null
+++ b/spice_auto.html
@@ -0,0 +1,153 @@
+
+
+
+
+
+
+Spice Javascript client
+ 
+ 
+ 
+ 
+ 
+ 
+ 
+ 
+ 
+ 
+ 
+ 
+ 
+ 
+ 
+ 
+
+
+
+
+
+
+
+
+
+var host = null, port = null;
+var sc;
+
+function spice_set_cookie(name, value, days) {
+var date, expires;
+date = new Date();
+date.setTime(date.getTime() + (days*24*60*60*1000));
+expires = "; expires=" + date.toGMTString();
+document.cookie = name + "=" + value + expires + "; path=/";
+};
+
+function spice_query_var(name, defvalue) {
+var match = RegExp('[?&]' + name + '=([^&]*)')
+  .exec(window.location.search);
+return match ?
+decodeURIComponent(match[1].replace(/\+/g, ' '))
+: defvalue;
+}
+
+function spice_error(e)
+{
+disconnect();
+}
+
+function connect()
+{
+var host, port, password, scheme = "ws://", uri;
+
+// By default, use the host and port of server that served 
this file
+host = spice_query_var('host', window.location.hostname);
+port = spice_query_var('port', window.location.port);
+
+// If a token variable is passed in, set the parameter in a 
cookie.
+// This is used by nova-spiceproxy.
+token = spice_query_var('token', null);
+if (token) {
+spice_set_cookie('token', token, 1)
+}
+
+password = spice_query_var('password', '');
+path = spice_query_var('path', 'websockify');
+
+if ((!host) || (!port)) {
+console.log("must specify host and port in URL");
+return;
+}
+
+if (sc) {
+sc.stop();
+}
+
+uri = scheme + host + ":" + port;
+
+try
+{
+sc = new SpiceMainConn({uri: uri, screen_id: 
"spice-screen", dump_id: "debug-div", 
+message_id: "message-div", password: password, 
onerror: spice_error });
+}
+catch (e)
+{
+alert(e.toString());
+disconnect();
+}
+
+}
+
+function disconnect()
+{
+console.log(">> disconnect");
+if (sc) {
+sc.stop();
+}
+console.log("<< disconnect");
+}
+
+connect();
+
+
+
+
+
+
+
+SPICE
+
+
+
+
+
+
+
+
+
+
+
+
+
+
-- 
1.7.11.7

___
Spice-devel mailing list
Spice-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/spice-devel