Signed-off-by: Daniel P. Berrange
---
Changed in v2:
- Newer submodule hash to pull in python 3 portability fixes
.gitmodules | 3 +
configure.ac | 11 --
src/Makefile.am | 30 ++--
src/keycodemapdb | 1 +
src/keymap-gen.pl | 214 ---
src/keymaps.csv
On Wed, 2017-02-15 at 18:01 +0100, Christophe Fergeau wrote:
> On Wed, Feb 15, 2017 at 11:39:32AM -0500, Frediano Ziglio wrote:
> > Hi,
> > question was raised recently on the ML and IRC.
> >
> > Some time ago we decided to use gboolean but some of us would like
> > to discuss again.
>
> So...
>
> On Wed, 2017-02-15 at 12:07 -0500, Frediano Ziglio wrote:
> > >
> > > On Thu, Feb 02, 2017 at 04:29:37PM +, Frediano Ziglio wrote:
> > > > Signed-off-by: Frediano Ziglio
> > > > ---
> > > > server/sound.c | 6 ++
> > > > 1 file changed, 2 insertions(+), 4 deletions(-)
> > > >
> > >
On Wed, 2017-02-15 at 12:07 -0500, Frediano Ziglio wrote:
> >
> > On Thu, Feb 02, 2017 at 04:29:37PM +, Frediano Ziglio wrote:
> > > Signed-off-by: Frediano Ziglio
> > > ---
> > > server/sound.c | 6 ++
> > > 1 file changed, 2 insertions(+), 4 deletions(-)
> > >
> > > diff --git a/serve
>
> On Thu, Feb 02, 2017 at 04:29:37PM +, Frediano Ziglio wrote:
> > Signed-off-by: Frediano Ziglio
> > ---
> > server/sound.c | 6 ++
> > 1 file changed, 2 insertions(+), 4 deletions(-)
> >
> > diff --git a/server/sound.c b/server/sound.c
> > index 7c36174..b692646 100644
> > --- a/ser
On Wed, Feb 15, 2017 at 11:39:32AM -0500, Frediano Ziglio wrote:
> Hi,
> question was raised recently on the ML and IRC.
>
> Some time ago we decided to use gboolean but some of us would like
> to discuss again.
So... As said a few times, I would have preferred to wait for a bit to
have that di
On Wed, Feb 15, 2017 at 11:39:32AM -0500, Frediano Ziglio wrote:
> Hi,
> question was raised recently on the ML and IRC.
>
> Some time ago we decided to use gboolean but some of us would like
> to discuss again.
>
> As any style changes there's no right or wrong, mainly personal
> opinions but
Yes, changed all, sending a v2
>
> On Wed, Feb 15, 2017 at 11:46:21AM -0500, Frediano Ziglio wrote:
> > >
> > > ACK, maybe rcc: in the short log
> > >
> >
> > I'd use red-channel-client: as consistent with the second...
> > or change both to rcc!
>
> I meant to suggest to change the second on
On Wed, Feb 15, 2017 at 11:44:53AM -0500, Frediano Ziglio wrote:
> >
> > On Wed, Feb 15, 2017 at 12:49:06PM +, Frediano Ziglio wrote:
> > > RedChannelClient is responsible for talking to the client so it knows
> > > how if is connected or not.
> >
> > s/how//
> >
> > I'd mention explicitly t
On Wed, Feb 15, 2017 at 11:46:21AM -0500, Frediano Ziglio wrote:
> >
> > ACK, maybe rcc: in the short log
> >
>
> I'd use red-channel-client: as consistent with the second...
> or change both to rcc!
I meant to suggest to change the second one to rcc but forgot. Only
reason for red-channel-clie
On Thu, Feb 02, 2017 at 04:29:37PM +, Frediano Ziglio wrote:
> Signed-off-by: Frediano Ziglio
> ---
> server/sound.c | 6 ++
> 1 file changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/server/sound.c b/server/sound.c
> index 7c36174..b692646 100644
> --- a/server/sound.c
> +++ b/s
>
> ACK, maybe rcc: in the short log
>
I'd use red-channel-client: as consistent with the second...
or change both to rcc!
Not strong opinions, on the logs there are both versions
Frediano
> On Wed, Feb 15, 2017 at 12:49:05PM +, Frediano Ziglio wrote:
> > red_channel seems a variable name
On Wed, Feb 15, 2017 at 12:49:06PM +, Frediano Ziglio wrote:
> RedChannelClient is responsible for talking to the client so it knows
> how if is connected or not.
s/how//
I'd mention explicitly that it's RedChannelClient::is_connected and
RedChannelClient::disconnect which are dropped (would
>
> On Wed, Feb 15, 2017 at 12:49:06PM +, Frediano Ziglio wrote:
> > RedChannelClient is responsible for talking to the client so it knows
> > how if is connected or not.
>
> s/how//
>
> I'd mention explicitly that it's RedChannelClient::is_connected and
> RedChannelClient::disconnect which
I'd change the log to something like:
"rcc: Make some functions/macros private
Some constants/macros/function prototypes are defined in
red-channel-client.h while they are only used by red-channel-client.c.
This commit moves them to the .c file to make them private"
Acked-by: Christophe Fergea
Hi,
question was raised recently on the ML and IRC.
Some time ago we decided to use gboolean but some of us would like
to discuss again.
As any style changes there's no right or wrong, mainly personal
opinions but I think consistency is quite important.
Some consideration (feel free to add/rem
ACK, maybe rcc: in the short log
On Wed, Feb 15, 2017 at 12:49:05PM +, Frediano Ziglio wrote:
> red_channel seems a variable name.
>
> Signed-off-by: Frediano Ziglio
> ---
> server/red-channel-client.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/server/red-chan
>
>
> > On 15 Feb 2017, at 12:45, Frediano Ziglio wrote:
> >
> >> @@ -134,7 +134,7 @@ static void red_qxl_display_migrate(RedChannelClient
> >> *rcc)
> >> }
> >> g_object_get(channel, "channel-type", &type, "id", &id, NULL);
> >> dispatcher = (Dispatcher *)g_object_get_data(G_OBJECT
Hi,
Most of them will never show up, and if they will, it always starts with
"Unable to authenticate" so i thought it's good enough , but no problem,
i can show the user only the relevant ones and log the others (or to
ignore them :] )
On 02/15/2017 05:00 PM, Christophe Fergeau wrote:
On W
On Wed, 2017-02-15 at 16:34 +0100, Christophe de Dinechin wrote:
> > On 15 Feb 2017, at 12:45, Frediano Ziglio
> > wrote:
> >
> > > @@ -134,7 +134,7 @@ static void
> > > red_qxl_display_migrate(RedChannelClient
> > > *rcc)
> > > }
> > > g_object_get(channel, "channel-type", &type, "id", &
On Wed, Feb 15, 2017 at 04:44:22PM +0100, Pavel Grunt wrote:
> nice, although the python script looks pretty heavy.
Different users of the keymap database have different output formats.
We don't want ever user re-implementing the same logic, as that's
just as bad what we have today. So the tool is
On Wed, 2017-02-15 at 08:51 +0100, Christophe Fergeau wrote:
> On Tue, Feb 14, 2017 at 12:14:56PM -0600, Jonathon Jongsma wrote:
> > To be honest, I could go either way on this one. I like the
> > consistency, but I don't like the fact that it makes things like
> > "git
> > blame" less useful. I'll
Hi,
nice, although the python script looks pretty heavy.
Anyway, in spice-gtk we support python3, imo it cannot go in without
the python3 support.
Thanks,
Pavel
On Wed, 2017-02-15 at 15:17 +, Daniel P. Berrange wrote:
> Signed-off-by: Daniel P. Berrange
> ---
> .gitmodules | 3 +
> On 15 Feb 2017, at 12:45, Frediano Ziglio wrote:
>
>> @@ -134,7 +134,7 @@ static void red_qxl_display_migrate(RedChannelClient
>> *rcc)
>> }
>> g_object_get(channel, "channel-type", &type, "id", &id, NULL);
>> dispatcher = (Dispatcher *)g_object_get_data(G_OBJECT(channel),
>> "
Signed-off-by: Daniel P. Berrange
---
.gitmodules | 3 +
configure.ac | 11 --
src/Makefile.am | 30 ++--
src/keycodemapdb | 1 +
src/keymap-gen.pl | 214 ---
src/keymaps.csv | 511 --
6 files changed, 1
On Wed, Feb 15, 2017 at 03:23:36PM +0100, Pavel Grunt wrote:
> On Wed, 2017-02-15 at 15:16 +0100, Christophe Fergeau wrote:
> > On Wed, Feb 15, 2017 at 09:14:10AM -0500, Frediano Ziglio wrote:
> > > >
> > > > On Wed, Feb 15, 2017 at 06:39:13AM -0500, Frediano Ziglio wrote:
> > > > >
> > > > > Pat
On Wed, Feb 15, 2017 at 02:56:36PM +0200, Snir Sheriber wrote:
> Hi,
> yes, the idea is to present errors which are generated on the sasl server
> side, in the err window on the user/sasl-client side (only errors- without
> sasl_ok, continue , interact) by sending the error number to the client and
On Wed, 2017-02-15 at 15:16 +0100, Christophe Fergeau wrote:
> On Wed, Feb 15, 2017 at 09:14:10AM -0500, Frediano Ziglio wrote:
> > >
> > > On Wed, Feb 15, 2017 at 06:39:13AM -0500, Frediano Ziglio wrote:
> > > >
> > > > Patch is good however the use of bool is inconsistent.
> > > > Although I _m
On Wed, Feb 15, 2017 at 09:14:10AM -0500, Frediano Ziglio wrote:
> >
> > On Wed, Feb 15, 2017 at 06:39:13AM -0500, Frediano Ziglio wrote:
> > >
> > > Patch is good however the use of bool is inconsistent.
> > > Although I _much_ prefer bool/true/false we decided for
> > > gboolean/TRUE/FALSE.
> >
>
> On Wed, Feb 15, 2017 at 06:39:13AM -0500, Frediano Ziglio wrote:
> >
> > Patch is good however the use of bool is inconsistent.
> > Although I _much_ prefer bool/true/false we decided for
> > gboolean/TRUE/FALSE.
>
> reds-stream.[ch] is already exclusively using bool instead of gboolean,
> s
On Wed, Feb 15, 2017 at 06:39:13AM -0500, Frediano Ziglio wrote:
>
> Patch is good however the use of bool is inconsistent.
> Although I _much_ prefer bool/true/false we decided for gboolean/TRUE/FALSE.
reds-stream.[ch] is already exclusively using bool instead of gboolean,
so I'd prefer to stick
>
> Hi,
>
> On Wed, 2017-02-15 at 12:37 +, Frediano Ziglio wrote:
> > The usage was removed with commit
> > 7ea1f2c133a8e57523078ba3112cec6a1d2f353e
> > ("sound: Use RedChannelClient to receive/send data").
> >
> > Signed-off-by: Frediano Ziglio
> > ---
> > server/sound.c | 4
> > 1 f
Hi,
On Wed, 2017-02-15 at 12:37 +, Frediano Ziglio wrote:
> The usage was removed with commit
> 7ea1f2c133a8e57523078ba3112cec6a1d2f353e
> ("sound: Use RedChannelClient to receive/send data").
>
> Signed-off-by: Frediano Ziglio
> ---
> server/sound.c | 4
> 1 file changed, 4 deletions(
Hi,
On 02/14/2017 06:42 PM, Christophe Fergeau wrote:
On Mon, Feb 13, 2017 at 03:49:44PM +0200, Snir Sheriber wrote:
Remove handling with failures in the SASL authentication
process to separate function and display the error message
as reported by the SASL client (could also display SASL
serve
red_channel seems a variable name.
Signed-off-by: Frediano Ziglio
---
server/red-channel-client.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/server/red-channel-client.h b/server/red-channel-client.h
index 4f1d481..75d6cc3 100644
--- a/server/red-channel-client.h
+++ b/se
Signed-off-by: Frediano Ziglio
---
server/red-channel-client.c | 25 -
server/red-channel-client.h | 23 ---
2 files changed, 24 insertions(+), 24 deletions(-)
diff --git a/server/red-channel-client.c b/server/red-channel-client.c
index 16e5446..a0e213
RedChannelClient is responsible for talking to the client so it knows
how if is connected or not.
About disconnect you think that we loose a bit of flexibility however on
disconnect the channel is triggered so additional needed process can be
handled in destructor.
These vfuncs where used by DummyC
The usage was removed with commit 7ea1f2c133a8e57523078ba3112cec6a1d2f353e
("sound: Use RedChannelClient to receive/send data").
Signed-off-by: Frediano Ziglio
---
server/sound.c | 4
1 file changed, 4 deletions(-)
diff --git a/server/sound.c b/server/sound.c
index 530a7ec..afe9c96 100644
On Wed, Feb 15, 2017 at 06:48:32AM -0500, Frediano Ziglio wrote:
> >
> > On Wed, Feb 15, 2017 at 11:31:59AM +, Frediano Ziglio wrote:
> > > TCP_NODELAY flag is set by default for all connection inside
> > > reds.c so there's no need to set again for the single
> > > client channel.
> >
> > Wo
Ping
>
> Signed-off-by: Frediano Ziglio
> ---
> server/sound.c | 6 ++
> 1 file changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/server/sound.c b/server/sound.c
> index 7c36174..b692646 100644
> --- a/server/sound.c
> +++ b/server/sound.c
> @@ -384,7 +384,6 @@ static int
> snd_pla
Concentrate filling of signal info in single procedure.
Fill signal info with specific or default frequency data
according to the global flag of VSync support.
Note that the state of this flobal flag must be defined only
on driver startup (based on OS version or any other information
available at D
Prepare all the procedures needed for VSync feature but
do not turn it on yet. Manual uncomment still required to
enable it.
Advanced users may enable VSync feature and report functional
problem with it. The driver is expected to pass all the formal
tests under HLK 1607 with device rev.3 and rev.4
These patches address failures in Windows Hardware Lab Tests:
Requirement for EDID data
Requirement for refresh rate of 75/60 Hz
Now the driver contains data in EDID format and reports it to the system
(requires manual uncomment)
In order to report refresh rate the driver reports support for
VSy
Solves failure of HLK "Test for EDID requirements"
EDID contains capabilities and manufacturer data of
the emulated display device. Parsed EDID data presented
in the source file.
Signed-off-by: Yuri Benditovich
---
qxldod/QxlDod.cpp | 106 --
1
Do not clear interrupt mask upon interrupt.
Instead clear pending interrupt status and write
QXL_IO_UPDATE_IRQ register (this drops interrupt level).
There are 3 advantages:
1. We do not need to wake the host to enable interrupt
mask in DPC procedure (1 wake per interrupt instead of 2)
2. The drive
When the video mode is changed and then the driver disabled and
enabled, it did not enumerate available video modes with lower
resolution than current one. All modes starting from 1024x768
should be available regardless what is current resolution
on driver startup
Signed-off-by: Yuri Benditovich
In case the driver supports VSync control feature, it
maintains timer for VSync interrupt indication.
In further commits this timer will be started upon
class driver request. The interrupt notification and
related DPC do not access device registers.
Signed-off-by: Yuri Benditovich
---
qxldod/Qxl
>
> On Wed, Feb 15, 2017 at 11:31:59AM +, Frediano Ziglio wrote:
> > TCP_NODELAY flag is set by default for all connection inside
> > reds.c so there's no need to set again for the single
> > client channel.
>
> Would have been nice to keep "Do not set not blocking flag twice"
> together with
On Wed, 2017-02-15 at 12:29 +0100, Christophe Fergeau wrote:
> Since c3d237 "gstreamer: Avoid memory copy if strides are different"
> is
> only needed when zero copy is disabled. This moves the function
> definition to an already existing #ifdef block.
Ack,
Pavel
> ---
>
> Changes since v1:
> - d
On Wed, Feb 15, 2017 at 11:31:58AM +, Frediano Ziglio wrote:
> Avoid possible dangling pointers.
Acked-by: Christophe Fergeau
>
> Signed-off-by: Frediano Ziglio
> ---
> server/red-channel-client.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/server/red-channel-client.c b/s
>
> Some debug messages were using spice_printerr(), which should be limited
> to actual errors.
>
> Signed-off-by: Christophe Fergeau
> ---
> server/inputs-channel.c | 4 ++--
> server/main-channel-client.c | 28 ++--
> server/main-channel.c| 10 +-
On Wed, Feb 15, 2017 at 11:31:59AM +, Frediano Ziglio wrote:
> TCP_NODELAY flag is set by default for all connection inside
> reds.c so there's no need to set again for the single
> client channel.
Would have been nice to keep "Do not set not blocking flag twice"
together with this patch, and
Acked-by: Christophe Fergeau
Christophe
On Wed, Feb 15, 2017 at 11:32:01AM +, Frediano Ziglio wrote:
> Most channels don't need to do specific settings for the client socket
> so avoid the need to do this setting making easier to setup the client
> channnel.
>
> Some improvements and commi
>
> From: Christophe Fergeau
>
> The code to enable/disable on a TCP socket is duplicated in multiple
> places in the code base, this commit replaces this duplicated code with
> a helper in RedsStream.
> ---
> server/common-graphics-channel.c | 13 ++---
> server/red-channel-client.c
Some debug messages were using spice_printerr(), which should be limited
to actual errors.
Signed-off-by: Christophe Fergeau
---
server/inputs-channel.c | 4 ++--
server/main-channel-client.c | 28 ++--
server/main-channel.c| 10 +-
server/red-channe
TCP_NODELAY flag is set by default for all connection inside
reds.c so there's no need to set again for the single
client channel.
Note that there are still some call to setsockopt to set this
option but in this case the flag can reset the flag.
Signed-off-by: Frediano Ziglio
---
server/inputs-
From: Christophe Fergeau
The code to enable/disable on a TCP socket is duplicated in multiple
places in the code base, this commit replaces this duplicated code with
a helper in RedsStream.
---
server/common-graphics-channel.c | 13 ++---
server/red-channel-client.c | 17 ++-
Avoid possible dangling pointers.
Signed-off-by: Frediano Ziglio
---
server/red-channel-client.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/server/red-channel-client.c b/server/red-channel-client.c
index 9ab22e4..32db186 100644
--- a/server/red-channel-client.c
+++ b/server/red-channe
Most channels don't need to do specific settings for the client socket
so avoid the need to do this setting making easier to setup the client
channnel.
Some improvements and commit subject suggested by Christophe Fergeau.
Signed-off-by: Frediano Ziglio
---
server/inputs-channel.c | 6 --
se
Since c3d237 "gstreamer: Avoid memory copy if strides are different" is
only needed when zero copy is disabled. This moves the function
definition to an already existing #ifdef block.
---
Changes since v1:
- do not add redundant #ifdef block
server/gstreamer-encoder.c | 61 ++
Currently, the network sockets opened by reds_init_net() are not closed
on destruction, in other words they are leaked.
Signed-off-by: Christophe Fergeau
Acked-by: Pavel Grunt
---
server/reds.c | 15 +--
1 file changed, 13 insertions(+), 2 deletions(-)
diff --git a/server/reds.c b/
This switches the test to using the GTest API, and add several tests
related to https://bugzilla.redhat.com/show_bug.cgi?id=1411194
This uses some API not available in glib 2.28, so this checks we have a
new enough glib before building this test, and disables warnings when
using too new glib API w
>
> On Wed, Feb 15, 2017 at 06:02:00AM -0500, Frediano Ziglio wrote:
> > >
> > > On Tue, Feb 14, 2017 at 03:55:25PM +0100, Christophe Fergeau wrote:
> > > > I would not provide any default implementation, and just change
> > > > red_channel_config_socket to
> > > >
> > >
> > > Thinking more abo
This allows to chain several test cases by using
test_new()/test_destroy().
Signed-off-by: Christophe Fergeau
Acked-by: Pavel Grunt
---
server/tests/test-display-base.c | 7 +++
server/tests/test-display-base.h | 1 +
2 files changed, 8 insertions(+)
diff --git a/server/tests/test-display-
Only changes since v3 is the removal of redundant initialization in patch 2/5,
and an additional patch changing some spice_printerr() messages to
spice_debug().
Christophe
___
Spice-devel mailing list
Spice-devel@lists.freedesktop.org
https://lists.fre
Using spice_info() gets in the way of tests using
g_test_expect_message() as all the messages emitted using
a non-debug log level must be listed as expected, otherwise we get a
critical about an expected message not having been logged.
Signed-off-by: Christophe Fergeau
---
server/cursor-channel.
On Wed, Feb 15, 2017 at 06:02:00AM -0500, Frediano Ziglio wrote:
> >
> > On Tue, Feb 14, 2017 at 03:55:25PM +0100, Christophe Fergeau wrote:
> > > I would not provide any default implementation, and just change
> > > red_channel_config_socket to
> > >
> >
> > Thinking more about this, I actually
>
> On Tue, Feb 14, 2017 at 03:55:25PM +0100, Christophe Fergeau wrote:
> > I would not provide any default implementation, and just change
> > red_channel_config_socket to
> >
>
> Thinking more about this, I actually would provide a default
> implementation, and move the setsockopt/fnctl code f
On Wed, Feb 15, 2017 at 10:48:07AM +0100, Victor Toso wrote:
> From: Michal Suchanek
>
> This allows running big endian and little endian guest side by side using
> cut & paste between them.
>
> There is a general design idea that swapping should come as close to
> virtio_read/virtio_write as poss
On Wed, Feb 15, 2017 at 10:48:06AM +0100, Victor Toso wrote:
> From: Christophe Fergeau
'vdagentd:' prefix here too?
Just a small note, this was also wrong before "vdagentd: early
return on bad message size" patch.
Acked-by: Victor Toso
>
> ---
> src/vdagentd/vdagentd.c | 4 ++--
> 1 file ch
On Wed, Feb 15, 2017 at 10:48:05AM +0100, Victor Toso wrote:
> From: Christophe Fergeau
Maybe include vdagentd: prefix in the commit log?
Acked-by: Victor Toso
>
> ---
> src/vdagentd/vdagentd.c | 14 ++
> 1 file changed, 6 insertions(+), 8 deletions(-)
>
> diff --git a/src/vdagent
Hi,
On Wed, Feb 15, 2017 at 10:48:04AM +0100, Victor Toso wrote:
> From: Victor Toso
>
> The payload size for each message should be the size of the expected
> struct or bigger when it contain arrays of no fixed size.
>
> This patch creates the vdagent_message_min_size[] static array with
> the e
On Tue, Feb 14, 2017 at 03:55:25PM +0100, Christophe Fergeau wrote:
> I would not provide any default implementation, and just change
> red_channel_config_socket to
>
Thinking more about this, I actually would provide a default
implementation, and move the setsockopt/fnctl code from
reds_init_cli
From: Michal Suchanek
This allows running big endian and little endian guest side by side using
cut & paste between them.
There is a general design idea that swapping should come as close to
virtio_read/virtio_write as possible. In particular, the protocol
between vdagent and vdagentd is guest-s
From: Victor Toso
Hi there!
This patches were also pending for some time and the patches were
distributed in two mail threads and for that reason I'm puting them
here in (another) thread :)
This first three patches were attached to Christophe's email [0], the
last patch is the v7 from Michal [1
From: Christophe Fergeau
---
src/vdagentd/vdagentd.c | 14 ++
1 file changed, 6 insertions(+), 8 deletions(-)
diff --git a/src/vdagentd/vdagentd.c b/src/vdagentd/vdagentd.c
index ea8482b..8d43700 100644
--- a/src/vdagentd/vdagentd.c
+++ b/src/vdagentd/vdagentd.c
@@ -391,6 +391,8 @@
From: Victor Toso
The payload size for each message should be the size of the expected
struct or bigger when it contain arrays of no fixed size.
This patch creates the vdagent_message_min_size[] static array with
the expected size for each message so we can check on
virtio_port_read_complete().
From: Christophe Fergeau
---
src/vdagentd/vdagentd.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/vdagentd/vdagentd.c b/src/vdagentd/vdagentd.c
index 8d43700..2329489 100644
--- a/src/vdagentd/vdagentd.c
+++ b/src/vdagentd/vdagentd.c
@@ -395,8 +395,6 @@ static gboo
Hi,
On Wed, 2017-02-15 at 09:39 +0100, Christophe Fergeau wrote:
> Since c3d237 "gstreamer: Avoid memory copy if strides are different"
> is
> only needed when zero copy is disabled.
> ---
> server/gstreamer-encoder.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/server/gstreamer-en
On Wed, 2017-02-15 at 09:39 +0100, Christophe Fergeau wrote:
> rate_control_is_active() is static and not used in gstreamer-
> encoder.c
Yeah, not needed since 97fcad82eb7d1f3bbd8d1163801b5a3db92944c2
Acked-by: Pavel Grunt
> ---
> server/gstreamer-encoder.c | 5 -
> 1 file changed, 5 delet
Since c3d237 "gstreamer: Avoid memory copy if strides are different" is
only needed when zero copy is disabled.
---
server/gstreamer-encoder.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/server/gstreamer-encoder.c b/server/gstreamer-encoder.c
index dd2926d..bb41425 100644
--- a/server/gs
rate_control_is_active() is static and not used in gstreamer-encoder.c
---
server/gstreamer-encoder.c | 5 -
1 file changed, 5 deletions(-)
diff --git a/server/gstreamer-encoder.c b/server/gstreamer-encoder.c
index 5c60a64..dd2926d 100644
--- a/server/gstreamer-encoder.c
+++ b/server/gstreame
>
> On Tue, Feb 14, 2017 at 12:14:56PM -0600, Jonathon Jongsma wrote:
> > To be honest, I could go either way on this one. I like the
> > consistency, but I don't like the fact that it makes things like "git
> > blame" less useful. I'll leave the decision to others ;)
>
> I've pushed the series w
Acked-by: Christophe Fergeau
On Fri, Feb 10, 2017 at 02:30:57PM +, Frediano Ziglio wrote:
> smartcard_channel_client_pipe_add_push was just calling
> red_channel_client_pipe_add_push without any cast or other
> changes.
>
> Signed-off-by: Frediano Ziglio
> ---
> server/smartcard.c | 15 +++
84 matches
Mail list logo