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-fsdevel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
