[PATCH] Add a check file for multipath.

[PATCH] Add a check file for multipath.

[Peter Jones]

If multipath isn't installed, don't use it.  If we're in hostonly mode,
only install the multipath module if it's used for / .  Otherwise, if
the user was dumb enough to install it, they get it during bootup.
---
 modules.d/90multipath/check |   27 +++++++++++++++++++++++++++
 1 files changed, 27 insertions(+), 0 deletions(-)
 create mode 100755 modules.d/90multipath/check

diff --git a/modules.d/90multipath/check b/modules.d/90multipath/check
new file mode 100755
index 0000000..f602e3e
--- /dev/null
+++ b/modules.d/90multipath/check
@@ -0,0 +1,27 @@
+#!/bin/bash
+
+# if there's no multipath binary, no go.
+which multipath >/dev/null 2>&1 || exit 1
+
+[[ $1 = -d ]] && echo dm_multipath
+
+. $dracutfunctions
+[[ $debug ]] && set -x
+
+is_mpath() {
+    [ -e /sys/dev/block/$1/dm/uuid ] || return 1
+    [[ $(cat /sys/dev/block/$1/dm/uuid) =~ ^mpath- ]] && return 0
+    return 1
+}
+
+# If we're in hostonly mode, do mpath if we appear to be using it for /
+if [[ $hostonly ]]; then
+    rootdev=$(find_root_block_device)
+    if [[ $rootdev ]]; then
+        check_block_and_slaves is_mpath "$rootdev" && exit 0
+    fi
+    exit 1
+fi
+
+# the user installed it, they get it...
+exit 0
-- 
1.6.5.2

Re: [PATCH] Add a check file for multipath.

[Harald Hoyer]

On 01/21/2010 09:41 PM, Peter Jones wrote:
> If multipath isn't installed, don't use it.  If we're in hostonly mode,
> only install the multipath module if it's used for / .  Otherwise, if
> the user was dumb enough to install it, they get it during bootup.
> ---
>   modules.d/90multipath/check |   27 +++++++++++++++++++++++++++
>   1 files changed, 27 insertions(+), 0 deletions(-)
>   create mode 100755 modules.d/90multipath/check
>
> diff --git a/modules.d/90multipath/check b/modules.d/90multipath/check
> new file mode 100755
> index 0000000..f602e3e
> --- /dev/null
> +++ b/modules.d/90multipath/check
> @@ -0,0 +1,27 @@
> +#!/bin/bash
> +
> +# if there's no multipath binary, no go.
> +which multipath>/dev/null 2>&1 || exit 1
> +
> +[[ $1 = -d ]]&&  echo dm_multipath

hmm, there is no module dm_multipath

> +
> +. $dracutfunctions
> +[[ $debug ]]&&  set -x
> +
> +is_mpath() {
> +    [ -e /sys/dev/block/$1/dm/uuid ] || return 1
> +    [[ $(cat /sys/dev/block/$1/dm/uuid) =~ ^mpath- ]]&&  return 0
> +    return 1
> +}
> +
> +# If we're in hostonly mode, do mpath if we appear to be using it for /
> +if [[ $hostonly ]]; then
> +    rootdev=$(find_root_block_device)
> +    if [[ $rootdev ]]; then
> +        check_block_and_slaves is_mpath "$rootdev"&&  exit 0
> +    fi
> +    exit 1
> +fi
> +
> +# the user installed it, they get it...
> +exit 0

Re: [PATCH] Add a check file for multipath.

[Peter Jones]

On 01/22/2010 04:12 AM, Harald Hoyer wrote:
> On 01/21/2010 09:41 PM, Peter Jones wrote:
>> If multipath isn't installed, don't use it.  If we're in hostonly mode,
>> only install the multipath module if it's used for / .  Otherwise, if
>> the user was dumb enough to install it, they get it during bootup.
>> ---
>>   modules.d/90multipath/check |   27 +++++++++++++++++++++++++++
>>   1 files changed, 27 insertions(+), 0 deletions(-)
>>   create mode 100755 modules.d/90multipath/check
>>
>> diff --git a/modules.d/90multipath/check b/modules.d/90multipath/check
>> new file mode 100755
>> index 0000000..f602e3e
>> --- /dev/null
>> +++ b/modules.d/90multipath/check
>> @@ -0,0 +1,27 @@
>> +#!/bin/bash
>> +
>> +# if there's no multipath binary, no go.
>> +which multipath>/dev/null 2>&1 || exit 1
>> +
>> +[[ $1 = -d ]]&&  echo dm_multipath
> 
> hmm, there is no module dm_multipath

Oh, I completely misread how this works and parsed it as kernel module deps,
rather than dracut module deps.  I'll send a revised version with a fix.

-- 
        Peter

I'd like to start a religion. That's where the money is.
		-- L. Ron Hubbard to Lloyd Eshbach, in 1949;
			quoted by Eshbach in _Over My Shoulder_.

[PATCH] Add a check file for multipath.

[Peter Jones]

If multipath isn't installed, don't use it.  If we're in hostonly mode,
only install the multipath module if it's used for / .  Otherwise, if
the user was dumb enough to install it, they get it during bootup.
---
 modules.d/90multipath/check |   27 +++++++++++++++++++++++++++
 1 files changed, 27 insertions(+), 0 deletions(-)
 create mode 100755 modules.d/90multipath/check

diff --git a/modules.d/90multipath/check b/modules.d/90multipath/check
new file mode 100755
index 0000000..d0cad4c
--- /dev/null
+++ b/modules.d/90multipath/check
@@ -0,0 +1,27 @@
+#!/bin/bash
+
+# if there's no multipath binary, no go.
+which multipath >/dev/null 2>&1 || exit 1
+
+[[ $1 = -d ]] && exit 0
+
+. $dracutfunctions
+[[ $debug ]] && set -x
+
+is_mpath() {
+    [ -e /sys/dev/block/$1/dm/uuid ] || return 1
+    [[ $(cat /sys/dev/block/$1/dm/uuid) =~ ^mpath- ]] && return 0
+    return 1
+}
+
+# If we're in hostonly mode, do mpath if we appear to be using it for /
+if [[ $hostonly ]]; then
+    rootdev=$(find_root_block_device)
+    if [[ $rootdev ]]; then
+        check_block_and_slaves is_mpath "$rootdev" && exit 0
+    fi
+    exit 1
+fi
+
+# the user installed it, they get it...
+exit 0
-- 
1.6.5.2

Re: [PATCH] Add a check file for multipath.

[Harald Hoyer]

On 01/22/2010 04:56 PM, Peter Jones wrote:
> If multipath isn't installed, don't use it.  If we're in hostonly mode,
> only install the multipath module if it's used for / .  Otherwise, if
> the user was dumb enough to install it, they get it during bootup.
> ---
>   modules.d/90multipath/check |   27 +++++++++++++++++++++++++++
>   1 files changed, 27 insertions(+), 0 deletions(-)
>   create mode 100755 modules.d/90multipath/check
>
> diff --git a/modules.d/90multipath/check b/modules.d/90multipath/check
> new file mode 100755
> index 0000000..d0cad4c
> --- /dev/null
> +++ b/modules.d/90multipath/check
> @@ -0,0 +1,27 @@
> +#!/bin/bash
> +
> +# if there's no multipath binary, no go.
> +which multipath>/dev/null 2>&1 || exit 1
> +
> +[[ $1 = -d ]]&&  exit 0
> +
> +. $dracutfunctions
> +[[ $debug ]]&&  set -x
> +
> +is_mpath() {
> +    [ -e /sys/dev/block/$1/dm/uuid ] || return 1
> +    [[ $(cat /sys/dev/block/$1/dm/uuid) =~ ^mpath- ]]&&  return 0
> +    return 1
> +}
> +
> +# If we're in hostonly mode, do mpath if we appear to be using it for /
> +if [[ $hostonly ]]; then
> +    rootdev=$(find_root_block_device)
> +    if [[ $rootdev ]]; then
> +        check_block_and_slaves is_mpath "$rootdev"&&  exit 0
> +    fi
> +    exit 1
> +fi
> +
> +# the user installed it, they get it...
> +exit 0


pushed, long time ago..

commit 3d45f9bba994ecd99d8bb7378c7a4dbac1d141fe