All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] ecryptfs-utils: Modify systemd service file to 'simple'.
@ 2013-06-19 10:53 Noor, Ahsan
  2013-06-20  2:31 ` Khem Raj
  0 siblings, 1 reply; 4+ messages in thread
From: Noor, Ahsan @ 2013-06-19 10:53 UTC (permalink / raw)
  To: holger.behrens, florin.sarbu; +Cc: yocto, Mikhail Durnev

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.service
@@ -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



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

* Re: [PATCH] ecryptfs-utils: Modify systemd service file to 'simple'.
  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
  0 siblings, 1 reply; 4+ messages in thread
From: Khem Raj @ 2013-06-20  2:31 UTC (permalink / raw)
  To: Noor, Ahsan; +Cc: yocto, Mikhail Durnev

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.service
> @@ -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



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

* Re: [PATCH] ecryptfs-utils: Modify systemd service file to 'simple'.
  2013-06-20  2:31 ` Khem Raj
@ 2013-06-20  4:17   ` Mikhail Durnev
  2013-06-20  4:49     ` Khem Raj
  0 siblings, 1 reply; 4+ messages in thread
From: Mikhail Durnev @ 2013-06-20  4:17 UTC (permalink / raw)
  To: 'Khem Raj', 'Noor, Ahsan'; +Cc: yocto

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





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

* Re: [PATCH] ecryptfs-utils: Modify systemd service file to 'simple'.
  2013-06-20  4:17   ` Mikhail Durnev
@ 2013-06-20  4:49     ` Khem Raj
  0 siblings, 0 replies; 4+ messages in thread
From: Khem Raj @ 2013-06-20  4:49 UTC (permalink / raw)
  To: Mikhail Durnev; +Cc: yocto@yoctoproject.org

On Wed, Jun 19, 2013 at 9:17 PM, Mikhail Durnev
<Mikhail_Durnev@mentor.com> wrote:
> 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.

thanks for details. Please add this to commit message.


>
> 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
>
>
>


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

end of thread, other threads:[~2013-06-20  4:49 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
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
2013-06-20  4:49     ` Khem Raj

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.