All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jes Sorensen <Jes.Sorensen@redhat.com>
To: NeilBrown <neilb@suse.de>
Cc: linux-raid@vger.kernel.org, dledford@redhat.com, harald@redhat.com
Subject: Re: [PATCH 0/2] add support for launching mdmon via systemd
Date: Fri, 18 Jan 2013 10:09:06 +0100	[thread overview]
Message-ID: <wrfjfw1yyi19.fsf@redhat.com> (raw)
In-Reply-To: <20130118200232.59d328da@notabene.brown> (NeilBrown's message of "Fri, 18 Jan 2013 20:02:32 +1100")

NeilBrown <neilb@suse.de> writes:
> On Fri, 18 Jan 2013 09:43:10 +0100 Jes.Sorensen@redhat.com wrote:
>
>> From: Jes Sorensen <Jes.Sorensen@redhat.com>
>> 
>> Hi,
>> 
>> It's been an ongoing issue for a while trying to figure out how to
>> handle launching mdmon from the initrd in Fedora and avoid it getting
>> killed by systemd due to it ending up in the wrong cgroup (udev).
>> 
>> We tried simply attaching mdmon to the cgroup of systemd but it didn't
>> really work, and in the end it seems to be more in line with the
>> systemd mentality to have systemd spawn it. With these patches and a
>> few pending changes to dracut, I am finally able to boot and reboot
>> systems with 1 or 2 IMSM arrays.
>> 
>> The patch for systemctl is implemented as a compile time option rather
>> than a command line option, since I think it is something that the
>> distro will decide. However we can make it a command line option if
>> preferred.
>> 
>> I also have matching systemd .service files for this, which I am happy
>> to contribute, but I am not sure if they are useful to other distros
>> or not? We haven't had a history of including .service files with
>> mdadm.
>
> Thanks.
> I'll have a proper look next week, however:
>  - I would rather it was a run-time option, not compile time.
>    I believe in giving choice to the admin.
>  - I would like to include the .service files.  I don't know if we can get
>    all distros to use the same one, but we certainly won't if we don't try.

Ok, I'll adapt it as a runtime option and also include the .service
files with the patch.

Cheers,
Jes

      reply	other threads:[~2013-01-18  9:09 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-01-18  8:43 [PATCH 0/2] add support for launching mdmon via systemd Jes.Sorensen
2013-01-18  8:43 ` [PATCH 1/2] Add support for launching mdmon via systemctl instead of fork/exec Jes.Sorensen
2013-01-18  8:43 ` [PATCH 2/2] execl() only returns in case of error Jes.Sorensen
2013-01-18  9:02 ` [PATCH 0/2] add support for launching mdmon via systemd NeilBrown
2013-01-18  9:09   ` Jes Sorensen [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=wrfjfw1yyi19.fsf@redhat.com \
    --to=jes.sorensen@redhat.com \
    --cc=dledford@redhat.com \
    --cc=harald@redhat.com \
    --cc=linux-raid@vger.kernel.org \
    --cc=neilb@suse.de \
    /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 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.