Reviewed-by: Christophe Fergeau
On Thu, Feb 21, 2019 at 12:43:28PM +0100, Marc-André Lureau wrote:
> Signed-off-by: Marc-André Lureau
> ---
> include/hw/virtio/virtio-gpu.h | 2 --
> hw/display/virtio-gpu.c| 3 +--
> 2 files changed, 1 insertion(+), 4 deletions(-)
I came up with the same fix while looking at that bug before seeing
Marc-André's patch.
Then I tested Marc-André's patch.
So for this patch,
Reviewed-by: Christophe Fergeau
Tested-by: Christophe Fergeau
On Thu, Feb 21, 2019 at 12:43:29PM +0100, Marc-André Lureau wrote:
> Now that 2d comma
Reviewed-by: Christophe Fergeau
On Thu, Feb 21, 2019 at 12:43:27PM +0100, Marc-André Lureau wrote:
> Signed-off-by: Marc-André Lureau
> ---
> include/hw/virtio/virtio-gpu.h | 1 -
> hw/display/virtio-gpu.c| 1 -
> 2 files changed, 2 deletions(-)
>
> diff --gi
Reviewed-by: Christophe Fergeau
On Thu, Feb 21, 2019 at 12:43:30PM +0100, Marc-André Lureau wrote:
> Let's check renderer_blocked instead directly.
>
> Signed-off-by: Marc-André Lureau
> ---
> include/hw/virtio/virtio-gpu.h | 1 -
> hw/display/virtio-gpu.c| 4 +---
The main goal of this patch series is to make logs output by
libspice-server.so go through qemu logging infrastructure so that their
format is the same as the rest of QEMU messages (in particular,
timestamps). This is done in the second patch, the first one is a
preparation patch.
Christophe
ect, so this call is
not conditional on the SPICE version.
Signed-off-by: Christophe Fergeau
Reviewed-by: Stefan Hajnoczi
---
Changes since v6:
- expanded on the bsd-user/linux-user/qemu-pr-helper changes in the
commit log
- fix obsolete mention of qemu_init_logging() in commit log
- improved out
qemu-io reimplements itself what
error_get_progname()/error_set_progname() already does.
This commit switches it to use this API from qemu-error.h
Signed-off-by: Christophe Fergeau
Reviewed-by: Eric Blake
Reviewed-by: Stefan Hajnoczi
---
No changes since v6
qemu-io.c | 14 ++
1
qemu-io reimplements itself what
error_get_progname()/error_set_progname() already does.
This commit switches it to use this API from qemu-error.h
Signed-off-by: Christophe Fergeau
---
qemu-io.c | 14 ++
1 file changed, 6 insertions(+), 8 deletions(-)
diff --git a/qemu-io.c b/qemu
ice versions, this is not going
to work as expected, but will not have any ill effect, so this call is
not conditional on the SPICE version.
Signed-off-by: Christophe Fergeau
---
bsd-user/main.c | 2 ++
include/qemu/error-report.h | 3 ++-
linux-user/main.c | 2 ++
qemu-im
On Thu, Jan 24, 2019 at 05:10:28PM +0100, Markus Armbruster wrote:
> Christophe Fergeau writes:
>
> > This commit adds a qemu_init_logging() helper which calls
> > g_log_set_default_handler() so that glib logs (g_log, g_warning, ...)
> > are handled similarly to oth
Hey,
On Thu, Jan 24, 2019 at 10:38:37AM +0100, Markus Armbruster wrote:
> Christophe Fergeau writes:
>
> > This commit adds a qemu_init_logging() helper which calls
> > g_log_set_default_handler() so that glib logs (g_log, g_warning, ...)
> > are handled similarly to oth
Hey,
On Thu, Jan 24, 2019 at 10:35:52AM +0100, Markus Armbruster wrote:
> Markus Armbruster writes:
>
> > Eric Blake writes:
> >
> >> On 1/2/19 12:01 PM, Christophe Fergeau wrote:
> >>> Adding Markus to cc: list, I forgot to do it when sending the patc
ons, this is not going
to work as expected, but will not have any ill effect, so this call is
not conditional on the SPICE version.
Signed-off-by: Christophe Fergeau
Reviewed-by: Daniel P. Berrangé
Reviewed-by: Stefan Hajnoczi
---
One more iteration of the patch as it hit CI failures
(ht
On Tue, Jan 15, 2019 at 02:09:12PM +, Stefan Hajnoczi wrote:
> > In these codepaths, I'm confident up to
> > void monitor_vfprintf(FILE *stream, const char *fmt, va_list ap)
> > {
> > if (cur_mon && !monitor_cur_is_qmp()) {
> > monitor_vprintf(cur_mon, fmt, ap);
> > } else {
>
On Fri, Jan 04, 2019 at 03:40:34PM +, Stefan Hajnoczi wrote:
> On Thu, Jan 03, 2019 at 03:14:55PM +0100, Christophe Fergeau wrote:
> > Hey,
> >
> > On Thu, Jan 03, 2019 at 10:54:25AM +, Stefan Hajnoczi wrote:
> > > On Fri, Dec 14, 2018 at 11:56:42AM +
On Mon, Jan 07, 2019 at 04:47:44PM +0100, Markus Armbruster wrote:
> Eric Blake writes:
>
> > On 1/2/19 12:01 PM, Christophe Fergeau wrote:
> >> Adding Markus to cc: list, I forgot to do it when sending the patch.
> >
> > Also worth backp
Hey,
On Thu, Jan 03, 2019 at 04:25:00PM -0600, Eric Blake wrote:
> On 12/27/18 8:51 AM, Niccolò Belli wrote:
> > On mercoledì 26 dicembre 2018 13:38:28 CET, Frediano Ziglio wrote:
> >> Yes, this looks like a format string error in the upper (not into
> >> spice) layer.
> >>
> >> This potentially
Hey,
On Thu, Jan 03, 2019 at 10:54:25AM +, Stefan Hajnoczi wrote:
> On Fri, Dec 14, 2018 at 11:56:42AM +0100, Christophe Fergeau wrote:
> > +static void qemu_log_func(const gchar *log_domain,
> > + GLogLevelFlags log_level,
> > +
Adding Markus to cc: list, I forgot to do it when sending the patch.
Christophe
On Wed, Jan 02, 2019 at 03:05:35PM +0100, Christophe Fergeau wrote:
> commit 8bca4613 added support for %% in json strings when interpolating,
> but in doing so, this broke handling of % when not interpo
commit 8bca4613 added support for %% in json strings when interpolating,
but in doing so, this broke handling of % when not interpolating as the
'%' is skipped in both cases.
This commit ensures we only try to handle %% when interpolating.
Signed-off-by: Christophe Fergeau
---
qobject/json
ons, this is not going
to work as expected, but will not have any ill effect, so this call is
not conditional on the SPICE version.
Signed-off-by: Christophe Fergeau
Reviewed-by: Daniel P. Berrangé
---
Changes since v3:
- Fixed style issue: removed 2 extra spaces before parens
(repor
ons, this is not going
to work as expected, but will not have any ill effect, so this call is
not conditional on the SPICE version.
Signed-off-by: Christophe Fergeau
---
Changes since v2:
- report G_LOG_LEVEL_CRITICAL using error_report() instead of
warn_report()
- print G_LOG_LEVEL_DEBUG messa
On Thu, Dec 13, 2018 at 11:52:12AM +, Daniel P. Berrangé wrote:
> On Thu, Dec 13, 2018 at 12:26:12PM +0100, Christophe Fergeau wrote:
> > This commit adds a qemu_init_logging() helper which calls
> > g_log_set_default_handler() so that glib logs (g_log, g_warning, ...)
all is
not conditional on the SPICE version.
Signed-off-by: Christophe Fergeau
---
Changes since v1:
- introduced a qemu_init_logging() helper, and call that in more
binaries than just vl.c
bsd-user/main.c | 1 +
include/qemu/error-report.h | 2 ++
linux-user/main.c | 1 +
q
on spice
version.
Since this added g_log_set_default_handler() will bridge glib logging
and QEMU logging, the call might fit better in a more generic place.
Signed-off-by: Christophe Fergeau
---
ui/spice-core.c | 25 +
1 file changed, 25 insertions(+)
diff --git a/ui
hey,
On Mon, Nov 26, 2018 at 03:30:36PM +, Frediano Ziglio wrote:
> Definitions were updated by spice-server in patch de66161 included
> in 0.12.6 released on 12th June 2015.
QEMU's configure only checks for spice-server 0.12.0:
$pkg_config --atleast-version=0.12.0 spice-server
if
This was added in 13f1243, but is missing from qemu-options.hx
Signed-off-by: Christophe Fergeau <cferg...@redhat.com>
---
qemu-options.hx | 11 ++-
1 file changed, 10 insertions(+), 1 deletion(-)
diff --git a/qemu-options.hx b/qemu-options.hx
index f11c4ac960..118784ceb7
, ...
This causes https://bugzilla.redhat.com/show_bug.cgi?id=1266484
This commit makes sure that we only emit
QXL_INTERRUPT_CLIENT_MONITORS_CONFIG when there are actual configuration
changes the guest should act on.
Signed-off-by: Christophe Fergeau <cferg...@redhat.com>
---
Changes since v1:
- mad
Currently if the client keeps sending the same monitor config to
QEMU/spice-server, QEMU will always raise
a QXL_INTERRUPT_CLIENT_MONITORS_CONFIG regardless of whether there was a
change or not.
Guest-side (with fedora 25), the kernel QXL KMS driver will also forward the
event to user-space
On Tue, Jul 19, 2016 at 09:41:22AM -0400, Frediano Ziglio wrote:
> > I don't think we have strong reasons to support software encoding, video
> > encoding is really expensive, and that mmap/copy is not going to be
> > marginal, so even less these 2 syscalls.
> >
>
> Using HW encoding is not easy
On Fri, Mar 18, 2016 at 09:17:53AM +0100, Gerd Hoffmann wrote:
> On Mo, 2016-03-14 at 12:41 +0100, Christophe Fergeau wrote:
> > Currently, virgl support has to go through a local unix socket, trying
> > to connect to a VM using -spice gl through spice://localhost:5900 will
On Tue, Mar 15, 2016 at 03:32:31PM +0100, Gerd Hoffmann wrote:
> > We can do something similar once gl+tcp is available.
>
> I don't expect adding gl+tcp support to spice needs changes in the
> spice-server API and qemu. So ifdef'ing on the spice-server version is
> bogous,
Hmm, I expected some
On Tue, Mar 15, 2016 at 02:09:47PM +0100, Gerd Hoffmann wrote:
> Hi,
>
> > Yes, long-term we will want to remove this once spice gets support for
> > this.
>
> How can qemu figure whenever spice supports gl+tcp or not?
gl support is already enabled through a spice-server version check
#if
On Mon, Mar 14, 2016 at 09:41:34AM -0600, Eric Blake wrote:
> On 03/14/2016 05:41 AM, Christophe Fergeau wrote:
> > Currently, virgl support has to go through a local unix socket, trying
> > to connect to a VM using -spice gl through spice://localhost:5900 will
> > only res
Hi,
On Mon, Mar 14, 2016 at 04:00:11PM +0100, Gerd Hoffmann wrote:
> On Mo, 2016-03-14 at 12:41 +0100, Christophe Fergeau wrote:
> > Currently, virgl support has to go through a local unix socket, trying
> > to connect to a VM using -spice gl through spice://localhost:5900 will
in spice-server, but currently QEMU does not call
into spice-server when parsing 'gl' on its command line, so we have to
do this check in QEMU instead.
Signed-off-by: Christophe Fergeau <cferg...@redhat.com>
---
ui/spice-core.c | 4
1 file changed, 4 insertions(+)
diff --git a/ui
On Fri, Jan 15, 2016 at 04:31:45PM -0700, Eric Blake wrote:
> On 01/12/2016 06:39 AM, Christophe Fergeau wrote:
> > Signed-off-by: Christophe Fergeau <cferg...@redhat.com>
> > ---
> > fsdev/virtfs-proxy-helper.texi | 2 +-
> > 1 file changed, 1 insertion(+),
There was a 'capbilities' typo in this man page. This commit
reformulates the sentence the typo was in to make it easier to grasp.
This is based on a suggestion from Eric Blake.
Signed-off-by: Christophe Fergeau <cferg...@redhat.com>
---
Changes since v1:
- rather than just fixing th
On Mon, Jan 18, 2016 at 11:11:42AM +0100, Gerd Hoffmann wrote:
> On Fr, 2016-01-15 at 16:33 -0700, Eric Blake wrote:
> > > This commit unconditionnally calls spice_server_set_sasl_appname()
> >
> > s/unconditionnally/unconditionally/
>
> Fixed up, no need to resend.
Thanks!
Christophe
Signed-off-by: Christophe Fergeau <cferg...@redhat.com>
---
fsdev/virtfs-proxy-helper.texi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/fsdev/virtfs-proxy-helper.texi b/fsdev/virtfs-proxy-helper.texi
index e60e3b9..1b905a0 100644
--- a/fsdev/virtfs-proxy-helper.texi
mu command line.
Signed-off-by: Christophe Fergeau <cferg...@redhat.com>
---
ui/spice-core.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/ui/spice-core.c b/ui/spice-core.c
index 3322bf2..77b209f 100644
--- a/ui/spice-core.c
+++ b/ui/spice-core.c
@@ -743,
o "spice" when this happens.
Signed-off-by: Christophe Fergeau <cferg...@redhat.com>
Reviewed-by: Daniel P. Berrange <berra...@redhat.com>
---
Changes since v2:
- Added mention of the oVirt bug which was caused by the change of behaviour
ui/spice-core.c | 4
1 file cha
Hey Gerd,
On Mon, Oct 12, 2015 at 03:43:51PM +0200, Gerd Hoffmann wrote:
> On Mo, 2015-10-12 at 13:25 +0200, Christophe Fergeau wrote:
> > Before commit b1ea7b79e1, it was possible to start with -spice
> > disable-ticketing, and then use the "set_password spice" comman
Hey,
On Fri, Aug 14, 2015 at 05:10:57PM +0200, Christophe Fergeau wrote:
> Before commit b1ea7b79e1, it was possible to start with -spice
> disable-ticketing, and then use the "set_password spice" command to
> enable ticketing with SPICE. Since commit b1ea7b79e1 this is n
On Fri, Aug 14, 2015 at 03:04:48PM +0100, Daniel P. Berrange wrote:
Hmm, is oVirt using this via libvirt ? If so, I guess we have to fix
it, as that would be a break in current usage.
Yes this is done through libvirt.
Before commit qemu-2.1.0-rc2~11^2, you could use virsh update-device
with
ticketing is disabled, 'auth' is none so
the attempt to set password fails.
This commit allows to call qemu_spice_set_ticket() when 'auth' is none
and changes 'auth' to spice when this happens.
Signed-off-by: Christophe Fergeau cferg...@redhat.com
Reviewed-by: Daniel P. Berrange berra...@redhat.com
On Fri, Aug 14, 2015 at 03:35:21PM +0100, Daniel P. Berrange wrote:
On Fri, Aug 14, 2015 at 02:47:15PM +0200, Christophe Fergeau wrote:
Before commit b1ea7b79e1, it was possible to start with -spice
disable-ticketing, and then use the set_password spice command to
enable ticketing
Before commit b1ea7b79e1, it was possible to start with -spice
disable-ticketing, and then use the set_password spice command to
enable ticketing with SPICE. Since commit b1ea7b79e1 this is no longer
possible as qemu_spice_set_ticket() will return an error unless the
'auth' type is spice. When
Hey,
On Fri, Aug 14, 2015 at 01:54:59PM +0100, Daniel P. Berrange wrote:
On Fri, Aug 14, 2015 at 02:47:15PM +0200, Christophe Fergeau wrote:
Before commit b1ea7b79e1, it was possible to start with -spice
disable-ticketing, and then use the set_password spice command to
enable ticketing
On Mon, Jul 20, 2015 at 09:43:23AM +0100, Frediano Ziglio wrote:
The new spice-server function to limit the number of monitors (0.12.6)
changed while development from spice_qxl_set_monitors_config_limit to
spice_qxl_max_monitors (accepted upstream).
By mistake I post patch with former name.
On Fri, Jun 12, 2015 at 03:05:10PM +0100, Frediano Ziglio wrote:
This patch allow to limit number of heads using qxl driver. By default
qxl driver is not limited on any kind on head use so can decide to use
as much heads.
libvirt has this as a video card parameter (actually set to 1 but not
On Tue, Jun 09, 2015 at 10:50:48AM +0100, Daniel P. Berrange wrote:
On Tue, Jun 09, 2015 at 09:49:44AM +0100, Frediano Ziglio wrote:
This patch allow to limit number of heads using qxl driver. By default
qxl driver is not limited on any kind on head use so can decide to use
as much heads.
Hey,
On Tue, Apr 14, 2015 at 09:14:53AM +0200, Gerd Hoffmann wrote:
Denis Kirjanov is busy getting spice run on ppc64 and trapped into this
one. Spice wire format is little endian, so we have to explicitly say
we want little endian when letting pixman convert the data for us.
This patch
Hey,
On Thu, Jan 08, 2015 at 11:50:13AM +0100, Javier Celaya wrote:
Hello
Recently, SPICE included the lz4 compression algorithm. This patch adds
a command line option to select it. However, SPICE_IMAGE_COMPRESS_LZ4 did not
exist before the commit that added this compression algorithm, so
Hey,
On Tue, Apr 29, 2014 at 10:02:26AM +0400, Michael Tokarev wrote:
Use glib-provided thread primitives in vscclient, not qemu
thread primitives. This way, vscclient becomes more stand-alone.
For what it's worth, this patch has a few non-threading related bits in
it because of the removal
On Tue, Apr 29, 2014 at 10:02:24AM +0400, Michael Tokarev wrote:
glib = 2.31 always enables thread support and g_thread_supported()
is #defined to 1, there's no need to call g_thread_init() anymore,
and it definitely does not need to report error which never happens.
Keep code for old 2.31
Hey, patch looks good
On Tue, Apr 29, 2014 at 10:02:27AM +0400, Michael Tokarev wrote:
Replace QemuMutex with GMutex and QemuCond with GCond
(with corresponding function changes), to make libcacard
independent of qemu internal functions.
Also replace single instance pstrcpy() in
- Mail original -
Il 26/03/2014 17:15, Fabio Fantoni ha scritto:
Time ago I have read somewhere that there is an option to
automatically spice port in qemu as for vnc.
I started to write a libxl patch to add this feature like the vnc one:
On Mon, Apr 07, 2014 at 11:59:06AM +0200, Fabio Fantoni wrote:
Today I did some tests also with hvm and spice and I found another
segfault with different backtrace to solve:
(gdb) c
Continuing.
*Program received signal SIGSEGV, Segmentation fault.**
**0x55855d30 in
Looks good, ACK.
Christophe
On Wed, Feb 19, 2014 at 11:40:50AM +0100, Gerd Hoffmann wrote:
Signed-off-by: Gerd Hoffmann kra...@redhat.com
---
hw/display/qxl.c | 8 +++-
1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/hw/display/qxl.c b/hw/display/qxl.c
index
-by: Christophe Fergeau cferg...@redhat.com
---
libcacard/Makefile | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libcacard/Makefile b/libcacard/Makefile
index 4d15da4..6b06448 100644
--- a/libcacard/Makefile
+++ b/libcacard/Makefile
@@ -25,7 +25,7 @@ vscclient$(EXESUF): libcacard
Similarly to the previous commit fixing VNC+SASL+QXL, when starting
QEMU with SPICE but no SASL, and also VNC with SASL, then
spice_server_init() will get called without a previous call to
spice_server_set_sasl_appname(), which will cause cyrus-sasl to
try to use /etc/sasl2/spice.conf
Similarly to the previous commit fixing VNC+SASL+QXL, when starting
QEMU with SPICE but no SASL, and also VNC with SASL, then
spice_server_init() will get called without a previous call to
spice_server_set_sasl_appname(), which will cause cyrus-sasl to
try to use /etc/sasl2/spice.conf
setting the appname
to qemu, which then causes the VNC code to try to use spice.conf
instead of qemu.conf.
Signed-off-by: Christophe Fergeau cferg...@redhat.com
---
ui/spice-core.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/ui/spice-core.c b/ui/spice-core.c
index 33ef837..d7566b0 100644
Hey Hans,
On Tue, Oct 08, 2013 at 04:27:38PM +0200, Hans de Goede wrote:
I'm having this weird problem with qemu master + spice/qxl using
guests. As soon as the guest starts Xorg, I get the following message
from qemu:
main-loop: WARNING: I/O thread spun for 1000 iterations
I've also seen
Several places in spice-core.c were using either g_malloc+snprintf
or snprintf+g_strdup to achieve the same result as g_strdup_printf.
Signed-off-by: Christophe Fergeau cferg...@redhat.com
---
ui/spice-core.c | 22 --
1 file changed, 8 insertions(+), 14 deletions(-)
diff
Several places in spice-core.c were using either g_malloc+snprintf
or snprintf+g_strdup to achieve the same result as g_strdup_printf.
Signed-off-by: Christophe Fergeau cferg...@redhat.com
---
Changes since v1:
- split lines 80 chars (checkpatch.pl now succeeds)
ui/spice-core.c | 28
On Mon, Sep 02, 2013 at 01:36:19PM +0200, Gerd Hoffmann wrote:
On Mo, 2013-09-02 at 11:53 +0200, Christophe Fergeau wrote:
Several places in spice-core.c were using either g_malloc+snprintf
or snprintf+g_strdup to achieve the same result as g_strdup_printf.
Patch looks good but fails
On Fri, Nov 16, 2012 at 06:02:33PM +0800, Dunrong Huang wrote:
I meet a weird problem:
If I I boot QEMU with virtio-serial being enabled and assign only one
cpu to windows VM, when I shutdown VM, QEMU hangs with using 100% cpu.
What version of virtio-serial were you using? The old
Hey,
On Fri, Nov 16, 2012 at 07:22:58PM +0800, Dunrong Huang wrote:
2012/11/16 Christophe Fergeau cferg...@redhat.com:
On Fri, Nov 16, 2012 at 06:02:33PM +0800, Dunrong Huang wrote:
I meet a weird problem:
If I I boot QEMU with virtio-serial being enabled and assign only one
cpu
When parsing its command line parameters, spice aborts when it
finds unexpected values, except for the 'streaming-video' option.
This happens because the parsing of the parameters for this option
is done using the 'name2enum' helper, which does not error out
on unknown values. Using the
Hey,
On Mon, Mar 19, 2012 at 02:29:41PM +0800, Charles.Tsai-蔡清海-研究發展部 wrote:
3. When we disable and enable the new VirtIO driver several times
on either Qemu 1.0 or Qemu 1.0.50, VirtIo driver failed to work
after enabling the new VirtIO driver.
For what it's worth, Alon Levy
On Mon, Mar 19, 2012 at 01:42:21PM +0200, Vadim Rozenfeld wrote:
On Monday, March 19, 2012 12:12:59 PM Christophe Fergeau wrote:
Hey,
On Mon, Mar 19, 2012 at 02:29:41PM +0800, Charles.Tsai-蔡清海-研究發展部 wrote:
3. When we disable and enable the new VirtIO driver several times
Without it the shell prompt doesn't appear on a new line after
qemu dies.
---
ui/spice-core.c |8
1 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/ui/spice-core.c b/ui/spice-core.c
index 1308a3d..6d240a3 100644
--- a/ui/spice-core.c
+++ b/ui/spice-core.c
@@ -568,15
It's currently possible to setup spice channels using TLS when
no TLS port has been specified (ie TLS is disabled). This cannot
work, so better to error out in such a situation.
---
ui/spice-core.c |8 +++-
1 files changed, 7 insertions(+), 1 deletions(-)
diff --git a/ui/spice-core.c
Error message reporting during spice startup wasn't consistent, it was done
with fprintf(stderr, ) but sometimes the message didn't have a trailing
\n. Using error_report make the intent of the message clearer and deal
with the final \n for us.
---
ui/spice-core.c | 22 +++---
1
It's currently possible to setup spice channels using TLS when
no TLS port has been specified (ie TLS is disabled). This cannot
work, so better to error out in such a situation.
---
ui/spice-core.c |7 ++-
1 files changed, 6 insertions(+), 1 deletions(-)
diff --git a/ui/spice-core.c
It's currently possible to setup spice channels using TLS when
no TLS port has been specified (ie TLS is disabled). This cannot
work, so better to error out in such a situation.
---
ui/spice-core.c |8 +++-
1 files changed, 7 insertions(+), 1 deletions(-)
diff --git a/ui/spice-core.c
On Mon, Oct 17, 2011 at 11:25:42AM +0200, Gerd Hoffmann wrote:
static const VMStateDescription vmstate_ps2_common = {
.name = PS2 Common State,
-.version_id = 3,
+.version_id = 4,
.minimum_version_id = 2,
.minimum_version_id_old = 2,
.fields =
Hey,
On Mon, Oct 17, 2011 at 01:37:34PM +0200, Juan Quintela wrote:
Make the ps2 device track its ledstate so that we can migrate it.
Otherwise it gets lost across migration, and spice-server gets
confused about the actual keyboard state and sends bogus
caps/scroll/num key events. This fixes
Make the ps2 device track its ledstate so that we can migrate it.
Otherwise it gets lost across migration, and spice-server gets
confused about the actual keyboard state and sends bogus
caps/scroll/num key events. This fixes RH bug #729294
Signed-off-by: Christophe Fergeau cferg...@redhat.com
Make the ps2 device track its ledstate so that we can migrate it.
Otherwise it gets lost across migration, and spice-server gets
confused about the actual keyboard state and sends bogus
caps/scroll/num key events.
Signed-off-by: Christophe Fergeau cferg...@redhat.com
---
hw/ps2.c | 13
Hey,
On Wed, Oct 12, 2011 at 06:35:28PM +0200, Christophe Fergeau wrote:
Make the ps2 device track its ledstate so that we can migrate it.
Otherwise it gets lost across migration, and spice-server gets
confused about the actual keyboard state and sends bogus
caps/scroll/num key events
Make the ps2 device track its ledstate so that we can migrate it.
Otherwise it gets lost across migration, and spice-server gets
confused about the actual keyboard state and sends bogus
caps/scroll/num key events. This fixes RH bug #729294
Signed-off-by: Christophe Fergeau cferg...@redhat.com
Ping?
On Mon, Jul 04, 2011 at 06:10:43PM +0200, Christophe Fergeau wrote:
vcard_emul_mirror_card and vcard_emul_init use
vcard_emul_alloc_arrays to allocate memory for temporary arrays
which will contain elements that in the end will be used one by
one in cac_card_init. The arrays themselves
vcard_emul_options now has repetitive code to read the current
token and advance to the next. After the previous changes,
this repetitive code can be moved in a NEXT_TOKEN macro to
avoid having this code duplicated.
Signed-off-by: Christophe Fergeau cferg...@redhat.com
Reviewed-by: Alon Levy al
copy_string reimplements strndup, this commit removes it and
replaces all copy_string uses with strndup.
Signed-off-by: Christophe Fergeau cferg...@redhat.com
Reviewed-by: Alon Levy al...@redhat.com
---
libcacard/vcard_emul_nss.c | 23 ++-
1 files changed, 6 insertions
. valgrind rightfully
complained about this. It also didn't handle empty fields correctly,
Signed-off-by: Christophe Fergeau cferg...@redhat.com
Reviewed-by: Alon Levy al...@redhat.com
---
libcacard/vcard_emul_nss.c | 45 +++
1 files changed, 28 insertions
the memory that was allocated for these arrays should be freed
after use or they will be leaked.
Signed-off-by: Christophe Fergeau cferg...@redhat.com
---
libcacard/vcard_emul_nss.c | 11 ++-
1 files changed, 10 insertions(+), 1 deletions(-)
diff --git a/libcacard/vcard_emul_nss.c b
2/2 is unchanged
Christophe Fergeau (4):
libcacard: fix soft=... parsing in vcard_emul_options
libcacard: introduce NEXT_TOKEN macro
libcacard: replace copy_string with strndup
libcacard: don't leak vcard_emul_alloc_arrays mem
libcacard/vcard_emul_nss.c | 124
vcard_emul_options now has repetitive code to read the current
token and advance to the next. After the previous changes,
this repetitive code can be moved in a NEXT_TOKEN macro to
avoid having this code duplicated.
Signed-off-by: Christophe Fergeau cferg...@redhat.com
Reviewed-by: Alon Levy al
copy_string reimplements strndup, this commit removes it and
replaces all copy_string uses with strndup.
Signed-off-by: Christophe Fergeau cferg...@redhat.com
Reviewed-by: Alon Levy al...@redhat.com
---
libcacard/vcard_emul_nss.c | 23 ++-
1 files changed, 6 insertions
. valgrind rightfully
complained about this. It also didn't handle empty fields correctly,
Signed-off-by: Christophe Fergeau cferg...@redhat.com
Reviewed-by: Alon Levy al...@redhat.com
---
libcacard/vcard_emul_nss.c | 45 +++
1 files changed, 28 insertions
will do what we expect.
Signed-off-by: Christophe Fergeau cferg...@redhat.com
Reviewed-by: Alon Levy al...@redhat.com
---
libcacard/vcard_emul_nss.c |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/libcacard/vcard_emul_nss.c b/libcacard/vcard_emul_nss.c
index f3db657
On Fri, Jul 22, 2011 at 01:33:26PM +0200, Christophe Fergeau wrote:
This is a resend of my series of libcacard fixes, no change since last
version apart from the added Reviewed-by tag.
Scratch that, I put the wrong patches in the series (which strongly
indicates that I should merge libcacard
vcard_emul_mirror_card and vcard_emul_init use
vcard_emul_alloc_arrays to allocate memory for temporary arrays
which will contain elements that in the end will be used one by
one in cac_card_init. The arrays themselves are never stored
anywhere, they are only used as temporary containers. Hence
Hi,
On Mon, Jun 27, 2011 at 12:34:44PM +0200, Alon Levy wrote:
Also add --pkgconfigdir and --includedir configure parameters.
---
configure | 10 ++
libcacard/Makefile| 25 ++---
libcacard/libcacard.pc.in | 13 +
3 files
On Fri, Jun 24, 2011 at 06:51:51PM +0200, Alon Levy wrote:
On Fri, Jun 24, 2011 at 04:37:39PM +0200, Christophe Fergeau wrote:
The previous parser had copy and paste errors when computing
vname_length and type_params_length, name was used instead
of respectively vname and type_params
. valgrind rightfully
complained about this. It also didn't handle empty fields correctly,
Signed-off-by: Christophe Fergeau cferg...@redhat.com
---
libcacard/vcard_emul_nss.c | 45 +++
1 files changed, 28 insertions(+), 17 deletions(-)
diff --git a/libcacard
copy_string reimplements strndup, this commit removes it and
replaces all copy_string uses with strndup.
Signed-off-by: Christophe Fergeau cferg...@redhat.com
---
libcacard/vcard_emul_nss.c | 23 ++-
1 files changed, 6 insertions(+), 17 deletions(-)
diff --git a/libcacard
1 - 100 of 125 matches
Mail list logo