On Tue, Dec 21, 2010 at 9:55 AM, Nick Piggin <npiggin@xxxxxxxxx> wrote: > On Tue, Dec 21, 2010 at 09:44:40AM +0100, Sedat Dilek wrote: >> Against linux-next (next-20101210) it should look like: > > Yep, I'll rebase my tree with the fix shortly. > >> >> $ 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 - > I just started the compilation due to slow kernel mirrors, I see this warning: $ egrep 'kernel/cgroup.o|kernel/cgroup.c' build_linux-next_next20101221.dileks.1.log CC kernel/cgroup.o /home/sd/src/linux-2.6/linux-2.6.37-rc6/debian/build/source_i386_none/kernel/cgroup.c: In function âcgroup_lookupâ: /home/sd/src/linux-2.6/linux-2.6.37-rc6/debian/build/source_i386_none/kernel/cgroup.c:2219:3: warning: initialization from incompatible pointer type [ kernel/cgroup.c ] ... static int cgroup_delete_dentry(struct dentry *dentry) { return 1; } static struct dentry *cgroup_lookup(struct inode *dir, struct dentry *dentry, struct nameidata *nd) { static const struct dentry_operations cgroup_dentry_operations = { .d_delete = cgroup_delete_dentry, <--- LINE #2219 .d_iput = cgroup_diput, }; if (dentry->d_name.len > NAME_MAX) return ERR_PTR(-ENAMETOOLONG); dentry->d_op = &cgroup_dentry_operations; d_add(dentry, NULL); return NULL; } ... Can you fix that with your rebasing, too? Thanks in advance. - Sedat - -- To unsubscribe from this list: send the line "unsubscribe linux-next" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html