[Tux3] Further sync for inode.c
OGAWA Hirofumi
hirofumi at mail.parknet.co.jp
Thu Nov 20 16:24:32 PST 2008
Daniel Phillips <phillips at phunq.net> writes:
>> --- tux3fs/fs/tux3/inode.c.orig 2008-11-21 08:01:31.000000000 +0900
>> +++ tux3fs/fs/tux3/inode.c 2008-11-21 08:08:22.000000000 +0900
>> @@ -194,7 +194,7 @@ int save_inode(struct inode *inode)
>> int tuxio(struct file *file, char *data, unsigned len, int write)
>> {
>> int err = 0;
>> - struct inode *inode = file->f_inode;
>> + struct inode *inode = file->f_dentry->d_inode;
>> loff_t pos = file->f_pos;
>> trace("%s %u bytes at %Lu, isize = 0x%Lx", write ? "write" : "read", len, (L)pos, (L)inode->i_size);
>> if (write && pos + len > MAX_FILESIZE)
>>
>> This functions may rewrite in kernel completely, I'm not sure. Well,
>> this difference wouldn't matter at least for now.
>
> Tuxio can go inside #ifndef __KERNEL__. Tuxio is the userspace version
> of generic file read/write in kernel, but we are going to use the generic_*
> functions for now, and maybe forever as they evolve and develop better
> interfaces.
Ok, I'll move it to inside __KERNEL__.
> Is there are reason to change tux_path[] to *tux_path?
We are allocate it by alloc_path(), so it's not array actually now.
--
OGAWA Hirofumi <hirofumi at mail.parknet.co.jp>
_______________________________________________
Tux3 mailing list
Tux3 at tux3.org
http://tux3.org/cgi-bin/mailman/listinfo/tux3
More information about the Tux3
mailing list