* [PATCH] When building with systemd based image, udev-hwdb postinstall script will fail because of not passing correct CPU option. So fix it by passing correct QEMU_OPTIONS.
@ 2014-11-27 11:17 Gaurang Shastri
2014-12-04 19:03 ` Iorga, Cristian
2014-12-04 19:11 ` Iorga, Cristian
0 siblings, 2 replies; 7+ messages in thread
From: Gaurang Shastri @ 2014-11-27 11:17 UTC (permalink / raw)
To: openembedded-core; +Cc: gmshastri
Signed-off-by: Gaurang Shastri <gshastri@juniper.net>
---
meta/classes/qemu.bbclass | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/meta/classes/qemu.bbclass b/meta/classes/qemu.bbclass
index b2cf85d..c373855 100644
--- a/meta/classes/qemu.bbclass
+++ b/meta/classes/qemu.bbclass
@@ -30,8 +30,9 @@ def qemu_run_binary(data, rootfs_path, binary):
libdir = rootfs_path + data.getVar("libdir", False)
base_libdir = rootfs_path + data.getVar("base_libdir", False)
oldest_kernel = data.getVar("OLDEST_KERNEL", True)
+ qemu_option = data.getVar("QEMU_OPTIONS",True)
- return "PSEUDO_UNLOAD=1 " + qemu_binary + " -r " + oldest_kernel + " -L " + rootfs_path\
+ return "PSEUDO_UNLOAD=1 " + qemu_binary + " " + qemu_option + " -r " + oldest_kernel + " -L " + rootfs_path\
+ " -E LD_LIBRARY_PATH=" + libdir + ":" + base_libdir + " "\
+ rootfs_path + binary
--
1.7.9.5
^ permalink raw reply related [flat|nested] 7+ messages in thread* Re: [PATCH] When building with systemd based image, udev-hwdb postinstall script will fail because of not passing correct CPU option. So fix it by passing correct QEMU_OPTIONS.
2014-11-27 11:17 [PATCH] When building with systemd based image, udev-hwdb postinstall script will fail because of not passing correct CPU option. So fix it by passing correct QEMU_OPTIONS Gaurang Shastri
@ 2014-12-04 19:03 ` Iorga, Cristian
2014-12-04 19:11 ` Iorga, Cristian
1 sibling, 0 replies; 7+ messages in thread
From: Iorga, Cristian @ 2014-12-04 19:03 UTC (permalink / raw)
To: Gaurang Shastri, openembedded-core@lists.openembedded.org
Cc: gmshastri@gmail.com
Hi Gaurang,
Are you sure that the patch solves the described issue?
Because I have tested with:
MACHINE=qemuppc bitbake core-image-sato
and I am still seeing the issue in rootfs log.
Can you please clarify how you tested this?
Regards,
Cristian Iorga
Yocto Project
Intel Corporation
-----Original Message-----
From: openembedded-core-bounces@lists.openembedded.org [mailto:openembedded-core-bounces@lists.openembedded.org] On Behalf Of Gaurang Shastri
Sent: Thursday, November 27, 2014 1:17 PM
To: openembedded-core@lists.openembedded.org
Cc: gmshastri@gmail.com
Subject: [OE-core] [PATCH] When building with systemd based image, udev-hwdb postinstall script will fail because of not passing correct CPU option. So fix it by passing correct QEMU_OPTIONS.
Signed-off-by: Gaurang Shastri <gshastri@juniper.net>
---
meta/classes/qemu.bbclass | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/meta/classes/qemu.bbclass b/meta/classes/qemu.bbclass index b2cf85d..c373855 100644
--- a/meta/classes/qemu.bbclass
+++ b/meta/classes/qemu.bbclass
@@ -30,8 +30,9 @@ def qemu_run_binary(data, rootfs_path, binary):
libdir = rootfs_path + data.getVar("libdir", False)
base_libdir = rootfs_path + data.getVar("base_libdir", False)
oldest_kernel = data.getVar("OLDEST_KERNEL", True)
+ qemu_option = data.getVar("QEMU_OPTIONS",True)
- return "PSEUDO_UNLOAD=1 " + qemu_binary + " -r " + oldest_kernel + " -L " + rootfs_path\
+ return "PSEUDO_UNLOAD=1 " + qemu_binary + " " + qemu_option + " -r
+ " + oldest_kernel + " -L " + rootfs_path\
+ " -E LD_LIBRARY_PATH=" + libdir + ":" + base_libdir + " "\
+ rootfs_path + binary
--
1.7.9.5
--
_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH] When building with systemd based image, udev-hwdb postinstall script will fail because of not passing correct CPU option. So fix it by passing correct QEMU_OPTIONS.
2014-11-27 11:17 [PATCH] When building with systemd based image, udev-hwdb postinstall script will fail because of not passing correct CPU option. So fix it by passing correct QEMU_OPTIONS Gaurang Shastri
2014-12-04 19:03 ` Iorga, Cristian
@ 2014-12-04 19:11 ` Iorga, Cristian
2014-12-04 19:49 ` Richard Purdie
1 sibling, 1 reply; 7+ messages in thread
From: Iorga, Cristian @ 2014-12-04 19:11 UTC (permalink / raw)
To: Gaurang Shastri, openembedded-core@lists.openembedded.org
Cc: gmshastri@gmail.com
Also, issue https://bugzilla.yoctoproject.org/show_bug.cgi?id=6687 (that I am trying to handle) seems quite similar to the issue that you have reported.
Regards,
Cristian Iorga
Yocto Project
Intel Corporation
-----Original Message-----
From: Iorga, Cristian
Sent: Thursday, December 4, 2014 9:04 PM
To: 'Gaurang Shastri'; openembedded-core@lists.openembedded.org
Cc: gmshastri@gmail.com
Subject: RE: [OE-core] [PATCH] When building with systemd based image, udev-hwdb postinstall script will fail because of not passing correct CPU option. So fix it by passing correct QEMU_OPTIONS.
Hi Gaurang,
Are you sure that the patch solves the described issue?
Because I have tested with:
MACHINE=qemuppc bitbake core-image-sato
and I am still seeing the issue in rootfs log.
Can you please clarify how you tested this?
Regards,
Cristian Iorga
Yocto Project
Intel Corporation
-----Original Message-----
From: openembedded-core-bounces@lists.openembedded.org [mailto:openembedded-core-bounces@lists.openembedded.org] On Behalf Of Gaurang Shastri
Sent: Thursday, November 27, 2014 1:17 PM
To: openembedded-core@lists.openembedded.org
Cc: gmshastri@gmail.com
Subject: [OE-core] [PATCH] When building with systemd based image, udev-hwdb postinstall script will fail because of not passing correct CPU option. So fix it by passing correct QEMU_OPTIONS.
Signed-off-by: Gaurang Shastri <gshastri@juniper.net>
---
meta/classes/qemu.bbclass | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/meta/classes/qemu.bbclass b/meta/classes/qemu.bbclass index b2cf85d..c373855 100644
--- a/meta/classes/qemu.bbclass
+++ b/meta/classes/qemu.bbclass
@@ -30,8 +30,9 @@ def qemu_run_binary(data, rootfs_path, binary):
libdir = rootfs_path + data.getVar("libdir", False)
base_libdir = rootfs_path + data.getVar("base_libdir", False)
oldest_kernel = data.getVar("OLDEST_KERNEL", True)
+ qemu_option = data.getVar("QEMU_OPTIONS",True)
- return "PSEUDO_UNLOAD=1 " + qemu_binary + " -r " + oldest_kernel + " -L " + rootfs_path\
+ return "PSEUDO_UNLOAD=1 " + qemu_binary + " " + qemu_option + " -r
+ " + oldest_kernel + " -L " + rootfs_path\
+ " -E LD_LIBRARY_PATH=" + libdir + ":" + base_libdir + " "\
+ rootfs_path + binary
--
1.7.9.5
--
_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH] When building with systemd based image, udev-hwdb postinstall script will fail because of not passing correct CPU option. So fix it by passing correct QEMU_OPTIONS.
2014-12-04 19:11 ` Iorga, Cristian
@ 2014-12-04 19:49 ` Richard Purdie
2014-12-04 21:48 ` Martin Jansa
0 siblings, 1 reply; 7+ messages in thread
From: Richard Purdie @ 2014-12-04 19:49 UTC (permalink / raw)
To: Iorga, Cristian
Cc: openembedded-core@lists.openembedded.org, gmshastri@gmail.com
I have a suspicion the whole QEMU_OPTIONS thing got totally broken by Martin's commit:
http://git.yoctoproject.org/cgit.cgi/poky/commit/?id=7efad8a1b56df6ee07c12ad360c0493d7b1d6d23
since the things being appended are not overrides, they're PACKAGE_ARCH
values. My change below may give us a way out of the current set of
problems.
From 8a8b35a02244cf04b6fb368bf1cb4c1edda8d7a6 Mon Sep 17 00:00:00 2001
From: Richard Purdie <richard.purdie@linuxfoundation.org>
Date: Thu, 4 Dec 2014 19:41:27 +0000
Subject: qemu/libc-package: Fix qemu option handling
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
diff --git a/meta/classes/libc-package.bbclass b/meta/classes/libc-package.bbclass
index c1bc399..eb76ff9 100644
--- a/meta/classes/libc-package.bbclass
+++ b/meta/classes/libc-package.bbclass
@@ -298,9 +298,7 @@ python package_do_split_gconvs () {
--inputfile=%s/i18n/locales/%s --charmap=%s %s" \
% (treedir, datadir, locale, encoding, name)
- qemu_options = d.getVar("QEMU_OPTIONS_%s" % d.getVar('PACKAGE_ARCH', True), True)
- if not qemu_options:
- qemu_options = d.getVar('QEMU_OPTIONS', True)
+ qemu_options = d.getVar('QEMU_OPTIONS', True)
cmd = "PSEUDO_RELOADED=YES PATH=\"%s\" I18NPATH=\"%s\" %s -L %s \
-E LD_LIBRARY_PATH=%s %s %s/bin/localedef %s" % \
diff --git a/meta/classes/qemu.bbclass b/meta/classes/qemu.bbclass
index b2cf85d..1c5736f 100644
--- a/meta/classes/qemu.bbclass
+++ b/meta/classes/qemu.bbclass
@@ -29,20 +29,20 @@ def qemu_run_binary(data, rootfs_path, binary):
libdir = rootfs_path + data.getVar("libdir", False)
base_libdir = rootfs_path + data.getVar("base_libdir", False)
- oldest_kernel = data.getVar("OLDEST_KERNEL", True)
+ qemu_options = data.getVar("QEMU_OPTIONS", True)
- return "PSEUDO_UNLOAD=1 " + qemu_binary + " -r " + oldest_kernel + " -L " + rootfs_path\
+ return "PSEUDO_UNLOAD=1 " + qemu_binary + " " + qemu_options + " -L " + rootfs_path\
+ " -E LD_LIBRARY_PATH=" + libdir + ":" + base_libdir + " "\
+ rootfs_path + binary
-QEMU_OPTIONS = "-r ${OLDEST_KERNEL}"
-QEMU_OPTIONS_append_iwmmxt = " -cpu pxa270-c5"
-QEMU_OPTIONS_append_armv6 = " -cpu arm1136"
-QEMU_OPTIONS_append_armv7a = " -cpu cortex-a8"
-QEMU_OPTIONS_append_e500v2 = " -cpu e500v2"
-QEMU_OPTIONS_append_e500mc = " -cpu e500mc"
-QEMU_OPTIONS_append_e5500 = " -cpu e5500"
-QEMU_OPTIONS_append_e5500-64b = " -cpu e5500"
-QEMU_OPTIONS_append_e6500 = " -cpu e6500"
-QEMU_OPTIONS_append_e6500-64b = " -cpu e6500"
-QEMU_OPTIONS_append_ppc7400 = " -cpu 7400"
+QEMU_OPTIONS = "-r ${OLDEST_KERNEL} ${@d.getVar("QEMU_EXTRAOPTIONS_%s" % d.getVar('PACKAGE_ARCH', True), True) or ""}"
+QEMU_EXTRAOPTIONS_iwmmxt = " -cpu pxa270-c5"
+QEMU_EXTRAOPTIONS_armv6 = " -cpu arm1136"
+QEMU_EXTRAOPTIONS_armv7a = " -cpu cortex-a8"
+QEMU_EXTRAOPTIONS_e500v2 = " -cpu e500v2"
+QEMU_EXTRAOPTIONS_e500mc = " -cpu e500mc"
+QEMU_EXTRAOPTIONS_e5500 = " -cpu e5500"
+QEMU_EXTRAOPTIONS_e5500-64b = " -cpu e5500"
+QEMU_EXTRAOPTIONS_e6500 = " -cpu e6500"
+QEMU_EXTRAOPTIONS_e6500-64b = " -cpu e6500"
+QEMU_EXTRAOPTIONS_ppc7400 = " -cpu 7400"
^ permalink raw reply related [flat|nested] 7+ messages in thread* Re: [PATCH] When building with systemd based image, udev-hwdb postinstall script will fail because of not passing correct CPU option. So fix it by passing correct QEMU_OPTIONS.
2014-12-04 19:49 ` Richard Purdie
@ 2014-12-04 21:48 ` Martin Jansa
2014-12-05 18:03 ` Richard Purdie
0 siblings, 1 reply; 7+ messages in thread
From: Martin Jansa @ 2014-12-04 21:48 UTC (permalink / raw)
To: Richard Purdie
Cc: openembedded-core@lists.openembedded.org, gmshastri@gmail.com
[-- Attachment #1: Type: text/plain, Size: 3808 bytes --]
On Thu, Dec 04, 2014 at 07:49:19PM +0000, Richard Purdie wrote:
> I have a suspicion the whole QEMU_OPTIONS thing got totally broken by Martin's commit:
>
> http://git.yoctoproject.org/cgit.cgi/poky/commit/?id=7efad8a1b56df6ee07c12ad360c0493d7b1d6d23
>
> since the things being appended are not overrides, they're PACKAGE_ARCH
> values. My change below may give us a way out of the current set of
> problems.
>
> From 8a8b35a02244cf04b6fb368bf1cb4c1edda8d7a6 Mon Sep 17 00:00:00 2001
> From: Richard Purdie <richard.purdie@linuxfoundation.org>
> Date: Thu, 4 Dec 2014 19:41:27 +0000
> Subject: qemu/libc-package: Fix qemu option handling
>
> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
>
> diff --git a/meta/classes/libc-package.bbclass b/meta/classes/libc-package.bbclass
> index c1bc399..eb76ff9 100644
> --- a/meta/classes/libc-package.bbclass
> +++ b/meta/classes/libc-package.bbclass
> @@ -298,9 +298,7 @@ python package_do_split_gconvs () {
> --inputfile=%s/i18n/locales/%s --charmap=%s %s" \
> % (treedir, datadir, locale, encoding, name)
>
> - qemu_options = d.getVar("QEMU_OPTIONS_%s" % d.getVar('PACKAGE_ARCH', True), True)
> - if not qemu_options:
> - qemu_options = d.getVar('QEMU_OPTIONS', True)
> + qemu_options = d.getVar('QEMU_OPTIONS', True)
>
> cmd = "PSEUDO_RELOADED=YES PATH=\"%s\" I18NPATH=\"%s\" %s -L %s \
> -E LD_LIBRARY_PATH=%s %s %s/bin/localedef %s" % \
> diff --git a/meta/classes/qemu.bbclass b/meta/classes/qemu.bbclass
> index b2cf85d..1c5736f 100644
> --- a/meta/classes/qemu.bbclass
> +++ b/meta/classes/qemu.bbclass
> @@ -29,20 +29,20 @@ def qemu_run_binary(data, rootfs_path, binary):
>
> libdir = rootfs_path + data.getVar("libdir", False)
> base_libdir = rootfs_path + data.getVar("base_libdir", False)
> - oldest_kernel = data.getVar("OLDEST_KERNEL", True)
> + qemu_options = data.getVar("QEMU_OPTIONS", True)
>
> - return "PSEUDO_UNLOAD=1 " + qemu_binary + " -r " + oldest_kernel + " -L " + rootfs_path\
> + return "PSEUDO_UNLOAD=1 " + qemu_binary + " " + qemu_options + " -L " + rootfs_path\
> + " -E LD_LIBRARY_PATH=" + libdir + ":" + base_libdir + " "\
> + rootfs_path + binary
>
> -QEMU_OPTIONS = "-r ${OLDEST_KERNEL}"
> -QEMU_OPTIONS_append_iwmmxt = " -cpu pxa270-c5"
> -QEMU_OPTIONS_append_armv6 = " -cpu arm1136"
> -QEMU_OPTIONS_append_armv7a = " -cpu cortex-a8"
> -QEMU_OPTIONS_append_e500v2 = " -cpu e500v2"
> -QEMU_OPTIONS_append_e500mc = " -cpu e500mc"
> -QEMU_OPTIONS_append_e5500 = " -cpu e5500"
> -QEMU_OPTIONS_append_e5500-64b = " -cpu e5500"
> -QEMU_OPTIONS_append_e6500 = " -cpu e6500"
> -QEMU_OPTIONS_append_e6500-64b = " -cpu e6500"
> -QEMU_OPTIONS_append_ppc7400 = " -cpu 7400"
> +QEMU_OPTIONS = "-r ${OLDEST_KERNEL} ${@d.getVar("QEMU_EXTRAOPTIONS_%s" % d.getVar('PACKAGE_ARCH', True), True) or ""}"
> +QEMU_EXTRAOPTIONS_iwmmxt = " -cpu pxa270-c5"
> +QEMU_EXTRAOPTIONS_armv6 = " -cpu arm1136"
> +QEMU_EXTRAOPTIONS_armv7a = " -cpu cortex-a8"
> +QEMU_EXTRAOPTIONS_e500v2 = " -cpu e500v2"
> +QEMU_EXTRAOPTIONS_e500mc = " -cpu e500mc"
> +QEMU_EXTRAOPTIONS_e5500 = " -cpu e5500"
> +QEMU_EXTRAOPTIONS_e5500-64b = " -cpu e5500"
> +QEMU_EXTRAOPTIONS_e6500 = " -cpu e6500"
> +QEMU_EXTRAOPTIONS_e6500-64b = " -cpu e6500"
> +QEMU_EXTRAOPTIONS_ppc7400 = " -cpu 7400"
Thanks for fixing that. Would it make sense to add comment saying that
nobody is expected to use "QEMU_EXTRAOPTIONS" directly and that they
aren't overrides (so that nobody makes the same mistake as I did).
--
Martin 'JaMa' Jansa jabber: Martin.Jansa@gmail.com
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 188 bytes --]
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH] When building with systemd based image, udev-hwdb postinstall script will fail because of not passing correct CPU option. So fix it by passing correct QEMU_OPTIONS.
2014-12-04 21:48 ` Martin Jansa
@ 2014-12-05 18:03 ` Richard Purdie
0 siblings, 0 replies; 7+ messages in thread
From: Richard Purdie @ 2014-12-05 18:03 UTC (permalink / raw)
To: Martin Jansa
Cc: openembedded-core@lists.openembedded.org, gmshastri@gmail.com
On Thu, 2014-12-04 at 22:48 +0100, Martin Jansa wrote:
> Thanks for fixing that. Would it make sense to add comment saying that
> nobody is expected to use "QEMU_EXTRAOPTIONS" directly and that they
> aren't overrides (so that nobody makes the same mistake as I did).
Yes, I've done that, and fixed up another reference in the
qemuwrapper-cross so things are more consistent...
Cheers,
Richard
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH] When building with systemd based image, udev-hwdb postinstall script will fail because of not passing correct CPU option. So fix it by passing correct QEMU_OPTIONS.
@ 2014-11-27 11:35 Gaurang Shastri
0 siblings, 0 replies; 7+ messages in thread
From: Gaurang Shastri @ 2014-11-27 11:35 UTC (permalink / raw)
To: openembedded-core; +Cc: gmshastri
Signed-off-by: Gaurang Shastri <gshastri@juniper.net>
---
meta/classes/qemu.bbclass | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/meta/classes/qemu.bbclass b/meta/classes/qemu.bbclass
index b2cf85d..c373855 100644
--- a/meta/classes/qemu.bbclass
+++ b/meta/classes/qemu.bbclass
@@ -30,8 +30,9 @@ def qemu_run_binary(data, rootfs_path, binary):
libdir = rootfs_path + data.getVar("libdir", False)
base_libdir = rootfs_path + data.getVar("base_libdir", False)
oldest_kernel = data.getVar("OLDEST_KERNEL", True)
+ qemu_option = data.getVar("QEMU_OPTIONS",True)
- return "PSEUDO_UNLOAD=1 " + qemu_binary + " -r " + oldest_kernel + " -L " + rootfs_path\
+ return "PSEUDO_UNLOAD=1 " + qemu_binary + " " + qemu_option + " -r " + oldest_kernel + " -L " + rootfs_path\
+ " -E LD_LIBRARY_PATH=" + libdir + ":" + base_libdir + " "\
+ rootfs_path + binary
--
1.7.9.5
^ permalink raw reply related [flat|nested] 7+ messages in thread
end of thread, other threads:[~2014-12-05 18:04 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-11-27 11:17 [PATCH] When building with systemd based image, udev-hwdb postinstall script will fail because of not passing correct CPU option. So fix it by passing correct QEMU_OPTIONS Gaurang Shastri
2014-12-04 19:03 ` Iorga, Cristian
2014-12-04 19:11 ` Iorga, Cristian
2014-12-04 19:49 ` Richard Purdie
2014-12-04 21:48 ` Martin Jansa
2014-12-05 18:03 ` Richard Purdie
-- strict thread matches above, loose matches on Subject: below --
2014-11-27 11:35 Gaurang Shastri
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox