Bug 28815 - MJPEG luminocity fixes for server major 1
Summary: MJPEG luminocity fixes for server major 1
Status: RESOLVED WONTFIX
Alias: None
Product: Spice
Classification: Unclassified
Component: spicec (deprecated) (show other bugs)
Version: unspecified
Hardware: All All
: medium normal
Assignee: Spice Bug List
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-06-29 05:43 UTC by Attila Sukosd
Modified: 2015-06-19 18:03 UTC (History)
1 user (show)

See Also:
i915 platform:
i915 features:


Attachments
Incorrect behaviour (362.11 KB, image/png)
2010-06-29 05:43 UTC, Attila Sukosd
Details
Correct behaviour (715.09 KB, image/png)
2010-06-29 05:43 UTC, Attila Sukosd
Details

Description Attila Sukosd 2010-06-29 05:43:08 UTC
Created attachment 36606 [details]
Incorrect behaviour

Using the latest (29/June/2010) development client with the stable v0.4 server, sometimes the video streams seem to have the blue and red channels inverted.
Comment 1 Attila Sukosd 2010-06-29 05:43:44 UTC
Created attachment 36607 [details]
Correct behaviour
Comment 2 Alexander Larsson 2010-06-29 07:58:01 UTC
There are two issues here. The old code that converted from rgb to yuv for conversion to jpeg had two bugs:

1) It swapped the red and blue channels. It reswapped it in the old client so it wasn't a problem there, but the yuv values were not quite right which could cause worse compression. This should be easy to fix in the new client by just swapping if peer_major is 1

2) It converted to the "traditional" yuv where y goes from 16-235 (PIX_FMT_YUV420P in ffmpeg), but jpeg uses the "full" YUV version where y goes from 0-255 (PIX_FMT_YUVJ420P in ffmpeg). This means that if you decode such a file correctly (assuming the full y range) the intensity will look kinda compress towards the middle intensity. This might be kinda hard to undo without being slow...
Comment 3 Alexander Larsson 2010-07-20 05:15:11 UTC
I fixed the channel swapping, the luminocity still needs fixing
Comment 4 aix 2011-04-14 02:02:46 UTC
I haven't seen that luminocity problem in 0.8 components. Does still it need to be fixed?
Comment 5 Alexander Larsson 2011-04-14 09:45:59 UTC
Server major 1 is the 0.4 version of the protocol, i.e. when you use 0.6/0.8 to talk to a 0.4 server. So, this will not be visible in pure 0.8 setups.
Comment 6 Victor Toso 2015-06-19 18:03:00 UTC
spicec is deprecated.

If you hit this bug, we highly recommended virt-viewer

http://virt-manager.org/download/
http://www.spice-space.org/download.html


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.