Add a function to look up an xrandr output for a given device display
id. This uses sysfs and the drm subsystem to lookup information about a
graphics device output. It then compares the drm output name to xrandr
output names to try to match that device output to an xrandr output.
This is necesary
Add a display_id field to the structure that we use to send down the
list of guest display resolutions to the vdagentd daemon. This allows us
to map the spice display id to the proper X display for determining
mouse locations, etc.
In the case where we have an mjpeg plugin running in the
When sending the guest xorg resolution to the vdagentd daemon, we get
the current resolution with this function, which returns a
VDAgentMonitorsConfig structure that must then be converted to the
structure that we send to the daemon. Rather than re-using this function
that returns the wrong type,
Instead of storing each device address and device display ID in the hash
table, simply use the lookup_xrandr_output_for_device_info() function to
look up the ID of the xrandr output and store that in the hash table.
---
src/vdagent/x11-priv.h | 2 +-
src/vdagent/x11-randr.c | 58
instead of using the spice display id directly as the xrandr output,
look it up using our new guest output map
---
src/vdagent/x11-randr.c | 81 -
1 file changed, 72 insertions(+), 9 deletions(-)
diff --git a/src/vdagent/x11-randr.c
There are basically three ways to refer to an output within vdagent:
- The index of the array of MonitorConfig message. This is essentially
a "spice display id"
- the index of the array of xrandr outputs. This is the "output index"
- the xrandr output id. This is the "output ID"
Rather than getting the current guest resolution in a
VDAgentMonitorsConfig struct and then translating it to a new struct
type for sending down to the daemon, simply use the new function that
was factored out in a previous commit and populate the message struct
directly.
---
This is a patch set that handles the PCI address and device dispay ID
sent down to the agent by the server, and uses that to maintain a map
for looking up a particular xrandr output for a given spice display id.
This patch series builds on the patch from Lukas titled "Receive the
>
> From: Victor Toso
>
> Small refactor to make each code block a bit more obvious.
>
> This code should (1) find the @buffer in the queue; (2) remove all old
> elements from queue. That perfectly fit in two loops in sequence, but
> they don't need to be nested and they don't need to use the
From: Victor Toso
Small refactor to make each code block a bit more obvious.
This code should (1) find the @buffer in the queue; (2) remove all old
elements from queue. That perfectly fit in two loops in sequence, but
they don't need to be nested and they don't need to use the same
pointer
From: Victor Toso
Odd minor versions are for development version, which means that
checks on 1.9.x should be considered for development while normal
environments should be running the stable.
Some timeline of stable releases:
1.14.4: Tue Oct 2 22:53:01 2018 +0100
1.14.0: Mon Mar 19 20:09:51
ping
>
> If test is run with root (for instance on Gitlab CI) creating
> a file on a directory with no permission will succeed and
> test will fail.
> Instead create a link trying to create a file in /proc/1
> directory (which fails also using root account).
>
> Signed-off-by: Frediano Ziglio
Hi,
On Wed, Jan 23, 2019 at 05:16:32PM +, Frediano Ziglio wrote:
> AsyncCommand was removed in eb6c8ba025.
>
> Signed-off-by: Frediano Ziglio
Acked-by: Victor Toso
> ---
> server/red-worker.h | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/server/red-worker.h
Hi,
On Wed, Jan 23, 2019 at 08:32:22PM +, Frediano Ziglio wrote:
> This patch is based on some work from Victor Toso (statistics) and
> Snir Sheriber (GStreamer probing).
> All GstBuffers are queued into decoding_queue and a probe is
> attached to the sink in order to understand when the
This patch is based on some work from Victor Toso (statistics) and
Snir Sheriber (GStreamer probing).
All GstBuffers are queued into decoding_queue and a probe is
attached to the sink in order to understand when the buffers
are decoded.
Previously we didn't add frames to decoding_queue in case of
Hi,
On Wed, Jan 23, 2019 at 11:28:57AM +, Frediano Ziglio wrote:
> This patch is based on some work from Victor Toso (statistics) and
> Snir Sheriber (GStreamer probing).
> All GstBuffers are queued into decoding_queue and a probe is
> attached to the sink in order to understand when the
Zlib structure take up more than 1MB and it is rarely used nowadays
as it is not much effective.
Initialise it only when necessary saving some memory in the normal
case.
Signed-off-by: Frediano Ziglio
---
server/image-encoders.c | 18 ++
1 file changed, 10 insertions(+), 8
On 1/23/19 3:02 PM, Frediano Ziglio wrote:
> Signed-off-by: Frediano Ziglio
> ---
> .gitlab-ci.yml | 9 ++---
> 1 file changed, 6 insertions(+), 3 deletions(-)
>
> diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
> index a757e7a3..9ce95c00 100644
> --- a/.gitlab-ci.yml
> +++ b/.gitlab-ci.yml
>
AsyncCommand was removed in eb6c8ba025.
Signed-off-by: Frediano Ziglio
---
server/red-worker.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/server/red-worker.h b/server/red-worker.h
index 68c5058d..767329da 100644
--- a/server/red-worker.h
+++ b/server/red-worker.h
@@ -26,7 +26,6 @@
Hi Daniel.
On Thu, Jan 17, 2019 at 10:03:34PM +0100, Daniel Vetter wrote:
> Having the probe helper stuff (which pretty much everyone needs) in
> the drm_crtc_helper.h file (which atomic drivers should never need) is
> confusing. Split them out.
>
> To make sure I actually achieved the goal here
Signed-off-by: Frediano Ziglio
---
.gitlab-ci.yml | 9 ++---
1 file changed, 6 insertions(+), 3 deletions(-)
diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index a757e7a3..9ce95c00 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -2,12 +2,15 @@ image: fedora:latest
before_script:
- >
> On Wed, Jan 23, 2019 at 08:57:36AM -0500, Frediano Ziglio wrote:
> > > On Wed, Jan 23, 2019 at 11:28:57AM +, Frediano Ziglio wrote:
> > > > This patch is based on some work from Victor Toso (statistics) and
> > > > Snir Sheriber (GStreamer probing).
> > > > All GstBuffers are queued into
Hi,
On Mon, Jan 14, 2019 at 02:20:15PM +, Frediano Ziglio wrote:
> Changes since v3:
> - split spice-common update;
> - update recorder commit to remove Meson error.
This was basically acked in v3, the spice-common one was small
suggestion; you decided to wait for the meson error to get
On Wed, Jan 23, 2019 at 08:57:36AM -0500, Frediano Ziglio wrote:
> > On Wed, Jan 23, 2019 at 11:28:57AM +, Frediano Ziglio wrote:
> > > This patch is based on some work from Victor Toso (statistics) and
> > > Snir Sheriber (GStreamer probing).
> > > All GstBuffers are queued into
ping
>
> Changes since v3:
> - split spice-common update;
> - update recorder commit to remove Meson error.
>
> Changes since v2:
> - rebased
>
> Frediano Ziglio (4):
> Integrate recorder library
> Update spice-common submodule
> Allows to enable recorder integration
> Trace streaming
> Hi,
>
> On Wed, Jan 23, 2019 at 11:28:57AM +, Frediano Ziglio wrote:
> > This patch is based on some work from Victor Toso (statistics) and
> > Snir Sheriber (GStreamer probing).
> > All GstBuffers are queued into decoding_queue and a probe is
> > attached to the sink in order to understand
Hi,
On Wed, Jan 23, 2019 at 11:28:57AM +, Frediano Ziglio wrote:
> This patch is based on some work from Victor Toso (statistics) and
> Snir Sheriber (GStreamer probing).
> All GstBuffers are queued into decoding_queue and a probe is
> attached to the sink in order to understand when the
Hi,
On Wed, Jan 23, 2019 at 08:11:17AM -0500, Frediano Ziglio wrote:
> >
> > Hi,
> >
> > On Wed, Jan 23, 2019 at 12:08:53AM +, Frediano Ziglio wrote:
> > > Separate the code from fetch_pending_sample that extracts the
> > > SpiceGstFrame relative to a give GstBuffer.
> > > This new function
Hey team,
It’s still time to wish you a happy new year, I guess… :-)
While looking for illustrations for an upcoming DevConf.cz talk, I cam across
the following article that compares three remote desktop protocols:
http://www.diva-portal.org/smash/get/diva2:530960/FULLTEXT01.pdf.
It was
>
> Hi,
>
> On Wed, Jan 23, 2019 at 12:08:53AM +, Frediano Ziglio wrote:
> > Separate the code from fetch_pending_sample that extracts the
> > SpiceGstFrame relative to a give GstBuffer.
> > This new function will be reused later.
> >
> > Signed-off-by: Frediano Ziglio
> > ---
> >
Hi,
On Wed, Jan 23, 2019 at 12:08:52AM +, Frediano Ziglio wrote:
> Add encoded_ and decoded_ prefix to understand which side
> they refer.
>
> Signed-off-by: Frediano Ziglio
Acked-by: Victor Toso
> ---
> src/channel-display-gst.c | 36 ++--
> 1 file
This patch is based on some work from Victor Toso (statistics) and
Snir Sheriber (GStreamer probing).
All GstBuffers are queued into decoding_queue and a probe is
attached to the sink in order to understand when the buffers
are decoded.
When a buffer is decoded the time spent and queue length are
This fixes Gitlab issue #9
(cfr https://gitlab.freedesktop.org/spice/usbredir/issues/9).
Signed-off-by: Frediano Ziglio
---
usbredirserver/usbredirserver.c | 7 +++
1 file changed, 7 insertions(+)
diff --git a/usbredirserver/usbredirserver.c b/usbredirserver/usbredirserver.c
index
Signed-off-by: Frediano Ziglio
---
src/spice-marshal.txt | 1 -
1 file changed, 1 deletion(-)
diff --git a/src/spice-marshal.txt b/src/spice-marshal.txt
index 92087c58..cf35790e 100644
--- a/src/spice-marshal.txt
+++ b/src/spice-marshal.txt
@@ -11,6 +11,5 @@ VOID:UINT,POINTER,UINT
d0cbd9618f0b removed the ability to use git-version-gen to
generate proper version string.
Generate .tarball-version file in the distribution file to
allow building from tarball.
Do not use MESON_SOURCE_ROOT calling git-version-gen command
as this won't be expanded.
Change directory in
35 matches
Mail list logo