mkinitrd unification across distributions
 help / color / mirror / Atom feed
From: Peter Jones <pjones-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
To: Hans de Goede <hdegoede-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
Cc: initramfs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: Re: [PATCH 4/4] Bring up multipath devices that are configured in multipath.conf.
Date: Wed, 25 Nov 2009 10:52:04 -0500	[thread overview]
Message-ID: <4B0D52A4.6000105@redhat.com> (raw)
In-Reply-To: <4B0D5306.8050706-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>

On 11/25/2009 10:53 AM, Hans de Goede wrote:
> Hi,
> 
> Patch 1-3 look good.
> 
> On 11/25/2009 04:25 PM, Peter Jones wrote:
>> This brings up multipath devices which are configured in multipath.conf
>> as we find them.
>> ---
>>   modules.d/90multipath/14-multipath-scan.rules |    9 +++++++++
>>   modules.d/90multipath/install                 |    3 +++
>>   modules.d/90multipath/multipath-scan.sh       |   16 ++++++++++++++++
>>   3 files changed, 28 insertions(+), 0 deletions(-)
>>   create mode 100644 modules.d/90multipath/14-multipath-scan.rules
>>   create mode 100644 modules.d/90multipath/multipath-scan.sh
>>
>> diff --git a/modules.d/90multipath/14-multipath-scan.rules
>> b/modules.d/90multipath/14-multipath-scan.rules
>> new file mode 100644
>> index 0000000..b4acf57
>> --- /dev/null
>> +++ b/modules.d/90multipath/14-multipath-scan.rules
>> @@ -0,0 +1,9 @@
>> +# scan for multipath devices
>> +SUBSYSTEM!="block", GOTO="end_mpath_scan"
>> +KERNEL!="sd*", GOTO="end_mpath_scan"
>> +PROGRAM=="/bin/sh -c 'for i in //holders/dm-[0-9]*; do [ -e $$i ]&& 
>> exit 0; done; exit 1;' ", GOTO="end_mpath_scan"
>> +RUN+="/sbin/modprobe dm-multipath"
>> +RUN+="/bin/echo 'queuing multipath scan'"
>> +RUN+="/sbin/initqueue --settled --onetime --unique --name=mpscan
>> /sbin/multipath-scan"
>> +RUN+="/bin/sh -c '>/tmp/.multipath-scan-%k;'"
>> +LABEL="end_mpath_scan"
> 
> Hmm, I just realized something, this rule will keep on triggering if a
> disk is
> not part of a multipath setup. As in that case it will never become a
> holder
> of a device mapper device.
> 
> And since things like opening /dev/sda rw (which /sbin/multipath might
> very well do), trigger a change event, this feels like an infinite loop
> to me.

It does not trigger a change event.

> Maybe this rule should have a
> ACTION!="add", GOTO="end_mpath_scan"
> 
> ?
> 
> I think it would be best to discuss this with Harald.
> 
> Regards,
> 
> Hans


-- 
        Peter

"I can imagine a world without war, without hate. I can imagine us
attacking it, because they'd never expect it."

      parent reply	other threads:[~2009-11-25 15:52 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-11-24 19:52 Add support for bringing up multipath root devices Peter Jones
     [not found] ` <1259092378-14143-1-git-send-email-pjones-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2009-11-24 19:52   ` [PATCH 1/4] Use glob for libdir when installing multipath libraries Peter Jones
     [not found]     ` <1259092378-14143-2-git-send-email-pjones-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2009-11-24 19:55       ` Peter Jones
     [not found]         ` <4B0C3A19.3010509-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2009-11-24 19:59           ` Peter Jones
2009-11-24 19:52   ` [PATCH 2/4] Add #!/bin/bash for consistency Peter Jones
2009-11-24 19:52   ` [PATCH 3/4] Make sure and get all the multipath related modules Peter Jones
2009-11-24 19:52   ` [PATCH 4/4] Bring up multipath devices that are configured in multipath.conf Peter Jones
     [not found] ` <4B0C40BC.5000107@redhat.com>
     [not found]   ` <4B0C40BC.5000107-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2009-11-25 10:21     ` [PATCH 1/4] Use glob for libdir when installing multipath libraries Hans de Goede
     [not found]       ` <4B0D0531.2050401-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2009-11-25 15:10         ` Peter Jones
     [not found] ` <4B0C40E4.6080006@redhat.com>
     [not found]   ` <4B0C40E4.6080006-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2009-11-25 10:22     ` [PATCH 2/4] Add #!/bin/bash for consistency Hans de Goede
     [not found] ` <4B0C40C7.7010206@redhat.com>
     [not found]   ` <4B0C40C7.7010206-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2009-11-25 10:23     ` [PATCH 3/4] Make sure and get all the multipath related modules Hans de Goede
     [not found] ` <4B0C40ED.9000909@redhat.com>
     [not found]   ` <4B0C40ED.9000909-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2009-11-25 10:35     ` [PATCH 4/4] Bring up multipath devices that are configured in multipath.conf Hans de Goede
     [not found]       ` <4B0D087E.1030502-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2009-11-25 15:17         ` Peter Jones
     [not found]           ` <4B0D4A9F.1090606-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2009-11-25 15:25             ` Add support for bringing up multipath root devices [version 2] Peter Jones
2009-11-25 15:25             ` [PATCH 1/4] Use glob for libdir when installing multipath libraries Peter Jones
     [not found]               ` <1259162752-22372-2-git-send-email-pjones-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2009-11-25 15:52                 ` Peter Jones
2009-11-25 15:25             ` [PATCH 2/4] Add #!/bin/bash for consistency Peter Jones
2009-11-25 15:25             ` [PATCH 3/4] Make sure and get all the multipath related modules Peter Jones
2009-11-25 15:25             ` [PATCH 4/4] Bring up multipath devices that are configured in multipath.conf Peter Jones
     [not found]               ` <1259162752-22372-5-git-send-email-pjones-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2009-11-25 15:53                 ` Hans de Goede
     [not found]                   ` <4B0D5306.8050706-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2009-11-25 15:52                     ` Peter Jones [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4B0D52A4.6000105@redhat.com \
    --to=pjones-h+wxahxf7alqt0dzr+alfa@public.gmane.org \
    --cc=hdegoede-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \
    --cc=initramfs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox