From: Anthony Liguori <aliguori@linux.vnet.ibm.com>
To: Alexander Graf <agraf@suse.de>
Cc: Corentin Chary <corentincj@iksaif.net>,
Qemu-development List <qemu-devel@nongnu.org>
Subject: [Qemu-devel] Re: [PATCH 2/5] vnc: JPEG should be disabled if the client don't set tight quality
Date: Fri, 04 Jun 2010 08:19:01 -0500 [thread overview]
Message-ID: <4C08FD45.8000404@linux.vnet.ibm.com> (raw)
In-Reply-To: <17E76748-6C1D-4A9A-8AD2-97AA128C189E@suse.de>
On 06/04/2010 08:16 AM, Alexander Graf wrote:
> On 04.06.2010, at 15:18, Corentin Chary wrote:
>
>
>> Disable JPEG compression by default and only enable it if the
>> VNC client has sent the requested quality.
>>
>> Signed-off-by: Corentin Chary<corentincj@iksaif.net>
>> ---
>> vnc.c | 2 +-
>> 1 files changed, 1 insertions(+), 1 deletions(-)
>>
>> diff --git a/vnc.c b/vnc.c
>> index ed0e096..9cf38d1 100644
>> --- a/vnc.c
>> +++ b/vnc.c
>> @@ -1644,7 +1644,7 @@ static void set_encodings(VncState *vs, int32_t *encodings, size_t n_encodings)
>> vs->features = 0;
>> vs->vnc_encoding = 0;
>> vs->tight_compression = 9;
>> - vs->tight_quality = 9;
>> + vs->tight_quality = -1; /* Lossless by default */
>>
> So what's the threshold? When do you start to be lossy?
>
When the client explicitly requests it.
Regards,
Anthony Liguori
> Alex
>
>
next prev parent reply other threads:[~2010-06-04 13:20 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-06-04 13:18 [Qemu-devel] [PATCH 0/5] vnc updates and ui move Corentin Chary
[not found] ` <1275657527-26032-5-git-send-email-corentincj@iksaif.net>
2010-06-04 13:18 ` [Qemu-devel] Re: [PATCH 4/5] ui: move all ui components in ui/ Alexander Graf
2010-06-04 14:23 ` Corentin Chary
2010-06-04 13:18 ` [Qemu-devel] [PATCH 1/5] vnc: tight: add JPEG and gradient subencoding with smooth image detection Corentin Chary
2010-06-04 13:18 ` [Qemu-devel] [PATCH 2/5] vnc: JPEG should be disabled if the client don't set tight quality Corentin Chary
2010-06-04 13:16 ` [Qemu-devel] " Alexander Graf
2010-06-04 13:19 ` Anthony Liguori [this message]
2010-06-04 13:20 ` Alexander Graf
2010-06-04 13:48 ` Corentin Chary
2010-06-04 13:52 ` Alexander Graf
2010-06-04 14:22 ` Corentin Chary
2010-06-04 13:18 ` [Qemu-devel] [PATCH 3/5] vnc: add lossless option Corentin Chary
2010-06-04 13:17 ` [Qemu-devel] " Alexander Graf
2010-06-04 13:20 ` Anthony Liguori
2010-06-04 13:18 ` [Qemu-devel] [PATCH 5/5] vnc: rename vnc-encoding-* vnc-enc-* Corentin Chary
2010-06-04 13:18 ` [Qemu-devel] " Alexander Graf
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4C08FD45.8000404@linux.vnet.ibm.com \
--to=aliguori@linux.vnet.ibm.com \
--cc=agraf@suse.de \
--cc=corentincj@iksaif.net \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).