linux-raid.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Luca Berra <bluca@comedia.it>
To: Linux RAID Mailing List <linux-raid@vger.kernel.org>
Subject: Re: [PATCH] enable auto=yes by default when using udev
Date: Tue, 4 Jul 2006 12:43:17 +0200	[thread overview]
Message-ID: <20060704104307.GB8357@percy.comedia.it> (raw)
In-Reply-To: <20060703224557.GB26362@percy.comedia.it>

On Tue, Jul 04, 2006 at 12:46:03AM +0200, Luca Berra wrote:
>On Mon, Jul 03, 2006 at 09:14:38AM +1000, Neil Brown wrote:
>>However....
>>
>>>+
>>>+	/* if we are using udev and auto is not set, mdadm will almost
>>>+	 * certainly fail, so we force it here.
>>>+	 */
>>>+	if (autof == 0 && access("/dev/.udevdb",F_OK) == 0)
>>>+		autof=2;
>>>+
>>
>>I'm worried that this test is not very robust.
>>On my Debian/unstable system running used, there is no
>>/dev/.udevdb
>>though there is a
>>/dev/.udev/db
>>
>>I guess I could test for both, but then udev might change
>>again.... I'd really like a more robust check.
>>
>is /dev/.udev/db a debianism?
no it is not
>in this case a check for both might suffice, else i will have to think
>harder about it.

-- 
Luca Berra -- bluca@comedia.it
        Communication Media & Services S.r.l.
 /"\
 \ /     ASCII RIBBON CAMPAIGN
  X        AGAINST HTML MAIL
 / \

  reply	other threads:[~2006-07-04 10:43 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-07-02 22:45 [PATCH] enable auto=yes by default when using udev Luca Berra
2006-07-02 23:14 ` Neil Brown
2006-07-02 23:29   ` Jason Lunz
2006-07-03  9:11     ` Mario 'BitKoenig' Holbe
2006-07-03 10:56   ` David Greaves
2006-07-03 11:13   ` Frank Blendinger
2006-07-03 22:46   ` Luca Berra
2006-07-04 10:43     ` Luca Berra [this message]
2006-07-04 10:29   ` Michael Tokarev
2006-07-04 10:47     ` Luca Berra
2006-07-04 12:19     ` Mario 'BitKoenig' Holbe
2006-07-18  5:19       ` Neil Brown
2006-07-18 10:07         ` Christian Pernegger
2006-07-17 19:42     ` Bill Davidsen

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=20060704104307.GB8357@percy.comedia.it \
    --to=bluca@comedia.it \
    --cc=linux-raid@vger.kernel.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;
as well as URLs for NNTP newsgroup(s).