Bug 14930 - connman - sysvinit - nfs/dhcp hangs in langdale/master
Summary: connman - sysvinit - nfs/dhcp hangs in langdale/master
Status: NEW
Alias: None
Product: OE-Core
Classification: Build System, Metadata & Runtime
Component: core (show other bugs)
Version: 4.1
Hardware: x86 Multiple
: Medium+ normal
Target Milestone: 6.99
Assignee: Randy MacLeod
QA Contact:
URL:
Whiteboard: backports
Depends on:
Blocks:
 
Reported: 2022-10-13 15:11 UTC by Robert Berger
Modified: 2026-05-07 15:19 UTC (History)
5 users (show)

See Also:
OS type for building Yocto: ---
Type of Regression: ---
Verified:
Documentation change: Don't know


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Robert Berger 2022-10-13 15:11:32 UTC
It looks like in langdale/master do_start() was stripped of essential functionality [1] 

[1] https://git.yoctoproject.org/poky/tree/meta/recipes-connectivity/connman/connman/connman?h=langdale

In kirkstone this functionality is still there[2]

[2] https://git.yoctoproject.org/poky/tree/meta/recipes-connectivity/connman/connman/connman?h=kirkstone

-->

nfsroot=0

exec 9<&0 < /proc/mounts
while read dev mtpt fstype rest; do
	if test $mtpt = "/" ; then
		case $fstype in
		    nfs | nfs4)
			nfsroot=1
			break
			;;
		    *)
			;;
		esac
	fi
done

do_start() {
	if test $nfsroot -eq 1 ; then
	    NET_DEVS=`cat /proc/net/dev | sed -ne 's/^\([a-zA-Z0-9 ]*\):.*$/\1/p'`
	    NET_ADDR=`cat /proc/cmdline | sed -ne 's/^.*ip=\([^ :]*\).*$/\1/p'`

	    if [ ! -z "$NET_ADDR" ]; then
		if [ "$NET_ADDR" = dhcp ]; then
		    ethn=`ifconfig | grep "^eth" | sed -e "s/\(eth[0-9]\)\(.*\)/\1/"`
		    if [ ! -z "$ethn" ]; then
			EXTRA_PARAM="$EXTRA_PARAM -I $ethn"
		    fi
		else
		    for i in $NET_DEVS; do
			ADDR=`ifconfig $i | sed 's/addr://g' | sed -ne 's/^.*inet \([0-9.]*\) .*$/\1/p'`
			if [ "$NET_ADDR" = "$ADDR" ]; then
			    EXTRA_PARAM="$EXTRA_PARAM -I $i"
			    break
			fi
		    done
		fi
	    fi
	fi
	if [ -f @DATADIR@/connman/wired-setup ] ; then
		. @DATADIR@/connman/wired-setup
	fi
	$DAEMON $EXTRA_PARAM
}

<--

Without this extra checks connman needs to be disabled if you boot with a rootfs over nfs, so I suggest to add it back in.
Comment 1 Randy MacLeod 2022-12-22 16:41:56 UTC
Zheng's co-op term is over so moving to unassigned.
Comment 2 Randy MacLeod 2023-07-26 21:22:04 UTC
Bulk move of 64 bugs to 4.3 M3 after a quick review.
If a bug is actually fixed, please add a commit link and resolve it.
-- Randy for YP bug team.
Comment 3 Trevor Gamblin 2023-09-14 16:52:33 UTC
Moved to M4.
Comment 4 Adrian 2023-09-22 10:12:06 UTC
Would it be worth a thought to replace conman with NetworkManager? Basically move conman to meta-oe and NetworkManager to poky?

I haven't compared the two in their current versions, but there is probably a lot more activity on the NetworkManager side than the conman side.
Comment 5 Randy MacLeod 2023-10-30 15:29:14 UTC
Bulk move to 5.0 M1. -- Randy
Comment 6 Randy MacLeod 2023-12-21 23:28:27 UTC
Bulk move of 52 5.0-M1 bugs to M2.
Comment 7 Randy MacLeod 2024-01-25 14:28:52 UTC
Bulk move of bugs owned by "unassigned" from 5.0-M2 to 5.0-M3.
Comment 8 Randy MacLeod 2024-02-22 19:53:24 UTC
Bulk move of bugs from 5.0-M3 to 5.99.
The bugs in this list are all unassigned and are ones that I consider to be nice to do some time but not essential track from milestone to milestone in 5.0 or 5.1. We agreed on this bug handling approach in the YP bug review on Feb 22, 2024.

If you disagree, please move back to a 5.[012...]-M* milestone.
Comment 9 Randy MacLeod 2026-05-07 15:19:39 UTC
Is the reproducible on current oe-core/master ?