* [meta-cgl][PATCH 1/2] Added device manager multipath support for root file system
@ 2016-05-10 10:05 Adrian Dudau
2016-05-10 10:05 ` [meta-cgl][PATCH 2/2] distro: Move IMAGE_CLASSES inheritance Adrian Dudau
2016-05-11 15:15 ` [meta-cgl][PATCH 1/2] Added device manager multipath support for root file system Alexandru Vaduva
0 siblings, 2 replies; 5+ messages in thread
From: Adrian Dudau @ 2016-05-10 10:05 UTC (permalink / raw)
To: yocto
From: Mats Liljegren <mats.liljegren@enea.com>
This is implemented using an initramfs, built by core-image-cgl-initramfs image
recipe. Multipath device configurations are done using kernel boot parameters.
The multipath-root-howto.md explains how to use this.
Also note that the meta-cgl/scripts/conf_setup.sh will now also add a BBMASK
entry to mask out the meta-virtualization's version of multipath-tools, since
we need the newer recipe in the meta-openembedded/meta-oe instead.
Signed-off-by: Mats Liljegren <mats.liljegren@enea.com>
Signed-off-by: Adrian Dudau <adrian.dudau@enea.com>
---
meta-cgl-common/images/core-image-cgl-initramfs.bb | 22 +++
meta-cgl-common/images/core-image-cgl.bb | 2 +
.../packagegroups/packagegroup-cgl-middleware.bb | 8 +-
.../recipes-core/initrdscripts/files/init-boot.sh | 179 +++++++++++++++++++++
.../initrdscripts/initramfs-cgl-boot_1.0.bb | 14 ++
scripts/conf_setup.sh | 2 +
6 files changed, 220 insertions(+), 7 deletions(-)
create mode 100644 meta-cgl-common/images/core-image-cgl-initramfs.bb
create mode 100644 meta-cgl-common/recipes-core/initrdscripts/files/init-boot.sh
create mode 100644 meta-cgl-common/recipes-core/initrdscripts/initramfs-cgl-boot_1.0.bb
create mode 100755 scripts/conf_setup.sh
diff --git a/meta-cgl-common/images/core-image-cgl-initramfs.bb b/meta-cgl-common/images/core-image-cgl-initramfs.bb
new file mode 100644
index 0000000..845fa07
--- /dev/null
+++ b/meta-cgl-common/images/core-image-cgl-initramfs.bb
@@ -0,0 +1,22 @@
+require core-image-cgl.bb
+
+# Recipe is based on core-image-minimal.bb
+DESCRIPTION = "Initramfs used to mount multipath device as root file system"
+
+PACKAGE_INSTALL = "initramfs-cgl-boot busybox base-passwd udev ${ROOTFS_BOOTSTRAP_INSTALL}"
+
+# Do not pollute the initrd image with rootfs features
+IMAGE_FEATURES = ""
+
+export IMAGE_BASENAME = "core-image-cgl-initramfs"
+IMAGE_LINGUAS = ""
+
+LICENSE = "MIT"
+
+IMAGE_FSTYPES = "cpio.gz.u-boot"
+IMAGE_CLASSES += "image_types_uboot"
+inherit core-image
+
+IMAGE_ROOTFS_SIZE = "8192"
+
+BAD_RECOMMENDATIONS += "busybox-syslog"
diff --git a/meta-cgl-common/images/core-image-cgl.bb b/meta-cgl-common/images/core-image-cgl.bb
index d12391b..5975601 100644
--- a/meta-cgl-common/images/core-image-cgl.bb
+++ b/meta-cgl-common/images/core-image-cgl.bb
@@ -21,6 +21,8 @@ IMAGE_INSTALL += "\
kernel-modules \
"
+IMAGE_FSTYPES += " ext3.gz"
+
# kexec-tools doesn't work on Mips
KEXECTOOLS_mips ?= ""
KEXECTOOLS_mipsel ?= ""
diff --git a/meta-cgl-common/packagegroups/packagegroup-cgl-middleware.bb b/meta-cgl-common/packagegroups/packagegroup-cgl-middleware.bb
index 448e038..b0c2cac 100644
--- a/meta-cgl-common/packagegroups/packagegroup-cgl-middleware.bb
+++ b/meta-cgl-common/packagegroups/packagegroup-cgl-middleware.bb
@@ -18,14 +18,8 @@ DHCP = " \
"
MULTIPATH_TOOLS = " \
- libmpathpersist \
- mpathpersist \
+ multipath-tools \
kpartx \
- libmultipath \
- multipath \
- multipathd \
- libmultipath-dev \
- libmpathpersist-dev \
"
RDEPENDS_packagegroup-cgl-middleware = "\
diff --git a/meta-cgl-common/recipes-core/initrdscripts/files/init-boot.sh b/meta-cgl-common/recipes-core/initrdscripts/files/init-boot.sh
new file mode 100644
index 0000000..fcadfc6
--- /dev/null
+++ b/meta-cgl-common/recipes-core/initrdscripts/files/init-boot.sh
@@ -0,0 +1,179 @@
+#!/bin/sh -eu
+
+# Fail function, either called explicitly or when shell will quit
+fail () {
+ # Avoid recursive traps
+ trap '' ERR EXIT
+
+ # If message provided, print it
+ [ -n '${1:-}' ] && echo $@
+
+ # Generic error message and shell access
+ echo "Error occured, giving a shell"
+ exec sh
+}
+
+trap fail ERR EXIT
+
+interrupt () {
+ echo "User interrupt received, giving a shell. When exiting shell, execution will continue."
+ sh
+}
+
+trap interrupt INT
+
+# Find session ID for an iSCSI disk given its IQN name
+iqn_to_sid () {
+ iscsiadm -m session | fgrep $1 | sed -r 's/.*\[([0-9])\].*/\1/'
+}
+
+# Find device name, without path, for an iSCSI disk given its IQN name
+iqn_to_dev () {
+ iscsiadm -m session -r $(iqn_to_sid $1) -P3 | sed -rn 's/.*Attached scsi disk ([a-zA-Z0-9_]+).*/\1/p'
+}
+
+# Parse input parameters expecting name=value pairs.
+# Name only matches known parameters.
+# On match, set variable <name> to value <value>.
+# E.g. given "parse_cmdline trythis="ok, do it"
+# the shell variable "trythis" will now have the value "ok, do it".
+parse_cmdline () {
+ iscsi_chap_user=""
+ iscsi_chap_pwd=""
+ iscsi_dev=""
+ iscsi_debug=0
+
+ while [ -n "${1:-}" ]; do
+ name="${1%%=*}"
+ val="${1#*=}"
+ case $name in
+ iscsi_chap_user|iscsi_chap_pw)
+ eval $name=\"$val\";;
+ iscsi_dev)
+ eval $name=\"$iscsi_dev $val\"
+ ;;
+ iscsi_debug)
+ set -x
+ iscsi_debug=1
+ ;;
+ esac
+ shift
+ done
+
+ [ -n "${iscsi_dev}" ] || fail "Mandatory kernel boot parameter 'iscsi_dev' not given."
+
+}
+
+PATH=/sbin:/bin:/usr/sbin:/usr/bin
+
+echo "Mounting /proc"
+mount -t proc proc /proc
+
+KERNEL_CMDLINE="$(cat /proc/cmdline)"
+
+echo "Parsing kernel parameters"
+parse_cmdline $KERNEL_CMDLINE
+
+echo "Mounting /sys"
+mount -t sysfs sysfs /sys
+
+# udev is needed for multipath
+echo "Starting udev"
+/etc/init.d/udev start
+
+# Add CHAP autenthication, if given as kernel boot parameters
+echo "Configuring iSCSI"
+[ -n "$iscsi_chap_user" -o -n "$iscsi_chap_pw" ] && cat <<EOF >> /etc/iscsi/iscsid.conf
+node.session.auth.authmethod = CHAP
+EOF
+[ -n "$iscsi_chap_user" ] && cat <<EOF >> /etc/iscsi/iscsid.conf
+node.session.auth.username = $iscsi_chap_user
+EOF
+[ -n "$iscsi_chap_pw" ] && cat <<EOF >> /etc/iscsi/iscsid.conf
+node.session.auth.password = $iscsi_chap_pw
+EOF
+
+echo >> /etc/iscsi/iscsid.conf
+
+echo "Starting iSCSI daemon"
+/etc/init.d/iscsid restart
+
+echo "Discovering iSCSI devices"
+
+for dev in $iscsi_dev; do
+ target_ip="${dev%%:*}"
+ dev_name="${dev#*:}"
+ echo "Logging in to iscsi devices: $dev_name, target: $target_ip"
+ iscsiadm --mode discoverydb --type sendtargets --discover --portal $target_ip
+ iscsiadm --mode node --targetname $dev_name --login --portal $target_ip
+done
+
+echo "Configuring multipath"
+cat <<EOF > /etc/multipath.conf
+defaults {
+ path_grouping_policy multibus
+ # If no path, then queue requests
+ no_path_retry queue
+}
+devices {
+ device {
+ vendor IET
+ product VIRTUAL-DISK
+ path_grouping_policy multibus
+ }
+}
+blacklist {
+ devnode ".*"
+}
+blacklist_exceptions {
+$(for dev in $iscsi_dev; do
+ echo " devnode \"^$(iqn_to_dev ${dev#*:})\""
+ done)
+ property ".*"
+}
+EOF
+
+echo "Starting multipath daemon"
+# Make sure lock file directory exists
+mkdir -p /var/lock/subsys/multipathd
+/etc/init.d/multipathd start
+
+mpath_template='/dev/disk/by-id/dm-uuid-mpath-*'
+
+echo "Waiting for mpath device to appear"
+while [ -z "$(ls $mpath_template 2>/dev/null)" ]; do
+ sleep 1
+done
+MPATH_DEV="$(ls $mpath_template)"
+echo "mpath device: $MPATH_DEV"
+
+echo "Mounting mpath device $MPATH_DEV"
+mount $MPATH_DEV /mnt
+
+echo "Stopping multipath daemon"
+# Should be using "/etc/init.d/multipathd stop", but did not work.
+# Got "killall: /sbin/multipathd: no process killed".
+# Kill the process based on the saved pid.
+kill $(cat /run/multipathd.pid)
+
+echo "Moving iscsi pid and lock files"
+cp /run/iscsid.pid /mnt/run/iscsid.pid
+cp /run/lock/iscsi/* /mnt/run/lock/iscsi
+
+# In case iscsi_debug is given, open a shell at this point
+if [ $iscsi_debug -eq 1 ]; then
+ echo "iscsi_debug given, opening a shell. When exiting shell, boot will continue."
+ sh
+fi
+
+# udev needs to be restart when real init runs, so stop it
+echo "Stopping udev"
+/etc/init.d/udev stop
+
+echo "Moving sys, proc and dev mounts to new root"
+mount --move /sys /mnt/sys
+mount --move /proc /mnt/proc
+mount --move /dev /mnt/dev
+
+echo "Switching to new root"
+exec switch_root /mnt /sbin/init $KERNEL_CMDLINE
diff --git a/meta-cgl-common/recipes-core/initrdscripts/initramfs-cgl-boot_1.0.bb b/meta-cgl-common/recipes-core/initrdscripts/initramfs-cgl-boot_1.0.bb
new file mode 100644
index 0000000..5265bf0
--- /dev/null
+++ b/meta-cgl-common/recipes-core/initrdscripts/initramfs-cgl-boot_1.0.bb
@@ -0,0 +1,14 @@
+SUMMARY = "Support for having multipath iSCSI devices as root file system"
+LICENSE = "MIT"
+LIC_FILES_CHKSUM = "file://${COREBASE}/meta-cgl/COPYING.MIT;md5=838c366f69b72c5df05c96dff79b35f2"
+SRC_URI = "file://init-boot.sh"
+
+do_install() {
+ install -m 0755 ${WORKDIR}/init-boot.sh ${D}/init
+}
+
+inherit allarch
+
+RDEPENDS_${PN} += "multipath-tools kpartx iscsi-initiator-utils"
+
+FILES_${PN} += " /init "
diff --git a/scripts/conf_setup.sh b/scripts/conf_setup.sh
new file mode 100755
index 0000000..aad8e71
--- /dev/null
+++ b/scripts/conf_setup.sh
@@ -0,0 +1,2 @@
+#!/bin/bash
+echo -e '\nBBMASK = "meta-virtualization/recipes-extended/multipath-tools/multipath-tools_git.bb"' >> conf/local.conf
--
1.9.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [meta-cgl][PATCH 2/2] distro: Move IMAGE_CLASSES inheritance
2016-05-10 10:05 [meta-cgl][PATCH 1/2] Added device manager multipath support for root file system Adrian Dudau
@ 2016-05-10 10:05 ` Adrian Dudau
2016-05-11 15:17 ` Alexandru Vaduva
2016-05-11 15:15 ` [meta-cgl][PATCH 1/2] Added device manager multipath support for root file system Alexandru Vaduva
1 sibling, 1 reply; 5+ messages in thread
From: Adrian Dudau @ 2016-05-10 10:05 UTC (permalink / raw)
To: yocto
The IMAGE_CLASSES variable affects all images, so it must be defined at
distro level. Move it from the core-image-cgl-initramfs where it has no
effect into the distro definition.
Signed-off-by: Adrian Dudau <adrian.dudau@enea.com>
---
meta-cgl-common/conf/distro/poky-cgl.conf | 2 ++
meta-cgl-common/images/core-image-cgl-initramfs.bb | 1 -
2 files changed, 2 insertions(+), 1 deletion(-)
diff --git a/meta-cgl-common/conf/distro/poky-cgl.conf b/meta-cgl-common/conf/distro/poky-cgl.conf
index aff5dec..1fd453e 100644
--- a/meta-cgl-common/conf/distro/poky-cgl.conf
+++ b/meta-cgl-common/conf/distro/poky-cgl.conf
@@ -3,3 +3,5 @@ require conf/distro/poky-lsb.conf
DISTRO = "poky-cgl"
DISTRO_FEATURES_append = " pam selinux"
+
+IMAGE_CLASSES += "image_types_uboot"
diff --git a/meta-cgl-common/images/core-image-cgl-initramfs.bb b/meta-cgl-common/images/core-image-cgl-initramfs.bb
index 845fa07..881f517 100644
--- a/meta-cgl-common/images/core-image-cgl-initramfs.bb
+++ b/meta-cgl-common/images/core-image-cgl-initramfs.bb
@@ -14,7 +14,6 @@ IMAGE_LINGUAS = ""
LICENSE = "MIT"
IMAGE_FSTYPES = "cpio.gz.u-boot"
-IMAGE_CLASSES += "image_types_uboot"
inherit core-image
IMAGE_ROOTFS_SIZE = "8192"
--
1.9.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [meta-cgl][PATCH 1/2] Added device manager multipath support for root file system
2016-05-10 10:05 [meta-cgl][PATCH 1/2] Added device manager multipath support for root file system Adrian Dudau
2016-05-10 10:05 ` [meta-cgl][PATCH 2/2] distro: Move IMAGE_CLASSES inheritance Adrian Dudau
@ 2016-05-11 15:15 ` Alexandru Vaduva
2016-06-02 9:44 ` Adrian Dudau
1 sibling, 1 reply; 5+ messages in thread
From: Alexandru Vaduva @ 2016-05-11 15:15 UTC (permalink / raw)
To: Adrian Dudau, yocto
[-- Attachment #1: Type: text/plain, Size: 10910 bytes --]
Hello Adi, Cannot this two patches be redone by: removingfrom "[PATCH 1/2] Added devicemanager multipath support for root file system" the line "+IMAGE_CLASSES+= "image_types_uboot"" onlykeeping in "[PATCH 2/2] distro: Move IMAGE_CLASSES inheritance" the line from " poky-cgl.conf".
Also this patch has both require and inherit and in myopinion the inherit line can be removed since the core-image-cgl also adds itindirectly.
Another objection for this patch is that it should be brokenin more pieces: multipath, images, initrdscripts and conf_setup or in anotherformat. Having it like this is too big and in case of a revert too muchfunctionality could be lost or replicated. Thanks,Alex V.
On Tuesday, May 10, 2016 1:08 PM, Adrian Dudau <adrian.dudau@enea.com> wrote:
From: Mats Liljegren <mats.liljegren@enea.com>
This is implemented using an initramfs, built by core-image-cgl-initramfs image
recipe. Multipath device configurations are done using kernel boot parameters.
The multipath-root-howto.md explains how to use this.
Also note that the meta-cgl/scripts/conf_setup.sh will now also add a BBMASK
entry to mask out the meta-virtualization's version of multipath-tools, since
we need the newer recipe in the meta-openembedded/meta-oe instead.
Signed-off-by: Mats Liljegren <mats.liljegren@enea.com>
Signed-off-by: Adrian Dudau <adrian.dudau@enea.com>
---
meta-cgl-common/images/core-image-cgl-initramfs.bb | 22 +++
meta-cgl-common/images/core-image-cgl.bb | 2 +
.../packagegroups/packagegroup-cgl-middleware.bb | 8 +-
.../recipes-core/initrdscripts/files/init-boot.sh | 179 +++++++++++++++++++++
.../initrdscripts/initramfs-cgl-boot_1.0.bb | 14 ++
scripts/conf_setup.sh | 2 +
6 files changed, 220 insertions(+), 7 deletions(-)
create mode 100644 meta-cgl-common/images/core-image-cgl-initramfs.bb
create mode 100644 meta-cgl-common/recipes-core/initrdscripts/files/init-boot.sh
create mode 100644 meta-cgl-common/recipes-core/initrdscripts/initramfs-cgl-boot_1.0.bb
create mode 100755 scripts/conf_setup.sh
diff --git a/meta-cgl-common/images/core-image-cgl-initramfs.bb b/meta-cgl-common/images/core-image-cgl-initramfs.bb
new file mode 100644
index 0000000..845fa07
--- /dev/null
+++ b/meta-cgl-common/images/core-image-cgl-initramfs.bb
@@ -0,0 +1,22 @@
+require core-image-cgl.bb
+
+# Recipe is based on core-image-minimal.bb
+DESCRIPTION = "Initramfs used to mount multipath device as root file system"
+
+PACKAGE_INSTALL = "initramfs-cgl-boot busybox base-passwd udev ${ROOTFS_BOOTSTRAP_INSTALL}"
+
+# Do not pollute the initrd image with rootfs features
+IMAGE_FEATURES = ""
+
+export IMAGE_BASENAME = "core-image-cgl-initramfs"
+IMAGE_LINGUAS = ""
+
+LICENSE = "MIT"
+
+IMAGE_FSTYPES = "cpio.gz.u-boot"
+IMAGE_CLASSES += "image_types_uboot"
+inherit core-image
+
+IMAGE_ROOTFS_SIZE = "8192"
+
+BAD_RECOMMENDATIONS += "busybox-syslog"
diff --git a/meta-cgl-common/images/core-image-cgl.bb b/meta-cgl-common/images/core-image-cgl.bb
index d12391b..5975601 100644
--- a/meta-cgl-common/images/core-image-cgl.bb
+++ b/meta-cgl-common/images/core-image-cgl.bb
@@ -21,6 +21,8 @@ IMAGE_INSTALL += "\
kernel-modules \
"
+IMAGE_FSTYPES += " ext3.gz"
+
# kexec-tools doesn't work on Mips
KEXECTOOLS_mips ?= ""
KEXECTOOLS_mipsel ?= ""
diff --git a/meta-cgl-common/packagegroups/packagegroup-cgl-middleware.bb b/meta-cgl-common/packagegroups/packagegroup-cgl-middleware.bb
index 448e038..b0c2cac 100644
--- a/meta-cgl-common/packagegroups/packagegroup-cgl-middleware.bb
+++ b/meta-cgl-common/packagegroups/packagegroup-cgl-middleware.bb
@@ -18,14 +18,8 @@ DHCP = " \
"
MULTIPATH_TOOLS = " \
- libmpathpersist \
- mpathpersist \
+ multipath-tools \
kpartx \
- libmultipath \
- multipath \
- multipathd \
- libmultipath-dev \
- libmpathpersist-dev \
"
RDEPENDS_packagegroup-cgl-middleware = "\
diff --git a/meta-cgl-common/recipes-core/initrdscripts/files/init-boot.sh b/meta-cgl-common/recipes-core/initrdscripts/files/init-boot.sh
new file mode 100644
index 0000000..fcadfc6
--- /dev/null
+++ b/meta-cgl-common/recipes-core/initrdscripts/files/init-boot.sh
@@ -0,0 +1,179 @@
+#!/bin/sh -eu
+
+# Fail function, either called explicitly or when shell will quit
+fail () {
+ # Avoid recursive traps
+ trap '' ERR EXIT
+
+ # If message provided, print it
+ [ -n '${1:-}' ] && echo $@
+
+ # Generic error message and shell access
+ echo "Error occured, giving a shell"
+ exec sh
+}
+
+trap fail ERR EXIT
+
+interrupt () {
+ echo "User interrupt received, giving a shell. When exiting shell, execution will continue."
+ sh
+}
+
+trap interrupt INT
+
+# Find session ID for an iSCSI disk given its IQN name
+iqn_to_sid () {
+ iscsiadm -m session | fgrep $1 | sed -r 's/.*\[([0-9])\].*/\1/'
+}
+
+# Find device name, without path, for an iSCSI disk given its IQN name
+iqn_to_dev () {
+ iscsiadm -m session -r $(iqn_to_sid $1) -P3 | sed -rn 's/.*Attached scsi disk ([a-zA-Z0-9_]+).*/\1/p'
+}
+
+# Parse input parameters expecting name=value pairs.
+# Name only matches known parameters.
+# On match, set variable <name> to value <value>.
+# E.g. given "parse_cmdline trythis="ok, do it"
+# the shell variable "trythis" will now have the value "ok, do it".
+parse_cmdline () {
+ iscsi_chap_user=""
+ iscsi_chap_pwd=""
+ iscsi_dev=""
+ iscsi_debug=0
+
+ while [ -n "${1:-}" ]; do
+ name="${1%%=*}"
+ val="${1#*=}"
+ case $name in
+ iscsi_chap_user|iscsi_chap_pw)
+ eval $name=\"$val\";;
+ iscsi_dev)
+ eval $name=\"$iscsi_dev $val\"
+ ;;
+ iscsi_debug)
+ set -x
+ iscsi_debug=1
+ ;;
+ esac
+ shift
+ done
+
+ [ -n "${iscsi_dev}" ] || fail "Mandatory kernel boot parameter 'iscsi_dev' not given."
+
+}
+
+PATH=/sbin:/bin:/usr/sbin:/usr/bin
+
+echo "Mounting /proc"
+mount -t proc proc /proc
+
+KERNEL_CMDLINE="$(cat /proc/cmdline)"
+
+echo "Parsing kernel parameters"
+parse_cmdline $KERNEL_CMDLINE
+
+echo "Mounting /sys"
+mount -t sysfs sysfs /sys
+
+# udev is needed for multipath
+echo "Starting udev"
+/etc/init.d/udev start
+
+# Add CHAP autenthication, if given as kernel boot parameters
+echo "Configuring iSCSI"
+[ -n "$iscsi_chap_user" -o -n "$iscsi_chap_pw" ] && cat <<EOF >> /etc/iscsi/iscsid.conf
+node.session.auth.authmethod = CHAP
+EOF
+[ -n "$iscsi_chap_user" ] && cat <<EOF >> /etc/iscsi/iscsid.conf
+node.session.auth.username = $iscsi_chap_user
+EOF
+[ -n "$iscsi_chap_pw" ] && cat <<EOF >> /etc/iscsi/iscsid.conf
+node.session.auth.password = $iscsi_chap_pw
+EOF
+
+echo >> /etc/iscsi/iscsid.conf
+
+echo "Starting iSCSI daemon"
+/etc/init.d/iscsid restart
+
+echo "Discovering iSCSI devices"
+
+for dev in $iscsi_dev; do
+ target_ip="${dev%%:*}"
+ dev_name="${dev#*:}"
+ echo "Logging in to iscsi devices: $dev_name, target: $target_ip"
+ iscsiadm --mode discoverydb --type sendtargets --discover --portal $target_ip
+ iscsiadm --mode node --targetname $dev_name --login --portal $target_ip
+done
+
+echo "Configuring multipath"
+cat <<EOF > /etc/multipath.conf
+defaults {
+ path_grouping_policy multibus
+ # If no path, then queue requests
+ no_path_retry queue
+}
+devices {
+ device {
+ vendor IET
+ product VIRTUAL-DISK
+ path_grouping_policy multibus
+ }
+}
+blacklist {
+ devnode ".*"
+}
+blacklist_exceptions {
+$(for dev in $iscsi_dev; do
+ echo " devnode \"^$(iqn_to_dev ${dev#*:})\""
+ done)
+ property ".*"
+}
+EOF
+
+echo "Starting multipath daemon"
+# Make sure lock file directory exists
+mkdir -p /var/lock/subsys/multipathd
+/etc/init.d/multipathd start
+
+mpath_template='/dev/disk/by-id/dm-uuid-mpath-*'
+
+echo "Waiting for mpath device to appear"
+while [ -z "$(ls $mpath_template 2>/dev/null)" ]; do
+ sleep 1
+done
+MPATH_DEV="$(ls $mpath_template)"
+echo "mpath device: $MPATH_DEV"
+
+echo "Mounting mpath device $MPATH_DEV"
+mount $MPATH_DEV /mnt
+
+echo "Stopping multipath daemon"
+# Should be using "/etc/init.d/multipathd stop", but did not work.
+# Got "killall: /sbin/multipathd: no process killed".
+# Kill the process based on the saved pid.
+kill $(cat /run/multipathd.pid)
+
+echo "Moving iscsi pid and lock files"
+cp /run/iscsid.pid /mnt/run/iscsid.pid
+cp /run/lock/iscsi/* /mnt/run/lock/iscsi
+
+# In case iscsi_debug is given, open a shell at this point
+if [ $iscsi_debug -eq 1 ]; then
+ echo "iscsi_debug given, opening a shell. When exiting shell, boot will continue."
+ sh
+fi
+
+# udev needs to be restart when real init runs, so stop it
+echo "Stopping udev"
+/etc/init.d/udev stop
+
+echo "Moving sys, proc and dev mounts to new root"
+mount --move /sys /mnt/sys
+mount --move /proc /mnt/proc
+mount --move /dev /mnt/dev
+
+echo "Switching to new root"
+exec switch_root /mnt /sbin/init $KERNEL_CMDLINE
diff --git a/meta-cgl-common/recipes-core/initrdscripts/initramfs-cgl-boot_1.0.bb b/meta-cgl-common/recipes-core/initrdscripts/initramfs-cgl-boot_1.0.bb
new file mode 100644
index 0000000..5265bf0
--- /dev/null
+++ b/meta-cgl-common/recipes-core/initrdscripts/initramfs-cgl-boot_1.0.bb
@@ -0,0 +1,14 @@
+SUMMARY = "Support for having multipath iSCSI devices as root file system"
+LICENSE = "MIT"
+LIC_FILES_CHKSUM = "file://${COREBASE}/meta-cgl/COPYING.MIT;md5=838c366f69b72c5df05c96dff79b35f2"
+SRC_URI = "file://init-boot.sh"
+
+do_install() {
+ install -m 0755 ${WORKDIR}/init-boot.sh ${D}/init
+}
+
+inherit allarch
+
+RDEPENDS_${PN} += "multipath-tools kpartx iscsi-initiator-utils"
+
+FILES_${PN} += " /init "
diff --git a/scripts/conf_setup.sh b/scripts/conf_setup.sh
new file mode 100755
index 0000000..aad8e71
--- /dev/null
+++ b/scripts/conf_setup.sh
@@ -0,0 +1,2 @@
+#!/bin/bash
+echo -e '\nBBMASK = "meta-virtualization/recipes-extended/multipath-tools/multipath-tools_git.bb"' >> conf/local.conf
--
1.9.1
--
_______________________________________________
yocto mailing list
yocto@yoctoproject.org
https://lists.yoctoproject.org/listinfo/yocto
[-- Attachment #2: Type: text/html, Size: 15090 bytes --]
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [meta-cgl][PATCH 2/2] distro: Move IMAGE_CLASSES inheritance
2016-05-10 10:05 ` [meta-cgl][PATCH 2/2] distro: Move IMAGE_CLASSES inheritance Adrian Dudau
@ 2016-05-11 15:17 ` Alexandru Vaduva
0 siblings, 0 replies; 5+ messages in thread
From: Alexandru Vaduva @ 2016-05-11 15:17 UTC (permalink / raw)
To: Adrian Dudau, yocto
[-- Attachment #1: Type: text/plain, Size: 1878 bytes --]
Hello Adi, Cannot this two patches be redone by: removingfrom "[PATCH 1/2] Added devicemanager multipath support for root file system" the line "+IMAGE_CLASSES+= "image_types_uboot"" onlykeeping in "[PATCH 2/2] distro: Move IMAGE_CLASSES inheritance" the line from " poky-cgl.conf".
Thanks,Alex V.
On Tuesday, May 10, 2016 1:06 PM, Adrian Dudau <adrian.dudau@enea.com> wrote:
The IMAGE_CLASSES variable affects all images, so it must be defined at
distro level. Move it from the core-image-cgl-initramfs where it has no
effect into the distro definition.
Signed-off-by: Adrian Dudau <adrian.dudau@enea.com>
---
meta-cgl-common/conf/distro/poky-cgl.conf | 2 ++
meta-cgl-common/images/core-image-cgl-initramfs.bb | 1 -
2 files changed, 2 insertions(+), 1 deletion(-)
diff --git a/meta-cgl-common/conf/distro/poky-cgl.conf b/meta-cgl-common/conf/distro/poky-cgl.conf
index aff5dec..1fd453e 100644
--- a/meta-cgl-common/conf/distro/poky-cgl.conf
+++ b/meta-cgl-common/conf/distro/poky-cgl.conf
@@ -3,3 +3,5 @@ require conf/distro/poky-lsb.conf
DISTRO = "poky-cgl"
DISTRO_FEATURES_append = " pam selinux"
+
+IMAGE_CLASSES += "image_types_uboot"
diff --git a/meta-cgl-common/images/core-image-cgl-initramfs.bb b/meta-cgl-common/images/core-image-cgl-initramfs.bb
index 845fa07..881f517 100644
--- a/meta-cgl-common/images/core-image-cgl-initramfs.bb
+++ b/meta-cgl-common/images/core-image-cgl-initramfs.bb
@@ -14,7 +14,6 @@ IMAGE_LINGUAS = ""
LICENSE = "MIT"
IMAGE_FSTYPES = "cpio.gz.u-boot"
-IMAGE_CLASSES += "image_types_uboot"
inherit core-image
IMAGE_ROOTFS_SIZE = "8192"
--
1.9.1
--
_______________________________________________
yocto mailing list
yocto@yoctoproject.org
https://lists.yoctoproject.org/listinfo/yocto
[-- Attachment #2: Type: text/html, Size: 4457 bytes --]
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [meta-cgl][PATCH 1/2] Added device manager multipath support for root file system
2016-05-11 15:15 ` [meta-cgl][PATCH 1/2] Added device manager multipath support for root file system Alexandru Vaduva
@ 2016-06-02 9:44 ` Adrian Dudau
0 siblings, 0 replies; 5+ messages in thread
From: Adrian Dudau @ 2016-06-02 9:44 UTC (permalink / raw)
To: vaduvajanalexandru; +Cc: yocto
On ons, 2016-05-11 at 15:15 +0000, Alexandru Vaduva wrote:
> Hello Adi,
>
> Cannot this two patches be redone by:
> removing from "[PATCH 1/2] Added device manager
> multipath support for root file system" the line "+IMAGE_CLASSES +=
> "image_types_uboot""
> only keeping in "[PATCH 2/2] distro: Move IMAGE_CLASSES
> inheritance" the line from " poky-cgl.conf".
>
>
> Also this patch has both require and inherit and in my opinion the
> inherit line can be removed since the core-image-cgl also adds it
> indirectly.
>
>
> Another objection for this patch is that it should be broken in more
> pieces: multipath, images, initrdscripts and conf_setup or in another
> format. Having it like this is too big and in case of a revert too
> much functionality could be lost or replicated.
You're probably right it would be good to rework this patch, but since
I'm not the original author and I don't have allocated time for it these
days it may be a while until I can get around to do it.
I'll merge the other two regarding Selinux though.
--Adrian
>
>
> Thanks,
> Alex V.
>
>
>
>
> On Tuesday, May 10, 2016 1:08 PM, Adrian Dudau <adrian.dudau@enea.com>
> wrote:
>
>
>
> From: Mats Liljegren <mats.liljegren@enea.com>
>
> This is implemented using an initramfs, built by
> core-image-cgl-initramfs image
> recipe. Multipath device configurations are done using kernel boot
> parameters.
> The multipath-root-howto.md explains how to use this.
>
> Also note that the meta-cgl/scripts/conf_setup.sh will now also add a
> BBMASK
> entry to mask out the meta-virtualization's version of
> multipath-tools, since
> we need the newer recipe in the meta-openembedded/meta-oe instead.
>
> Signed-off-by: Mats Liljegren <mats.liljegren@enea.com>
> Signed-off-by: Adrian Dudau <adrian.dudau@enea.com>
> ---
> meta-cgl-common/images/core-image-cgl-initramfs.bb | 22 +++
> meta-cgl-common/images/core-image-cgl.bb | 2 +
> .../packagegroups/packagegroup-cgl-middleware.bb | 8 +-
> .../recipes-core/initrdscripts/files/init-boot.sh | 179
> +++++++++++++++++++++
> .../initrdscripts/initramfs-cgl-boot_1.0.bb | 14 ++
> scripts/conf_setup.sh | 2 +
> 6 files changed, 220 insertions(+), 7 deletions(-)
> create mode 100644 meta-cgl-common/images/core-image-cgl-initramfs.bb
> create mode 100644
> meta-cgl-common/recipes-core/initrdscripts/files/init-boot.sh
> create mode 100644
> meta-cgl-common/recipes-core/initrdscripts/initramfs-cgl-boot_1.0.bb
> create mode 100755 scripts/conf_setup.sh
>
> diff --git a/meta-cgl-common/images/core-image-cgl-initramfs.bb
> b/meta-cgl-common/images/core-image-cgl-initramfs.bb
> new file mode 100644
> index 0000000..845fa07
> --- /dev/null
> +++ b/meta-cgl-common/images/core-image-cgl-initramfs.bb
> @@ -0,0 +1,22 @@
> +require core-image-cgl.bb
> +
> +# Recipe is based on core-image-minimal.bb
> +DESCRIPTION = "Initramfs used to mount multipath device as root file
> system"
> +
> +PACKAGE_INSTALL = "initramfs-cgl-boot busybox base-passwd udev
> ${ROOTFS_BOOTSTRAP_INSTALL}"
> +
> +# Do not pollute the initrd image with rootfs features
> +IMAGE_FEATURES = ""
> +
> +export IMAGE_BASENAME = "core-image-cgl-initramfs"
> +IMAGE_LINGUAS = ""
> +
> +LICENSE = "MIT"
> +
> +IMAGE_FSTYPES = "cpio.gz.u-boot"
> +IMAGE_CLASSES += "image_types_uboot"
> +inherit core-image
> +
> +IMAGE_ROOTFS_SIZE = "8192"
> +
> +BAD_RECOMMENDATIONS += "busybox-syslog"
> diff --git a/meta-cgl-common/images/core-image-cgl.bb
> b/meta-cgl-common/images/core-image-cgl.bb
> index d12391b..5975601 100644
> --- a/meta-cgl-common/images/core-image-cgl.bb
> +++ b/meta-cgl-common/images/core-image-cgl.bb
> @@ -21,6 +21,8 @@ IMAGE_INSTALL += "\
> kernel-modules \
> "
>
> +IMAGE_FSTYPES += " ext3.gz"
> +
> # kexec-tools doesn't work on Mips
> KEXECTOOLS_mips ?= ""
> KEXECTOOLS_mipsel ?= ""
> diff --git
> a/meta-cgl-common/packagegroups/packagegroup-cgl-middleware.bb
> b/meta-cgl-common/packagegroups/packagegroup-cgl-middleware.bb
> index 448e038..b0c2cac 100644
> --- a/meta-cgl-common/packagegroups/packagegroup-cgl-middleware.bb
> +++ b/meta-cgl-common/packagegroups/packagegroup-cgl-middleware.bb
> @@ -18,14 +18,8 @@ DHCP = " \
> "
>
> MULTIPATH_TOOLS = " \
> - libmpathpersist \
> - mpathpersist \
> + multipath-tools \
> kpartx \
> - libmultipath \
> - multipath \
> - multipathd \
> - libmultipath-dev \
> - libmpathpersist-dev \
> "
>
> RDEPENDS_packagegroup-cgl-middleware = "\
> diff --git
> a/meta-cgl-common/recipes-core/initrdscripts/files/init-boot.sh
> b/meta-cgl-common/recipes-core/initrdscripts/files/init-boot.sh
> new file mode 100644
> index 0000000..fcadfc6
> --- /dev/null
> +++ b/meta-cgl-common/recipes-core/initrdscripts/files/init-boot.sh
> @@ -0,0 +1,179 @@
> +#!/bin/sh -eu
> +
> +# Fail function, either called explicitly or when shell will quit
> +fail () {
> + # Avoid recursive traps
> + trap '' ERR EXIT
> +
> + # If message provided, print it
> + [ -n '${1:-}' ] && echo $@
> +
> + # Generic error message and shell access
> + echo "Error occured, giving a shell"
> + exec sh
> +}
> +
> +trap fail ERR EXIT
> +
> +interrupt () {
> + echo "User interrupt received, giving a shell. When exiting
> shell, execution will continue."
> + sh
> +}
> +
> +trap interrupt INT
> +
> +# Find session ID for an iSCSI disk given its IQN name
> +iqn_to_sid () {
> + iscsiadm -m session | fgrep $1 | sed -r 's/.*\[([0-9])\].*/\1/'
> +}
> +
> +# Find device name, without path, for an iSCSI disk given its IQN
> name
> +iqn_to_dev () {
> + iscsiadm -m session -r $(iqn_to_sid $1) -P3 | sed -rn
> 's/.*Attached scsi disk ([a-zA-Z0-9_]+).*/\1/p'
> +}
> +
> +# Parse input parameters expecting name=value pairs.
> +# Name only matches known parameters.
> +# On match, set variable <name> to value <value>.
> +# E.g. given "parse_cmdline trythis="ok, do it"
> +# the shell variable "trythis" will now have the value "ok, do it".
> +parse_cmdline () {
> + iscsi_chap_user=""
> + iscsi_chap_pwd=""
> + iscsi_dev=""
> + iscsi_debug=0
> +
> + while [ -n "${1:-}" ]; do
> + name="${1%%=*}"
> + val="${1#*=}"
> + case $name in
> + iscsi_chap_user|iscsi_chap_pw)
> + eval $name=\"$val\";;
> + iscsi_dev)
> + eval $name=\"$iscsi_dev $val\"
> + ;;
> + iscsi_debug)
> + set -x
> + iscsi_debug=1
> + ;;
> + esac
> + shift
> + done
> +
> + [ -n "${iscsi_dev}" ] || fail "Mandatory kernel boot parameter
> 'iscsi_dev' not given."
> +
> +}
> +
> +PATH=/sbin:/bin:/usr/sbin:/usr/bin
> +
> +echo "Mounting /proc"
> +mount -t proc proc /proc
> +
> +KERNEL_CMDLINE="$(cat /proc/cmdline)"
> +
> +echo "Parsing kernel parameters"
> +parse_cmdline $KERNEL_CMDLINE
> +
> +echo "Mounting /sys"
> +mount -t sysfs sysfs /sys
> +
> +# udev is needed for multipath
> +echo "Starting udev"
> +/etc/init.d/udev start
> +
> +# Add CHAP autenthication, if given as kernel boot parameters
> +echo "Configuring iSCSI"
> +[ -n "$iscsi_chap_user" -o -n "$iscsi_chap_pw" ] && cat <<EOF
> >> /etc/iscsi/iscsid.conf
> +node.session.auth.authmethod = CHAP
> +EOF
> +[ -n "$iscsi_chap_user" ] && cat <<EOF >> /etc/iscsi/iscsid.conf
> +node.session.auth.username = $iscsi_chap_user
> +EOF
> +[ -n "$iscsi_chap_pw" ] && cat <<EOF >> /etc/iscsi/iscsid.conf
> +node.session.auth.password = $iscsi_chap_pw
> +EOF
> +
> +echo >> /etc/iscsi/iscsid.conf
> +
> +echo "Starting iSCSI daemon"
> +/etc/init.d/iscsid restart
> +
> +echo "Discovering iSCSI devices"
> +
> +for dev in $iscsi_dev; do
> + target_ip="${dev%%:*}"
> + dev_name="${dev#*:}"
> + echo "Logging in to iscsi devices: $dev_name, target: $target_ip"
> + iscsiadm --mode discoverydb --type sendtargets --discover
> --portal $target_ip
> + iscsiadm --mode node --targetname $dev_name --login --portal
> $target_ip
> +done
> +
> +echo "Configuring multipath"
> +cat <<EOF > /etc/multipath.conf
> +defaults {
> + path_grouping_policy multibus
> + # If no path, then queue requests
> + no_path_retry queue
> +}
> +devices {
> + device {
> + vendor IET
> + product VIRTUAL-DISK
> + path_grouping_policy multibus
> + }
> +}
> +blacklist {
> + devnode ".*"
> +}
> +blacklist_exceptions {
> +$(for dev in $iscsi_dev; do
> + echo " devnode \"^$(iqn_to_dev ${dev#*:})\""
> + done)
> + property ".*"
> +}
> +EOF
> +
> +echo "Starting multipath daemon"
> +# Make sure lock file directory exists
> +mkdir -p /var/lock/subsys/multipathd
> +/etc/init.d/multipathd start
> +
> +mpath_template='/dev/disk/by-id/dm-uuid-mpath-*'
> +
> +echo "Waiting for mpath device to appear"
> +while [ -z "$(ls $mpath_template 2>/dev/null)" ]; do
> + sleep 1
> +done
> +MPATH_DEV="$(ls $mpath_template)"
> +echo "mpath device: $MPATH_DEV"
> +
> +echo "Mounting mpath device $MPATH_DEV"
> +mount $MPATH_DEV /mnt
> +
> +echo "Stopping multipath daemon"
> +# Should be using "/etc/init.d/multipathd stop", but did not work.
> +# Got "killall: /sbin/multipathd: no process killed".
> +# Kill the process based on the saved pid.
> +kill $(cat /run/multipathd.pid)
> +
> +echo "Moving iscsi pid and lock files"
> +cp /run/iscsid.pid /mnt/run/iscsid.pid
> +cp /run/lock/iscsi/* /mnt/run/lock/iscsi
> +
> +# In case iscsi_debug is given, open a shell at this point
> +if [ $iscsi_debug -eq 1 ]; then
> + echo "iscsi_debug given, opening a shell. When exiting shell,
> boot will continue."
> + sh
> +fi
> +
> +# udev needs to be restart when real init runs, so stop it
> +echo "Stopping udev"
> +/etc/init.d/udev stop
> +
> +echo "Moving sys, proc and dev mounts to new root"
> +mount --move /sys /mnt/sys
> +mount --move /proc /mnt/proc
> +mount --move /dev /mnt/dev
> +
> +echo "Switching to new root"
> +exec switch_root /mnt /sbin/init $KERNEL_CMDLINE
> diff --git
> a/meta-cgl-common/recipes-core/initrdscripts/initramfs-cgl-boot_1.0.bb
> b/meta-cgl-common/recipes-core/initrdscripts/initramfs-cgl-boot_1.0.bb
> new file mode 100644
> index 0000000..5265bf0
> --- /dev/null
> +++
> b/meta-cgl-common/recipes-core/initrdscripts/initramfs-cgl-boot_1.0.bb
> @@ -0,0 +1,14 @@
> +SUMMARY = "Support for having multipath iSCSI devices as root file
> system"
> +LICENSE = "MIT"
> +LIC_FILES_CHKSUM =
> "file://${COREBASE}/meta-cgl/COPYING.MIT;md5=838c366f69b72c5df05c96dff79b35f2"
> +SRC_URI = "file://init-boot.sh"
> +
> +do_install() {
> + install -m 0755 ${WORKDIR}/init-boot.sh ${D}/init
> +}
> +
> +inherit allarch
> +
> +RDEPENDS_${PN} += "multipath-tools kpartx iscsi-initiator-utils"
> +
> +FILES_${PN} += " /init "
> diff --git a/scripts/conf_setup.sh b/scripts/conf_setup.sh
> new file mode 100755
> index 0000000..aad8e71
> --- /dev/null
> +++ b/scripts/conf_setup.sh
> @@ -0,0 +1,2 @@
> +#!/bin/bash
> +echo -e '\nBBMASK =
> "meta-virtualization/recipes-extended/multipath-tools/multipath-tools_git.bb"' >> conf/local.conf
> --
> 1.9.1
>
> --
> _______________________________________________
> yocto mailing list
> yocto@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/yocto
>
>
>
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2016-06-02 9:44 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-05-10 10:05 [meta-cgl][PATCH 1/2] Added device manager multipath support for root file system Adrian Dudau
2016-05-10 10:05 ` [meta-cgl][PATCH 2/2] distro: Move IMAGE_CLASSES inheritance Adrian Dudau
2016-05-11 15:17 ` Alexandru Vaduva
2016-05-11 15:15 ` [meta-cgl][PATCH 1/2] Added device manager multipath support for root file system Alexandru Vaduva
2016-06-02 9:44 ` Adrian Dudau
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.