[systemd-devel] [PATCH] core: Don't require cgroups xattr support

[Cristian Rodríguez]

El 26/06/14 01:58, Zbigniew Jędrzejewski-Szmek escribió:
> On Wed, Jun 25, 2014 at 11:45:45AM +0000, Tom Hirst wrote:
>> Failure to mount cgroups with xattr should not be fatal
>> ---
>>  src/core/mount-setup.c | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/src/core/mount-setup.c b/src/core/mount-setup.c
>> index 991bfdf..206f89a 100644
>> --- a/src/core/mount-setup.c
>> +++ b/src/core/mount-setup.c
>> @@ -94,7 +94,7 @@ static const MountPoint mount_table[] = {
>>          { "tmpfs",      "/sys/fs/cgroup",            "tmpfs",      "mode=755", MS_NOSUID|MS_NOEXEC|MS_NODEV|MS_STRICTATIME,
>>            NULL,       MNT_FATAL|MNT_IN_CONTAINER },
>>          { "cgroup",     "/sys/fs/cgroup/systemd",    "cgroup",     "none,name=systemd,xattr", MS_NOSUID|MS_NOEXEC|MS_NODEV,
>> -          NULL,       MNT_FATAL|MNT_IN_CONTAINER },
>> +          NULL,       MNT_IN_CONTAINER },
>>          { "cgroup",     "/sys/fs/cgroup/systemd",    "cgroup",     "none,name=systemd", MS_NOSUID|MS_NOEXEC|MS_NODEV,
>>            NULL,       MNT_FATAL|MNT_IN_CONTAINER },
>>          { "pstore",     "/sys/fs/pstore",            "pstore",     NULL, MS_NOSUID|MS_NOEXEC|MS_NODEV,
> 
> Applied.


Btw..Adding xattr as a mount option is no longer required, it is the
default since cgroups were converted to kernfs.


-- 
Cristian
"I don't know the key to success, but the key to failure is trying to
please everybody."