lkml.org 
[lkml]   [2010]   [Oct]   [5]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [V9fs-developer] [PATCH 1/3] fs/9p: Update TLCREATE to allow create without open
On 10/5/2010 10:15 AM, Eric Van Hensbergen wrote:
> Ouch. Protocol change.
>
> Also - not sure I understand what's going on here. Why does mknod go
> through TLCREATE, shouldn't we have our
> own protocol messages for mknod?//
>
> Why add an additional flag field instead of just using the existing
> flags field since we are only talking about a bit?
>
> -eric
>
>
> On Tue, Oct 5, 2010 at 1:01 PM, Aneesh Kumar K.V
> <aneesh.kumar@linux.vnet.ibm.com> wrote:
>> We need this so that we can support regular file creation
>> with mknod
>>
>> Signed-off-by: Aneesh Kumar K.V<aneesh.kumar@linux.vnet.ibm.com>
>> ---
>> fs/9p/v9fs.h | 2 ++
>> fs/9p/vfs_inode.c | 13 +++++++------
>> include/net/9p/client.h | 2 +-
>> net/9p/client.c | 6 +++---
>> 4 files changed, 13 insertions(+), 10 deletions(-)
>>
>> diff --git a/fs/9p/v9fs.h b/fs/9p/v9fs.h
>> index 8bb7792..6209a29 100644
>> --- a/fs/9p/v9fs.h
>> +++ b/fs/9p/v9fs.h
>> @@ -126,6 +126,8 @@ void v9fs_session_begin_cancel(struct v9fs_session_info *v9ses);
>> #define V9FS_DEFUID (-2)
>> #define V9FS_DEFGID (-2)
>>
>> +#define P9_LOOKUP_OPEN 0x1
>> +
>> static inline struct v9fs_session_info *v9fs_inode2v9ses(struct inode *inode)
>> {
>> return (inode->i_sb->s_fs_info);
>> diff --git a/fs/9p/vfs_inode.c b/fs/9p/vfs_inode.c
>> index 2f904a8..1ec9075 100644
>> --- a/fs/9p/vfs_inode.c
>> +++ b/fs/9p/vfs_inode.c
>> @@ -668,8 +668,9 @@ v9fs_vfs_create_dotl(struct inode *dir, struct dentry *dentry, int omode,
>> int err = 0;
>> char *name = NULL;
>> gid_t gid;
>> - int flags;
>> + int flags = 0;
>> mode_t mode;
>> + int lookup_flags = 0;

I think we decided to send -1 with the flags instead of doing protocol change here.

Thanks,
JV

>> struct v9fs_session_info *v9ses;
>> struct p9_fid *fid = NULL;
>> struct p9_fid *dfid, *ofid;
>> @@ -679,11 +680,10 @@ v9fs_vfs_create_dotl(struct inode *dir, struct dentry *dentry, int omode,
>> struct posix_acl *pacl = NULL, *dacl = NULL;
>>
>> v9ses = v9fs_inode2v9ses(dir);
>> - if (nd&& nd->flags& LOOKUP_OPEN)
>> + if (nd&& nd->flags& LOOKUP_OPEN) {
>> flags = nd->intent.open.flags - 1;
>> - else
>> - flags = O_RDWR;
>> -
>> + lookup_flags = P9_LOOKUP_OPEN;
>> + }
>> name = (char *) dentry->d_name.name;
>> P9_DPRINTK(P9_DEBUG_VFS, "v9fs_vfs_create_dotl: name:%s flags:0x%x "
>> "mode:0x%x\n", name, flags, omode);
>> @@ -713,7 +713,8 @@ v9fs_vfs_create_dotl(struct inode *dir, struct dentry *dentry, int omode,
>> "Failed to get acl values in creat %d\n", err);
>> goto error;
>> }
>> - err = p9_client_create_dotl(ofid, name, flags, mode, gid,&qid);
>> + err = p9_client_create_dotl(ofid, name, flags, mode,
>> + gid, lookup_flags,&qid);
>> if (err< 0) {
>> P9_DPRINTK(P9_DEBUG_VFS,
>> "p9_client_open_dotl failed in creat %d\n",
>> diff --git a/include/net/9p/client.h b/include/net/9p/client.h
>> index fb358f5..b0b97ec 100644
>> --- a/include/net/9p/client.h
>> +++ b/include/net/9p/client.h
>> @@ -230,7 +230,7 @@ int p9_client_link(struct p9_fid *fid, struct p9_fid *oldfid, char *newname);
>> int p9_client_symlink(struct p9_fid *fid, char *name, char *symname, gid_t gid,
>> struct p9_qid *qid);
>> int p9_client_create_dotl(struct p9_fid *ofid, char *name, u32 flags, u32 mode,
>> - gid_t gid, struct p9_qid *qid);
>> + gid_t gid, int lookup_flags, struct p9_qid *qid);
>> int p9_client_clunk(struct p9_fid *fid);
>> int p9_client_remove(struct p9_fid *fid);
>> int p9_client_read(struct p9_fid *fid, char *data, char __user *udata,
>> diff --git a/net/9p/client.c b/net/9p/client.c
>> index 4b90b82..9950275 100644
>> --- a/net/9p/client.c
>> +++ b/net/9p/client.c
>> @@ -1070,7 +1070,7 @@ error:
>> EXPORT_SYMBOL(p9_client_open);
>>
>> int p9_client_create_dotl(struct p9_fid *ofid, char *name, u32 flags, u32 mode,
>> - gid_t gid, struct p9_qid *qid)
>> + gid_t gid, int lookup_flags, struct p9_qid *qid)
>> {
>> int err = 0;
>> struct p9_client *clnt;
>> @@ -1085,8 +1085,8 @@ int p9_client_create_dotl(struct p9_fid *ofid, char *name, u32 flags, u32 mode,
>> if (ofid->mode != -1)
>> return -EINVAL;
>>
>> - req = p9_client_rpc(clnt, P9_TLCREATE, "dsddd", ofid->fid, name, flags,
>> - mode, gid);
>> + req = p9_client_rpc(clnt, P9_TLCREATE, "dsdddd", ofid->fid, name, flags,
>> + mode, gid, lookup_flags);
>> if (IS_ERR(req)) {
>> err = PTR_ERR(req);
>> goto error;
>> --
>> 1.7.0.4
>>
>>
>> ------------------------------------------------------------------------------
>> Beautiful is writing same markup. Internet Explorer 9 supports
>> standards for HTML5, CSS3, SVG 1.1, ECMAScript5, and DOM L2& L3.
>> Spend less time writing and rewriting code and more time creating great
>> experiences on the web. Be a part of the beta today.
>> http://p.sf.net/sfu/beautyoftheweb
>> _______________________________________________
>> V9fs-developer mailing list
>> V9fs-developer@lists.sourceforge.net
>> https://lists.sourceforge.net/lists/listinfo/v9fs-developer
>>
>
> ------------------------------------------------------------------------------
> Beautiful is writing same markup. Internet Explorer 9 supports
> standards for HTML5, CSS3, SVG 1.1, ECMAScript5, and DOM L2& L3.
> Spend less time writing and rewriting code and more time creating great
> experiences on the web. Be a part of the beta today.
> http://p.sf.net/sfu/beautyoftheweb
> _______________________________________________
> V9fs-developer mailing list
> V9fs-developer@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/v9fs-developer




\
 
 \ /
  Last update: 2010-10-05 20:23    [W:0.158 / U:0.092 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site