* [PATCH V3 0/2] connman 1.10 upgrade and qemu networking fix
@ 2013-01-11 10:53 Cristian Iorga
2013-01-11 10:53 ` [PATCH V3 1/2] connman: upgrade to 1.10 Cristian Iorga
2013-01-11 10:53 ` [PATCH V3 2/2] connman-conf: configures connman in qemu machines Cristian Iorga
0 siblings, 2 replies; 7+ messages in thread
From: Cristian Iorga @ 2013-01-11 10:53 UTC (permalink / raw)
To: openembedded-core
This set of patches will:
- update connman to 1.10
- solve the issue of lost connectivity with qemu machines
connman will clear previous configuration for a network
interface, rendering qemu machines eth0 unusable, because
there is no DHCP server to be used, and the static configuration
of eth0 will be cleared when connman starts.
connman-conf solves this.
Cristian Iorga (2):
connman: upgrade to 1.10
connman-conf: configures connman in qemu machines
meta/conf/layer.conf | 1 +
meta/conf/machine/include/qemu.inc | 2 +
meta/recipes-connectivity/connman/connman-conf.bb | 18 ++++++
.../connman/connman-conf/qemuall/main.conf | 3 +
...If-there-is-no-d_type-support-use-fstatat.patch | 61 --------------------
...If-there-is-no-d_type-support-use-fstatat.patch | 38 +++++++-----
.../connman/inet-fix-ip-cleanup-functions.patch | 40 +++++++++++++
.../connman/{connman_1.4.bb => connman_1.10.bb} | 8 ++-
8 files changed, 91 insertions(+), 80 deletions(-)
create mode 100644 meta/recipes-connectivity/connman/connman-conf.bb
create mode 100644 meta/recipes-connectivity/connman/connman-conf/qemuall/main.conf
delete mode 100644 meta/recipes-connectivity/connman/connman/0001-timezone.c-If-there-is-no-d_type-support-use-fstatat.patch
create mode 100644 meta/recipes-connectivity/connman/connman/inet-fix-ip-cleanup-functions.patch
rename meta/recipes-connectivity/connman/{connman_1.4.bb => connman_1.10.bb} (71%)
--
1.7.10.4
^ permalink raw reply [flat|nested] 7+ messages in thread* [PATCH V3 1/2] connman: upgrade to 1.10 2013-01-11 10:53 [PATCH V3 0/2] connman 1.10 upgrade and qemu networking fix Cristian Iorga @ 2013-01-11 10:53 ` Cristian Iorga 2013-01-11 10:53 ` [PATCH V3 2/2] connman-conf: configures connman in qemu machines Cristian Iorga 1 sibling, 0 replies; 7+ messages in thread From: Cristian Iorga @ 2013-01-11 10:53 UTC (permalink / raw) To: openembedded-core 0002-storage.c-If-there-is-no-d_type-support-use-fstatat.patch - adapted to the new version 0001-timezone.c-If-there-is-no-d_type-support-use-fstatat.patch - patch removed (it is included in the new version) inet-fix-ip-cleanup-functions.patch: added - fix for ip cleanup functions Signed-off-by: Constantin Musca <constantinx.musca@intel.com> Signed-off-by: Cristian Iorga <cristian.iorga@intel.com> --- ...If-there-is-no-d_type-support-use-fstatat.patch | 61 -------------------- ...If-there-is-no-d_type-support-use-fstatat.patch | 38 +++++++----- .../connman/inet-fix-ip-cleanup-functions.patch | 40 +++++++++++++ .../connman/{connman_1.4.bb => connman_1.10.bb} | 8 ++- 4 files changed, 67 insertions(+), 80 deletions(-) delete mode 100644 meta/recipes-connectivity/connman/connman/0001-timezone.c-If-there-is-no-d_type-support-use-fstatat.patch create mode 100644 meta/recipes-connectivity/connman/connman/inet-fix-ip-cleanup-functions.patch rename meta/recipes-connectivity/connman/{connman_1.4.bb => connman_1.10.bb} (71%) diff --git a/meta/recipes-connectivity/connman/connman/0001-timezone.c-If-there-is-no-d_type-support-use-fstatat.patch b/meta/recipes-connectivity/connman/connman/0001-timezone.c-If-there-is-no-d_type-support-use-fstatat.patch deleted file mode 100644 index 3bced52..0000000 --- a/meta/recipes-connectivity/connman/connman/0001-timezone.c-If-there-is-no-d_type-support-use-fstatat.patch +++ /dev/null @@ -1,61 +0,0 @@ -From f2094e6b2e4542adf458d8fa58d7bccd5edb762e Mon Sep 17 00:00:00 2001 -From: Andrei Gherzan <andrei@gherzan.ro> -Date: Tue, 17 Jul 2012 17:27:39 +0300 -Subject: [PATCH V3 1/2] timezone.c: If there is no d_type support use - fstatat() - -This is useful for filesystems where d_type is always DT_UNKNOWN. -In this case use fstatat() function. - -Signed-off-by: Andrei Gherzan <andrei.gherzan@windriver.com> -Upstream-Status: Submitted - ---- - src/timezone.c | 24 ++++++++++++++++++++++++ - 1 file changed, 24 insertions(+) - -diff --git a/src/timezone.c b/src/timezone.c -index 173d658..f951f6b 100644 ---- a/src/timezone.c -+++ b/src/timezone.c -@@ -157,6 +157,8 @@ static char *find_origin(void *src_map, struct stat *src_st, - DIR *dir; - struct dirent *d; - char *str, pathname[PATH_MAX]; -+ struct stat buf; -+ int ret; - - if (subpath == NULL) - strncpy(pathname, basepath, sizeof(pathname)); -@@ -205,6 +207,28 @@ static char *find_origin(void *src_map, struct stat *src_st, - return str; - } - break; -+ case DT_UNKNOWN: -+ /* -+ * If there is no d_type support use fstatat() -+ * to check if directory -+ */ -+ ret = fstatat(dirfd(dir), d->d_name, &buf, 0); -+ if (ret < 0) -+ continue; -+ if (!(buf.st_mode & S_IFDIR)) -+ continue; -+ if (subpath == NULL) -+ strncpy(pathname, d->d_name, sizeof(pathname)); -+ else -+ snprintf(pathname, sizeof(pathname), -+ "%s/%s", subpath, d->d_name); -+ -+ str = find_origin(src_map, src_st, basepath, pathname); -+ if (str != NULL) { -+ closedir(dir); -+ return str; -+ } -+ break; - } - } - --- -1.7.9.5 - diff --git a/meta/recipes-connectivity/connman/connman/0002-storage.c-If-there-is-no-d_type-support-use-fstatat.patch b/meta/recipes-connectivity/connman/connman/0002-storage.c-If-there-is-no-d_type-support-use-fstatat.patch index d3205c0..7315545 100644 --- a/meta/recipes-connectivity/connman/connman/0002-storage.c-If-there-is-no-d_type-support-use-fstatat.patch +++ b/meta/recipes-connectivity/connman/connman/0002-storage.c-If-there-is-no-d_type-support-use-fstatat.patch @@ -8,20 +8,33 @@ This is useful for filesystems where d_type is always DT_UNKNOWN. In this case use fstatat() function. Signed-off-by: Andrei Gherzan <andrei.gherzan@windriver.com> +Signed-off-by: Constantin Musca <constantinx.musca@intel.com> Upstream-Status: Submitted --- src/storage.c | 19 +++++++++++++++++++ 1 file changed, 19 insertions(+) -diff --git a/src/storage.c b/src/storage.c -index 47bd0cb..0491a52 100644 ---- a/src/storage.c -+++ b/src/storage.c -@@ -206,6 +206,25 @@ gchar **connman_storage_get_services() +Index: git/src/storage.c +=================================================================== +--- git.orig/src/storage.c ++++ git/src/storage.c +@@ -193,7 +193,6 @@ gchar **connman_storage_get_services() - g_string_append_printf(result, "%s/", d->d_name); - break; + switch (d->d_type) { + case DT_DIR: +- case DT_UNKNOWN: + /* + * If the settings file is not found, then + * assume this directory is not a services dir. +@@ -203,6 +202,25 @@ gchar **connman_storage_get_services() + ret = stat(str, &buf); + g_free(str); + if (ret < 0) ++ continue; ++ ++ g_string_append_printf(result, "%s/", d->d_name); ++ break; + case DT_UNKNOWN: + /* + * If there is no d_type support use fstatat() @@ -37,13 +50,6 @@ index 47bd0cb..0491a52 100644 + ret = stat(str, &buf); + g_free(str); + if (ret < 0) -+ continue; -+ -+ g_string_append_printf(result, "%s/", d->d_name); -+ break; - } - } + continue; --- -1.7.9.5 - + g_string_append_printf(result, "%s/", d->d_name); diff --git a/meta/recipes-connectivity/connman/connman/inet-fix-ip-cleanup-functions.patch b/meta/recipes-connectivity/connman/connman/inet-fix-ip-cleanup-functions.patch new file mode 100644 index 0000000..3071549 --- /dev/null +++ b/meta/recipes-connectivity/connman/connman/inet-fix-ip-cleanup-functions.patch @@ -0,0 +1,40 @@ +From 100353e10f60a50ca1ba78daa6bc4dfebf5b3297 Mon Sep 17 00:00:00 2001 +From: Constantin Musca <constantinx.musca@intel.com> +Date: Wed, 5 Dec 2012 15:07:21 +0200 +Subject: [PATCH] inet: fix ip cleanup functions + +Upstream-Status: Pending +Signed-off-by: Constantin Musca <constantinx.musca@intel.com> +--- + src/inet.c | 8 ++++++++ + 1 file changed, 8 insertions(+) + +diff --git a/src/inet.c b/src/inet.c +index be69aca..e76a1f1 100644 +--- a/src/inet.c ++++ b/src/inet.c +@@ -626,6 +626,10 @@ int connman_inet_clear_ipv6_address(int index, const char *address, + + DBG("index %d address %s prefix_len %d", index, address, prefix_len); + ++ if (address == NULL) { ++ return -1; ++ } ++ + err = __connman_inet_modify_address(RTM_DELADDR, 0, index, AF_INET6, + address, NULL, prefix_len, NULL); + if (err < 0) { +@@ -649,6 +653,10 @@ int connman_inet_clear_address(int index, struct connman_ipaddress *ipaddress) + + DBG("index %d address %s prefix_len %d", index, address, prefix_len); + ++ if (address == NULL) { ++ return -1; ++ } ++ + err = __connman_inet_modify_address(RTM_DELADDR, 0, index, AF_INET, + address, peer, prefix_len, broadcast); + if (err < 0) { +-- +1.7.11.7 + diff --git a/meta/recipes-connectivity/connman/connman_1.4.bb b/meta/recipes-connectivity/connman/connman_1.10.bb similarity index 71% rename from meta/recipes-connectivity/connman/connman_1.4.bb rename to meta/recipes-connectivity/connman/connman_1.10.bb index bd8ba65..10d5255 100644 --- a/meta/recipes-connectivity/connman/connman_1.4.bb +++ b/meta/recipes-connectivity/connman/connman_1.10.bb @@ -1,12 +1,14 @@ require connman.inc -# 1.4 tag -SRCREV = "f701bbca259f1f35e68d338f31f5373f75f3da5f" +# 1.10 tag +SRCREV = "3bda76eb442c9395db1dba17352c3e78376671cc" SRC_URI = "git://git.kernel.org/pub/scm/network/connman/connman.git \ file://0001-plugin.h-Change-visibility-to-default-for-debug-symb.patch \ file://add_xuser_dbus_permission.patch \ file://connman \ file://0002-storage.c-If-there-is-no-d_type-support-use-fstatat.patch \ - file://0001-timezone.c-If-there-is-no-d_type-support-use-fstatat.patch" + file://inet-fix-ip-cleanup-functions.patch" S = "${WORKDIR}/git" PR = "${INC_PR}.0" + +RRECOMMENDS_${PN} = "connman-conf" -- 1.7.10.4 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH V3 2/2] connman-conf: configures connman in qemu machines 2013-01-11 10:53 [PATCH V3 0/2] connman 1.10 upgrade and qemu networking fix Cristian Iorga 2013-01-11 10:53 ` [PATCH V3 1/2] connman: upgrade to 1.10 Cristian Iorga @ 2013-01-11 10:53 ` Cristian Iorga 2013-01-11 11:27 ` Martin Jansa 2013-01-11 12:22 ` Enrico Scholz 1 sibling, 2 replies; 7+ messages in thread From: Cristian Iorga @ 2013-01-11 10:53 UTC (permalink / raw) To: openembedded-core For qemu machines, connman should not configure virtual wired interfaces. Wired interfaces (eth0) are assigned static IP addresses by the virtual machine manager (qemu). This packages places the eth0 interface in a list of blacklisted interfaces, forbiding connman administering wired interfaces for qemu machines. Fixes [YOCTO #3227]. Signed-off-by: Cristian Iorga <cristian.iorga@intel.com> --- meta/conf/layer.conf | 1 + meta/conf/machine/include/qemu.inc | 2 ++ meta/recipes-connectivity/connman/connman-conf.bb | 18 ++++++++++++++++++ .../connman/connman-conf/qemuall/main.conf | 3 +++ 4 files changed, 24 insertions(+) create mode 100644 meta/recipes-connectivity/connman/connman-conf.bb create mode 100644 meta/recipes-connectivity/connman/connman-conf/qemuall/main.conf diff --git a/meta/conf/layer.conf b/meta/conf/layer.conf index 78ceae9..3259e5c 100644 --- a/meta/conf/layer.conf +++ b/meta/conf/layer.conf @@ -22,6 +22,7 @@ SIGGEN_EXCLUDERECIPES_ABISAFE += " \ shadow-securetty \ opkg-config-base \ netbase \ + connman-conf \ formfactor \ xserver-xf86-config \ pointercal \ diff --git a/meta/conf/machine/include/qemu.inc b/meta/conf/machine/include/qemu.inc index 5d59a7f..c1012ab 100644 --- a/meta/conf/machine/include/qemu.inc +++ b/meta/conf/machine/include/qemu.inc @@ -10,6 +10,8 @@ XSERVER ?= "xserver-xorg \ MACHINE_FEATURES = "apm alsa pcmcia bluetooth irda usbgadget screen" +MACHINEOVERRIDES .= ":qemuall" + IMAGE_FSTYPES += "tar.bz2 ext3" ROOT_FLASH_SIZE = "280" diff --git a/meta/recipes-connectivity/connman/connman-conf.bb b/meta/recipes-connectivity/connman/connman-conf.bb new file mode 100644 index 0000000..7c7ef26 --- /dev/null +++ b/meta/recipes-connectivity/connman/connman-conf.bb @@ -0,0 +1,18 @@ +#connman config to ignore wired interfaces on qemu machines + +LICENSE = "GPLv2" +LIC_FILES_CHKSUM = "file://${COREBASE}/meta/files/common-licenses/GPL-2.0;md5=801f80980d171dd6425610833a22dbe6" + +SRC_URI_append_qemuall = " file://main.conf" + +PR = "r0" + +PACKAGE_ARCH = "${MACHINE_ARCH}" + +do_install() { + #Blacklist ethn network interface in case of qemu* machines + if test -e ${WORKDIR}/main.conf; then + install -d ${D}${sysconfdir}/connman + install -m 0644 ${WORKDIR}/main.conf ${D}${sysconfdir}/connman + fi +} diff --git a/meta/recipes-connectivity/connman/connman-conf/qemuall/main.conf b/meta/recipes-connectivity/connman/connman-conf/qemuall/main.conf new file mode 100644 index 0000000..01973e7 --- /dev/null +++ b/meta/recipes-connectivity/connman/connman-conf/qemuall/main.conf @@ -0,0 +1,3 @@ +[General] + +NetworkInterfaceBlacklist = eth -- 1.7.10.4 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH V3 2/2] connman-conf: configures connman in qemu machines 2013-01-11 10:53 ` [PATCH V3 2/2] connman-conf: configures connman in qemu machines Cristian Iorga @ 2013-01-11 11:27 ` Martin Jansa 2013-01-11 12:01 ` Iorga, Cristian 2013-01-11 12:22 ` Enrico Scholz 1 sibling, 1 reply; 7+ messages in thread From: Martin Jansa @ 2013-01-11 11:27 UTC (permalink / raw) To: Cristian Iorga; +Cc: openembedded-core [-- Attachment #1: Type: text/plain, Size: 3392 bytes --] On Fri, Jan 11, 2013 at 12:53:38PM +0200, Cristian Iorga wrote: > For qemu machines, connman should not configure > virtual wired interfaces. Wired interfaces (eth0) > are assigned static IP addresses by the virtual machine > manager (qemu). This packages places the eth0 interface in > a list of blacklisted interfaces, forbiding connman > administering wired interfaces for qemu machines. This should be applied before 1/2 which is RRECOMMENDING this. Cheers, > Fixes [YOCTO #3227]. > > Signed-off-by: Cristian Iorga <cristian.iorga@intel.com> > --- > meta/conf/layer.conf | 1 + > meta/conf/machine/include/qemu.inc | 2 ++ > meta/recipes-connectivity/connman/connman-conf.bb | 18 ++++++++++++++++++ > .../connman/connman-conf/qemuall/main.conf | 3 +++ > 4 files changed, 24 insertions(+) > create mode 100644 meta/recipes-connectivity/connman/connman-conf.bb > create mode 100644 meta/recipes-connectivity/connman/connman-conf/qemuall/main.conf > > diff --git a/meta/conf/layer.conf b/meta/conf/layer.conf > index 78ceae9..3259e5c 100644 > --- a/meta/conf/layer.conf > +++ b/meta/conf/layer.conf > @@ -22,6 +22,7 @@ SIGGEN_EXCLUDERECIPES_ABISAFE += " \ > shadow-securetty \ > opkg-config-base \ > netbase \ > + connman-conf \ > formfactor \ > xserver-xf86-config \ > pointercal \ > diff --git a/meta/conf/machine/include/qemu.inc b/meta/conf/machine/include/qemu.inc > index 5d59a7f..c1012ab 100644 > --- a/meta/conf/machine/include/qemu.inc > +++ b/meta/conf/machine/include/qemu.inc > @@ -10,6 +10,8 @@ XSERVER ?= "xserver-xorg \ > > MACHINE_FEATURES = "apm alsa pcmcia bluetooth irda usbgadget screen" > > +MACHINEOVERRIDES .= ":qemuall" > + > IMAGE_FSTYPES += "tar.bz2 ext3" > > ROOT_FLASH_SIZE = "280" > diff --git a/meta/recipes-connectivity/connman/connman-conf.bb b/meta/recipes-connectivity/connman/connman-conf.bb > new file mode 100644 > index 0000000..7c7ef26 > --- /dev/null > +++ b/meta/recipes-connectivity/connman/connman-conf.bb > @@ -0,0 +1,18 @@ > +#connman config to ignore wired interfaces on qemu machines > + > +LICENSE = "GPLv2" > +LIC_FILES_CHKSUM = "file://${COREBASE}/meta/files/common-licenses/GPL-2.0;md5=801f80980d171dd6425610833a22dbe6" > + > +SRC_URI_append_qemuall = " file://main.conf" > + > +PR = "r0" > + > +PACKAGE_ARCH = "${MACHINE_ARCH}" > + > +do_install() { > + #Blacklist ethn network interface in case of qemu* machines > + if test -e ${WORKDIR}/main.conf; then > + install -d ${D}${sysconfdir}/connman > + install -m 0644 ${WORKDIR}/main.conf ${D}${sysconfdir}/connman > + fi > +} > diff --git a/meta/recipes-connectivity/connman/connman-conf/qemuall/main.conf b/meta/recipes-connectivity/connman/connman-conf/qemuall/main.conf > new file mode 100644 > index 0000000..01973e7 > --- /dev/null > +++ b/meta/recipes-connectivity/connman/connman-conf/qemuall/main.conf > @@ -0,0 +1,3 @@ > +[General] > + > +NetworkInterfaceBlacklist = eth > -- > 1.7.10.4 > > > _______________________________________________ > Openembedded-core mailing list > Openembedded-core@lists.openembedded.org > http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core -- Martin 'JaMa' Jansa jabber: Martin.Jansa@gmail.com [-- Attachment #2: Digital signature --] [-- Type: application/pgp-signature, Size: 205 bytes --] ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH V3 2/2] connman-conf: configures connman in qemu machines 2013-01-11 11:27 ` Martin Jansa @ 2013-01-11 12:01 ` Iorga, Cristian 0 siblings, 0 replies; 7+ messages in thread From: Iorga, Cristian @ 2013-01-11 12:01 UTC (permalink / raw) To: Martin Jansa; +Cc: openembedded-core@lists.openembedded.org Yes, I think I missed that. Should I resend the patches? -----Original Message----- From: Martin Jansa [mailto:martin.jansa@gmail.com] Sent: Friday, January 11, 2013 1:28 PM To: Iorga, Cristian Cc: openembedded-core@lists.openembedded.org Subject: Re: [OE-core] [PATCH V3 2/2] connman-conf: configures connman in qemu machines On Fri, Jan 11, 2013 at 12:53:38PM +0200, Cristian Iorga wrote: > For qemu machines, connman should not configure virtual wired > interfaces. Wired interfaces (eth0) are assigned static IP addresses > by the virtual machine manager (qemu). This packages places the eth0 > interface in a list of blacklisted interfaces, forbiding connman > administering wired interfaces for qemu machines. This should be applied before 1/2 which is RRECOMMENDING this. Cheers, > Fixes [YOCTO #3227]. > > Signed-off-by: Cristian Iorga <cristian.iorga@intel.com> > --- > meta/conf/layer.conf | 1 + > meta/conf/machine/include/qemu.inc | 2 ++ > meta/recipes-connectivity/connman/connman-conf.bb | 18 ++++++++++++++++++ > .../connman/connman-conf/qemuall/main.conf | 3 +++ > 4 files changed, 24 insertions(+) > create mode 100644 meta/recipes-connectivity/connman/connman-conf.bb > create mode 100644 > meta/recipes-connectivity/connman/connman-conf/qemuall/main.conf > > diff --git a/meta/conf/layer.conf b/meta/conf/layer.conf index > 78ceae9..3259e5c 100644 > --- a/meta/conf/layer.conf > +++ b/meta/conf/layer.conf > @@ -22,6 +22,7 @@ SIGGEN_EXCLUDERECIPES_ABISAFE += " \ > shadow-securetty \ > opkg-config-base \ > netbase \ > + connman-conf \ > formfactor \ > xserver-xf86-config \ > pointercal \ > diff --git a/meta/conf/machine/include/qemu.inc > b/meta/conf/machine/include/qemu.inc > index 5d59a7f..c1012ab 100644 > --- a/meta/conf/machine/include/qemu.inc > +++ b/meta/conf/machine/include/qemu.inc > @@ -10,6 +10,8 @@ XSERVER ?= "xserver-xorg \ > > MACHINE_FEATURES = "apm alsa pcmcia bluetooth irda usbgadget screen" > > +MACHINEOVERRIDES .= ":qemuall" > + > IMAGE_FSTYPES += "tar.bz2 ext3" > > ROOT_FLASH_SIZE = "280" > diff --git a/meta/recipes-connectivity/connman/connman-conf.bb > b/meta/recipes-connectivity/connman/connman-conf.bb > new file mode 100644 > index 0000000..7c7ef26 > --- /dev/null > +++ b/meta/recipes-connectivity/connman/connman-conf.bb > @@ -0,0 +1,18 @@ > +#connman config to ignore wired interfaces on qemu machines > + > +LICENSE = "GPLv2" > +LIC_FILES_CHKSUM = "file://${COREBASE}/meta/files/common-licenses/GPL-2.0;md5=801f80980d171dd6425610833a22dbe6" > + > +SRC_URI_append_qemuall = " file://main.conf" > + > +PR = "r0" > + > +PACKAGE_ARCH = "${MACHINE_ARCH}" > + > +do_install() { > + #Blacklist ethn network interface in case of qemu* machines > + if test -e ${WORKDIR}/main.conf; then > + install -d ${D}${sysconfdir}/connman > + install -m 0644 ${WORKDIR}/main.conf ${D}${sysconfdir}/connman > + fi > +} > diff --git > a/meta/recipes-connectivity/connman/connman-conf/qemuall/main.conf > b/meta/recipes-connectivity/connman/connman-conf/qemuall/main.conf > new file mode 100644 > index 0000000..01973e7 > --- /dev/null > +++ b/meta/recipes-connectivity/connman/connman-conf/qemuall/main.conf > @@ -0,0 +1,3 @@ > +[General] > + > +NetworkInterfaceBlacklist = eth > -- > 1.7.10.4 > > > _______________________________________________ > Openembedded-core mailing list > Openembedded-core@lists.openembedded.org > http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core -- Martin 'JaMa' Jansa jabber: Martin.Jansa@gmail.com ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH V3 2/2] connman-conf: configures connman in qemu machines 2013-01-11 10:53 ` [PATCH V3 2/2] connman-conf: configures connman in qemu machines Cristian Iorga 2013-01-11 11:27 ` Martin Jansa @ 2013-01-11 12:22 ` Enrico Scholz 2013-01-11 13:00 ` Iorga, Cristian 1 sibling, 1 reply; 7+ messages in thread From: Enrico Scholz @ 2013-01-11 12:22 UTC (permalink / raw) To: openembedded-core; +Cc: Cristian Iorga Cristian Iorga <cristian.iorga-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org> writes: > For qemu machines, connman should not configure virtual wired interfaces. This is not a qemu-only problem but affects all platforms with nfs rootfs, isn't it? Setup whether to disable configuration of interface should be done dynamically and based on e.g. whether rootfs is on nfs and/or if kernel cmdline contains 'ip=...'. > Wired interfaces (eth0) are assigned static IP addresses by the virtual > machine manager (qemu). This packages places the eth0 interface in a > list of blacklisted interfaces, forbiding connman administering wired > interfaces for qemu machines. Unfortunatly, this breaks DNS in the machine because DNS server information are not retrieved from DHCP anymore :( Enrico ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH V3 2/2] connman-conf: configures connman in qemu machines 2013-01-11 12:22 ` Enrico Scholz @ 2013-01-11 13:00 ` Iorga, Cristian 0 siblings, 0 replies; 7+ messages in thread From: Iorga, Cristian @ 2013-01-11 13:00 UTC (permalink / raw) To: Enrico Scholz, openembedded-core@lists.openembedded.org; +Cc: Cristian Iorga Hello Enrico, see my answers below. Regards, Cristian -----Original Message----- From: Enrico Scholz [mailto:enrico.scholz@sigma-chemnitz.de] Sent: Friday, January 11, 2013 2:23 PM To: openembedded-core@lists.openembedded.org Cc: Cristian Iorga Subject: Re: [PATCH V3 2/2] connman-conf: configures connman in qemu machines Cristian Iorga <cristian.iorga-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org> writes: > For qemu machines, connman should not configure virtual wired interfaces. This is not a qemu-only problem but affects all platforms with nfs rootfs, isn't it? In this particular case, it's only related to qemu in the network environment presented to a virtual machine. There is a separate patch for the $nfsroot issue. Setup whether to disable configuration of interface should be done dynamically and based on e.g. whether rootfs is on nfs and/or if kernel cmdline contains 'ip=...'. In my opinion, that's debatable. Qemu images for sure won't have a DHCP to send request too (as the setup is done now); as such, the build system can decide in advance to blacklist ethn to connman. To me, the $nfsroot fix is OK. It is done dynamically done, and on need by need base. > Wired interfaces (eth0) are assigned static IP addresses by the > virtual machine manager (qemu). This packages places the eth0 > interface in a list of blacklisted interfaces, forbiding connman > administering wired interfaces for qemu machines. Unfortunatly, this breaks DNS in the machine because DNS server information are not retrieved from DHCP anymore :( In which machine? qemu*? Is there a DHCP server to work with? That's the main problem that I am seeing, that there is no DHCP server, and, in the end, connman will assign a Zeroconf IP address to eth0. As far as I know, there will be no DNS server set if Zeroconf is used. Enrico ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2013-01-11 13:16 UTC | newest] Thread overview: 7+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2013-01-11 10:53 [PATCH V3 0/2] connman 1.10 upgrade and qemu networking fix Cristian Iorga 2013-01-11 10:53 ` [PATCH V3 1/2] connman: upgrade to 1.10 Cristian Iorga 2013-01-11 10:53 ` [PATCH V3 2/2] connman-conf: configures connman in qemu machines Cristian Iorga 2013-01-11 11:27 ` Martin Jansa 2013-01-11 12:01 ` Iorga, Cristian 2013-01-11 12:22 ` Enrico Scholz 2013-01-11 13:00 ` Iorga, Cristian
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.