All of lore.kernel.org
 help / color / mirror / Atom feed
* [DRACUT PATCH] lvm_scan: use '--ignoremonitoring' rather than '--monitor n'
@ 2010-03-10 19:24 ` Mike Snitzer
  0 siblings, 0 replies; 6+ messages in thread
From: Mike Snitzer @ 2010-03-10 19:24 UTC (permalink / raw)
  To: harald-H+wXaHxf7aLQT0dZR+AlfA
  Cc: initramfs-u79uwXL29TY76Z2rM5mHXA,
	lvm-devel-H+wXaHxf7aLQT0dZR+AlfA

lvchange and vgchange '--monitor n' will not prevent lvm from
attempting to dlopen the libdevmapper-event library.

dracut git commit 47ab3b6c5e introduced the use of '--monitor n' but
'--ignoremonitoring' is needed now that the libdevmapper-event library
isn't copied into the initramfs (ever since 0fae59d6eb3)

Signed-off-by: Mike Snitzer <snitzer-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
---
 modules.d/90lvm/lvm_scan.sh |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/modules.d/90lvm/lvm_scan.sh b/modules.d/90lvm/lvm_scan.sh
index 85cd78e..f4f7b4a 100755
--- a/modules.d/90lvm/lvm_scan.sh
+++ b/modules.d/90lvm/lvm_scan.sh
@@ -60,13 +60,13 @@ nopoll=$(
 if [ -n "$LVS" ] ; then
     info "Scanning devices $lvmdevs for LVM logical volumes $LVS"
     lvm lvscan --ignorelockingfailure 2>&1 | vinfo
-    lvm lvchange -ay --ignorelockingfailure $nopoll --monitor n $LVS 2>&1 | vinfo    
+    lvm lvchange -ay --ignorelockingfailure $nopoll --ignoremonitoring $LVS 2>&1 | vinfo
 fi
 
 if [ -z "$LVS" -o -n "$VGS" ]; then
     info "Scanning devices $lvmdevs for LVM volume groups $VGS"
     lvm vgscan --ignorelockingfailure 2>&1 | vinfo
-    lvm vgchange -ay --ignorelockingfailure $nopoll --monitor n $VGS 2>&1 | vinfo
+    lvm vgchange -ay --ignorelockingfailure $nopoll --ignoremonitoring $VGS 2>&1 | vinfo
 fi
 
 if [ "$lvmwritten" ]; then

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [DRACUT PATCH] lvm_scan: use '--ignoremonitoring' rather than '--monitor n'
@ 2010-03-10 19:24 ` Mike Snitzer
  0 siblings, 0 replies; 6+ messages in thread
From: Mike Snitzer @ 2010-03-10 19:24 UTC (permalink / raw)
  To: lvm-devel

lvchange and vgchange '--monitor n' will not prevent lvm from
attempting to dlopen the libdevmapper-event library.

dracut git commit 47ab3b6c5e introduced the use of '--monitor n' but
'--ignoremonitoring' is needed now that the libdevmapper-event library
isn't copied into the initramfs (ever since 0fae59d6eb3)

Signed-off-by: Mike Snitzer <snitzer@redhat.com>
---
 modules.d/90lvm/lvm_scan.sh |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/modules.d/90lvm/lvm_scan.sh b/modules.d/90lvm/lvm_scan.sh
index 85cd78e..f4f7b4a 100755
--- a/modules.d/90lvm/lvm_scan.sh
+++ b/modules.d/90lvm/lvm_scan.sh
@@ -60,13 +60,13 @@ nopoll=$(
 if [ -n "$LVS" ] ; then
     info "Scanning devices $lvmdevs for LVM logical volumes $LVS"
     lvm lvscan --ignorelockingfailure 2>&1 | vinfo
-    lvm lvchange -ay --ignorelockingfailure $nopoll --monitor n $LVS 2>&1 | vinfo    
+    lvm lvchange -ay --ignorelockingfailure $nopoll --ignoremonitoring $LVS 2>&1 | vinfo
 fi
 
 if [ -z "$LVS" -o -n "$VGS" ]; then
     info "Scanning devices $lvmdevs for LVM volume groups $VGS"
     lvm vgscan --ignorelockingfailure 2>&1 | vinfo
-    lvm vgchange -ay --ignorelockingfailure $nopoll --monitor n $VGS 2>&1 | vinfo
+    lvm vgchange -ay --ignorelockingfailure $nopoll --ignoremonitoring $VGS 2>&1 | vinfo
 fi
 
 if [ "$lvmwritten" ]; then



^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [DRACUT PATCH] lvm_scan: use '--ignoremonitoring' rather than '--monitor n'
  2010-03-10 19:24 ` Mike Snitzer
@ 2010-03-10 19:31     ` Mike Snitzer
  -1 siblings, 0 replies; 6+ messages in thread
From: Mike Snitzer @ 2010-03-10 19:31 UTC (permalink / raw)
  To: harald-H+wXaHxf7aLQT0dZR+AlfA
  Cc: initramfs-u79uwXL29TY76Z2rM5mHXA,
	lvm-devel-H+wXaHxf7aLQT0dZR+AlfA

On Wed, Mar 10 2010 at  2:24pm -0500,
Mike Snitzer <snitzer-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> wrote:

> lvchange and vgchange '--monitor n' will not prevent lvm from
> attempting to dlopen the libdevmapper-event library.
> 
> dracut git commit 47ab3b6c5e introduced the use of '--monitor n' but
> '--ignoremonitoring' is needed now that the libdevmapper-event library
> isn't copied into the initramfs (ever since 0fae59d6eb3)

And just to be clear:
'--monitor n' is "stop monitoring this (VG or LV)"
'--ignoremonitoring' is "don't do any monitoring".

We'll be fixing up lvm's vgchange and lvchange man pages so that they
are more precise on what '--monitor n' means.

Mike

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [DRACUT PATCH] lvm_scan: use '--ignoremonitoring' rather than '--monitor n'
@ 2010-03-10 19:31     ` Mike Snitzer
  0 siblings, 0 replies; 6+ messages in thread
From: Mike Snitzer @ 2010-03-10 19:31 UTC (permalink / raw)
  To: lvm-devel

On Wed, Mar 10 2010 at  2:24pm -0500,
Mike Snitzer <snitzer@redhat.com> wrote:

> lvchange and vgchange '--monitor n' will not prevent lvm from
> attempting to dlopen the libdevmapper-event library.
> 
> dracut git commit 47ab3b6c5e introduced the use of '--monitor n' but
> '--ignoremonitoring' is needed now that the libdevmapper-event library
> isn't copied into the initramfs (ever since 0fae59d6eb3)

And just to be clear:
'--monitor n' is "stop monitoring this (VG or LV)"
'--ignoremonitoring' is "don't do any monitoring".

We'll be fixing up lvm's vgchange and lvchange man pages so that they
are more precise on what '--monitor n' means.

Mike



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [DRACUT PATCH] lvm_scan: use '--ignoremonitoring' rather than '--monitor n'
  2010-03-10 19:24 ` Mike Snitzer
@ 2010-03-17 10:59     ` Harald Hoyer
  -1 siblings, 0 replies; 6+ messages in thread
From: Harald Hoyer @ 2010-03-17 10:59 UTC (permalink / raw)
  To: Mike Snitzer
  Cc: initramfs-u79uwXL29TY76Z2rM5mHXA,
	lvm-devel-H+wXaHxf7aLQT0dZR+AlfA

On 03/10/2010 08:24 PM, Mike Snitzer wrote:
> lvchange and vgchange '--monitor n' will not prevent lvm from
> attempting to dlopen the libdevmapper-event library.
>
> dracut git commit 47ab3b6c5e introduced the use of '--monitor n' but
> '--ignoremonitoring' is needed now that the libdevmapper-event library
> isn't copied into the initramfs (ever since 0fae59d6eb3)
>
> Signed-off-by: Mike Snitzer<snitzer-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
> ---
>   modules.d/90lvm/lvm_scan.sh |    4 ++--
>   1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/modules.d/90lvm/lvm_scan.sh b/modules.d/90lvm/lvm_scan.sh
> index 85cd78e..f4f7b4a 100755
> --- a/modules.d/90lvm/lvm_scan.sh
> +++ b/modules.d/90lvm/lvm_scan.sh
> @@ -60,13 +60,13 @@ nopoll=$(
>   if [ -n "$LVS" ] ; then
>       info "Scanning devices $lvmdevs for LVM logical volumes $LVS"
>       lvm lvscan --ignorelockingfailure 2>&1 | vinfo
> -    lvm lvchange -ay --ignorelockingfailure $nopoll --monitor n $LVS 2>&1 | vinfo
> +    lvm lvchange -ay --ignorelockingfailure $nopoll --ignoremonitoring $LVS 2>&1 | vinfo
>   fi
>
>   if [ -z "$LVS" -o -n "$VGS" ]; then
>       info "Scanning devices $lvmdevs for LVM volume groups $VGS"
>       lvm vgscan --ignorelockingfailure 2>&1 | vinfo
> -    lvm vgchange -ay --ignorelockingfailure $nopoll --monitor n $VGS 2>&1 | vinfo
> +    lvm vgchange -ay --ignorelockingfailure $nopoll --ignoremonitoring $VGS 2>&1 | vinfo
>   fi
>
>   if [ "$lvmwritten" ]; then
> --
> To unsubscribe from this list: send the line "unsubscribe initramfs" in
> the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

pushed

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [DRACUT PATCH] lvm_scan: use '--ignoremonitoring' rather than '--monitor n'
@ 2010-03-17 10:59     ` Harald Hoyer
  0 siblings, 0 replies; 6+ messages in thread
From: Harald Hoyer @ 2010-03-17 10:59 UTC (permalink / raw)
  To: lvm-devel

On 03/10/2010 08:24 PM, Mike Snitzer wrote:
> lvchange and vgchange '--monitor n' will not prevent lvm from
> attempting to dlopen the libdevmapper-event library.
>
> dracut git commit 47ab3b6c5e introduced the use of '--monitor n' but
> '--ignoremonitoring' is needed now that the libdevmapper-event library
> isn't copied into the initramfs (ever since 0fae59d6eb3)
>
> Signed-off-by: Mike Snitzer<snitzer@redhat.com>
> ---
>   modules.d/90lvm/lvm_scan.sh |    4 ++--
>   1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/modules.d/90lvm/lvm_scan.sh b/modules.d/90lvm/lvm_scan.sh
> index 85cd78e..f4f7b4a 100755
> --- a/modules.d/90lvm/lvm_scan.sh
> +++ b/modules.d/90lvm/lvm_scan.sh
> @@ -60,13 +60,13 @@ nopoll=$(
>   if [ -n "$LVS" ] ; then
>       info "Scanning devices $lvmdevs for LVM logical volumes $LVS"
>       lvm lvscan --ignorelockingfailure 2>&1 | vinfo
> -    lvm lvchange -ay --ignorelockingfailure $nopoll --monitor n $LVS 2>&1 | vinfo
> +    lvm lvchange -ay --ignorelockingfailure $nopoll --ignoremonitoring $LVS 2>&1 | vinfo
>   fi
>
>   if [ -z "$LVS" -o -n "$VGS" ]; then
>       info "Scanning devices $lvmdevs for LVM volume groups $VGS"
>       lvm vgscan --ignorelockingfailure 2>&1 | vinfo
> -    lvm vgchange -ay --ignorelockingfailure $nopoll --monitor n $VGS 2>&1 | vinfo
> +    lvm vgchange -ay --ignorelockingfailure $nopoll --ignoremonitoring $VGS 2>&1 | vinfo
>   fi
>
>   if [ "$lvmwritten" ]; then
> --
> To unsubscribe from this list: send the line "unsubscribe initramfs" in
> the body of a message to majordomo at vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

pushed



^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2010-03-17 10:59 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-03-10 19:24 [DRACUT PATCH] lvm_scan: use '--ignoremonitoring' rather than '--monitor n' Mike Snitzer
2010-03-10 19:24 ` Mike Snitzer
     [not found] ` <1268249058-23510-1-git-send-email-snitzer-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2010-03-10 19:31   ` Mike Snitzer
2010-03-10 19:31     ` Mike Snitzer
2010-03-17 10:59   ` Harald Hoyer
2010-03-17 10:59     ` Harald Hoyer

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.