Hi,
On Thu, Jan 17, 2019 at 03:04:32AM +0400, marcandre.lur...@redhat.com wrote:
> From: Marc-André Lureau
>
> autofoo build-sys defines WORDS_BIGENDIAN, and spice-common code uses it.
>
> Later, I think it would make sense to switch to G_BIG_ENDIAN instead.
Indeed,
> Fixes:
> https://gitlab.
Hi,
On Thu, Jan 17, 2019 at 03:08:43AM +0400, marcandre.lur...@redhat.com wrote:
> From: Marc-André Lureau
>
> WORDS_BIGENDIAN is defined by autofoo macro.
> meson doesn't define it. Let's use the GLib defines instead.
We will probably need to fix spice-common too.
> Signed-off-by: Marc-André
Hey Iordan
On Thu, Jan 17, 2019 at 3:45 AM i iordanov wrote:
>
> Hello,
>
> Happy new year!
>
> I tried to build spice-gtk 0.34 and 0.35 with OpenSSL 1.1.0, and I got some
> linking errors:
>
> jni/src/../libs/deps/x86/root/lib/libspice-client-glib-2.0.a(spice-channel.o):spice-channel.c:function
Hello,
Happy new year!
I tried to build spice-gtk 0.34 and 0.35 with OpenSSL 1.1.0, and I got some
linking errors:
jni/src/../libs/deps/x86/root/lib/libspice-client-glib-2.0.a(spice-channel.o):spice-channel.c:function
spice_channel_class_intern_init: error: undefined reference to
'SSL_library_in
From: Marc-André Lureau
../subprojects/spice-common/common/quic.c: In function 'fill_model_structures':
../subprojects/spice-common/common/quic.c:695:55: error: comparison of integer
expressions of different signedness: 'int' and 'unsigned int'
[-Werror=sign-compare]
spice_assert(free_coun
From: Marc-André Lureau
WORDS_BIGENDIAN is defined by autofoo macro.
meson doesn't define it. Let's use the GLib defines instead.
Signed-off-by: Marc-André Lureau
---
tools/spicy-screenshot.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/spicy-screenshot.c b/tools/s
On Thu, Jan 17, 2019 at 3:04 AM wrote:
>
> From: Marc-André Lureau
>
> WORDS_BIGENDIAN is defined by autofoo macro.
> meson doesn't define it. Let's use the GLib define instead.
>
> Signed-off-by: Marc-André Lureau
> ---
> tools/spicy-screenshot.c | 2 +-
> 1 file changed, 1 insertion(+), 1 del
From: Marc-André Lureau
autofoo build-sys defines WORDS_BIGENDIAN, and spice-common code uses it.
Later, I think it would make sense to switch to G_BIG_ENDIAN instead.
Fixes:
https://gitlab.freedesktop.org/spice/spice-common/issues/2
Signed-off-by: Marc-André Lureau
---
meson.build | 3 +++
From: Marc-André Lureau
WORDS_BIGENDIAN is defined by autofoo macro.
meson doesn't define it. Let's use the GLib define instead.
Signed-off-by: Marc-André Lureau
---
tools/spicy-screenshot.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/spicy-screenshot.c b/tools/sp
On Wed, Jan 16, 2019 at 6:08 AM Marc-André Lureau
wrote:
> On Wed, Jan 16, 2019 at 2:47 PM james harvey wrote:
> > Am I understanding correctly that the Jan 10 patches which completely
> > fix spice for me and others experiencing clipboard problems aren't
> > being added in any form, and that ins
Hi Daniel.
> v5: Actually try to sort them, and while at it, sort all the ones I
> touch.
Applied this variant on top of drm-misc and did a build test.
Looked good for ia64, x86 and alpha.
Took a closer look at the changes to atmel_hlcd - and they looked OK.
But I noticed that atmel_hlcdc uses
On Mon, Jan 07, 2019 at 03:50:12PM +0200, Yuri Benditovich wrote:
> On Mon, Nov 26, 2018 at 7:41 PM Christophe Fergeau
> wrote:
> > In _SpiceUsbDeviceManagerPrivate, you replaced
> > #ifndef G_OS_WIN32
> > libusb_device *libdev;
> > #endif
> >
> > with
> >
> > #ifndef G_OS_WIN32
> > SpiceU
Hi,
I'm picking up the Virgl remote support patchset with the goal of
adapting it for use with Xspice.
If I'm not mistaken the latest version of this patchset is in the
virgl-spice branches of the following repositories:
https://cgit.freedesktop.org/~fziglio/spice-protocol/log/?h=virgl-spice
On Wed, Jan 16, 2019 at 12:28:00PM +0100, Gerd Hoffmann wrote:
> > > +static int qxl_check_mode(struct qxl_device *qdev,
> > > + unsigned int width,
> > > + unsigned int height)
> > > +{
> > > + if (width * height * 4 > qdev->vram_size)
> >
> > Is someone checki
Currently we set timestamps as buffer's PTS, this value may be changed by
the pipeline in some cases and cause an unexpected buffer warnings (when
GstVideoOverlay is not used). Use GstReferenceTimestampMeta when
synchronization is made by spice.
Before applying this patch you can reproduce the war
Adds an interface method to the FrameCapture class to get the device
display info (device address and device display id) for each display of
the graphics device that is captured.
Also adds functions to the API implementing this functionality for X11
in variants with and without DRM (the non-DRM ve
Receives the GraphicsDeviceInfo message from the streaming agent and
stores the data in a list on the streaming device.
Signed-off-by: Lukáš Hrázký
---
server/display-limits.h| 3 ++
server/red-qxl.c | 2 +-
server/red-stream-device.c | 78 +-
Adds the graphics device info from the streaming device(s) to the
VDAgentGraphicsDeviceInfo message sent to the vd_agent.
Signed-off-by: Lukáš Hrázký
---
server/red-stream-device.c | 18 +++
server/red-stream-device.h | 7 +
server/reds.c | 63 ++
Sends the device address and device display IDs to the vdagent. The
message is sent either in reaction to the SPICE_MSGC_MAIN_AGENT_START
message or when the graphics device info changes.
Signed-off-by: Lukáš Hrázký
---
server/red-qxl.c | 20 +
server/red-qxl.h | 3 ++
ser
The message contains information about the graphics device and monitor
belonging to a particular video stream (which maps to a channel) from
the streaming agent.
Signed-off-by: Lukáš Hrázký
Acked-by: Frediano Ziglio
---
spice/stream-device.h | 31 +++
1 file changed,
The graphics_device_info message contains the device display ID
information (device address and device display ID). Stores the data in a
hash table in vdagent.
Signed-off-by: Lukáš Hrázký
---
src/vdagent/vdagent.c| 3 ++
src/vdagent/x11-priv.h | 1 +
src/vdagent/x11-randr.c
Adds serialization of the GraphicsDeviceInfo message and sends it to the
server when it starts to stream.
Signed-off-by: Lukáš Hrázký
---
src/spice-streaming-agent.cpp | 54 +--
1 file changed, 45 insertions(+), 9 deletions(-)
diff --git a/src/spice-streaming-age
The message serves for passing the device address and device display ID
information for all display channels from SPICE server to the vd_agent.
Signed-off-by: Lukáš Hrázký
Acked-by: Frediano Ziglio
---
spice/vd_agent.h | 23 +++
1 file changed, 23 insertions(+)
diff --git a
Hello,
this series introduces a new mechanism to identify monitors across the
host and guest boundary, using the pair (device_address and
device_display_id). device_address is a HW address of a graphics device
(a PCI address to be specific, though other address domains can be
used as well), device
>
> Hi,
>
> On 1/16/19 12:35 PM, marcandre.lur...@redhat.com wrote:
> > From: Marc-André Lureau
> >
> > Signed-off-by: Marc-André Lureau
> > ---
> > NEWS | 25 +
> > 1 file changed, 25 insertions(+)
> >
> > diff --git a/NEWS b/NEWS
> > index 3f54f77..1693621 100644
> >
From: Marc-André Lureau
- Fix the following warnings:
./spice-gtk-sections.txt:467: warning: No declaration found for
SPICE_GTK_CHECK_VERSION.
./spice-gtk-sections.txt:468: warning: No declaration found for
SPICE_GTK_MAJOR_VERSION.
./spice-gtk-sections.txt:469: warning: No declaration found for
Hi,
On 1/16/19 12:35 PM, marcandre.lur...@redhat.com wrote:
From: Marc-André Lureau
Signed-off-by: Marc-André Lureau
---
NEWS | 25 +
1 file changed, 25 insertions(+)
diff --git a/NEWS b/NEWS
index 3f54f77..1693621 100644
--- a/NEWS
+++ b/NEWS
@@ -1,3 +1,28 @@
+v0.
On Tue, 2019-01-15 at 08:31 -0500, Frediano Ziglio wrote:
> >
> > Receives the GraphicsDeviceInfo message from the streaming agent and
> > stores the data in a list on the streaming device.
> >
> > Signed-off-by: Lukáš Hrázký
> > ---
> > server/display-limits.h| 3 ++
> > server/red-qxl.c
>
> Hi
>
> On Wed, Jan 16, 2019 at 3:38 PM Frediano Ziglio wrote:
> >
> > >
> > > Hi
> > >
> > > On Wed, Jan 16, 2019 at 3:14 PM Frediano Ziglio
> > > wrote:
> > > >
> > > > >
> > > > > Hi,
> > > > >
> > > > > On Wed, Jan 16, 2019 at 12:43:06PM +0400, Marc-André Lureau wrote:
> > > > > > Hi
> >
Hi
On Wed, Jan 16, 2019 at 3:38 PM Frediano Ziglio wrote:
>
> >
> > Hi
> >
> > On Wed, Jan 16, 2019 at 3:14 PM Frediano Ziglio wrote:
> > >
> > > >
> > > > Hi,
> > > >
> > > > On Wed, Jan 16, 2019 at 12:43:06PM +0400, Marc-André Lureau wrote:
> > > > > Hi
> > > > >
> > > > >
> > > > > On Fri, Ja
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
---
tests/test-file-xfe
>
> Hi
>
> On Wed, Jan 16, 2019 at 3:14 PM Frediano Ziglio wrote:
> >
> > >
> > > Hi,
> > >
> > > On Wed, Jan 16, 2019 at 12:43:06PM +0400, Marc-André Lureau wrote:
> > > > Hi
> > > >
> > > >
> > > > On Fri, Jan 11, 2019 at 3:03 PM wrote:
> > > > >
> > > > > From: Marc-André Lureau
> > > > >
>
Hi
On Wed, Jan 16, 2019 at 3:14 PM Frediano Ziglio wrote:
>
> >
> > Hi,
> >
> > On Wed, Jan 16, 2019 at 12:43:06PM +0400, Marc-André Lureau wrote:
> > > Hi
> > >
> > >
> > > On Fri, Jan 11, 2019 at 3:03 PM wrote:
> > > >
> > > > From: Marc-André Lureau
> > > >
> > > > There is a racy bug in pul
> > +static int qxl_check_mode(struct qxl_device *qdev,
> > + unsigned int width,
> > + unsigned int height)
> > +{
> > + if (width * height * 4 > qdev->vram_size)
>
> Is someone checking for integer overflows already?
Need to have a look. This is just .
>
> Hi,
>
> On Wed, Jan 16, 2019 at 12:43:06PM +0400, Marc-André Lureau wrote:
> > Hi
> >
> >
> > On Fri, Jan 11, 2019 at 3:03 PM wrote:
> > >
> > > From: Marc-André Lureau
> > >
> > > There is a racy bug in pulsesrc that we can't easily workaround:
> > > https://gitlab.freedesktop.org/gstrea
Hi
On Wed, Jan 16, 2019 at 2:47 PM james harvey wrote:
>
> On Wed, Jan 16, 2019 at 4:10 AM Victor Toso wrote:
> >
> > From: Victor Toso
> >
> > Similar to 172c521271a3d - if we don't, the agent might be waiting for
> > data till some timeout happens in the system, blocking copy-paste
> > featur
Hi,
On Wed, Jan 16, 2019 at 02:33:49PM +0400, Marc-André Lureau wrote:
> Hi
>
> On Wed, Jan 16, 2019 at 1:18 PM Victor Toso wrote:
> >
> > Hi,
> >
> > On Wed, Jan 16, 2019 at 01:07:27PM +0400, Marc-André Lureau wrote:
> > > > Not true. Possible client-side bug, reproducible on X11, as I
> > > >
Hi,
On Wed, Jan 16, 2019 at 05:46:58AM -0500, james harvey wrote:
> On Wed, Jan 16, 2019 at 4:10 AM Victor Toso wrote:
> >
> > From: Victor Toso
> >
> > Similar to 172c521271a3d - if we don't, the agent might be waiting for
> > data till some timeout happens in the system, blocking copy-paste
>
On Wed, Jan 16, 2019 at 4:10 AM Victor Toso wrote:
>
> From: Victor Toso
>
> Similar to 172c521271a3d - if we don't, the agent might be waiting for
> data till some timeout happens in the system, blocking copy-paste
> feature and possibly freezing applications.
>
> A way to reproduce is copy-past
From: Marc-André Lureau
Signed-off-by: Marc-André Lureau
---
NEWS | 25 +
1 file changed, 25 insertions(+)
diff --git a/NEWS b/NEWS
index 3f54f77..1693621 100644
--- a/NEWS
+++ b/NEWS
@@ -1,3 +1,28 @@
+v0.36
+=
+
+- Add meson build: autotools will be removed in a fu
Hi
On Wed, Jan 16, 2019 at 1:18 PM Victor Toso wrote:
>
> Hi,
>
> On Wed, Jan 16, 2019 at 01:07:27PM +0400, Marc-André Lureau wrote:
> > > Not true. Possible client-side bug, reproducible on X11, as I
> > > discussed before. As mentioned in the comment, we are sending
> > > wrong selection-grab t
Hi
On Wed, Jan 16, 2019 at 10:03 AM Victor Toso wrote:
>
> Hi,
>
> On Wed, Jan 16, 2019 at 12:08:31AM +0400, Marc-André Lureau wrote:
> > On Tue, Jan 15, 2019 at 8:11 PM Victor Toso wrote:
> > >
> > > From: Victor Toso
> > >
> > > Hi,
> > >
> > > Several iteractions trying to avoid some bug in
>
> Make easier to test file creation
>
> Signed-off-by: Frediano Ziglio
> ---
> src/vdagent/file-xfers.c | 88
> src/vdagent/file-xfers.h | 1 +
> 2 files changed, 53 insertions(+), 36 deletions(-)
>
> diff --git a/src/vdagent/file-xfers.c b/src/vdage
Hi,
On Wed, Jan 16, 2019 at 09:49:22AM +, Frediano Ziglio wrote:
> Signed-off-by: Frediano Ziglio
Acked-by: Victor Toso
> ---
> Changes since v1:
> - move file to a new tests/ directory;
> - add missing .gitignore changes;
> - improve test to catch errors in open() other than EEXISTS.
> ---
Signed-off-by: Frediano Ziglio
---
Changes since v1:
- move file to a new tests/ directory;
- add missing .gitignore changes;
- improve test to catch errors in open() other than EEXISTS.
---
.gitignore | 17 +
Makefile.am | 22
tests/test-file-xfers.c
>
> Add a helper functions to check video modes. Also add a helper to check
> framebuffer buffer objects, using the former for consistency. That way
> we should not fail in qxl_primary_atomic_check() because video modes
> which are too big will not be added to the mode list in the first place.
>
Hi,
On Wed, Jan 16, 2019 at 04:15:50AM -0500, Frediano Ziglio wrote:
> >
> > Hi,
> >
> > For some reason I miss in my inbox the 6/7 patch. Looking at,
> > seems that mailman got it.
> >
> >
> > https://lists.freedesktop.org/archives/spice-devel/2019-January/047268.html
> >
> > My only sug
Hi,
On Wed, Jan 16, 2019 at 01:07:27PM +0400, Marc-André Lureau wrote:
> > Not true. Possible client-side bug, reproducible on X11, as I
> > discussed before. As mentioned in the comment, we are sending
> > wrong selection-grab to the guest.
>
> Sorry, I don't follow "we are sending wrong selecti
>
> Hi,
>
> For some reason I miss in my inbox the 6/7 patch. Looking at,
> seems that mailman got it.
>
>
> https://lists.freedesktop.org/archives/spice-devel/2019-January/047268.html
>
> My only suggestion is to move the test to root directory instead
> of srcdir. I'd prefer also to use
Hi,
On Wed, Jan 16, 2019 at 12:43:06PM +0400, Marc-André Lureau wrote:
> Hi
>
>
> On Fri, Jan 11, 2019 at 3:03 PM wrote:
> >
> > From: Marc-André Lureau
> >
> > There is a racy bug in pulsesrc that we can't easily workaround:
> > https://gitlab.freedesktop.org/gstreamer/gst-plugins-good/merge_
From: Victor Toso
Similar to 172c521271a3d - if we don't, the agent might be waiting for
data till some timeout happens in the system, blocking copy-paste
feature and possibly freezing applications.
A way to reproduce is copy-paste big clipboard data between two spice
clients.
Also add some doc
Hi
On Wed, Jan 16, 2019 at 12:56 PM Victor Toso wrote:
>
> Hi,
>
> On Wed, Jan 16, 2019 at 12:40:36PM +0400, Marc-André Lureau wrote:
> > Hi
> >
> > On Wed, Jan 16, 2019 at 11:44 AM Victor Toso wrote:
> > >
> > > From: Victor Toso
> > >
> > > Similar to 172c521271a3d - if we don't, the agent mi
Hi,
On Wed, Jan 16, 2019 at 12:40:36PM +0400, Marc-André Lureau wrote:
> Hi
>
> On Wed, Jan 16, 2019 at 11:44 AM Victor Toso wrote:
> >
> > From: Victor Toso
> >
> > Similar to 172c521271a3d - if we don't, the agent might be waiting for
> > data till some timeout happens in the system, blocking
Hi
On Fri, Jan 11, 2019 at 3:03 PM wrote:
>
> From: Marc-André Lureau
>
> There is a racy bug in pulsesrc that we can't easily workaround:
> https://gitlab.freedesktop.org/gstreamer/gst-plugins-good/merge_requests/69
>
> It is fixed with 1.15, and will be backported to upcoming 1.14.5.
>
> Puls
On 1/15/19 2:26 PM, Neil Armstrong wrote:
On 15/01/2019 11:41, 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 her
Commit 098268a33c7c8008ccec9050aea8f0763f1c06d5 (systemd: Update path
in unit file) was a really incomplete change. Let's change every
occurrence of /var/run to /run.
Signed-off-by: Christian Hesse
---
README | 2 +-
data/spice-vdagentd | 4 ++--
data
Hi Daniel,
Thank you for the patch.
On Tuesday, 15 January 2019 12:41:37 EET 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 actuall
On Tue, Jan 15, 2019 at 5:41 AM 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 I went through all
>
Hi
On Wed, Jan 16, 2019 at 11:44 AM Victor Toso wrote:
>
> From: Victor Toso
>
> Similar to 172c521271a3d - if we don't, the agent might be waiting for
> data till some timeout happens in the system, blocking copy-paste
> feature and possibly freezing applications.
>
> A way to reproduce is copy
Add a helper function to add custom video modes to a connector.
Signed-off-by: Gerd Hoffmann
---
drivers/gpu/drm/qxl/qxl_display.c | 84 +++
1 file changed, 49 insertions(+), 35 deletions(-)
diff --git a/drivers/gpu/drm/qxl/qxl_display.c
b/drivers/gpu/drm/qx
Add a helper functions to check video modes. Also add a helper to check
framebuffer buffer objects, using the former for consistency. That way
we should not fail in qxl_primary_atomic_check() because video modes
which are too big will not be added to the mode list in the first place.
Signed-off-
Add all standard modes from the kernel's video mode data base.
Keep a few non-standard modes in the qxl mode list.
Signed-off-by: Gerd Hoffmann
---
drivers/gpu/drm/qxl/qxl_display.c | 27 +++
1 file changed, 7 insertions(+), 20 deletions(-)
diff --git a/drivers/gpu/drm/q
Hi,
For some reason I miss in my inbox the 6/7 patch. Looking at,
seems that mailman got it.
https://lists.freedesktop.org/archives/spice-devel/2019-January/047268.html
My only suggestion is to move the test to root directory instead
of srcdir. I'd prefer also to use the glib test framework
On Wed, Jan 16, 2019 at 11:44 AM Victor Toso wrote:
>
> From: Victor Toso
>
> Should first check selection value before accessing those arrays.
> The get_clipboard_from_selection() function does that for us.
>
> Signed-off-by: Victor Toso
ack
> ---
> src/spice-gtk-session.c | 5 ++---
> 1 fi
64 matches
Mail list logo