[oe] [meta-networking][PATCH] spice, spice-protocol: Uprev to 0.14.0
Hongzhi, Song
hongzhi.song at windriver.com
Thu May 30 09:14:20 UTC 2019
The error I met:
| i686-poky-linux-gcc: warning: switch '-Wchkp' is no longer supported
| In file included from ../../git/subprojects/spice-common/common/mem.h:22,
| from
../../git/subprojects/spice-common/common/marshaller.h:25,
| from ../../git/server/reds.h:24,
| from ../../git/server/red-replay-qxl.c:27:
| ../../git/server/red-replay-qxl.c: In function 'read_binary':
| ../../git/subprojects/spice-common/common/log.h:80:62: error: format
'%i' expects argument of type 'int', but argument 6 has type 'long
unsigned int' [-Werror=format=]
| 80 | spice_log(G_LOG_LEVEL_ERROR, SPICE_STRLOC, __FUNCTION__,
"" format, ## __VA_ARGS__); \
| | ^~
| ../../git/server/red-replay-qxl.c:269:13: note: in expansion of macro
'spice_error'
| 269 | spice_error("inflate error %d (disc: %"
G_GSSIZE_FORMAT ")",
| | ^~~~~~~~~~~
My error looks like different from yours.
--Hongzhi
On 5/30/19 3:23 PM, Martin Jansa wrote:
> What kind of do_compile failure you're seeing Hongzhi?
>
> On Ubuntu-18.04 (with gcc-7.4.0) I'm seeing nativesdk-spice failing
> with couple address-of-packed-member errors like:
>
> ../../git/server/red-parse-qxl.c:1022:38: error: taking address of
> packed member of 'struct QXLDrawable' may result in an unaligned
> pointer value [-Werror=address-of-packed-member]
> 1022 | red->release_info_ext.info <http://release_info_ext.info>
> = &qxl->release_info;
> | ^~~~~~~~~~~~~~~~~~
>
> On Thu, May 30, 2019 at 6:57 AM Hongzhi, Song
> <hongzhi.song at windriver.com <mailto:hongzhi.song at windriver.com>> wrote:
>
> Spice has do_compile failure on v0.14
>
> Thanks,
>
> --Hongzhi
>
>
> On 5/24/19 1:47 AM, Khem Raj wrote:
> > Drop the patch for fixing printf formatting, its fixed differently
> > upstream
> >
> > Forward port pthread_t printf patch
> >
> > Add -Wno-address-of-packed-member needed with gcc9
> >
> > replace _append += with just _append syntax
> >
> > Signed-off-by: Khem Raj <raj.khem at gmail.com
> <mailto:raj.khem at gmail.com>>
> > ---
> > .../spice/spice-protocol_git.bb <http://spice-protocol_git.bb>
> | 4 +--
> > ...0001-Convert-pthread_t-to-be-numeric.patch | 25 +++++--------
> > .../0001-spice-fix-compile-fail-problem.patch | 36
> -------------------
> > .../recipes-support/spice/spice_git.bb <http://spice_git.bb>
> | 11 +++---
> > 4 files changed, 17 insertions(+), 59 deletions(-)
> > delete mode 100644
> meta-networking/recipes-support/spice/spice/0001-spice-fix-compile-fail-problem.patch
> >
> > diff --git
> a/meta-networking/recipes-support/spice/spice-protocol_git.bb
> <http://spice-protocol_git.bb>
> b/meta-networking/recipes-support/spice/spice-protocol_git.bb
> <http://spice-protocol_git.bb>
> > index 30d0a76752..be9dba32f1 100644
> > ---
> a/meta-networking/recipes-support/spice/spice-protocol_git.bb
> <http://spice-protocol_git.bb>
> > +++
> b/meta-networking/recipes-support/spice/spice-protocol_git.bb
> <http://spice-protocol_git.bb>
> > @@ -13,9 +13,9 @@ architectures."
> > LICENSE = "BSD"
> > LIC_FILES_CHKSUM =
> "file://COPYING;md5=b37311cb5604f3e5cc2fb0fd23527e95"
> >
> > -PV = "0.12.13+git${SRCPV}"
> > +PV = "0.14.0+git${SRCPV}"
> >
> > -SRCREV = "87441524f4e7b79658e42bd8f1f6c3e3c8649aa5"
> > +SRCREV = "f72ece993aeaf23f77e2845562b20e5563e52ba0"
> >
> > SRC_URI = " \
> > git://anongit.freedesktop.org/spice/spice-protocol
> <http://anongit.freedesktop.org/spice/spice-protocol> \
> > diff --git
> a/meta-networking/recipes-support/spice/spice/0001-Convert-pthread_t-to-be-numeric.patch
> b/meta-networking/recipes-support/spice/spice/0001-Convert-pthread_t-to-be-numeric.patch
> > index 505b7c890b..e63cbe3b72 100644
> > ---
> a/meta-networking/recipes-support/spice/spice/0001-Convert-pthread_t-to-be-numeric.patch
> > +++
> b/meta-networking/recipes-support/spice/spice/0001-Convert-pthread_t-to-be-numeric.patch
> > @@ -14,21 +14,19 @@ Upstream-Status: Pending
> > server/red-client.c | 6 +++---
> > 2 files changed, 6 insertions(+), 5 deletions(-)
> >
> > -diff --git a/server/red-channel.c b/server/red-channel.c
> > -index 1b38f04d..11dc667b 100644
> > --- a/server/red-channel.c
> > +++ b/server/red-channel.c
> > -@@ -192,7 +192,7 @@ red_channel_constructed(GObject *object)
> > +@@ -204,7 +204,7 @@ red_channel_constructed(GObject *object)
> > {
> > RedChannel *self = RED_CHANNEL(object);
> >
> > -- red_channel_debug(self, "thread_id 0x%lx",
> self->priv->thread_id);
> > -+ red_channel_debug(self, "thread_id 0x%lx", (unsigned
> long)self->priv->thread_id);
> > +- red_channel_debug(self, "thread_id 0x%" G_GSIZE_MODIFIER
> "x", self->priv->thread_id);
> > ++ red_channel_debug(self, "thread_id 0x%" G_GSIZE_MODIFIER
> "x", (unsigned long)self->priv->thread_id);
> >
> > RedChannelClass *klass = RED_CHANNEL_GET_CLASS(self);
> >
> > -@@ -475,7 +475,8 @@ void red_channel_remove_client(RedChannel
> *channel, RedChannelClient *rcc)
> > - red_channel_warning(channel, "channel->thread_id
> (0x%lx) != pthread_self (0x%lx)."
> > +@@ -479,7 +479,8 @@ void red_channel_remove_client(RedChanne
> > + "pthread_self (0x%"
> G_GSIZE_MODIFIER "x)."
> > "If one of the threads is !=
> io-thread && != vcpu-thread, "
> > "this might be a BUG",
> > - channel->priv->thread_id, pthread_self());
> > @@ -37,12 +35,10 @@ index 1b38f04d..11dc667b 100644
> > }
> > spice_return_if_fail(channel);
> > link = g_list_find(channel->priv->clients, rcc);
> > -diff --git a/server/red-client.c b/server/red-client.c
> > -index ddfc5400..76986640 100644
> > --- a/server/red-client.c
> > +++ b/server/red-client.c
> > -@@ -180,7 +180,7 @@ void red_client_migrate(RedClient *client)
> > - spice_warning("client->thread_id (0x%lx) !=
> pthread_self (0x%lx)."
> > +@@ -180,7 +180,7 @@ void red_client_migrate(RedClient *clien
> > + "pthread_self (0x%" G_GSIZE_MODIFIER "x)."
> > "If one of the threads is != io-thread
> && != vcpu-thread,"
> > " this might be a BUG",
> > - client->thread_id, pthread_self());
> > @@ -50,8 +46,8 @@ index ddfc5400..76986640 100644
> > }
> > FOREACH_CHANNEL_CLIENT(client, rcc) {
> > if (red_channel_client_is_connected(rcc)) {
> > -@@ -199,8 +199,8 @@ void red_client_destroy(RedClient *client)
> > - spice_warning("client->thread_id (0x%lx) !=
> pthread_self (0x%lx)."
> > +@@ -199,8 +199,8 @@ void red_client_destroy(RedClient *clien
> > + "pthread_self (0x%" G_GSIZE_MODIFIER "x)."
> > "If one of the threads is != io-thread
> && != vcpu-thread,"
> > " this might be a BUG",
> > - client->thread_id,
> > @@ -61,6 +57,3 @@ index ddfc5400..76986640 100644
> > }
> > red_client_set_disconnecting(client);
> > FOREACH_CHANNEL_CLIENT(client, rcc) {
> > ---
> > -2.17.1
> > -
> > diff --git
> a/meta-networking/recipes-support/spice/spice/0001-spice-fix-compile-fail-problem.patch
> b/meta-networking/recipes-support/spice/spice/0001-spice-fix-compile-fail-problem.patch
> > deleted file mode 100644
> > index 1f9d5fdd7b..0000000000
> > ---
> a/meta-networking/recipes-support/spice/spice/0001-spice-fix-compile-fail-problem.patch
> > +++ /dev/null
> > @@ -1,36 +0,0 @@
> > -From 7023732c65b4dc509c46a54fb7715da275b5597f Mon Sep 17
> 00:00:00 2001
> > -From: Changqing Li <changqing.li at windriver.com
> <mailto:changqing.li at windriver.com>>
> > -Date: Thu, 13 Sep 2018 12:39:44 +0800
> > -Subject: [PATCH] spice: fix compile fail problem
> > -
> > -compile error:
> > -format '%d' expects argument of type 'int', but argument 6 has
> > -type 'long unsigned int' [-Werror=format=]
> > -
> > -spice compile failed on 32bit system, since upstream commit
> > -9541cd2fe(in V0.14.1) change %ld to %PRIdPTR, %PRIdPTR is %d,
> but argument
> > -strm.total_out is uLong.
> > -
> > -Upstream-Status:
> Submitted[https://github.com/freedesktop/spice/pull/1]
> > -
> > -Signed-off-by: Changqing Li <changqing.li at windriver.com
> <mailto:changqing.li at windriver.com>>
> > ----
> > - server/red-replay-qxl.c | 2 +-
> > - 1 file changed, 1 insertion(+), 1 deletion(-)
> > -
> > -diff --git a/server/red-replay-qxl.c b/server/red-replay-qxl.c
> > -index 1fce76c..bd33b58 100644
> > ---- a/server/red-replay-qxl.c
> > -+++ b/server/red-replay-qxl.c
> > -@@ -266,7 +266,7 @@ static replay_t read_binary(SpiceReplay
> *replay, const char *prefix, size_t *siz
> > - exit(1);
> > - }
> > - if ((ret = inflate(&strm, Z_NO_FLUSH)) != Z_STREAM_END) {
> > -- spice_error("inflate error %d (disc: %" PRIdPTR
> ")", ret, *size - strm.total_out);
> > -+ spice_error("inflate error %d (disc: %ld)", ret,
> *size - strm.total_out);
> > - if (ret == Z_DATA_ERROR) {
> > - /* last operation may be wrong. since we do
> the recording
> > - * in red_worker, when there is a shutdown
> from the vcpu/io thread
> > ---
> > -2.7.4
> > -
> > diff --git a/meta-networking/recipes-support/spice/spice_git.bb
> <http://spice_git.bb>
> b/meta-networking/recipes-support/spice/spice_git.bb
> <http://spice_git.bb>
> > index 1ad46c2c31..6249b0422b 100644
> > --- a/meta-networking/recipes-support/spice/spice_git.bb
> <http://spice_git.bb>
> > +++ b/meta-networking/recipes-support/spice/spice_git.bb
> <http://spice_git.bb>
> > @@ -13,10 +13,10 @@ architectures."
> > LICENSE = "BSD & LGPLv2.1+"
> > LIC_FILES_CHKSUM =
> "file://COPYING;md5=4fbd65380cdd255951079008b364516c"
> >
> > -PV = "0.14.1+git${SRCPV}"
> > +PV = "0.14.2+git${SRCPV}"
> >
> > -SRCREV_spice = "eaa07ef15cfc3bf57a69da2576af66f028787774"
> > -SRCREV_spice-common = "6b93b3fce8909b836ef1d1434d191900d8aa00be"
> > +SRCREV_spice = "7cbd70b931db76c69c89c2d9d5d704f67381a81b"
> > +SRCREV_spice-common = "4fc4c2db36c7f07b906e9a326a9d3dc0ae6a2671"
> >
> > SRCREV_FORMAT = "spice_spice-common"
> >
> > @@ -24,7 +24,6 @@ SRC_URI = " \
> > git://anongit.freedesktop.org/spice/spice;name=spice
> <http://anongit.freedesktop.org/spice/spice;name=spice> \
> >
> git://anongit.freedesktop.org/spice/spice-common;destsuffix=git/subprojects/spice-common;name=spice-common
> <http://anongit.freedesktop.org/spice/spice-common;destsuffix=git/subprojects/spice-common;name=spice-common>
> \
> > file://0001-Convert-pthread_t-to-be-numeric.patch \
> > - file://0001-spice-fix-compile-fail-problem.patch \
> > "
> >
> > S = "${WORKDIR}/git"
> > @@ -37,6 +36,8 @@ DEPENDS_append_class-nativesdk =
> "nativesdk-openssl"
> > export PYTHON="${STAGING_BINDIR_NATIVE}/python-native/python"
> > export
> PYTHONPATH="${PKG_CONFIG_SYSROOT_DIR}${libdir}/python2.7/site-packages"
> >
> > +CFLAGS += "-Wno-address-of-packed-member"
> > +
> > PACKAGECONFIG_class-native = ""
> > PACKAGECONFIG_class-nativesdk = ""
> > PACKAGECONFIG ?= "sasl"
> > @@ -54,4 +55,4 @@ COMPATIBLE_HOST = '(x86_64|i.86).*-linux'
> >
> > BBCLASSEXTEND = "native nativesdk"
> >
> > -EXTRA_OECONF_toolchain-clang += "--disable-werror"
> > +EXTRA_OECONF_append_toolchain-clang = " --disable-werror"
> --
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel at lists.openembedded.org
> <mailto:Openembedded-devel at lists.openembedded.org>
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
>
More information about the Openembedded-devel
mailing list