All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0/2] Fix Xen compilation
@ 2020-11-09 14:47 Bertrand Marquis
  2020-11-09 14:49 ` [PATCH 1/2] xen: Fix python libraries installation on multilib Bertrand Marquis
                   ` (3 more replies)
  0 siblings, 4 replies; 10+ messages in thread
From: Bertrand Marquis @ 2020-11-09 14:47 UTC (permalink / raw)
  To: meta-virtualization; +Cc: christopher.clark6

This serie contain 2 patches to fix Xen compilation:
- on arm32 by adding a patch already merged in Xen master to fix gcc10
  compilation issues.
- on x86 to fix an issue happening when compiling xen-tools with older
  xen versions.

Those would be good candidates to include in gatesgarth.

Bertrand Marquis (2):
  xen: Fix python libraries installation on multilib
  xen: Fix xenpmd compilation error on arm32

 ...ls-xenpmd-Fix-gcc10-snprintf-warning.patch | 60 +++++++++++++++++++
 recipes-extended/xen/xen-tools.inc            |  2 +
 recipes-extended/xen/xen-tools_4.14.bb        |  1 +
 recipes-extended/xen/xen-tools_git.bb         |  1 +
 4 files changed, 64 insertions(+)
 create mode 100644 recipes-extended/xen/files/0001-tools-xenpmd-Fix-gcc10-snprintf-warning.patch

-- 
2.17.1


^ permalink raw reply	[flat|nested] 10+ messages in thread

* [PATCH 1/2] xen: Fix python libraries installation on multilib
  2020-11-09 14:47 [PATCH 0/2] Fix Xen compilation Bertrand Marquis
@ 2020-11-09 14:49 ` Bertrand Marquis
  2020-11-10  7:47   ` [meta-virtualization] " Christopher Clark
  2020-11-09 14:49 ` [PATCH 2/2] xen: Fix xenpmd compilation error on arm32 Bertrand Marquis
                   ` (2 subsequent siblings)
  3 siblings, 1 reply; 10+ messages in thread
From: Bertrand Marquis @ 2020-11-09 14:49 UTC (permalink / raw)
  To: meta-virtualization

Fix install path for xen python libraries when multilib configuration is
activated (which is mandatory to compile xen for x86_64).
Depending on the version of Xen, the libraries are installed on lib or
on the nonarch lib directory so add both.

Signed-off-by: Bertrand Marquis <bertrand.marquis@arm.com>
Change-Id: I1f7f81a4e9e1420bce55d1e4bfe03d98ac2c93a3
---
 recipes-extended/xen/xen-tools.inc | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/recipes-extended/xen/xen-tools.inc b/recipes-extended/xen/xen-tools.inc
index 736a766..0c69fec 100644
--- a/recipes-extended/xen/xen-tools.inc
+++ b/recipes-extended/xen/xen-tools.inc
@@ -496,7 +496,9 @@ FILES_${PN}-pygrub = "\
     ${libdir}/xen/bin/pygrub \
     "
 
+# Depending on the version of Xen libdir or nonarch libdir is used
 FILES_${PN}-python = "\
+    ${libdir}/${PYTHON_DIR} \
     ${nonarch_libdir}/${PYTHON_DIR} \
     "
 
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 10+ messages in thread

* [PATCH 2/2] xen: Fix xenpmd compilation error on arm32
  2020-11-09 14:47 [PATCH 0/2] Fix Xen compilation Bertrand Marquis
  2020-11-09 14:49 ` [PATCH 1/2] xen: Fix python libraries installation on multilib Bertrand Marquis
@ 2020-11-09 14:49 ` Bertrand Marquis
  2020-11-10 12:32   ` [meta-virtualization] " Christopher Clark
  2020-11-10 12:37 ` [meta-virtualization] [PATCH 0/2] Fix Xen compilation Christopher Clark
  2020-11-15  4:00 ` Bruce Ashfield
  3 siblings, 1 reply; 10+ messages in thread
From: Bertrand Marquis @ 2020-11-09 14:49 UTC (permalink / raw)
  To: meta-virtualization

Add patch merged in current Xen master to solve compilation errors on
xenmpd when Xen is compiled for arm32.

Signed-off-by: Bertrand Marquis <bertrand.marquis@arm.com>
Change-Id: Ifd94aa30e1b3a1016156ead395688f594ad2711d
---
 ...ls-xenpmd-Fix-gcc10-snprintf-warning.patch | 60 +++++++++++++++++++
 recipes-extended/xen/xen-tools_4.14.bb        |  1 +
 recipes-extended/xen/xen-tools_git.bb         |  1 +
 3 files changed, 62 insertions(+)
 create mode 100644 recipes-extended/xen/files/0001-tools-xenpmd-Fix-gcc10-snprintf-warning.patch

diff --git a/recipes-extended/xen/files/0001-tools-xenpmd-Fix-gcc10-snprintf-warning.patch b/recipes-extended/xen/files/0001-tools-xenpmd-Fix-gcc10-snprintf-warning.patch
new file mode 100644
index 0000000..0a136c9
--- /dev/null
+++ b/recipes-extended/xen/files/0001-tools-xenpmd-Fix-gcc10-snprintf-warning.patch
@@ -0,0 +1,60 @@
+From 0dfddb2116e3757f77a691a3fe335173088d69dc Mon Sep 17 00:00:00 2001
+Message-Id: <0dfddb2116e3757f77a691a3fe335173088d69dc.1604734077.git.bertrand.marquis@arm.com>
+From: Bertrand Marquis <bertrand.marquis@arm.com>
+Date: Thu, 15 Oct 2020 10:16:09 +0100
+Subject: [PATCH] tools/xenpmd: Fix gcc10 snprintf warning
+
+Add a check for snprintf return code and ignore the entry if we get an
+error. This should in fact never happen and is more a trick to make gcc
+happy and prevent compilation errors.
+
+This is solving the following gcc warning when compiling for arm32 host
+platforms with optimization activated:
+xenpmd.c:92:37: error: '%s' directive output may be truncated writing
+between 4 and 2147483645 bytes into a region of size 271
+[-Werror=format-truncation=]
+
+This is also solving the following Debian bug:
+https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=970802
+
+Signed-off-by: Bertrand Marquis <bertrand.marquis@arm.com>
+Acked-by: Wei Liu <wl@xen.org>
+---
+Upstream-status: Backport from 4.15
+---
+ tools/xenpmd/xenpmd.c | 9 +++++++--
+ 1 file changed, 7 insertions(+), 2 deletions(-)
+
+diff --git a/tools/xenpmd/xenpmd.c b/tools/xenpmd/xenpmd.c
+index 35fd1c931a..12b82cf43e 100644
+--- a/tools/xenpmd/xenpmd.c
++++ b/tools/xenpmd/xenpmd.c
+@@ -102,6 +102,7 @@ FILE *get_next_battery_file(DIR *battery_dir,
+     FILE *file = 0;
+     struct dirent *dir_entries;
+     char file_name[284];
++    int ret;
+     
+     do 
+     {
+@@ -111,11 +112,15 @@ FILE *get_next_battery_file(DIR *battery_dir,
+         if ( strlen(dir_entries->d_name) < 4 )
+             continue;
+         if ( battery_info_type == BIF ) 
+-            snprintf(file_name, sizeof(file_name), BATTERY_INFO_FILE_PATH,
++            ret = snprintf(file_name, sizeof(file_name), BATTERY_INFO_FILE_PATH,
+                      dir_entries->d_name);
+         else 
+-            snprintf(file_name, sizeof(file_name), BATTERY_STATE_FILE_PATH,
++            ret = snprintf(file_name, sizeof(file_name), BATTERY_STATE_FILE_PATH,
+                      dir_entries->d_name);
++        /* This should not happen but is needed to pass gcc checks */
++        if (ret < 0)
++            continue;
++        file_name[sizeof(file_name) - 1] = '\0';
+         file = fopen(file_name, "r");
+     } while ( !file );
+ 
+-- 
+2.17.1
+
diff --git a/recipes-extended/xen/xen-tools_4.14.bb b/recipes-extended/xen/xen-tools_4.14.bb
index 4119190..73a52bc 100644
--- a/recipes-extended/xen/xen-tools_4.14.bb
+++ b/recipes-extended/xen/xen-tools_4.14.bb
@@ -7,6 +7,7 @@ SRC_URI = " \
     git://xenbits.xen.org/xen.git;branch=${XEN_BRANCH} \
     file://0001-python-pygrub-pass-DISTUTILS-xen-4.14.patch \
     file://0001-xen-build-temporarily-inhibit-Werror-4.14.patch \
+    file://0001-tools-xenpmd-Fix-gcc10-snprintf-warning.patch \
     "
 
 LIC_FILES_CHKSUM ?= "file://COPYING;md5=419739e325a50f3d7b4501338e44a4e5"
diff --git a/recipes-extended/xen/xen-tools_git.bb b/recipes-extended/xen/xen-tools_git.bb
index 71aeeed..1f6be1d 100644
--- a/recipes-extended/xen/xen-tools_git.bb
+++ b/recipes-extended/xen/xen-tools_git.bb
@@ -7,6 +7,7 @@ SRC_URI = " \
     git://xenbits.xen.org/xen.git;branch=${XEN_BRANCH} \
     file://0001-python-pygrub-pass-DISTUTILS-xen-4.14.patch \
     file://0001-xen-build-temporarily-inhibit-Werror-4.14.patch \
+    file://0001-tools-xenpmd-Fix-gcc10-snprintf-warning.patch \
     "
 
 LIC_FILES_CHKSUM ?= "file://COPYING;md5=419739e325a50f3d7b4501338e44a4e5"
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 10+ messages in thread

* Re: [meta-virtualization] [PATCH 1/2] xen: Fix python libraries installation on multilib
  2020-11-09 14:49 ` [PATCH 1/2] xen: Fix python libraries installation on multilib Bertrand Marquis
@ 2020-11-10  7:47   ` Christopher Clark
  2020-11-10  8:31     ` Bertrand Marquis
  0 siblings, 1 reply; 10+ messages in thread
From: Christopher Clark @ 2020-11-10  7:47 UTC (permalink / raw)
  To: Bertrand Marquis; +Cc: meta-virtualization

On Mon, Nov 9, 2020 at 6:49 AM Bertrand Marquis
<bertrand.marquis@arm.com> wrote:
>
> Fix install path for xen python libraries when multilib configuration is
> activated (which is mandatory to compile xen for x86_64).
> Depending on the version of Xen, the libraries are installed on lib or
> on the nonarch lib directory so add both.

Thanks, Bertrand -- I have positive results for this with
compile-tests with both Xen 4.14 and 4.15 for x86_64.

> Signed-off-by: Bertrand Marquis <bertrand.marquis@arm.com>
> Change-Id: I1f7f81a4e9e1420bce55d1e4bfe03d98ac2c93a3
> ---
>  recipes-extended/xen/xen-tools.inc | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/recipes-extended/xen/xen-tools.inc b/recipes-extended/xen/xen-tools.inc
> index 736a766..0c69fec 100644
> --- a/recipes-extended/xen/xen-tools.inc
> +++ b/recipes-extended/xen/xen-tools.inc
> @@ -496,7 +496,9 @@ FILES_${PN}-pygrub = "\
>      ${libdir}/xen/bin/pygrub \
>      "
>
> +# Depending on the version of Xen libdir or nonarch libdir is used
>  FILES_${PN}-python = "\
> +    ${libdir}/${PYTHON_DIR} \
>      ${nonarch_libdir}/${PYTHON_DIR} \
>      "

Do you know whether a similar change is also needed for the python
nonarch_libdir items in the -dbg package, when building older versions
of Xen?

Christopher

>
> --
> 2.17.1
>
>
> 
>

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [meta-virtualization] [PATCH 1/2] xen: Fix python libraries installation on multilib
  2020-11-10  7:47   ` [meta-virtualization] " Christopher Clark
@ 2020-11-10  8:31     ` Bertrand Marquis
  2020-11-10 12:18       ` Christopher Clark
  0 siblings, 1 reply; 10+ messages in thread
From: Bertrand Marquis @ 2020-11-10  8:31 UTC (permalink / raw)
  To: Christopher Clark; +Cc: meta-virtualization

Hi Christopher,

> On 10 Nov 2020, at 07:47, Christopher Clark <christopher.w.clark@gmail.com> wrote:
>
> On Mon, Nov 9, 2020 at 6:49 AM Bertrand Marquis
> <bertrand.marquis@arm.com> wrote:
>>
>> Fix install path for xen python libraries when multilib configuration is
>> activated (which is mandatory to compile xen for x86_64).
>> Depending on the version of Xen, the libraries are installed on lib or
>> on the nonarch lib directory so add both.
>
> Thanks, Bertrand -- I have positive results for this with
> compile-tests with both Xen 4.14 and 4.15 for x86_64.

I tried on x86_64 and also on arm32 and arm64 and all compile tests are now passing.

>
>> Signed-off-by: Bertrand Marquis <bertrand.marquis@arm.com>
>> Change-Id: I1f7f81a4e9e1420bce55d1e4bfe03d98ac2c93a3
>> ---
>> recipes-extended/xen/xen-tools.inc | 2 ++
>> 1 file changed, 2 insertions(+)
>>
>> diff --git a/recipes-extended/xen/xen-tools.inc b/recipes-extended/xen/xen-tools.inc
>> index 736a766..0c69fec 100644
>> --- a/recipes-extended/xen/xen-tools.inc
>> +++ b/recipes-extended/xen/xen-tools.inc
>> @@ -496,7 +496,9 @@ FILES_${PN}-pygrub = "\
>>     ${libdir}/xen/bin/pygrub \
>>     "
>>
>> +# Depending on the version of Xen libdir or nonarch libdir is used
>> FILES_${PN}-python = "\
>> +    ${libdir}/${PYTHON_DIR} \
>>     ${nonarch_libdir}/${PYTHON_DIR} \
>>     "
>
> Do you know whether a similar change is also needed for the python
> nonarch_libdir items in the -dbg package, when building older versions
> of Xen?

No from my tests this is not needed.
Just after gatesgarth branch is released i will push patches to update Xen SRCREV
in both 4.14 and master (which will also allow to remove some patches).
I will also spend some time checking dunfell but right now i have no results i can share.

Cheers
Bertrand

>
> Christopher
>
>>
>> --
>> 2.17.1
>>
>>
>> 
>>

IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [meta-virtualization] [PATCH 1/2] xen: Fix python libraries installation on multilib
  2020-11-10  8:31     ` Bertrand Marquis
@ 2020-11-10 12:18       ` Christopher Clark
  0 siblings, 0 replies; 10+ messages in thread
From: Christopher Clark @ 2020-11-10 12:18 UTC (permalink / raw)
  To: Bertrand Marquis; +Cc: meta-virtualization

On Tue, Nov 10, 2020 at 12:31 AM Bertrand Marquis
<bertrand.marquis@arm.com> wrote:
>
> Hi Christopher,
>
> > On 10 Nov 2020, at 07:47, Christopher Clark <christopher.w.clark@gmail.com> wrote:
> >
> > On Mon, Nov 9, 2020 at 6:49 AM Bertrand Marquis
> > <bertrand.marquis@arm.com> wrote:
> >>
> >> Fix install path for xen python libraries when multilib configuration is
> >> activated (which is mandatory to compile xen for x86_64).
> >> Depending on the version of Xen, the libraries are installed on lib or
> >> on the nonarch lib directory so add both.
> >
> > Thanks, Bertrand -- I have positive results for this with
> > compile-tests with both Xen 4.14 and 4.15 for x86_64.
>
> I tried on x86_64 and also on arm32 and arm64 and all compile tests are now passing.
>
> >
> >> Signed-off-by: Bertrand Marquis <bertrand.marquis@arm.com>

Reviewed-by: Christopher Clark <christopher.clark6@baesystems.com>

> >> Change-Id: I1f7f81a4e9e1420bce55d1e4bfe03d98ac2c93a3
> >> ---
> >> recipes-extended/xen/xen-tools.inc | 2 ++
> >> 1 file changed, 2 insertions(+)
> >>
> >> diff --git a/recipes-extended/xen/xen-tools.inc b/recipes-extended/xen/xen-tools.inc
> >> index 736a766..0c69fec 100644
> >> --- a/recipes-extended/xen/xen-tools.inc
> >> +++ b/recipes-extended/xen/xen-tools.inc
> >> @@ -496,7 +496,9 @@ FILES_${PN}-pygrub = "\
> >>     ${libdir}/xen/bin/pygrub \
> >>     "
> >>
> >> +# Depending on the version of Xen libdir or nonarch libdir is used
> >> FILES_${PN}-python = "\
> >> +    ${libdir}/${PYTHON_DIR} \
> >>     ${nonarch_libdir}/${PYTHON_DIR} \
> >>     "
> >
> > Do you know whether a similar change is also needed for the python
> > nonarch_libdir items in the -dbg package, when building older versions
> > of Xen?
>
> No from my tests this is not needed.
> Just after gatesgarth branch is released i will push patches to update Xen SRCREV
> in both 4.14 and master (which will also allow to remove some patches).
> I will also spend some time checking dunfell but right now i have no results i can share.

Excellent - thanks.

Christopher

>
> Cheers
> Bertrand
>
> >
> > Christopher
> >
> >>
> >> --
> >> 2.17.1
> >>
> >>
> >>
> >>
>
> IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.
>
> 
>

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [meta-virtualization] [PATCH 2/2] xen: Fix xenpmd compilation error on arm32
  2020-11-09 14:49 ` [PATCH 2/2] xen: Fix xenpmd compilation error on arm32 Bertrand Marquis
@ 2020-11-10 12:32   ` Christopher Clark
  0 siblings, 0 replies; 10+ messages in thread
From: Christopher Clark @ 2020-11-10 12:32 UTC (permalink / raw)
  To: Bertrand Marquis; +Cc: meta-virtualization

On Mon, Nov 9, 2020 at 6:49 AM Bertrand Marquis
<bertrand.marquis@arm.com> wrote:
>
> Add patch merged in current Xen master to solve compilation errors on
> xenmpd when Xen is compiled for arm32.
>
> Signed-off-by: Bertrand Marquis <bertrand.marquis@arm.com>

Reviewed-by: Christopher Clark <christopher.clark6@baesystems.com>

> Change-Id: Ifd94aa30e1b3a1016156ead395688f594ad2711d
> ---
>  ...ls-xenpmd-Fix-gcc10-snprintf-warning.patch | 60 +++++++++++++++++++
>  recipes-extended/xen/xen-tools_4.14.bb        |  1 +
>  recipes-extended/xen/xen-tools_git.bb         |  1 +
>  3 files changed, 62 insertions(+)
>  create mode 100644 recipes-extended/xen/files/0001-tools-xenpmd-Fix-gcc10-snprintf-warning.patch
>
> diff --git a/recipes-extended/xen/files/0001-tools-xenpmd-Fix-gcc10-snprintf-warning.patch b/recipes-extended/xen/files/0001-tools-xenpmd-Fix-gcc10-snprintf-warning.patch
> new file mode 100644
> index 0000000..0a136c9
> --- /dev/null
> +++ b/recipes-extended/xen/files/0001-tools-xenpmd-Fix-gcc10-snprintf-warning.patch
> @@ -0,0 +1,60 @@
> +From 0dfddb2116e3757f77a691a3fe335173088d69dc Mon Sep 17 00:00:00 2001
> +Message-Id: <0dfddb2116e3757f77a691a3fe335173088d69dc.1604734077.git.bertrand.marquis@arm.com>
> +From: Bertrand Marquis <bertrand.marquis@arm.com>
> +Date: Thu, 15 Oct 2020 10:16:09 +0100
> +Subject: [PATCH] tools/xenpmd: Fix gcc10 snprintf warning
> +
> +Add a check for snprintf return code and ignore the entry if we get an
> +error. This should in fact never happen and is more a trick to make gcc
> +happy and prevent compilation errors.
> +
> +This is solving the following gcc warning when compiling for arm32 host
> +platforms with optimization activated:
> +xenpmd.c:92:37: error: '%s' directive output may be truncated writing
> +between 4 and 2147483645 bytes into a region of size 271
> +[-Werror=format-truncation=]
> +
> +This is also solving the following Debian bug:
> +https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=970802
> +
> +Signed-off-by: Bertrand Marquis <bertrand.marquis@arm.com>
> +Acked-by: Wei Liu <wl@xen.org>
> +---
> +Upstream-status: Backport from 4.15
> +---
> + tools/xenpmd/xenpmd.c | 9 +++++++--
> + 1 file changed, 7 insertions(+), 2 deletions(-)
> +
> +diff --git a/tools/xenpmd/xenpmd.c b/tools/xenpmd/xenpmd.c
> +index 35fd1c931a..12b82cf43e 100644
> +--- a/tools/xenpmd/xenpmd.c
> ++++ b/tools/xenpmd/xenpmd.c
> +@@ -102,6 +102,7 @@ FILE *get_next_battery_file(DIR *battery_dir,
> +     FILE *file = 0;
> +     struct dirent *dir_entries;
> +     char file_name[284];
> ++    int ret;
> +
> +     do
> +     {
> +@@ -111,11 +112,15 @@ FILE *get_next_battery_file(DIR *battery_dir,
> +         if ( strlen(dir_entries->d_name) < 4 )
> +             continue;
> +         if ( battery_info_type == BIF )
> +-            snprintf(file_name, sizeof(file_name), BATTERY_INFO_FILE_PATH,
> ++            ret = snprintf(file_name, sizeof(file_name), BATTERY_INFO_FILE_PATH,
> +                      dir_entries->d_name);
> +         else
> +-            snprintf(file_name, sizeof(file_name), BATTERY_STATE_FILE_PATH,
> ++            ret = snprintf(file_name, sizeof(file_name), BATTERY_STATE_FILE_PATH,
> +                      dir_entries->d_name);
> ++        /* This should not happen but is needed to pass gcc checks */
> ++        if (ret < 0)
> ++            continue;
> ++        file_name[sizeof(file_name) - 1] = '\0';
> +         file = fopen(file_name, "r");
> +     } while ( !file );
> +
> +--
> +2.17.1
> +
> diff --git a/recipes-extended/xen/xen-tools_4.14.bb b/recipes-extended/xen/xen-tools_4.14.bb
> index 4119190..73a52bc 100644
> --- a/recipes-extended/xen/xen-tools_4.14.bb
> +++ b/recipes-extended/xen/xen-tools_4.14.bb
> @@ -7,6 +7,7 @@ SRC_URI = " \
>      git://xenbits.xen.org/xen.git;branch=${XEN_BRANCH} \
>      file://0001-python-pygrub-pass-DISTUTILS-xen-4.14.patch \
>      file://0001-xen-build-temporarily-inhibit-Werror-4.14.patch \
> +    file://0001-tools-xenpmd-Fix-gcc10-snprintf-warning.patch \
>      "
>
>  LIC_FILES_CHKSUM ?= "file://COPYING;md5=419739e325a50f3d7b4501338e44a4e5"
> diff --git a/recipes-extended/xen/xen-tools_git.bb b/recipes-extended/xen/xen-tools_git.bb
> index 71aeeed..1f6be1d 100644
> --- a/recipes-extended/xen/xen-tools_git.bb
> +++ b/recipes-extended/xen/xen-tools_git.bb
> @@ -7,6 +7,7 @@ SRC_URI = " \
>      git://xenbits.xen.org/xen.git;branch=${XEN_BRANCH} \
>      file://0001-python-pygrub-pass-DISTUTILS-xen-4.14.patch \
>      file://0001-xen-build-temporarily-inhibit-Werror-4.14.patch \
> +    file://0001-tools-xenpmd-Fix-gcc10-snprintf-warning.patch \
>      "
>
>  LIC_FILES_CHKSUM ?= "file://COPYING;md5=419739e325a50f3d7b4501338e44a4e5"
> --
> 2.17.1
>
>
> 
>

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [meta-virtualization] [PATCH 0/2] Fix Xen compilation
  2020-11-09 14:47 [PATCH 0/2] Fix Xen compilation Bertrand Marquis
  2020-11-09 14:49 ` [PATCH 1/2] xen: Fix python libraries installation on multilib Bertrand Marquis
  2020-11-09 14:49 ` [PATCH 2/2] xen: Fix xenpmd compilation error on arm32 Bertrand Marquis
@ 2020-11-10 12:37 ` Christopher Clark
  2020-11-15  4:00 ` Bruce Ashfield
  3 siblings, 0 replies; 10+ messages in thread
From: Christopher Clark @ 2020-11-10 12:37 UTC (permalink / raw)
  To: Bertrand Marquis; +Cc: meta-virtualization, Christopher Clark

On Mon, Nov 9, 2020 at 6:47 AM Bertrand Marquis
<bertrand.marquis@arm.com> wrote:
>
> This serie contain 2 patches to fix Xen compilation:
> - on arm32 by adding a patch already merged in Xen master to fix gcc10
>   compilation issues.
> - on x86 to fix an issue happening when compiling xen-tools with older
>   xen versions.

Reviewed - thanks.

> Those would be good candidates to include in gatesgarth.

Agreed.

Christopher

>
> Bertrand Marquis (2):
>   xen: Fix python libraries installation on multilib
>   xen: Fix xenpmd compilation error on arm32
>
>  ...ls-xenpmd-Fix-gcc10-snprintf-warning.patch | 60 +++++++++++++++++++
>  recipes-extended/xen/xen-tools.inc            |  2 +
>  recipes-extended/xen/xen-tools_4.14.bb        |  1 +
>  recipes-extended/xen/xen-tools_git.bb         |  1 +
>  4 files changed, 64 insertions(+)
>  create mode 100644 recipes-extended/xen/files/0001-tools-xenpmd-Fix-gcc10-snprintf-warning.patch
>
> --
> 2.17.1
>
>
> 
>

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [meta-virtualization] [PATCH 0/2] Fix Xen compilation
  2020-11-09 14:47 [PATCH 0/2] Fix Xen compilation Bertrand Marquis
                   ` (2 preceding siblings ...)
  2020-11-10 12:37 ` [meta-virtualization] [PATCH 0/2] Fix Xen compilation Christopher Clark
@ 2020-11-15  4:00 ` Bruce Ashfield
  2020-11-16  8:46   ` Bertrand Marquis
  3 siblings, 1 reply; 10+ messages in thread
From: Bruce Ashfield @ 2020-11-15  4:00 UTC (permalink / raw)
  To: Bertrand Marquis; +Cc: meta-virtualization, christopher.clark6

In message: [meta-virtualization] [PATCH 0/2] Fix Xen compilation
on 09/11/2020 Bertrand Marquis wrote:

> This serie contain 2 patches to fix Xen compilation:
> - on arm32 by adding a patch already merged in Xen master to fix gcc10
>   compilation issues.
> - on x86 to fix an issue happening when compiling xen-tools with older
>   xen versions.
> 
> Those would be good candidates to include in gatesgarth.

merged.

And yes, I haven't branched gatesgarth yet, so they will be part of that.

Bruce

> 
> Bertrand Marquis (2):
>   xen: Fix python libraries installation on multilib
>   xen: Fix xenpmd compilation error on arm32
> 
>  ...ls-xenpmd-Fix-gcc10-snprintf-warning.patch | 60 +++++++++++++++++++
>  recipes-extended/xen/xen-tools.inc            |  2 +
>  recipes-extended/xen/xen-tools_4.14.bb        |  1 +
>  recipes-extended/xen/xen-tools_git.bb         |  1 +
>  4 files changed, 64 insertions(+)
>  create mode 100644 recipes-extended/xen/files/0001-tools-xenpmd-Fix-gcc10-snprintf-warning.patch
> 
> -- 
> 2.17.1
> 

> 
> 
> 


^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [meta-virtualization] [PATCH 0/2] Fix Xen compilation
  2020-11-15  4:00 ` Bruce Ashfield
@ 2020-11-16  8:46   ` Bertrand Marquis
  0 siblings, 0 replies; 10+ messages in thread
From: Bertrand Marquis @ 2020-11-16  8:46 UTC (permalink / raw)
  To: Bruce Ashfield; +Cc: meta-virtualization, christopher.clark6



> On 15 Nov 2020, at 04:00, Bruce Ashfield via lists.yoctoproject.org <bruce.ashfield=gmail.com@lists.yoctoproject.org> wrote:
>
> In message: [meta-virtualization] [PATCH 0/2] Fix Xen compilation
> on 09/11/2020 Bertrand Marquis wrote:
>
>> This serie contain 2 patches to fix Xen compilation:
>> - on arm32 by adding a patch already merged in Xen master to fix gcc10
>>  compilation issues.
>> - on x86 to fix an issue happening when compiling xen-tools with older
>>  xen versions.
>>
>> Those would be good candidates to include in gatesgarth.
>
> merged.

Thanks

>
> And yes, I haven't branched gatesgarth yet, so they will be part of that.

Perfect.

Cheers
Bertrand

>
> Bruce
>
>>
>> Bertrand Marquis (2):
>>  xen: Fix python libraries installation on multilib
>>  xen: Fix xenpmd compilation error on arm32
>>
>> ...ls-xenpmd-Fix-gcc10-snprintf-warning.patch | 60 +++++++++++++++++++
>> recipes-extended/xen/xen-tools.inc            |  2 +
>> recipes-extended/xen/xen-tools_4.14.bb        |  1 +
>> recipes-extended/xen/xen-tools_git.bb         |  1 +
>> 4 files changed, 64 insertions(+)
>> create mode 100644 recipes-extended/xen/files/0001-tools-xenpmd-Fix-gcc10-snprintf-warning.patch
>>
>> --
>> 2.17.1
>>
>
>>
>>
>>
>
>
> 
>

IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.

^ permalink raw reply	[flat|nested] 10+ messages in thread

end of thread, other threads:[~2020-11-16  8:46 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-11-09 14:47 [PATCH 0/2] Fix Xen compilation Bertrand Marquis
2020-11-09 14:49 ` [PATCH 1/2] xen: Fix python libraries installation on multilib Bertrand Marquis
2020-11-10  7:47   ` [meta-virtualization] " Christopher Clark
2020-11-10  8:31     ` Bertrand Marquis
2020-11-10 12:18       ` Christopher Clark
2020-11-09 14:49 ` [PATCH 2/2] xen: Fix xenpmd compilation error on arm32 Bertrand Marquis
2020-11-10 12:32   ` [meta-virtualization] " Christopher Clark
2020-11-10 12:37 ` [meta-virtualization] [PATCH 0/2] Fix Xen compilation Christopher Clark
2020-11-15  4:00 ` Bruce Ashfield
2020-11-16  8:46   ` Bertrand Marquis

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.