All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mikhail Durnev <Mikhail_Durnev@mentor.com>
To: 'Khem Raj' <raj.khem@gmail.com>, "'Noor, Ahsan'" <noor_ahsan@mentor.com>
Cc: yocto@yoctoproject.org
Subject: Re: [PATCH] ecryptfs-utils: Modify systemd service file to 'simple'.
Date: Thu, 20 Jun 2013 15:17:03 +1100	[thread overview]
Message-ID: <119b01ce6d6d$06c6ae70$14540b50$@com> (raw)
In-Reply-To: <746BB1A1-A592-4FFB-9A3F-B9D6C904451A@gmail.com>

ecryptfsd is a resident program, i.e. daemon. According to systemd
documentation, Type=oneshot/RemainAfterExit=yes should be used for programs
that do not remain working after exit. But for daemons we should use
Type=forked to indicate that the service remains running in background.
ecryptfs provides option -f to run in foreground. This option is preferred
when the service is started from init/systemd. That is why we use
Type=simple (default) to indicate that the service runs in foreground. To
manage the service properly systemd uses its type. E.g. if ecryptfsd
silently dies, systemd will recognize its failure in case of simple or
forked, but not in case of oneshot.

Best regards,
Mikhail

-----Original Message-----
From: Khem Raj [mailto:raj.khem@gmail.com] 
Sent: Thursday, 20 June, 2013 13:32
To: Noor, Ahsan
Cc: holger.behrens@windriver.com; florin.sarbu@windriver.com;
yocto@yoctoproject.org; Mikhail Durnev
Subject: Re: [yocto] [PATCH] ecryptfs-utils: Modify systemd service file to
'simple'.

reasoning why it is being turned into simple service would be nice

On Jun 19, 2013, at 3:53 AM, "Noor, Ahsan" <noor_ahsan@mentor.com> wrote:

> From: Noor <noor_ahsan@mentor.com>
> 
> Signed-off-by: Mikhail Durnev <mikhail_durnev@mentor.com>
> Signed-off-by: Noor Ahsan <noor_ahsan@mentor.com>
> ---
> .../ecryptfs-utils/ecryptfs-utils/ecryptfs.service |    4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
> 
> diff --git 
> a/recipes-support-ivi/ecryptfs-utils/ecryptfs-utils/ecryptfs.service 
> b/recipes-support-ivi/ecryptfs-utils/ecryptfs-utils/ecryptfs.service
> index ba12aa3..52f3397 100644
> --- 
> a/recipes-support-ivi/ecryptfs-utils/ecryptfs-utils/ecryptfs.service
> +++ b/recipes-support-ivi/ecryptfs-utils/ecryptfs-utils/ecryptfs.servi
> +++ ce
> @@ -3,9 +3,7 @@ Description=A userspace daemon that runs as the user 
> perform file operations und After=udev.service
> 
> [Service]
> -Type=oneshot
> -ExecStart=/usr/bin/ecryptfsd
> -RemainAfterExit=yes
> +ExecStart=/usr/bin/ecryptfsd -f
> 
> [Install]
> WantedBy=multi-user.target
> --
> 1.7.9.5
> 
> _______________________________________________
> yocto mailing list
> yocto@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/yocto





  reply	other threads:[~2013-06-20  4:17 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-06-19 10:53 [PATCH] ecryptfs-utils: Modify systemd service file to 'simple' Noor, Ahsan
2013-06-20  2:31 ` Khem Raj
2013-06-20  4:17   ` Mikhail Durnev [this message]
2013-06-20  4:49     ` Khem Raj

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='119b01ce6d6d$06c6ae70$14540b50$@com' \
    --to=mikhail_durnev@mentor.com \
    --cc=noor_ahsan@mentor.com \
    --cc=raj.khem@gmail.com \
    --cc=yocto@yoctoproject.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 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.