[systemd-devel] [patch] cgroup fs: avoid switching ->d_op on live dentry

Sedat Dilek sedat.dilek at googlemail.com
Tue Dec 21 07:12:03 PST 2010


On Tue, Dec 21, 2010 at 9:44 AM, Sedat Dilek <sedat.dilek at googlemail.com> wrote:
> Against linux-next (next-20101210) it should look like:
>
> $ diff -Naur cgroup-fix/cgroup-fs-avoid-switching-d_op-on-live-dentry.patch
> cgroup-fix/cgroup-fs-avoid-switching-d_op-on-live-dentry-v2.patch
> --- cgroup-fix/cgroup-fs-avoid-switching-d_op-on-live-dentry.patch
>  2010-12-21 09:31:38.649601964 +0100
> +++ cgroup-fix/cgroup-fs-avoid-switching-d_op-on-live-dentry-v2.patch
>  2010-12-21 09:40:21.151033232 +0100
> @@ -83,7 +83,7 @@
>                inode->i_size = 0;
>                inode->i_fop = &cgroup_file_operations;
>        }
> --      dentry->d_op = &cgroup_dops;
> +-      d_set_d_op(dentry, &cgroup_dops);
>        d_instantiate(dentry, inode);
>        dget(dentry);   /* Extra count - pin the dentry in core */
>        return 0;
>
> - Sedat -
>

YAY!!!

# dmesg | grep systemd
[    0.000000] Kernel command line:
BOOT_IMAGE=/boot/vmlinuz-2.6.37-rc6-686
root=UUID=1ceb69a7-ecf4-47e9-a231-b74e0f0a9b62 ro ini
[    4.124563] systemd[1]: systemd 15 running in system mode. (+PAM
-LIBWRAP -AUDIT +SELINUX +SYSVINIT -LIBCRYPTSETUP; debian)
[    4.246090] systemd[1]: Set hostname to <tbox>.
[    6.942949] systemd-logger[290]: Got error on stream: No such
process
[    9.011378] systemd-fsck[524]: /dev/sda3: sauber, 128062/640848
Dateien, 1839774/2560359 Blöcke
[   15.186112] systemd[1]: ifupdown-clean.service: control process
exited, code=exited status=209
[   15.186941] systemd[1]: Unit ifupdown-clean.service entered failed state.
[   15.187363] systemd[1]: mountoverflowtmp.service: control process
exited, code=exited status=209
[   15.206223] systemd[1]: Unit mountoverflowtmp.service entered failed state.
[   15.215739] systemd[1]: resolvconf.service: control process exited,
code=exited status=209
[   15.235257] systemd[1]: Unit resolvconf.service entered failed state.

Feel free to add:

       Reported-and-tested-by: Sedat Dilek <sedat.dilek at gmail.com>

(Tested with the above v2 patch against linux-next (next-20101221) -
see file attachment).

- Sedat -
-------------- next part --------------
A non-text attachment was scrubbed...
Name: cgroup-fs-avoid-switching-d_op-on-live-dentry-v2.patch
Type: plain/text
Size: 3276 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/systemd-devel/attachments/20101221/57b14755/attachment.bin>


More information about the systemd-devel mailing list