quot;is quite the same"
>>> and "that of coefficient position".
>> "is quite the same" isn't idiomatic English either. I can't
>> tell if the author meant "is quite similar" or "is exactly
>> the same", so we probabl
eter Lieven" a écrit :
> On 13.03.2012 16:06, Alexander Graf wrote:
>
>> On 13.03.2012, at 16:05, Corentin Chary wrote:
>>
>> On Tue, Mar 13, 2012 at 12:29 PM, Peter Lieven wrote:
>>>
>>>> On 11.02.2012 09:55, Corentin Chary wrote
On Wed, Mar 14, 2012 at 11:16 PM, Stefan Weil wrote:
> Am 14.03.2012 22:46, schrieb Anthony Liguori:
>
>> On 03/14/2012 01:58 AM, Corentin Chary wrote:
>>>
>>> Hi Anthony,
>>>
>>> Please merge these two patchs from another age, they fix crash in t
From: Corentin Chary
The threaded VNC servers messed up with QEMU fd handlers without
any kind of locking, and that can cause some nasty race conditions.
Using qemu_mutex_lock_iothread() won't work because vnc_dpy_cpy(),
which will wait for the current job queue to finish, can be called
From: Stefan Weil
This fixes memory reads and writes which exceeded the upper limit
of allocated memory vd->guest.ds->data and vd->server->data.
Cc: Anthony Liguori
Signed-off-by: Stefan Weil
---
ui/vnc.c |3 +++
1 files changed, 3 insertions(+), 0 deletions(-)
diff --git a/ui/vnc.c b/ui
Hi Anthony,
Please merge these two patchs from another age, they fix crash in the VNC
server (the iohandler one is only for the threaded server).
Thanks,
Corentin Chary (1):
vnc: don't mess up with iohandlers in the vnc thread
Stefan Weil (1):
vnc: Limit r/w access to size of allo
On Tue, Mar 13, 2012 at 12:29 PM, Peter Lieven wrote:
> On 11.02.2012 09:55, Corentin Chary wrote:
>>
>> On Thu, Feb 9, 2012 at 7:08 PM, Peter Lieven wrote:
>>>
>>> Hi,
>>>
>>> is anyone aware if there are still problems when enabling the threa
On Mon, Feb 13, 2012 at 10:24 AM, Peter Lieven wrote:
>
> Am 11.02.2012 um 09:55 schrieb Corentin Chary:
>
>> On Thu, Feb 9, 2012 at 7:08 PM, Peter Lieven wrote:
>>> Hi,
>>>
>>> is anyone aware if there are still problems when enabling the threaded v
go but somehow
get lost on the mailing list. I rebased the tree but did not have the
time (yet) to test them.
http://git.iksaif.net/?p=qemu.git;a=shortlog;h=refs/heads/wip
Feel free to try them. If QEMU segfault again, please send a full gdb
backtrace / valgrind trace / way to reproduce :).
Thanks,
--
Corentin Chary
http://xf.iksaif.net
The patch looks right, maybe you should send it directly to the qemu
mailing list.
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/806656
Title:
Tight PNG VNC encoding is sent even when --disable-vnc
how would coroutines help ?
Do you plan to rewrite the server using coroutines instead of threads
?
Thanks
--
Corentin Chary
http://xf.iksaif.net
Aurelien, could you also take a look at
http://patchwork.ozlabs.org/patch/87717/ ?
Thanks,
--
Corentin Chary
http://xf.iksaif.net
alid symbol in the latest tip.
>
> Regards,
Patch updated here: http://patchwork.ozlabs.org/patch/87722/
--
Corentin Chary
http://xf.iksaif.net
is whole patch review thing is about :).
>
> Alex
>
Hi,
Also, I'd suggest you to start here: http://wiki.qemu.org/Contribute/StartHere .
You can also pick a GSOC 2010/2011 project, and start working on it
outside of GSOC context if you want.
And keep in mind that if you want to write code send patchs for QEMU,
nobody can stop you :).
--
Corentin Chary
http://xf.iksaif.net
int strategy)
> {
> z_streamp zstream = &vs->tight.stream[stream_id];
> - int previous_out;
> + uLong previous_out;
>
> if (bytes < VNC_TIGHT_MIN_TO_COMPRESS) {
> vnc_write(vs, vs->tight.tight.buffer, vs->tight.tight.offset);
>
>
Hi Ulrich,
Looks a lot like "vnc: tight: Fix crash after 2GB of output", right ?
--
Corentin Chary
http://xf.iksaif.net
From: Wen Congyang
This patch fix the following regression:
1. we should use bitmap_set() and bitmap_clear() to replace vnc_set_bits().
Signed-off-by: Wen Congyang
Signed-off-by: Corentin Chary
---
ui/vnc.c |8 ++--
1 files changed, 6 insertions(+), 2 deletions(-)
diff --git a/ui
ff-by: Corentin Chary
---
ui/vnc-enc-tight.c |5 +++--
ui/vnc-enc-zlib.c |4 ++--
2 files changed, 5 insertions(+), 4 deletions(-)
diff --git a/ui/vnc-enc-tight.c b/ui/vnc-enc-tight.c
index 2522936..87fdf35 100644
--- a/ui/vnc-enc-tight.c
+++ b/ui/vnc-enc-tight.c
@@ -868,8 +868,8 @@ stati
Hi Anthony,
Here are some vnc related patchs that should be merged.
Thanks,
Corentin Chary (1):
vnc: don't mess up with iohandlers in the vnc thread
Michael Tokarev (1):
vnc: tight: Fix crash after 2GB of output
Stefan Weil (1):
vnc: Limit r/w access to size of allocated memory
From: Stefan Weil
This fixes memory reads and writes which exceeded the upper limit
of allocated memory vd->guest.ds->data and vd->server->data.
Cc: Anthony Liguori
Signed-off-by: Stefan Weil
Signed-off-by: Corentin Chary
---
ui/vnc.c |3 +++
1 files changed, 3 inse
held.
Instead, we now store the data in a temporary buffer, and use a bottom
half to notify the main thread that new data is available.
vnc_[un]lock_ouput() is still needed to access VncState members like
abort, csock or jobs_buffer.
Signed-off-by: Corentin Chary
---
ui/vnc-jobs-async.c |
On Thu, Mar 10, 2011 at 3:13 PM, Corentin Chary
wrote:
> The threaded VNC servers messed up with QEMU fd handlers without
> any kind of locking, and that can cause some nasty race conditions.
>
> Using qemu_mutex_lock_iothread() won't work because vnc_dpy_cpy(),
> which will
viadeoBonjour,
Je vous invite à rejoindre mon réseau de contacts sur Viadeo.
Cordialement,Corentin Chary Confirmer que vous connaissez Corentin
http://www.viadeo.com/action/index.jsp?actionId=0021xn1x7dm9pjk8&urlId=0021v1e4bjtkdcp0&ut
held.
Instead, we now store the data in a temporary buffer, and use a bottom
half to notify the main thread that new data is available.
vnc_[un]lock_ouput() is still needed to access VncState members like
abort, csock or jobs_buffer.
Signed-off-by: Corentin Chary
---
ui/vnc-jobs-async.c |
On Thu, Mar 10, 2011 at 1:45 PM, Anthony Liguori wrote:
> On 03/10/2011 07:06 AM, Paolo Bonzini wrote:
>>
>> On 03/10/2011 01:59 PM, Corentin Chary wrote:
>>>
>>> Instead, we now store the data in a temporary buffer, and use a socket
>>> pair to notify
ed to access VncState members like
abort, csock or jobs_buffer.
Thanks to Jan Kiszka for helping me solve this issue.
Signed-off-by: Corentin Chary
---
ui/vnc-jobs-async.c | 50 --
ui/vnc-jobs-sync.c |4
ui/vnc-jobs.h |1 +
ui/
Signed-off-by: Corentin Chary
---
osdep.c | 83 +
qemu_socket.h |1 +
2 files changed, 84 insertions(+), 0 deletions(-)
diff --git a/osdep.c b/osdep.c
index 327583b..93bfbe0 100644
--- a/osdep.c
+++ b/osdep.c
@@ -147,6 +147,89
On Wed, Mar 9, 2011 at 1:51 PM, Paolo Bonzini wrote:
> On 03/09/2011 02:21 PM, Corentin Chary wrote:
>>
>> The threaded VNC servers messed up with QEMU fd handlers without
>> any kind of locking, and that can cause some nasty race conditions.
>>
>> The IO-Th
On Wed, Mar 9, 2011 at 1:21 PM, Corentin Chary wrote:
> The threaded VNC servers messed up with QEMU fd handlers without
> any kind of locking, and that can cause some nasty race conditions.
>
> The IO-Thread provides appropriate locking primitives to avoid that.
> T
around the two faulty calls.
Thanks to Jan Kiszka for helping me solve this issue.
Cc: Jan Kiszka
Signed-off-by: Corentin Chary
---
The previous patch was total crap, introduced race conditions,
and probably crashs on client disconnections.
configure |9 +
ui/vnc-jobs
u can try to define some global mutex acting like iothread
locks, but that doesn't sounds like an easy fix.
--
Corentin Chary
http://xf.iksaif.net
around the two faulty calls.
Thanks to Jan Kiszka for helping me solve this issue.
Cc: Jan Kiszka
Signed-off-by: Corentin Chary
---
configure |9 +
ui/vnc-jobs-async.c |4
2 files changed, 13 insertions(+), 0 deletions(-)
diff --git a/configure b/configure
index
mutex to protect handler updates against
> the main select loop, this won't help here. So vnc-threading likely
> needs to be made dependent on --enable-io-thread.
Plus proper lock_iothread() calls right ?
If yes I'll send a patch for that (or you can do it, as you want).
Thanks,
--
Corentin Chary
http://xf.iksaif.net
aded mode? Only the vnc thread(s), or also other qemu
> threads? In the former case, just avoid setting a write handler at all.
Cheap stuff is done by the main thread (cursor, etc...). The thread
only do framebuffer updates.
--
Corentin Chary
http://xf.iksaif.net
e in a temporary buffer, and only send it to
the network (real vnc_write calls with csock correctly set) once it's
successfully finished.
The is only two functions calls that break this isolation are the two
that I pointed out earlier.
--
Corentin Chary
http://xf.iksaif.net
, vs.output.offset);
+/* unlock */
disconnected:
/* Copy persistent encoding data */
@@ -267,7 +269,9 @@ disconnected:
vnc_unlock_output(job->vs);
if (flush) {
+/* lock */
vnc_flush(job->vs);
+ /* unlock */
}
vnc_lock_queue(queue)
--
Corentin Chary
http://xf.iksaif.net
hing that was already fixed sometime ago.
What version exactly are you using ? Could you try with the one from
stable-0.14 git branch ?
Thanks,
--
Corentin Chary
http://xf.iksaif.net
> Can confirm screen update problems (wrong colors, e.g. red blocks).
With wich client, encoding, pixel format ? Does using another client
fix the issue ? Could you provide more informations (wireshark dumps,
screenshots, etc...) ?
Thanks,
--
Corentin Chary
http://xf.iksaif.net
ting
http://wiki.qemu.org/Contribute/SubmitAPatch:
"Send patches inline so they are easy to reply to with review
comments. Do not put patches in attachments".
I mainly asked yo to re-send it so Anthony has less chance to miss it
if he is cc and if the patch is inline :), that's all.
Thanks,
--
Corentin Chary
http://xf.iksaif.net
On Sat, Mar 5, 2011 at 6:56 AM, Michael Tokarev wrote:
> 05.03.2011 00:08, Corentin Chary wrote:
>> On Fri, Mar 4, 2011 at 12:46 PM, Michael Tokarev wrote:
> []
>>> The attached slightly different patch fixes both places and fixes
>>> them for good (hopefully an
On Fri, Mar 4, 2011 at 12:46 PM, Michael Tokarev wrote:
> 04.03.2011 11:56, Corentin Chary wrote:
>>>>
>>>> bytes = zstream->total_out - previous_out;
>>
>> Good catch
>>
>>> total_out isn't used by zlib internally, so if the re
rong for 64 bit
> hosts because of a rounding effect: for these hosts, VNC_MAX_WIDTH is no
> longer a multiple of (16 * BITS_PER_LONG), so the rounded value of
> VNC_DIRTY_WORDS was too small.
>
> Fix both declarations by using the macro which is designed for this
> purpose.
&
use the resulting value directly as "bytes", without
> saving its previous value in previous_out. Something like
> the attached patch does.
If you're certain that total_out is not used by zlib, could you also
send a patch for zlib encoding please ? (vnc-enc-zlib.c)
Thanks,
--
Corentin Chary
http://xf.iksaif.net
deup(x, y) (((x) + ((y) - 1)) / (y))
osdep.h:#define DIV_ROUND_UP(n,d) (((n) + (d) - 1) / (d))
--
Corentin Chary
http://xf.iksaif.net
e num of words to bitmap_intersects().
>
> Changes from v1 to v2:
> 1. fix the third argument of bitmap_clear()
>
> Signed-off-by: Wen Congyang
Acked-by: Corentin Chary
> ---
> ui/vnc.c | 11 ---
> 1 files changed, 8 insertions(+), 3 deletions(-)
>
> diff --git
dows specific!
>
>
> The third argument of bitmap_clear() is number of bits to be cleared, but we
> pass
> the end bits to be cleared to bitmap_clear().
>
> Signed-off-by: Wen Congyang
> Reported-by: Stefan Weil
Acked-by: Corentin Chary
> ---
> ui/vn
h appropriate Signed-off-by: and changelog for the other patch
- using a const size_t width = ds_get_width(vs->ds) / 16; in both
patchs to make the call more explicit
Thanks,
--
Corentin Chary
http://xf.iksaif.net
Queue, resulting in a big and ugly memleak.
Signed-off-by: Corentin Chary
---
I believe this is a (slightly) better patch than Bruce's one, because
it reduce memory allocations by using always the same buffer.
ui/vnc-jobs-async.c |2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff
On Wed, Feb 23, 2011 at 11:23 PM, Anthony Liguori
wrote:
> On 02/04/2011 02:05 AM, Corentin Chary wrote:
>>
>> agraf reported that qemu_mutex_destroy(vs->output_mutex) while failing
>> in vnc_disconnect_finish().
>>
>> It's because vnc_worker_thread_loop
ng some letter => not seen. When you press
> enter => OK.
Did you try http://patchwork.ozlabs.org/patch/84517/ ?
What vnc settings are you using ? What VNC encoding did you client choose ?
Thanks,
--
Corentin Chary
http://xf.iksaif.net
- if (bitmap_intersects(vd->guest.dirty[y], width_mask,
> VNC_DIRTY_WORDS)) {
> + if (bitmap_intersects(vd->guest.dirty[y], width_mask,
> + VNC_DIRTY_WORDS * BITS_PER_LONG)) {
> int x;
> uint8_t *guest_ptr;
> uint8_t *server_ptr;
Good catch,
Thanks,
Acked-by: Corentin Chary
--
Corentin Chary
http://xf.iksaif.net
ed timersub when not available.
Since I don't have any working mingw toolchain, cool you make a patch
for those two ?
Thanks,
--
Corentin Chary
http://xf.iksaif.net
./bitops.h:192: warning: ‘old’ is used uninitialized in this function
Signed-off-by: Corentin Chary
---
bitops.h |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/bitops.h b/bitops.h
index ae7bcb1..e2b9df3 100644
--- a/bitops.h
+++ b/bitops.h
@@ -187,7 +187,7 @@ static
On Thu, Feb 24, 2011 at 4:04 PM, Peter Maydell wrote:
> Add some missing #ifdefs to fix compilation failures in the !CONFIG_VNC_JPEG
> case introduced by commit ce702e93.
Ooops.
Acked-By: Corentin Chary
--
Corentin Chary
http://xf.iksaif.net
rs]
FreeNX / NX can only be run from userspace, there is no way to
implement that into QEMU.
Simply install it on your guests if you want to use it.
--
Corentin Chary
http://xf.iksaif.net
The force_jpeg threshold was too low.
Signed-off-by: Corentin Chary
---
ui/vnc-enc-tight.c | 20 ++--
1 files changed, 10 insertions(+), 10 deletions(-)
diff --git a/ui/vnc-enc-tight.c b/ui/vnc-enc-tight.c
index 81024d5..82c1e96 100644
--- a/ui/vnc-enc-tight.c
+++ b/ui/vnc
Switch to bitmap.h and bitops.h instead of redefining our own bitmap
helpers.
Signed-off-by: Corentin Chary
---
ui/vnc.c | 91 ++---
ui/vnc.h |7 +++--
2 files changed, 25 insertions(+), 73 deletions(-)
diff --git a/ui/vnc.c b/ui
This option allow to disable adaptive behaviors in some encodings.
Signed-off-by: Corentin Chary
---
qemu-options.hx|9 +
ui/vnc-enc-tight.c |2 +-
ui/vnc.c | 13 +
ui/vnc.h |1 +
4 files changed, 20 insertions(+), 5 deletions(-)
diff
Respect client size if it doesn't not support desktop resizing.
Signed-off-by: Corentin Chary
---
ui/vnc.c | 10 +-
1 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/ui/vnc.c b/ui/vnc.c
index 6c57b0e..86c23207 100644
--- a/ui/vnc.c
+++ b/ui/vnc.c
@@ -795,12 +7
The for loop in send_lossy_rect was totally wrong, and we can't
call vnc_set_bits() because it does not really do what it should.
Use vnc_set_bit() directly instead.
Signed-off-by: Corentin Chary
---
ui/vnc.c | 12
1 files changed, 8 insertions(+), 4 deletions(-)
diff --git
Signed-off-by: Corentin Chary
---
ui/vnc-enc-tight.c |4 ++--
ui/vnc-enc-zrle.c |3 ++-
2 files changed, 4 insertions(+), 3 deletions(-)
diff --git a/ui/vnc-enc-tight.c b/ui/vnc-enc-tight.c
index ad9a9a8..81024d5 100644
--- a/ui/vnc-enc-tight.c
+++ b/ui/vnc-enc-tight.c
@@ -1546,7
.
bitops_ffsl() and bitops_flsl() are here because bitops/bitmap works
on unsigned long, not int, and we can't use current code because:
* ffs only works on int
* qemu_fls only works on int
* ffsl is a GNU extension
Signed-off-by: Corentin Chary
---
Makefile.objs |1 +
bitmap.c |
This allow to use palette on the stack instead of always
allocating them.
Signed-off-by: Corentin Chary
---
ui/vnc-palette.c |8 +++-
ui/vnc-palette.h |1 +
2 files changed, 8 insertions(+), 1 deletions(-)
diff --git a/ui/vnc-palette.c b/ui/vnc-palette.c
index c47420b..f93250b
Use the new update frequency infrastructure to use jpeg for regions with
high update frequency.
Signed-off-by: Corentin Chary
---
ui/vnc-enc-tight.c | 75 +++-
1 files changed, 62 insertions(+), 13 deletions(-)
diff --git a/ui/vnc-enc-tight.c b
We now that the palette will never have more than 256
elements. Let's use a pool to reduce malloc calls.
Signed-off-by: Corentin Chary
---
ui/vnc-palette.c | 18 ++
ui/vnc-palette.h |3 ++-
2 files changed, 4 insertions(+), 17 deletions(-)
diff --git a/ui/vnc-palett
rect was not updated in the last 2 second, then the frequency
became 0, and all the stored timestamp are reseted.
Signed-off-by: Corentin Chary
---
ui/vnc.c | 101 ++
ui/vnc.h | 19 +++
2 files changed, 120 insertions(+), 0
This should not change the current behavior, but if any new
encoding try to use the tight quality, it will always be set
to -1 when lossy encodings are disabled.
Signed-off-by: Corentin Chary
---
ui/vnc.c |4 +++-
1 files changed, 3 insertions(+), 1 deletions(-)
diff --git a/ui/vnc.c b/ui
If an adaptive encoding has choosen to send a lossy update
based on the result of vnc_update_freq(), then it should advertise
it with vnc_sent_lossy_rect(). This will allow to automatically refresh
this rect once it's static again.
Signed-off-by: Corentin Chary
---
ui/vnc-jobs-async.c |
These two helpers are needed for zrle and zywrle.
Signed-off-by: Corentin Chary
---
ui/vnc-palette.c | 32
ui/vnc-palette.h |3 +++
2 files changed, 35 insertions(+), 0 deletions(-)
diff --git a/ui/vnc-palette.c b/ui/vnc-palette.c
index f93250b..c478060
From: Corentin Chary
Hi,
Since v2:
- Fixed some styles issues
- Rebased to current master
- Fixed a Makefile issue (using .c instead of .o)
I rebased the series against current master, it contains:
- Adaptive Tight Encoding: send lossy or lossless updates depending on the
update frequency
agraf reported that qemu_mutex_destroy(vs->output_mutex) while failing
in vnc_disconnect_finish().
It's because vnc_worker_thread_loop() tries to unlock the mutex while
not locked. The unlocking call doesn't fail (pthread bug ?), but
the destroy call does.
Signed-off-by: Corentin C
vnc: qemu can die if the client is disconnected while updating screen
http://patchwork.ozlabs.org/patch/80334/
(what about a "stable" alias instead of [PING x.x] ? Like sta...@kernel.org)
Thanks,
--
Corentin Chary
http://xf.iksaif.net
On Tue, Jan 25, 2011 at 10:03 AM, Stefan Hajnoczi wrote:
> On Tue, Jan 25, 2011 at 8:33 AM, Corentin Chary
> wrote:
>> From: Yoshiaki Tamura
>>
>> Currently qemu_set_fd_handler2() is only setting ioh->deleted upon
>> deleting. This may cause
s, it'll be
called, and then crashes because of lack of resources. This patch
fixes it.
Signed-off-by: Yoshiaki Tamura
Reviewed-by: Corentin Chary
---
vl.c |2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/vl.c b/vl.c
index 14255c4..7a26bea 100644
--- a/vl.c
+++ b/vl
From: Corentin Chary
agraf reported that qemu_mutex_destroy(vs->output_mutex) was failing
in vnc_disconnect_finish() when the vnc client was disconnected while
updating the screen.
It's because vnc_worker_thread_loop() tries to unlock the mutex while
not locked.
Signed-off-by: Corent
Hi Anthony,
If you want to enable vnc threaded server by default, you should
really merge these two lost patchs :).
Thanks,
Corentin Chary (1):
vnc: qemu can die if the client is disconnected while updating screen
Yoshiaki Tamura (1):
vl.c: set NULL upon deleting handlers in
d bug ?), but
the destroy call does.
Signed-off-by: Corentin Chary
---
ui/vnc-jobs-async.c |4
1 files changed, 4 insertions(+), 0 deletions(-)
diff --git a/ui/vnc-jobs-async.c b/ui/vnc-jobs-async.c
index 6e9cf08..0b5d750 100644
--- a/ui/vnc-jobs-async.c
+++ b/ui/vnc-jobs-async.c
@@ -
__ffs(1): 0
ffs(1): 1
This behavior is needed for all the other function to work (and makes
more sense that the normal ffs).
But .. maybe I should just use ffs() - 1
--
Corentin Chary
http://xf.iksaif.net
.
Signed-off-by: Corentin Chary
---
Makefile.objs |1 +
bitmap.c | 255 +
bitmap.h | 222 ++
bitops.c | 142 ++
bitops.h | 272
Signed-off-by: Corentin Chary
---
ui/vnc-enc-tight.c |4 ++--
ui/vnc-enc-zrle.c |3 ++-
2 files changed, 4 insertions(+), 3 deletions(-)
diff --git a/ui/vnc-enc-tight.c b/ui/vnc-enc-tight.c
index ad9a9a8..81024d5 100644
--- a/ui/vnc-enc-tight.c
+++ b/ui/vnc-enc-tight.c
@@ -1546,7
This option allow to disable adaptive behaviors in some encodings.
Signed-off-by: Corentin Chary
---
qemu-options.hx|9 +
ui/vnc-enc-tight.c |2 +-
ui/vnc.c | 13 +
ui/vnc.h |1 +
4 files changed, 20 insertions(+), 5 deletions(-)
diff
This allow to use palette on the stack instead of always
allocating them.
Signed-off-by: Corentin Chary
---
ui/vnc-palette.c |8 +++-
ui/vnc-palette.h |1 +
2 files changed, 8 insertions(+), 1 deletions(-)
diff --git a/ui/vnc-palette.c b/ui/vnc-palette.c
index c47420b..f93250b
Respect client size if it doesn't not support desktop resizing.
Signed-off-by: Corentin Chary
---
ui/vnc.c | 10 +-
1 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/ui/vnc.c b/ui/vnc.c
index 600ea75..9189014 100644
--- a/ui/vnc.c
+++ b/ui/vnc.c
@@ -795,12 +7
Use the new update frequency infrastructure to use jpeg for regions with
high update frequency.
Signed-off-by: Corentin Chary
---
ui/vnc-enc-tight.c | 75 +++-
1 files changed, 62 insertions(+), 13 deletions(-)
diff --git a/ui/vnc-enc-tight.c b
rect was not updated in the last 2 second, then the frequency
became 0, and all the stored timestamp are reseted.
Signed-off-by: Corentin Chary
---
ui/vnc.c | 101 ++
ui/vnc.h | 19 +++
2 files changed, 120 insertions(+), 0
The force_jpeg threshold was too low.
Signed-off-by: Corentin Chary
---
ui/vnc-enc-tight.c | 20 ++--
1 files changed, 10 insertions(+), 10 deletions(-)
diff --git a/ui/vnc-enc-tight.c b/ui/vnc-enc-tight.c
index 81024d5..82c1e96 100644
--- a/ui/vnc-enc-tight.c
+++ b/ui/vnc
We now that the palette will never have more than 256
elements. Let's use a pool to reduce malloc calls.
Signed-off-by: Corentin Chary
---
ui/vnc-palette.c | 18 ++
ui/vnc-palette.h |3 ++-
2 files changed, 4 insertions(+), 17 deletions(-)
diff --git a/ui/vnc-palett
Switch to bitmap.h and bitops.h instead of redefining our own bitmap
helpers.
Signed-off-by: Corentin Chary
---
ui/vnc.c | 91 ++---
ui/vnc.h |7 +++--
2 files changed, 25 insertions(+), 73 deletions(-)
diff --git a/ui/vnc.c b/ui
If an adaptive encoding has choosen to send a lossy update
based on the result of vnc_update_freq(), then it should advertise
it with vnc_sent_lossy_rect(). This will allow to automatically refresh
this rect once it's static again.
Signed-off-by: Corentin Chary
---
ui/vnc-jobs-async.c |
This should not change the current behavior, but if any new
encoding try to use the tight quality, it will always be set
to -1 when lossy encodings are disabled.
Signed-off-by: Corentin Chary
---
ui/vnc.c |4 +++-
1 files changed, 3 insertions(+), 1 deletions(-)
diff --git a/ui/vnc.c b/ui
The for loop in send_lossy_rect was totally wrong, and we can't
call vnc_set_bits() because it does not really do what it should.
Use vnc_set_bit() directly instead.
Signed-off-by: Corentin Chary
---
ui/vnc.c | 12
1 files changed, 8 insertions(+), 4 deletions(-)
diff --git
These two helpers are needed for zrle and zywrle.
Signed-off-by: Corentin Chary
---
ui/vnc-palette.c | 33 +
ui/vnc-palette.h |3 +++
2 files changed, 36 insertions(+), 0 deletions(-)
diff --git a/ui/vnc-palette.c b/ui/vnc-palette.c
index f93250b..d691a0c
other encoding for lossless
updates,
like zlib or tight.
- Bitmap module: create bitmap.h and bitops.h, and remove duplicate code
from vnc.c
It was my last series from GSoC 2010 context, if necessary I can send different
series for adaptive vnc, zrle and bitmap stuff.
Thanks,
Corentin Char
gt;
There is two patch for that
http://patchwork.ozlabs.org/patch/62420/
http://patchwork.ozlabs.org/patch/62263/
On of these patch is already in master:
http://git.qemu.org/qemu.git/commit/?id=ac71103dc6b408775ae72067790ab367912f75ec
--
Corentin Chary
http://xf.iksaif.net
or? Most open source stuff is multiplatform, and serious
>> commercial work needs something faster than tcg.
>>
>
> *-user is useful to GCC developers (and used by those who work on the ARM
> backend to run the GCC testsuite).
>
> Paolo
>
>
>
Isn't that also used by scratchbox ?
--
Corentin Chary
http://xf.iksaif.net
ioh->deleted = 1;
> + ioh->fd_read = NULL;
> + ioh->fd_write = NULL;
> break;
> }
> }
> --
> 1.7.1.1
>
>
Good catch, thanks,
Reviewed-by: Corentin Chary
--
Corentin Chary
http://xf.iksaif.net
;
> - } else {
> + } else if (vs->csock != -1) {
> qemu_set_fd_handler2(vs->csock, NULL, vnc_client_read, NULL, vs);
> }
> vnc_unlock_output(vs);
> --
> 1.7.1.1
>
>
Acked-By: Corentin Chary
--
Corentin Chary
http://xf.iksaif.net
The force_jpeg threshold was too low.
Signed-off-by: Corentin Chary
---
ui/vnc-enc-tight.c | 20 ++--
1 file changed, 10 insertions(+), 10 deletions(-)
diff --git a/ui/vnc-enc-tight.c b/ui/vnc-enc-tight.c
index 9f83235..b0181ff 100644
--- a/ui/vnc-enc-tight.c
+++ b/ui/vnc-enc
On Wed, Aug 11, 2010 at 3:06 PM, Chris Krumme
wrote:
> Hello Corentin,
>
> On 08/11/2010 12:49 AM, Corentin Chary wrote:
>>
>> The force_jpeg threshold was too low.
>>
>> Signed-off-by: Corentin Chary
>> ---
>> qemu-thread.c | 1 +
>> u
Switch to bitmap.h and bitops.h instead of redefining our own bitmap
helpers.
Signed-off-by: Corentin Chary
---
ui/vnc.c | 91 ++---
ui/vnc.h |7 +++--
2 files changed, 25 insertions(+), 73 deletions(-)
diff --git a/ui/vnc.c b/ui
This option allow to disable adaptive behaviors in some encodings.
Signed-off-by: Corentin Chary
---
qemu-options.hx|9 +
ui/vnc-enc-tight.c |2 +-
ui/vnc.c | 13 +
ui/vnc.h |1 +
4 files changed, 20 insertions(+), 5 deletions(-)
diff
1 - 100 of 259 matches
Mail list logo