[Devel] Re: [RFC v14-rc][PATCH 14/23] A new file type (CR_FD_OBJREF) for a file descriptor already setup

Oren Laadan orenl at cs.columbia.edu
Fri Mar 20 14:02:41 PDT 2009



Dave Hansen wrote:
> On Fri, 2009-03-20 at 14:47 -0400, Oren Laadan wrote:
>>
>> +/* cr_write_fd_file - for regular files, directories, symbolic links
>> */
>> +static int cr_write_fd_file(struct cr_ctx *ctx, struct file *file)
>> +{
>> +       return cr_write_fname(ctx, &file->f_path, &ctx->fs_mnt);
>> +}
>> +
>> +/* cr_inode_to_fdtype - determine the fd type given an inode */
>> +static inline enum fd_type cr_inode_to_fdtype(struct inode *inode)
>> +{
>> +       switch (inode->i_mode & S_IFMT) {
>> +       case S_IFREG:
>> +               return CR_FD_FILE;
>> +       case S_IFDIR:
>> +               return CR_FD_DIR;
>> +       }
>> +       /* file type unsupported */
>> +       return -EBADF;
>> +}
> 
> These are cleanups and abstractions of a previous patch.  Please either
> put them in that patch or separate them from the CR_FD_OBJREF patch.
> They don't belong together.
> 
> -- Dave

Sounds good. Will split.

Oren.


_______________________________________________
Containers mailing list
Containers at lists.linux-foundation.org
https://lists.linux-foundation.org/mailman/listinfo/containers




More information about the Devel mailing list