t to allow larger framebuffers,
second bugs out on too large primary create request, and third resets the
current_async to allow recovery from bad state via reset. I'm dropping the qxl
info patch for now, lack of time.
Alon
Alon Levy (3):
qxl: add fb_size_mb and fb_size
qxl: refuse to
Hi,
I am playing with a new Lenovo tablet, x220t, and I've come up against
what I think is a current gnome-control-center limitation, which I
would like to solve. Namely, when I rotate my main display, i.e. the
LVDS1 output that is the LCD panel of the laptop, the tablet doesn't
rotate with
On Sun, Jun 10, 2012 at 05:11:06PM +0200, Hans de Goede wrote:
> Hi,
>
> On 06/10/2012 04:44 PM, Alon Levy wrote:
>
>
>
> >>>Obligatory migration data:
> >>>-
> >>>(1) agent/spicevmc/smartcard write buffer. i.e., dat
On Sun, Jun 10, 2012 at 04:32:54PM +0200, Hans de Goede wrote:
> Hi,
>
> On 06/10/2012 11:05 AM, Yonit Halperin wrote:
> >Hi,
> >
> >As the qemu team rejected integrating spice connection migration in qemu
> >migration process, we remain with a solution that will involve libvirt, and
> >passing
On Sun, Jun 10, 2012 at 01:04:32PM +, Blue Swirl wrote:
> Link trace objects to fix these errors:
> LINK vscclient
> oslib-posix.o: In function `trace_qemu_vfree':
> /src/qemu/obj-amd64/./trace.h:39: undefined reference to `trace1'
> oslib-posix.o: In function `trace_qemu_memalign':
> /src/q
On Sat, Jun 09, 2012 at 06:16:25PM -0400, Marc-André Lureau wrote:
> Hi
>
> - Mensaje original -
>
> > I agree that we need to change it, but I find that page very
> > confusing.
> > According to point 2 at the bottom, since we are backward compatible
> > but
> > added interfaces, we shou
On Thu, Jun 07, 2012 at 03:35:00PM +0200, Marc-André Lureau wrote:
> On Thu, Jun 7, 2012 at 1:12 PM, Alon Levy wrote:
> > ---
> > NEWS | 27 +++
> > configure.ac | 4 ++--
> > 2 files changed, 29 insertions(+), 2 deletions(-)
>
On Sat, Jun 09, 2012 at 05:23:37PM +0300, Alon Levy wrote:
This is for xinput.. messed the $SUBJECT.
> Uses the rotation & translation currently set according to RandR.
>
> Signed-off-by: Alon Levy
> ---
> Tested with a x220t (lenovo tablet). The comments below "corre
On Thu, Jun 07, 2012 at 05:08:15PM -0500, Ian Pilcher wrote:
> On 06/07/2012 08:22 AM, Cole Robinson wrote:
> > I think virt-viewer supports it, and it wouldn't be a ton of work, so if any
> > enterprising person wanted to take a stab at a patch I'd be happy to help
> > shepherd it along.
>
> We c
Uses the rotation & translation currently set according to RandR.
Signed-off-by: Alon Levy
---
Tested with a x220t (lenovo tablet). The comments below "correct in working
zone", I believe are not related to the wrong matrix, you can see for instance
that 'RR_Rotate_270 |
---
NEWS | 27 +++
configure.ac |4 ++--
2 files changed, 29 insertions(+), 2 deletions(-)
diff --git a/NEWS b/NEWS
index 2deba57..5201bcb 100644
--- a/NEWS
+++ b/NEWS
@@ -1,3 +1,30 @@
+Major changes in 0.11.0:
+* !Development Release!
+* 8817549..d905a1f
+*
On Wed, Jun 06, 2012 at 03:03:37PM +0200, Gerd Hoffmann wrote:
> Hi,
>
> >> I literally have danpb's event introspection patches from Luiz's PULL
> >> request testing on my system right now to be pushed.
>
> Good.
>
> > this?: [PATCH 29/29] Add 'query-events' command to QMP to query async even
Fix copied from libvirt, commit by Eric Blake.
glibc 2.15 (on Fedora 17) coupled with explicit disabling of
optimization during development dies a painful death:
/usr/include/features.h:314:4: error: #warning _FORTIFY_SOURCE requires
compiling with optimization (-O) [-Werror=cpp]
cc1: all warn
On Wed, Jun 06, 2012 at 08:17:29PM +0800, Anthony Liguori wrote:
> On 06/06/2012 08:15 PM, Alon Levy wrote:
> >On Wed, Jun 06, 2012 at 08:08:40PM +0800, Anthony Liguori wrote:
> >>On 06/06/2012 08:01 PM, Yonit Halperin wrote:
> >>>On 06/06/2012 12:22 PM, Anthony Liguo
On Wed, Jun 06, 2012 at 08:08:40PM +0800, Anthony Liguori wrote:
> On 06/06/2012 08:01 PM, Yonit Halperin wrote:
> >On 06/06/2012 12:22 PM, Anthony Liguori wrote:
> >>
> >>So send a QMP event and call it a day.
> >>
> >Using a QMP event is making spice seamless migration dependent on libvirt
> >ver
On Sun, Jun 03, 2012 at 09:54:28PM +0200, Cedric Sodhi wrote:
> Hello,
>
> There is an EETI eGalax 0eef:a001 Touchscreen and a Wacpm 056a:0090
> ISDV4 stylus device.
>
> Evtest output for a simple "tap" on the eGALAX is attached.
>
> With Xorg 1.12.2, inputproto 2.2 on gentoo, "xinput list egala
On Wed, Jun 06, 2012 at 07:05:44PM +0800, Anthony Liguori wrote:
> On 06/06/2012 06:54 PM, Alon Levy wrote:
> >On Wed, Jun 06, 2012 at 05:22:21PM +0800, Anthony Liguori wrote:
> >>On 06/06/2012 05:10 PM, Yonit Halperin wrote:
> >>Spice client migration has nothing
On Wed, Jun 06, 2012 at 05:22:21PM +0800, Anthony Liguori wrote:
> On 06/06/2012 05:10 PM, Yonit Halperin wrote:
> >Hi,
> >
> >I would like to add some more points to Gerd's explanation:
> >On 06/05/2012 04:15 PM, Gerd Hoffmann wrote:
> >>Hi,
> >>
> >>>Absolutely not. This is hideously ugly and aff
On Tue, Jun 05, 2012 at 10:38:09AM -0500, Jeremy White wrote:
> > For instance, I use a french keyboard and I found issues for it on
> > spice-html.
>
> *blush* Yes, sorry, the keyboard code is truly awful. It's a matter of
> translating web key codes into scan codes; I didn't find a good way to
---
server/spice.h |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/server/spice.h b/server/spice.h
index 77aec92..a6b3c3d 100644
--- a/server/spice.h
+++ b/server/spice.h
@@ -111,7 +111,7 @@ struct QXLRect;
struct QXLWorker {
uint32_t minor_version;
uint32_t major
On Mon, Jun 04, 2012 at 08:42:17AM -0500, Jeremy White wrote:
> >> However, while the X server main loop has a facility that lets us add fds
> >> to it's
> >> select read mask, it has no such facility to let us add fds to it's
> >> *write* select mask.
> >
> > Right. This is the problem. I think
On Mon, Jun 04, 2012 at 02:39:17PM +0200, Christophe Fergeau wrote:
> On Mon, Jun 04, 2012 at 03:26:02PM +0300, Alon Levy wrote:
> > On Mon, Jun 04, 2012 at 11:46:02AM +0200, Christophe Fergeau wrote:
> > > On Thu, May 31, 2012 at 10:04:05AM +0300, Yonit Halperin wrote:
> >
On Mon, Jun 04, 2012 at 11:33:22AM +0200, Christophe Fergeau wrote:
> Makes sense to me though I've only looked quickly through it. Since it
> works for you, I'm in favour of committing it if noone disagrees.
I agree. Would you do a review and ack?
>
> Christophe
>
> On Sat, Jun 02, 2012 at 03:
On Mon, Jun 04, 2012 at 11:46:02AM +0200, Christophe Fergeau wrote:
> On Thu, May 31, 2012 at 10:04:05AM +0300, Yonit Halperin wrote:
> > Added ref count for RedChannel and RedChannelClient.
>
> Just a side note, longer term we should probably introduce a
> SpiceRefCounted base class which would c
On Sat, Jun 02, 2012 at 01:24:29PM -0500, Jeremy White wrote:
> I spent a fair amount of energy trying to understand the issue with EAGAIN
> and the xspice server. I've sent in one patch which I believe to be
> correct.
>
> I had some further thoughts I wanted to share as well.
>
> To reprise, t
On Thu, May 31, 2012 at 05:44:23PM +0300, Uri Lublin wrote:
> From: Nahum Shalman
>
ACK.
Now you're making me feel back about the few patches I already
cherry-picked to 0.10 without asking anyone.
> Consolidate two separate chunks of library hunting that depend on the
> same check.
>
> Check
adds preprocessor definitions DEBUG_QXL_MEM & DEBUG_QXL_MEM_VERBOSE
---
v1->v2: add static in front of qxl_mem_unverifiable in the default case of
undefined DEBUG_QXL_MEM.
src/qxl.h|6 ++
src/qxl_driver.c |8
src/qxl_mem.c| 54 ++
adds preprocessor definitions DEBUG_QXL_MEM & DEBUG_QXL_MEM_VERBOSE
---
v1->v2: add static in front of qxl_mem_unverifiable in the default case of
undefined DEBUG_QXL_MEM.
src/qxl.h|6 ++
src/qxl_driver.c |8
src/qxl_mem.c| 54 ++
In summary, on vt enter we still:
reset
recreate memory slots
clear our mspace allocators
and then do what switch mode below says
On vt leave we still:
reset (this is redundant since the first VGA access will trigger a
reset on the device side)
On switch mode however we only:
destroy prima
In summary, on vt enter we still:
reset
recreate memory slots
clear our mspace allocators
and then do what switch mode below says
On vt leave we still:
reset (this is redundant since the first VGA access will trigger a
reset on the device side)
On switch mode however we only:
destroy prima
---
src/qxl.h |9 +---
src/qxl_driver.c | 65 -
src/qxl_surface.c |2 +-
3 files changed, 67 insertions(+), 9 deletions(-)
diff --git a/src/qxl.h b/src/qxl.h
index 56e03a3..819c98d 100644
--- a/src/qxl.h
+++ b/src/qxl.h
@@
---
src/qxl.h |9 +---
src/qxl_driver.c | 65 -
src/qxl_surface.c |2 +-
3 files changed, 67 insertions(+), 9 deletions(-)
diff --git a/src/qxl.h b/src/qxl.h
index 56e03a3..819c98d 100644
--- a/src/qxl.h
+++ b/src/qxl.h
@@
adds preprocessor definitions DEBUG_QXL_MEM & DEBUG_QXL_MEM_VERBOSE
---
src/qxl.h|6 ++
src/qxl_driver.c |8
src/qxl_mem.c| 54 +-
3 files changed, 67 insertions(+), 1 deletion(-)
diff --git a/src/qxl.h b/src/q
adds preprocessor definitions DEBUG_QXL_MEM & DEBUG_QXL_MEM_VERBOSE
---
src/qxl.h|6 ++
src/qxl_driver.c |8
src/qxl_mem.c| 54 +-
3 files changed, 67 insertions(+), 1 deletion(-)
diff --git a/src/qxl.h b/src/q
---
src/qxl.h|8
src/qxl_driver.c |4
2 files changed, 12 insertions(+)
diff --git a/src/qxl.h b/src/qxl.h
index c6d3793..56e03a3 100644
--- a/src/qxl.h
+++ b/src/qxl.h
@@ -126,6 +126,12 @@ enum {
OPTION_COUNT,
};
+enum {
+QXL_DEVICE_PRIMARY_UNDEFINED,
+
---
src/qxl.h|8
src/qxl_driver.c |4
2 files changed, 12 insertions(+)
diff --git a/src/qxl.h b/src/qxl.h
index c6d3793..56e03a3 100644
--- a/src/qxl.h
+++ b/src/qxl.h
@@ -126,6 +126,12 @@ enum {
OPTION_COUNT,
};
+enum {
+QXL_DEVICE_PRIMARY_UNDEFINED,
+
---
src/qxl_surface.c |6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/qxl_surface.c b/src/qxl_surface.c
index 976c339..6414351 100644
--- a/src/qxl_surface.c
+++ b/src/qxl_surface.c
@@ -635,7 +635,7 @@ qxl_surface_create (surface_cache_t *cache,
if (
---
src/qxl_surface.c |6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/qxl_surface.c b/src/qxl_surface.c
index 976c339..6414351 100644
--- a/src/qxl_surface.c
+++ b/src/qxl_surface.c
@@ -635,7 +635,7 @@ qxl_surface_create (surface_cache_t *cache,
if (
---
src/mspace.c | 33 +++--
src/mspace.h |5 -
2 files changed, 31 insertions(+), 7 deletions(-)
diff --git a/src/mspace.c b/src/mspace.c
index 17d976a..822aade 100644
--- a/src/mspace.c
+++ b/src/mspace.c
@@ -1479,7 +1479,8 @@ static struct mallinfo interna
---
src/mspace.c | 33 +++--
src/mspace.h |5 -
2 files changed, 31 insertions(+), 7 deletions(-)
diff --git a/src/mspace.c b/src/mspace.c
index 17d976a..822aade 100644
--- a/src/mspace.c
+++ b/src/mspace.c
@@ -1479,7 +1479,8 @@ static struct mallinfo interna
---
src/qxl_driver.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/qxl_driver.c b/src/qxl_driver.c
index c128231..3c8dcca 100644
--- a/src/qxl_driver.c
+++ b/src/qxl_driver.c
@@ -1214,7 +1214,7 @@ qxl_leave_vt(int scrnIndex, int flags)
qxl->vt_surfaces = qxl_sur
---
src/qxl_driver.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/qxl_driver.c b/src/qxl_driver.c
index c128231..3c8dcca 100644
--- a/src/qxl_driver.c
+++ b/src/qxl_driver.c
@@ -1214,7 +1214,7 @@ qxl_leave_vt(int scrnIndex, int flags)
qxl->vt_surfaces = qxl_sur
---
src/qxl_surface.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/qxl_surface.c b/src/qxl_surface.c
index d999b4d..976c339 100644
--- a/src/qxl_surface.c
+++ b/src/qxl_surface.c
@@ -551,7 +551,7 @@ surface_send_create (surface_cache_t *cache,
/* the final + stri
---
src/qxl_surface.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/qxl_surface.c b/src/qxl_surface.c
index d999b4d..976c339 100644
--- a/src/qxl_surface.c
+++ b/src/qxl_surface.c
@@ -551,7 +551,7 @@ surface_send_create (surface_cache_t *cache,
/* the final + stri
xf86CrtcConfigInit to allow later
xf86_hide_cursors
in qxl_leave_vt to avoid some pixmap references until qxl_enter_vt.
I have not tested these patches with a Xinerama setup, I'd appreciate any tests
or pointing out something that wouldn't work for that scenario
(xf86CrtcConfigInit ?)
Alo
xf86CrtcConfigInit to allow later
xf86_hide_cursors
in qxl_leave_vt to avoid some pixmap references until qxl_enter_vt.
I have not tested these patches with a Xinerama setup, I'd appreciate any tests
or pointing out something that wouldn't work for that scenario
(xf86CrtcConfigInit ?)
Alo
---
src/qxl_surface.c | 46 ++
1 file changed, 46 insertions(+)
diff --git a/src/qxl_surface.c b/src/qxl_surface.c
index fd62576..669a022 100644
--- a/src/qxl_surface.c
+++ b/src/qxl_surface.c
@@ -49,6 +49,12 @@
#include "qxl.h"
+#ifdef DEBUG_SUR
---
src/qxl_surface.c | 46 ++
1 file changed, 46 insertions(+)
diff --git a/src/qxl_surface.c b/src/qxl_surface.c
index fd62576..669a022 100644
--- a/src/qxl_surface.c
+++ b/src/qxl_surface.c
@@ -49,6 +49,12 @@
#include "qxl.h"
+#ifdef DEBUG_SUR
This is not enough to prevent any qxl_destroy_pixmap call during vt
switch, but it prevents those triggered by CursorDisplayCursor.
Note: a matching xf86_show_cursors call doesn't hurt, but is not
required, so not adding it.
It is still possible to access freed memory by the following trigger:
=
This is not enough to prevent any qxl_destroy_pixmap call during vt
switch, but it prevents those triggered by CursorDisplayCursor.
Note: a matching xf86_show_cursors call doesn't hurt, but is not
required, so not adding it.
It is still possible to access freed memory by the following trigger:
=
---
src/qxl.h|1 +
src/qxl_driver.c | 15 +++
2 files changed, 16 insertions(+)
diff --git a/src/qxl.h b/src/qxl.h
index 819c98d..aa6645c 100644
--- a/src/qxl.h
+++ b/src/qxl.h
@@ -435,6 +435,7 @@ inline void qxl_mem_unverifiable(struct qxl_mem *mem) {}
void qxl_update_
---
src/qxl.h|1 +
src/qxl_driver.c | 15 +++
2 files changed, 16 insertions(+)
diff --git a/src/qxl.h b/src/qxl.h
index 819c98d..aa6645c 100644
--- a/src/qxl.h
+++ b/src/qxl.h
@@ -435,6 +435,7 @@ inline void qxl_mem_unverifiable(struct qxl_mem *mem) {}
void qxl_update_
---
src/uxa/uxa-damage.c |1 -
1 file changed, 1 deletion(-)
diff --git a/src/uxa/uxa-damage.c b/src/uxa/uxa-damage.c
index 586c466..3e4c075 100644
--- a/src/uxa/uxa-damage.c
+++ b/src/uxa/uxa-damage.c
@@ -30,7 +30,6 @@
#include
#include"scrnintstr.h"
#include"windowstr.h"
-#in
---
src/uxa/uxa-damage.c |1 -
1 file changed, 1 deletion(-)
diff --git a/src/uxa/uxa-damage.c b/src/uxa/uxa-damage.c
index 586c466..3e4c075 100644
--- a/src/uxa/uxa-damage.c
+++ b/src/uxa/uxa-damage.c
@@ -30,7 +30,6 @@
#include
#include"scrnintstr.h"
#include"windowstr.h"
-#in
---
src/qxl_driver.c | 19 +++
1 file changed, 19 insertions(+)
diff --git a/src/qxl_driver.c b/src/qxl_driver.c
index 3c8dcca..a3ec1f8 100644
--- a/src/qxl_driver.c
+++ b/src/qxl_driver.c
@@ -41,6 +41,7 @@
#include
+#include "mspace.h"
#include "qxl.h"
#include "assert.
---
src/qxl_driver.c | 19 +++
1 file changed, 19 insertions(+)
diff --git a/src/qxl_driver.c b/src/qxl_driver.c
index 3c8dcca..a3ec1f8 100644
--- a/src/qxl_driver.c
+++ b/src/qxl_driver.c
@@ -41,6 +41,7 @@
#include
+#include "mspace.h"
#include "qxl.h"
#include "assert.
---
src/qxl_surface.c |2 --
1 file changed, 2 deletions(-)
diff --git a/src/qxl_surface.c b/src/qxl_surface.c
index a8901f0..cb0de1d 100644
--- a/src/qxl_surface.c
+++ b/src/qxl_surface.c
@@ -357,8 +357,6 @@ make_surface_cmd (surface_cache_t *cache, uint32_t id,
QXLSurfaceCmdType type)
---
src/qxl_surface.c |2 --
1 file changed, 2 deletions(-)
diff --git a/src/qxl_surface.c b/src/qxl_surface.c
index a8901f0..cb0de1d 100644
--- a/src/qxl_surface.c
+++ b/src/qxl_surface.c
@@ -357,8 +357,6 @@ make_surface_cmd (surface_cache_t *cache, uint32_t id,
QXLSurfaceCmdType type)
---
src/uxa/uxa-accel.c |2 --
src/uxa/uxa.c |2 --
2 files changed, 4 deletions(-)
diff --git a/src/uxa/uxa-accel.c b/src/uxa/uxa-accel.c
index 62034c2..e456e6c 100644
--- a/src/uxa/uxa-accel.c
+++ b/src/uxa/uxa-accel.c
@@ -31,8 +31,6 @@
#include
#endif
#include "uxa-priv.h"
-#
---
src/uxa/uxa-accel.c |2 --
src/uxa/uxa.c |2 --
2 files changed, 4 deletions(-)
diff --git a/src/uxa/uxa-accel.c b/src/uxa/uxa-accel.c
index 62034c2..e456e6c 100644
--- a/src/uxa/uxa-accel.c
+++ b/src/uxa/uxa-accel.c
@@ -31,8 +31,6 @@
#include
#endif
#include "uxa-priv.h"
-#
---
src/qxl_driver.c |8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/src/qxl_driver.c b/src/qxl_driver.c
index 4c08637..c128231 100644
--- a/src/qxl_driver.c
+++ b/src/qxl_driver.c
@@ -525,7 +525,7 @@ setup_slot(qxl_screen_t *qxl, uint8_t slot_index_offset,
}
sta
---
src/qxl_driver.c |8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/src/qxl_driver.c b/src/qxl_driver.c
index 4c08637..c128231 100644
--- a/src/qxl_driver.c
+++ b/src/qxl_driver.c
@@ -525,7 +525,7 @@ setup_slot(qxl_screen_t *qxl, uint8_t slot_index_offset,
}
sta
---
src/qxl_driver.c |1 +
1 file changed, 1 insertion(+)
diff --git a/src/qxl_driver.c b/src/qxl_driver.c
index a3ec1f8..1dbde5b 100644
--- a/src/qxl_driver.c
+++ b/src/qxl_driver.c
@@ -1448,6 +1448,7 @@ qxl_pre_init(ScrnInfoPtr pScrn, int flags)
if (!pScrn->driverPrivate)
pScrn
---
src/qxl_driver.c |1 +
1 file changed, 1 insertion(+)
diff --git a/src/qxl_driver.c b/src/qxl_driver.c
index a3ec1f8..1dbde5b 100644
--- a/src/qxl_driver.c
+++ b/src/qxl_driver.c
@@ -1448,6 +1448,7 @@ qxl_pre_init(ScrnInfoPtr pScrn, int flags)
if (!pScrn->driverPrivate)
pScrn
The primary surface, i.e. qxl->primary, the only surface with id==0, is
allocated in qxl_surface_cache_create_primary with prev==next==NULL.
Unlinking it was producing a wrong cache->free_surfaces == NULL. This
was not a problem because unlinking the primary only happened in
switch_host, which then
The primary surface, i.e. qxl->primary, the only surface with id==0, is
allocated in qxl_surface_cache_create_primary with prev==next==NULL.
Unlinking it was producing a wrong cache->free_surfaces == NULL. This
was not a problem because unlinking the primary only happened in
switch_host, which then
Prevent access to freed memory when:
1. qxl_leave_vt/qxl_surface_cache_evacuate_all freed cache->all_surfaces
2. ProcRenderDispatch/damageDestroyPixmap/qxl_destroy_pixmap/qxl_surface_kill
access a surface that pointed inside the all_surfaces array
Solution in this patch:
1. never free all_surfaces
Prevent access to freed memory when:
1. qxl_leave_vt/qxl_surface_cache_evacuate_all freed cache->all_surfaces
2. ProcRenderDispatch/damageDestroyPixmap/qxl_destroy_pixmap/qxl_surface_kill
access a surface that pointed inside the all_surfaces array
Solution in this patch:
1. never free all_surfaces
On Thu, May 31, 2012 at 10:04:05AM +0300, Yonit Halperin wrote:
> Added ref count for RedChannel and RedChannelClient.
>
> red_channel.c/red_peer_handle_incoming call to
> handler->cb->handle_message might lead to the release of the channel
> client, and the following call to handler->cb->release_
On Wed, May 30, 2012 at 12:02:36PM +0300, Yonit Halperin wrote:
> Signed-off-by: Yonit Halperin
One empty line that snuck in below.
> ---
> input.c |2 +-
> migration.c |2 +-
> notify.c | 79
> +++---
> notify.h | 5
On Wed, May 30, 2012 at 12:02:36PM +0300, Yonit Halperin wrote:
> Signed-off-by: Yonit Halperin
One empty line that snuck in below.
> ---
> input.c |2 +-
> migration.c |2 +-
> notify.c | 79
> +++---
> notify.h | 5
On Tue, May 29, 2012 at 01:44:35PM -0300, Luiz Capitulino wrote:
> On Tue, 29 May 2012 17:51:50 +0300
> Alon Levy wrote:
>
> > On Tue, May 29, 2012 at 10:38:20AM -0300, Luiz Capitulino wrote:
> > > On Tue, 29 May 2012 09:25:40 +0200
> > > Gerd Hof
source/anti/apple/
>
> There is no IGLU Cabal! Home‐made Cabals eventually superseded the power
> and
> influence of the original IGLU Cabal, which was considered a cutting edge
> development at its time.
>
> Please reply to list if it's a mailing list post - http://shlo
On Tue, May 29, 2012 at 10:38:20AM -0300, Luiz Capitulino wrote:
> On Tue, 29 May 2012 09:25:40 +0200
> Gerd Hoffmann wrote:
>
> > Hi,
> >
> > >> How would that work? I have QXLInfo that only makes sense when the
> > >> information is about a qxl device. Can't have opaque data in a QMP
> > >>
On Tue, May 29, 2012 at 07:06:05PM +0800, Zhou Peng wrote:
> Hi all,
>
> As we know, 64M is default for qxl ram size.
The default memory is 128MiB, not 64MiB - it's the sum of two 64 MiB
bars, the size of each is settable. There is no help in the man page
that I can find, the best you could do to
On Tue, May 29, 2012 at 09:25:40AM +0200, Gerd Hoffmann wrote:
> Hi,
>
> >> How would that work? I have QXLInfo that only makes sense when the
> >> information is about a qxl device. Can't have opaque data in a QMP
> >> response, so would this be a "info display qxl" "info display cirrus"
> >> e
On Tue, May 29, 2012 at 02:58:19AM +, Charles.Tsai-蔡清海-研究發展部 wrote:
Hi Vadim,
Could you have a look?
Thanks,
Alon
> Bug description:
> A sever bug was found on 64-bit Windows 7 VM which crashed after running
> idle for a while(~ 2-3 hours).
> When we checked the kvm process, it was kill
On Mon, May 28, 2012 at 12:19:57PM +0300, Alon Levy wrote:
> On Thu, May 24, 2012 at 01:24:05PM -0500, Jeremy White wrote:
> > > Actually, for WAN, we require ACK for 40 messages, but we allow sending
> > > up to 80, without getting an ack for the first 40.
> > > F
On Mon, May 28, 2012 at 10:13:28AM -0300, Luiz Capitulino wrote:
> On Mon, 28 May 2012 12:08:13 +0300
> Alon Levy wrote:
>
> > On Fri, May 25, 2012 at 11:43:11AM -0300, Luiz Capitulino wrote:
> > > On Thu, 24 May 2012 19:22:52 +0300
> > > Alon Levy wrote:
> &
ach
client, called the pipe. There is a cap on the linked list size, 50. If
the size is 50, we don't add any more and wait for sending to succeed,
or for the connection to drop, or for a timeout to happen.
>
> 2012/5/28 Alon Levy
>
> > On Fri, May 25, 2012 at 02:58:34PM +08
st
> side :(
The later is fixable with hw based compression.
>
>
> On Mon, May 28, 2012 at 11:19 AM, Alon Levy wrote:
>
> > On Thu, May 24, 2012 at 01:24:05PM -0500, Jeremy White wrote:
> > > > Actually, for WAN, we require ACK for 40 messages, but we allow sendin
On Fri, May 25, 2012 at 02:58:34PM +0800, wangfeng wangfeng wrote:
> Try the following command:
> export SPICE_DEBUG_ALLOW_MC=1
But please be aware that this will (given enough time) crash the server
if any of the clients reach a full pipe condition.
>
>
>
> 2012/5/25 蒋媛园
>
> > Hi~
> >I
On Fri, May 25, 2012 at 09:07:43AM +0200, Alexander Larsson wrote:
> On Wed, 2012-05-23 at 15:20 -0500, Jeremy White wrote:
>
> > Also, as a crazy idea, has anyone considered implementing a pure
> > streaming video driver? That is, what if we had a frame buffer driver,
> > and then a thread that
On Thu, May 24, 2012 at 01:24:05PM -0500, Jeremy White wrote:
> > Actually, for WAN, we require ACK for 40 messages, but we allow sending
> > up to 80, without getting an ack for the first 40.
> > From my experience with Windows guest, it sounds like the DRAW_FILL
> > commands might be related to a
On Fri, May 25, 2012 at 11:43:11AM -0300, Luiz Capitulino wrote:
> On Thu, 24 May 2012 19:22:52 +0300
> Alon Levy wrote:
>
> > For all devices print id, mode and guest_bug status.
>
> Is qxl really tied to spice? In the meaning that it's impossible to use it
>
On Fri, May 25, 2012 at 09:23:13AM +0200, Alexander Larsson wrote:
> On Mon, 2012-05-07 at 09:28 +0300, Alon Levy wrote:
> > Hi,
> >
> > Currently we support multiple monitors by having:
> > single pci = single display channel = single client window
> >
&g
series does the right
> thing (or is very near to).
>
Reviewed-by: Alon Levy
> console.c| 7 ++--
> console.h| 5 +--
> cutils.c | 100
> +++
> hmp-commands.hx | 5 ++-
> hmp.c
On Fri, May 25, 2012 at 04:41:19PM -0300, Luiz Capitulino wrote:
One small note below.
> Also activates error reporting from devices.
>
> Signed-off-by: Luiz Capitulino
> ---
> console.c| 7 ---
> console.h| 1 -
> hmp-commands.hx | 5 ++---
> hmp.c| 9
For all devices print id, mode and guest_bug status.
Known problems: Prints devices from highest id to lowest.
Signed-off-by: Alon Levy
---
This one builds. Fixed qapi-schema to say additions are for 1.2
hmp.c| 11 +++
hw/qxl.c | 22 ++
qapi
soft_reset is called from any of:
* QXL_IO_RESET
* vga io
* pci reset handler
Signed-off-by: Alon Levy
---
hw/qxl.c | 13 -
hw/qxl.h |3 +++
2 files changed, 15 insertions(+), 1 deletion(-)
diff --git a/hw/qxl.c b/hw/qxl.c
index 5a7be60..b5e53ce 100644
--- a/hw/qxl.c
+++ b
Tested with linux guest. Not sure how to check actual performance affect
of this. Checked with the previously send traceevent that the kvm ioctl
to start/stop dirty logging is being called.
(KVM_SET_USER_MEMORY_REGION).
Signed-off-by: Alon Levy
---
These are the two patches to queue instead of
On Thu, May 24, 2012 at 02:58:40PM +0200, Gerd Hoffmann wrote:
> On 05/24/12 11:38, Alon Levy wrote:
> > For all devices print id, mode and guest_bug status.
>
> Queued up 1+2+4
Please unqueue "qxl: stop dirty loging when not in vga mode" and I'll
send it again w
On Thu, May 24, 2012 at 06:09:51PM +0300, Alon Levy wrote:
> For all devices print id, mode and guest_bug status.
>
> Known problems: Prints devices from highest id to lowest.
>
> Signed-off-by: Alon Levy
> ---
> Fixed the documentation to match the command (no more qxl0)
For all devices print id, mode and guest_bug status.
Known problems: Prints devices from highest id to lowest.
Signed-off-by: Alon Levy
---
Fixed the documentation to match the command (no more qxl0).
Sending to QMP maintainers per Kraxel's request for an additional ack.
Alon
dea58
But that's only a miniport, and under MS-PL. Can't find anything else.
>
> Thanks a lot!
>
> /Mario
>
> On Mon, 21 May 2012 14:28:22 +0300, Alon Levy wrote:
> >On Mon, May 21, 2012 at 10:47:05AM +0200, Mario wrote:
> >>Alon, thank you for your reply.
For all devices print id, mode and guest_bug status.
Known problems: Prints devices from highest id to lowest.
Signed-off-by: Alon Levy
---
hmp.c| 11 +++
hw/qxl.c | 22 ++
qapi-schema.json | 47
On Thu, May 24, 2012 at 12:38:11PM +0300, Alon Levy wrote:
> We can't initialize QXLDevSurfaceCreate field by field because it has a
> pa hole, and so 4 bytes remain uninitialized when building on x86-64, so
> just memset.
This was supposed to be in reply to "<4fbdf
Tested with linux guest. Not sure how to check actual performance affect
of this. Checked with the previously send traceevent that the kvm ioctl
to start/stop dirty logging is being called.
(KVM_SET_USER_MEMORY_REGION).
Signed-off-by: Alon Levy
---
hw/qxl.c |2 ++
1 file changed, 2
Signed-off-by: Alon Levy
---
hw/qxl.c | 49 +++--
hw/qxl.h |3 ++-
2 files changed, 29 insertions(+), 23 deletions(-)
diff --git a/hw/qxl.c b/hw/qxl.c
index 3da3399..8777ba9 100644
--- a/hw/qxl.c
+++ b/hw/qxl.c
@@ -30,7 +30,7 @@
/*
* NOTE
We can't initialize QXLDevSurfaceCreate field by field because it has a
pa hole, and so 4 bytes remain uninitialized when building on x86-64, so
just memset.
Signed-off-by: Alon Levy
---
ui/spice-display.c |2 ++
1 file changed, 2 insertions(+)
diff --git a/ui/spice-display.c b/ui/
1101 - 1200 of 4874 matches
Mail list logo
