Performance analysis of multi-core decoder

Optimised shortcut for decoding on single CPU systems

Limit the number of decoder threads to 4

They just end up burning CPU fighting each other without much
improvement to the decoding time beyond four threads.

Throttle overlapping screen updates

We need to make sure the display server has finished reading our
previous update before we overwrite the buffer with the next update.

Consolidate how to run the FLTK loop in one place

Propagate exceptions from worker threads back to main thread

Estimate CPU core usage in performance tests

This is based on comparing CPU time with elapsed time, which means
that the tests can be heavily influenced by other load on the test
machine.

Protect PlatformPixelBuffer from simultaneous access

The damage tracking region needs to be protected from multiple
threads accessing it at once. The rest should be fine though.

Create one decode thread for each CPU

We can start creating more decoding threads now that we handle
rect ordering properly. No point in creating more threads than
there are CPUs though.

Add option to enable thread sanitizer

Allow parallel decoding of Tight JPEG rects

The cost is some more load on malloc(), but it should be small
compared to the gains of spreading the CPU heavy JPEG decoding
over multiple CPUs.

Allow conditional dependencies between rects when decoding

Some encodings only cause dependencies between rects some of the
time. Make sure we can allow parallel decoding of those rect that
aren't dependent on each other.

Decode rectangles in order if needed

Some encodings must be handled in the order they are received.
Make sure we respect this in the decode manager.

Decode overlapping rectangles in order

Add flags member to decoders

Allows us to add attributes later that affect of the decoder
manager deals with the decoders.

Make the decoder multi-threaded

This implements the basic infrastructure for multi-threaded
decoding of rects. However there is just one thread reading data
and one thread decoding it. More logic is needed to safely decode
multiple rects at the same time.

Let CConnection intercept more callbacks

We need more callbacks for correct operation of multi-threaded
rect decoding.

Add abstraction classes for system thread primitives

Raise system requirement on Windows to Vista/2008

We need the newer versions of Windows to get access to better
thread primitives.

Fix copy-paste error for the secTypeX509Vnc security type

Clear up ZlibInStream::reset() behaviour

It previously only did a reset of the ZlibInStream object, not the
underlying zlib stream. It also had the side effect of flushing
the underlying stream and disassociating from it.

Clear things up by changing the naming, and introducing a proper
reset function (which is needed by the Tight decoder).

Split decoders into a read and decode step

We need to split these steps up in preparation for multi-core
support. Reading needs to be done in a serial manner, whilst
decoding can be done in parallel.

This also involved a rather large cleanup of the Tight decoder.

Add efficient method to copy data between two streams

Remove unneeded virtual qualifiers on stream methods

Remove CMsgReader::getImageBuf() as nothing uses it

Decouple decoders from CConnection

Make ConnParams useful when const

Delegate decoder object management to a separate class

Done in preparation for multi-core decoding. Keeps the complexity
out of the other classes. This also moves ownership of the
framebuffer in to CConnection. It's the CConnection object that is
aware of the threads and how to synchronise with them. Therefore
the ownership of the framebuffer must also be there to make sure
it isn't deleted whilst threads are working.

Clean out unused dependencies from CConnection.h

Include kernel time in CPU usage measurement

Time spent executing in the kernel is as much of a performance problem
as time spent in userspace. Make sure both numbers are included.

Remove old Windows CE code

Use mingw's gettimeofday()

mingw has a perfectly functional gettimeofday() so use that instead
of having multiple copies of our own version.

Remove unused deleteReaderAndWriter() method

option -xstartup added

option -xstartup added

Merge branch 'patch-1' of https://github.com/Siot/tigervnc

Xvnc.man: remove -i option

Xvnc does not understand -i as an alias to -interface anymore (since commit f8e3b34c69)
but it is still listed in the man page.

Fix man accordingly

Signed-off-by: Kir Kolyshkin <kir@openvz.org>

Merge branch 'master' of https://github.com/graysky2/tigervnc

Allow BUILD_TIMESTAMP to be set statically

Allow static linking against only the GCC libs

Merge branch 'jpeg_includes' of https://github.com/dcommander/tigervnc

Added -noxstartup

new option -noxstartup to not load any xstartup script file

provide a systemd user service unit for vncserver

update manpage teaching about ~/.vnc/config

update vncserver to parse a config file

Flush entire JPEG buffer, ignoring state

libjpeg doesn't update the buffer state before calling the
empty_output_buffer() callback so we need to flush everything,
not just the apparent size.

Ensure that libjpeg-turbo headers are included

On some systems, the build was picking up jpeglib.h from the system
include directories, and if the system's version of libjpeg[-turbo] used
a different API/ABI version than the one specified in JPEG_LIBRARY, this
led to a "Wrong JPEG library version" error at run time.

Merge pull request #219 from astrand/master

Meta no longer sends Super.

Meta no longer sends Super.

Since 83e019f599f409d60c12a3c0096f6b6d228d9fb1.

Merge branch 'vncserverfix' of https://github.com/michalsrb/tigervnc

Merge pull request #215 from bphinz/master

Improved x509 auth for java viewer

Fix exit routine in java viewer

Some exceptions were causing all viewers to exit, not just the
one that threw the exception.

More cleanup of x509 exception handling in java viewer

Also removed some unnecessary functions & variables in CSecurityTLS

Cleanup exception handling for x509* auth types in java viewer

Merge pull request #214 from bphinz/java-HostnameVerifier

Added hostname verification for x509 authentication types to java viewer

Added hostname verification for x509 authentication types

vncserver: Clean pid files of dead processes.

When Xvnc fails to start, delete the pid file. Clean pid files of dead processes
when listing them as well.

Enable Windows specific debug code in debug builds

Properly free temporary cursor buffer

Don't shadow member variable

Avoid dropping const qualifier needlessly

Use correct type for format string

Clean up listening sockets on error

Return TcpListener pointers rather than objects

It is easier to control object life time and avoid magical socket
duplication by having a single TcpListener object to pass around.
We have to be more careful about deleting the object though.

Return correct error code from bind()

Closing the socket might overwrite errno so we need to save
its value.

Don't leak socket on setsockopt() errors

Add address sanitizer switch

Useful for debugging memory leaks and access violations. It is
not available on Windows though, and there is some problem compiling
ObjectiveC++ with it turned on.

Make sure fallback block handler remains enabled

Commit f8e3b34 introduced a regression where the fallback write block
handler would cease working after the first time it was called,
potentially stalling writes.

Give context for strings used in menus

Makes it easier to see which hot-keys might conflict with each
other, and allow some variation in translation in the future.

Add support for pgettext()

Primarily gives it a shorthand, but also make gcc tolerate its
use for format strings.

Remove duplicate gettext_noop() definition

It's defined in gettext.h so no need for us to duplicate the work.

Update gettext.h to a more current version

Only include translation comments in .po files

Terminate the viewer even when waiting for data

It should be possible to exit the viewer even if the network has
stalled in the middle of a transfer.

Display partial updates on slow transfers

Normally we only display screen changes once we have the updates for
the entire screen. This may give the impression that the viewer is
hung though. So display the partial data if the update is taking to
long to arrive.

Track statistics for CopyRect

Merge pull request #198 from bphinz/bugfix/x509

Don't exit completely if single viewer declines server cert

Don't exit completely if single viewer declines server cert

Prevents viewer from exiting completely just because user
declined to trust the server certificate for a single connection.
Copied from TurboVNC.

Fixes for X509 CA certificate handling

95f39a5 introduced a regression whereby an exception would be thrown
when a PEM encoded CA certificate file containing one or more blank
lines was read in under Apple Java 6.  Additionally, CA certs were
being appended to the ~/.vnc/x509_savedcerts.pem file even if they
were already included in it.  Also fixes a possible FileNotFoundException
if the x509_savedcerts.pem file didn't exist.

Update German translation

Use REGION_INTERSECT() rather than trying to compute things manually

REGION_INIT() does not handle an empty or invalid BoxRec, so this
method makes sure we don't feed bad rects further in to the process.

Make vncHooksComposite() and vncHooksGlyphs() static

They should never have been global symbols to begin with.

Remove REGION_NULL fallback

It's been in the Xorg code base for ages. Any version that lacks it is
too old for the rest of our code to work anyway.

Refuse to start Xvnc if we failed to initialise a screen

Starting Xvnc without having any VNC functionality is pretty much
pointless. So terminate when that happens, making the situation easier
to detect for startup scripts.

Fix Viewport initial sizing issue on OS X

The layout manager on OS X sometimes sets the preferredSize to 0,
so only call pack if that's not the case.

Handle CA chain certificates

Allow the Java client to read CA certificates containing multiple
certs concatenated together.

Improve handling of x509 authentication

Fixes #193 and also adds automatic certificate saving feature like
the Fltk viewer.  Also pulls in CA certificates from all trusted
authorities known to the JRE for the case when the server is using
a certificate from a top-level authority.

Merge pull request #194 from bphinz/master

Compile legacy Xvnc with relative rpath for portability

Compile legacy Xvnc with relative rpath for portability

Build with rpath link so that Xvnc finds our libGL even if
it's not installed in "/".  Also applied dridir and xkbcompdir
patches so that the correct DRI driver gets loaded.

Add keyboard debug log to servers

Makes it easier to troubleshoot keyboard issues by printing exactly
what key events the server is getting from the client.

Merge pull request #190 from bphinz/master

Java viewer updates

Fix UI logic in Java viewer

Fixes an issue where the FullScreenAllMonitors checkbox in the
OptionsDialog was not activated.

Disable window functions for embedded applet

Disable desktopSize option for embedded applets

Add multiple monitor spanning feature to java viewer

Adds support for spanning multiple monitors in "Extended" mode
to the Java viewer. Allows for spanning when maximizing in
addition to just full-screen mode. Seems a bit unpredictable on
MS Windows 7 (ie: depends on window placement, which screen is
set as primary, etc.), but this appears to be the behavior of
the OS itself.

Java viewer UI updates

Make the layout of the options dialog more consistent with the
native viewer.

Fix desktopSize parameter issue

desktopSize preference was being applied even if the checkbox was
unselected in the dialog is a value had previously been stored in
the preferences file.

Merge remote-tracking branch 'upstream/master'

Show error dialog when ACL pattern isn't accepted