diff options
Diffstat (limited to 'fs/9p')
-rw-r--r-- | fs/9p/9p.c | 314 | ||||
-rw-r--r-- | fs/9p/9p.h | 77 | ||||
-rw-r--r-- | fs/9p/Makefile | 10 | ||||
-rw-r--r-- | fs/9p/conv.c | 884 | ||||
-rw-r--r-- | fs/9p/conv.h | 35 | ||||
-rw-r--r-- | fs/9p/debug.h | 23 | ||||
-rw-r--r-- | fs/9p/error.c | 10 | ||||
-rw-r--r-- | fs/9p/error.h | 2 | ||||
-rw-r--r-- | fs/9p/fid.c | 5 | ||||
-rw-r--r-- | fs/9p/mux.c | 1145 | ||||
-rw-r--r-- | fs/9p/mux.h | 41 | ||||
-rw-r--r-- | fs/9p/trans_fd.c | 53 | ||||
-rw-r--r-- | fs/9p/trans_sock.c | 161 | ||||
-rw-r--r-- | fs/9p/transport.h | 4 | ||||
-rw-r--r-- | fs/9p/v9fs.c | 59 | ||||
-rw-r--r-- | fs/9p/v9fs.h | 17 | ||||
-rw-r--r-- | fs/9p/v9fs_vfs.h | 5 | ||||
-rw-r--r-- | fs/9p/vfs_dentry.c | 15 | ||||
-rw-r--r-- | fs/9p/vfs_dir.c | 47 | ||||
-rw-r--r-- | fs/9p/vfs_file.c | 28 | ||||
-rw-r--r-- | fs/9p/vfs_inode.c | 619 | ||||
-rw-r--r-- | fs/9p/vfs_super.c | 14 |
22 files changed, 2100 insertions, 1468 deletions
diff --git a/fs/9p/9p.c b/fs/9p/9p.c index e847f504a47c..1a6d08761f39 100644 --- a/fs/9p/9p.c +++ b/fs/9p/9p.c | |||
@@ -1,8 +1,9 @@ | |||
1 | /* | 1 | /* |
2 | * linux/fs/9p/9p.c | 2 | * linux/fs/9p/9p.c |
3 | * | 3 | * |
4 | * This file contains functions 9P2000 functions | 4 | * This file contains functions to perform synchronous 9P calls |
5 | * | 5 | * |
6 | * Copyright (C) 2004 by Latchesar Ionkov <lucho@ionkov.net> | ||
6 | * Copyright (C) 2004 by Eric Van Hensbergen <ericvh@gmail.com> | 7 | * Copyright (C) 2004 by Eric Van Hensbergen <ericvh@gmail.com> |
7 | * Copyright (C) 2002 by Ron Minnich <rminnich@lanl.gov> | 8 | * Copyright (C) 2002 by Ron Minnich <rminnich@lanl.gov> |
8 | * | 9 | * |
@@ -33,6 +34,7 @@ | |||
33 | #include "debug.h" | 34 | #include "debug.h" |
34 | #include "v9fs.h" | 35 | #include "v9fs.h" |
35 | #include "9p.h" | 36 | #include "9p.h" |
37 | #include "conv.h" | ||
36 | #include "mux.h" | 38 | #include "mux.h" |
37 | 39 | ||
38 | /** | 40 | /** |
@@ -46,16 +48,21 @@ | |||
46 | 48 | ||
47 | int | 49 | int |
48 | v9fs_t_version(struct v9fs_session_info *v9ses, u32 msize, | 50 | v9fs_t_version(struct v9fs_session_info *v9ses, u32 msize, |
49 | char *version, struct v9fs_fcall **fcall) | 51 | char *version, struct v9fs_fcall **rcp) |
50 | { | 52 | { |
51 | struct v9fs_fcall msg; | 53 | int ret; |
54 | struct v9fs_fcall *tc; | ||
52 | 55 | ||
53 | dprintk(DEBUG_9P, "msize: %d version: %s\n", msize, version); | 56 | dprintk(DEBUG_9P, "msize: %d version: %s\n", msize, version); |
54 | msg.id = TVERSION; | 57 | tc = v9fs_create_tversion(msize, version); |
55 | msg.params.tversion.msize = msize; | ||
56 | msg.params.tversion.version = version; | ||
57 | 58 | ||
58 | return v9fs_mux_rpc(v9ses, &msg, fcall); | 59 | if (!IS_ERR(tc)) { |
60 | ret = v9fs_mux_rpc(v9ses->mux, tc, rcp); | ||
61 | kfree(tc); | ||
62 | } else | ||
63 | ret = PTR_ERR(tc); | ||
64 | |||
65 | return ret; | ||
59 | } | 66 | } |
60 | 67 | ||
61 | /** | 68 | /** |
@@ -71,19 +78,45 @@ v9fs_t_version(struct v9fs_session_info *v9ses, u32 msize, | |||
71 | 78 | ||
72 | int | 79 | int |
73 | v9fs_t_attach(struct v9fs_session_info *v9ses, char *uname, char *aname, | 80 | v9fs_t_attach(struct v9fs_session_info *v9ses, char *uname, char *aname, |
74 | u32 fid, u32 afid, struct v9fs_fcall **fcall) | 81 | u32 fid, u32 afid, struct v9fs_fcall **rcp) |
75 | { | 82 | { |
76 | struct v9fs_fcall msg; | 83 | int ret; |
84 | struct v9fs_fcall* tc; | ||
77 | 85 | ||
78 | dprintk(DEBUG_9P, "uname '%s' aname '%s' fid %d afid %d\n", uname, | 86 | dprintk(DEBUG_9P, "uname '%s' aname '%s' fid %d afid %d\n", uname, |
79 | aname, fid, afid); | 87 | aname, fid, afid); |
80 | msg.id = TATTACH; | ||
81 | msg.params.tattach.fid = fid; | ||
82 | msg.params.tattach.afid = afid; | ||
83 | msg.params.tattach.uname = uname; | ||
84 | msg.params.tattach.aname = aname; | ||
85 | 88 | ||
86 | return v9fs_mux_rpc(v9ses, &msg, fcall); | 89 | tc = v9fs_create_tattach(fid, afid, uname, aname); |
90 | if (!IS_ERR(tc)) { | ||
91 | ret = v9fs_mux_rpc(v9ses->mux, tc, rcp); | ||
92 | kfree(tc); | ||
93 | } else | ||
94 | ret = PTR_ERR(tc); | ||
95 | |||
96 | return ret; | ||
97 | } | ||
98 | |||
99 | static void v9fs_t_clunk_cb(void *a, struct v9fs_fcall *tc, | ||
100 | struct v9fs_fcall *rc, int err) | ||
101 | { | ||
102 | int fid; | ||
103 | struct v9fs_session_info *v9ses; | ||
104 | |||
105 | if (err) | ||
106 | return; | ||
107 | |||
108 | fid = tc->params.tclunk.fid; | ||
109 | kfree(tc); | ||
110 | |||
111 | if (!rc) | ||
112 | return; | ||
113 | |||
114 | dprintk(DEBUG_9P, "tcall id %d rcall id %d\n", tc->id, rc->id); | ||
115 | v9ses = a; | ||
116 | if (rc->id == RCLUNK) | ||
117 | v9fs_put_idpool(fid, &v9ses->fidpool); | ||
118 | |||
119 | kfree(rc); | ||
87 | } | 120 | } |
88 | 121 | ||
89 | /** | 122 | /** |
@@ -95,16 +128,25 @@ v9fs_t_attach(struct v9fs_session_info *v9ses, char *uname, char *aname, | |||
95 | */ | 128 | */ |
96 | 129 | ||
97 | int | 130 | int |
98 | v9fs_t_clunk(struct v9fs_session_info *v9ses, u32 fid, | 131 | v9fs_t_clunk(struct v9fs_session_info *v9ses, u32 fid) |
99 | struct v9fs_fcall **fcall) | ||
100 | { | 132 | { |
101 | struct v9fs_fcall msg; | 133 | int ret; |
134 | struct v9fs_fcall *tc, *rc; | ||
102 | 135 | ||
103 | dprintk(DEBUG_9P, "fid %d\n", fid); | 136 | dprintk(DEBUG_9P, "fid %d\n", fid); |
104 | msg.id = TCLUNK; | ||
105 | msg.params.tclunk.fid = fid; | ||
106 | 137 | ||
107 | return v9fs_mux_rpc(v9ses, &msg, fcall); | 138 | rc = NULL; |
139 | tc = v9fs_create_tclunk(fid); | ||
140 | if (!IS_ERR(tc)) | ||
141 | ret = v9fs_mux_rpc(v9ses->mux, tc, &rc); | ||
142 | else | ||
143 | ret = PTR_ERR(tc); | ||
144 | |||
145 | if (ret) | ||
146 | dprintk(DEBUG_ERROR, "failed fid %d err %d\n", fid, ret); | ||
147 | |||
148 | v9fs_t_clunk_cb(v9ses, tc, rc, ret); | ||
149 | return ret; | ||
108 | } | 150 | } |
109 | 151 | ||
110 | /** | 152 | /** |
@@ -114,14 +156,21 @@ v9fs_t_clunk(struct v9fs_session_info *v9ses, u32 fid, | |||
114 | * | 156 | * |
115 | */ | 157 | */ |
116 | 158 | ||
117 | int v9fs_t_flush(struct v9fs_session_info *v9ses, u16 tag) | 159 | int v9fs_t_flush(struct v9fs_session_info *v9ses, u16 oldtag) |
118 | { | 160 | { |
119 | struct v9fs_fcall msg; | 161 | int ret; |
162 | struct v9fs_fcall *tc; | ||
163 | |||
164 | dprintk(DEBUG_9P, "oldtag %d\n", oldtag); | ||
165 | |||
166 | tc = v9fs_create_tflush(oldtag); | ||
167 | if (!IS_ERR(tc)) { | ||
168 | ret = v9fs_mux_rpc(v9ses->mux, tc, NULL); | ||
169 | kfree(tc); | ||
170 | } else | ||
171 | ret = PTR_ERR(tc); | ||
120 | 172 | ||
121 | dprintk(DEBUG_9P, "oldtag %d\n", tag); | 173 | return ret; |
122 | msg.id = TFLUSH; | ||
123 | msg.params.tflush.oldtag = tag; | ||
124 | return v9fs_mux_rpc(v9ses, &msg, NULL); | ||
125 | } | 174 | } |
126 | 175 | ||
127 | /** | 176 | /** |
@@ -133,17 +182,22 @@ int v9fs_t_flush(struct v9fs_session_info *v9ses, u16 tag) | |||
133 | */ | 182 | */ |
134 | 183 | ||
135 | int | 184 | int |
136 | v9fs_t_stat(struct v9fs_session_info *v9ses, u32 fid, struct v9fs_fcall **fcall) | 185 | v9fs_t_stat(struct v9fs_session_info *v9ses, u32 fid, struct v9fs_fcall **rcp) |
137 | { | 186 | { |
138 | struct v9fs_fcall msg; | 187 | int ret; |
188 | struct v9fs_fcall *tc; | ||
139 | 189 | ||
140 | dprintk(DEBUG_9P, "fid %d\n", fid); | 190 | dprintk(DEBUG_9P, "fid %d\n", fid); |
141 | if (fcall) | ||
142 | *fcall = NULL; | ||
143 | 191 | ||
144 | msg.id = TSTAT; | 192 | ret = -ENOMEM; |
145 | msg.params.tstat.fid = fid; | 193 | tc = v9fs_create_tstat(fid); |
146 | return v9fs_mux_rpc(v9ses, &msg, fcall); | 194 | if (!IS_ERR(tc)) { |
195 | ret = v9fs_mux_rpc(v9ses->mux, tc, rcp); | ||
196 | kfree(tc); | ||
197 | } else | ||
198 | ret = PTR_ERR(tc); | ||
199 | |||
200 | return ret; | ||
147 | } | 201 | } |
148 | 202 | ||
149 | /** | 203 | /** |
@@ -157,16 +211,21 @@ v9fs_t_stat(struct v9fs_session_info *v9ses, u32 fid, struct v9fs_fcall **fcall) | |||
157 | 211 | ||
158 | int | 212 | int |
159 | v9fs_t_wstat(struct v9fs_session_info *v9ses, u32 fid, | 213 | v9fs_t_wstat(struct v9fs_session_info *v9ses, u32 fid, |
160 | struct v9fs_stat *stat, struct v9fs_fcall **fcall) | 214 | struct v9fs_wstat *wstat, struct v9fs_fcall **rcp) |
161 | { | 215 | { |
162 | struct v9fs_fcall msg; | 216 | int ret; |
217 | struct v9fs_fcall *tc; | ||
218 | |||
219 | dprintk(DEBUG_9P, "fid %d\n", fid); | ||
163 | 220 | ||
164 | dprintk(DEBUG_9P, "fid %d length %d\n", fid, (int)stat->length); | 221 | tc = v9fs_create_twstat(fid, wstat, v9ses->extended); |
165 | msg.id = TWSTAT; | 222 | if (!IS_ERR(tc)) { |
166 | msg.params.twstat.fid = fid; | 223 | ret = v9fs_mux_rpc(v9ses->mux, tc, rcp); |
167 | msg.params.twstat.stat = stat; | 224 | kfree(tc); |
225 | } else | ||
226 | ret = PTR_ERR(tc); | ||
168 | 227 | ||
169 | return v9fs_mux_rpc(v9ses, &msg, fcall); | 228 | return ret; |
170 | } | 229 | } |
171 | 230 | ||
172 | /** | 231 | /** |
@@ -183,23 +242,27 @@ v9fs_t_wstat(struct v9fs_session_info *v9ses, u32 fid, | |||
183 | 242 | ||
184 | int | 243 | int |
185 | v9fs_t_walk(struct v9fs_session_info *v9ses, u32 fid, u32 newfid, | 244 | v9fs_t_walk(struct v9fs_session_info *v9ses, u32 fid, u32 newfid, |
186 | char *name, struct v9fs_fcall **fcall) | 245 | char *name, struct v9fs_fcall **rcp) |
187 | { | 246 | { |
188 | struct v9fs_fcall msg; | 247 | int ret; |
248 | struct v9fs_fcall *tc; | ||
249 | int nwname; | ||
189 | 250 | ||
190 | dprintk(DEBUG_9P, "fid %d newfid %d wname '%s'\n", fid, newfid, name); | 251 | dprintk(DEBUG_9P, "fid %d newfid %d wname '%s'\n", fid, newfid, name); |
191 | msg.id = TWALK; | 252 | |
192 | msg.params.twalk.fid = fid; | 253 | if (name) |
193 | msg.params.twalk.newfid = newfid; | 254 | nwname = 1; |
194 | 255 | else | |
195 | if (name) { | 256 | nwname = 0; |
196 | msg.params.twalk.nwname = 1; | 257 | |
197 | msg.params.twalk.wnames = &name; | 258 | tc = v9fs_create_twalk(fid, newfid, nwname, &name); |
198 | } else { | 259 | if (!IS_ERR(tc)) { |
199 | msg.params.twalk.nwname = 0; | 260 | ret = v9fs_mux_rpc(v9ses->mux, tc, rcp); |
200 | } | 261 | kfree(tc); |
201 | 262 | } else | |
202 | return v9fs_mux_rpc(v9ses, &msg, fcall); | 263 | ret = PTR_ERR(tc); |
264 | |||
265 | return ret; | ||
203 | } | 266 | } |
204 | 267 | ||
205 | /** | 268 | /** |
@@ -214,19 +277,21 @@ v9fs_t_walk(struct v9fs_session_info *v9ses, u32 fid, u32 newfid, | |||
214 | 277 | ||
215 | int | 278 | int |
216 | v9fs_t_open(struct v9fs_session_info *v9ses, u32 fid, u8 mode, | 279 | v9fs_t_open(struct v9fs_session_info *v9ses, u32 fid, u8 mode, |
217 | struct v9fs_fcall **fcall) | 280 | struct v9fs_fcall **rcp) |
218 | { | 281 | { |
219 | struct v9fs_fcall msg; | 282 | int ret; |
220 | long errorno = -1; | 283 | struct v9fs_fcall *tc; |
221 | 284 | ||
222 | dprintk(DEBUG_9P, "fid %d mode %d\n", fid, mode); | 285 | dprintk(DEBUG_9P, "fid %d mode %d\n", fid, mode); |
223 | msg.id = TOPEN; | ||
224 | msg.params.topen.fid = fid; | ||
225 | msg.params.topen.mode = mode; | ||
226 | 286 | ||
227 | errorno = v9fs_mux_rpc(v9ses, &msg, fcall); | 287 | tc = v9fs_create_topen(fid, mode); |
288 | if (!IS_ERR(tc)) { | ||
289 | ret = v9fs_mux_rpc(v9ses->mux, tc, rcp); | ||
290 | kfree(tc); | ||
291 | } else | ||
292 | ret = PTR_ERR(tc); | ||
228 | 293 | ||
229 | return errorno; | 294 | return ret; |
230 | } | 295 | } |
231 | 296 | ||
232 | /** | 297 | /** |
@@ -239,14 +304,21 @@ v9fs_t_open(struct v9fs_session_info *v9ses, u32 fid, u8 mode, | |||
239 | 304 | ||
240 | int | 305 | int |
241 | v9fs_t_remove(struct v9fs_session_info *v9ses, u32 fid, | 306 | v9fs_t_remove(struct v9fs_session_info *v9ses, u32 fid, |
242 | struct v9fs_fcall **fcall) | 307 | struct v9fs_fcall **rcp) |
243 | { | 308 | { |
244 | struct v9fs_fcall msg; | 309 | int ret; |
310 | struct v9fs_fcall *tc; | ||
245 | 311 | ||
246 | dprintk(DEBUG_9P, "fid %d\n", fid); | 312 | dprintk(DEBUG_9P, "fid %d\n", fid); |
247 | msg.id = TREMOVE; | 313 | |
248 | msg.params.tremove.fid = fid; | 314 | tc = v9fs_create_tremove(fid); |
249 | return v9fs_mux_rpc(v9ses, &msg, fcall); | 315 | if (!IS_ERR(tc)) { |
316 | ret = v9fs_mux_rpc(v9ses->mux, tc, rcp); | ||
317 | kfree(tc); | ||
318 | } else | ||
319 | ret = PTR_ERR(tc); | ||
320 | |||
321 | return ret; | ||
250 | } | 322 | } |
251 | 323 | ||
252 | /** | 324 | /** |
@@ -262,20 +334,22 @@ v9fs_t_remove(struct v9fs_session_info *v9ses, u32 fid, | |||
262 | 334 | ||
263 | int | 335 | int |
264 | v9fs_t_create(struct v9fs_session_info *v9ses, u32 fid, char *name, | 336 | v9fs_t_create(struct v9fs_session_info *v9ses, u32 fid, char *name, |
265 | u32 perm, u8 mode, struct v9fs_fcall **fcall) | 337 | u32 perm, u8 mode, struct v9fs_fcall **rcp) |
266 | { | 338 | { |
267 | struct v9fs_fcall msg; | 339 | int ret; |
340 | struct v9fs_fcall *tc; | ||
268 | 341 | ||
269 | dprintk(DEBUG_9P, "fid %d name '%s' perm %x mode %d\n", | 342 | dprintk(DEBUG_9P, "fid %d name '%s' perm %x mode %d\n", |
270 | fid, name, perm, mode); | 343 | fid, name, perm, mode); |
271 | 344 | ||
272 | msg.id = TCREATE; | 345 | tc = v9fs_create_tcreate(fid, name, perm, mode); |
273 | msg.params.tcreate.fid = fid; | 346 | if (!IS_ERR(tc)) { |
274 | msg.params.tcreate.name = name; | 347 | ret = v9fs_mux_rpc(v9ses->mux, tc, rcp); |
275 | msg.params.tcreate.perm = perm; | 348 | kfree(tc); |
276 | msg.params.tcreate.mode = mode; | 349 | } else |
350 | ret = PTR_ERR(tc); | ||
277 | 351 | ||
278 | return v9fs_mux_rpc(v9ses, &msg, fcall); | 352 | return ret; |
279 | } | 353 | } |
280 | 354 | ||
281 | /** | 355 | /** |
@@ -290,31 +364,29 @@ v9fs_t_create(struct v9fs_session_info *v9ses, u32 fid, char *name, | |||
290 | 364 | ||
291 | int | 365 | int |
292 | v9fs_t_read(struct v9fs_session_info *v9ses, u32 fid, u64 offset, | 366 | v9fs_t_read(struct v9fs_session_info *v9ses, u32 fid, u64 offset, |
293 | u32 count, struct v9fs_fcall **fcall) | 367 | u32 count, struct v9fs_fcall **rcp) |
294 | { | 368 | { |
295 | struct v9fs_fcall msg; | 369 | int ret; |
296 | struct v9fs_fcall *rc = NULL; | 370 | struct v9fs_fcall *tc, *rc; |
297 | long errorno = -1; | 371 | |
298 | 372 | dprintk(DEBUG_9P, "fid %d offset 0x%llux count 0x%x\n", fid, | |
299 | dprintk(DEBUG_9P, "fid %d offset 0x%lx count 0x%x\n", fid, | 373 | (long long unsigned) offset, count); |
300 | (long unsigned int)offset, count); | 374 | |
301 | msg.id = TREAD; | 375 | tc = v9fs_create_tread(fid, offset, count); |
302 | msg.params.tread.fid = fid; | 376 | if (!IS_ERR(tc)) { |
303 | msg.params.tread.offset = offset; | 377 | ret = v9fs_mux_rpc(v9ses->mux, tc, &rc); |
304 | msg.params.tread.count = count; | 378 | if (!ret) |
305 | errorno = v9fs_mux_rpc(v9ses, &msg, &rc); | 379 | ret = rc->params.rread.count; |
306 | 380 | if (rcp) | |
307 | if (!errorno) { | 381 | *rcp = rc; |
308 | errorno = rc->params.rread.count; | 382 | else |
309 | dump_data(rc->params.rread.data, rc->params.rread.count); | 383 | kfree(rc); |
310 | } | 384 | |
311 | 385 | kfree(tc); | |
312 | if (fcall) | 386 | } else |
313 | *fcall = rc; | 387 | ret = PTR_ERR(tc); |
314 | else | 388 | |
315 | kfree(rc); | 389 | return ret; |
316 | |||
317 | return errorno; | ||
318 | } | 390 | } |
319 | 391 | ||
320 | /** | 392 | /** |
@@ -328,32 +400,30 @@ v9fs_t_read(struct v9fs_session_info *v9ses, u32 fid, u64 offset, | |||
328 | */ | 400 | */ |
329 | 401 | ||
330 | int | 402 | int |
331 | v9fs_t_write(struct v9fs_session_info *v9ses, u32 fid, | 403 | v9fs_t_write(struct v9fs_session_info *v9ses, u32 fid, u64 offset, u32 count, |
332 | u64 offset, u32 count, void *data, struct v9fs_fcall **fcall) | 404 | const char __user *data, struct v9fs_fcall **rcp) |
333 | { | 405 | { |
334 | struct v9fs_fcall msg; | 406 | int ret; |
335 | struct v9fs_fcall *rc = NULL; | 407 | struct v9fs_fcall *tc, *rc; |
336 | long errorno = -1; | ||
337 | 408 | ||
338 | dprintk(DEBUG_9P, "fid %d offset 0x%llx count 0x%x\n", fid, | 409 | dprintk(DEBUG_9P, "fid %d offset 0x%llux count 0x%x\n", fid, |
339 | (unsigned long long)offset, count); | 410 | (long long unsigned) offset, count); |
340 | dump_data(data, count); | ||
341 | 411 | ||
342 | msg.id = TWRITE; | 412 | tc = v9fs_create_twrite(fid, offset, count, data); |
343 | msg.params.twrite.fid = fid; | 413 | if (!IS_ERR(tc)) { |
344 | msg.params.twrite.offset = offset; | 414 | ret = v9fs_mux_rpc(v9ses->mux, tc, &rc); |
345 | msg.params.twrite.count = count; | ||
346 | msg.params.twrite.data = data; | ||
347 | 415 | ||
348 | errorno = v9fs_mux_rpc(v9ses, &msg, &rc); | 416 | if (!ret) |
417 | ret = rc->params.rwrite.count; | ||
418 | if (rcp) | ||
419 | *rcp = rc; | ||
420 | else | ||
421 | kfree(rc); | ||
349 | 422 | ||
350 | if (!errorno) | 423 | kfree(tc); |
351 | errorno = rc->params.rwrite.count; | 424 | } else |
425 | ret = PTR_ERR(tc); | ||
352 | 426 | ||
353 | if (fcall) | 427 | return ret; |
354 | *fcall = rc; | ||
355 | else | ||
356 | kfree(rc); | ||
357 | |||
358 | return errorno; | ||
359 | } | 428 | } |
429 | |||
diff --git a/fs/9p/9p.h b/fs/9p/9p.h index f55424216be2..0cd374d94717 100644 --- a/fs/9p/9p.h +++ b/fs/9p/9p.h | |||
@@ -3,6 +3,7 @@ | |||
3 | * | 3 | * |
4 | * 9P protocol definitions. | 4 | * 9P protocol definitions. |
5 | * | 5 | * |
6 | * Copyright (C) 2005 by Latchesar Ionkov <lucho@ionkov.net> | ||
6 | * Copyright (C) 2004 by Eric Van Hensbergen <ericvh@gmail.com> | 7 | * Copyright (C) 2004 by Eric Van Hensbergen <ericvh@gmail.com> |
7 | * Copyright (C) 2002 by Ron Minnich <rminnich@lanl.gov> | 8 | * Copyright (C) 2002 by Ron Minnich <rminnich@lanl.gov> |
8 | * | 9 | * |
@@ -100,9 +101,18 @@ enum { | |||
100 | V9FS_QTFILE = 0x00, | 101 | V9FS_QTFILE = 0x00, |
101 | }; | 102 | }; |
102 | 103 | ||
104 | #define V9FS_NOTAG (u16)(~0) | ||
105 | #define V9FS_NOFID (u32)(~0) | ||
106 | #define V9FS_MAXWELEM 16 | ||
107 | |||
103 | /* ample room for Twrite/Rread header (iounit) */ | 108 | /* ample room for Twrite/Rread header (iounit) */ |
104 | #define V9FS_IOHDRSZ 24 | 109 | #define V9FS_IOHDRSZ 24 |
105 | 110 | ||
111 | struct v9fs_str { | ||
112 | u16 len; | ||
113 | char *str; | ||
114 | }; | ||
115 | |||
106 | /* qids are the unique ID for a file (like an inode */ | 116 | /* qids are the unique ID for a file (like an inode */ |
107 | struct v9fs_qid { | 117 | struct v9fs_qid { |
108 | u8 type; | 118 | u8 type; |
@@ -120,6 +130,29 @@ struct v9fs_stat { | |||
120 | u32 atime; | 130 | u32 atime; |
121 | u32 mtime; | 131 | u32 mtime; |
122 | u64 length; | 132 | u64 length; |
133 | struct v9fs_str name; | ||
134 | struct v9fs_str uid; | ||
135 | struct v9fs_str gid; | ||
136 | struct v9fs_str muid; | ||
137 | struct v9fs_str extension; /* 9p2000.u extensions */ | ||
138 | u32 n_uid; /* 9p2000.u extensions */ | ||
139 | u32 n_gid; /* 9p2000.u extensions */ | ||
140 | u32 n_muid; /* 9p2000.u extensions */ | ||
141 | }; | ||
142 | |||
143 | /* file metadata (stat) structure used to create Twstat message | ||
144 | The is similar to v9fs_stat, but the strings don't point to | ||
145 | the same memory block and should be freed separately | ||
146 | */ | ||
147 | struct v9fs_wstat { | ||
148 | u16 size; | ||
149 | u16 type; | ||
150 | u32 dev; | ||
151 | struct v9fs_qid qid; | ||
152 | u32 mode; | ||
153 | u32 atime; | ||
154 | u32 mtime; | ||
155 | u64 length; | ||
123 | char *name; | 156 | char *name; |
124 | char *uid; | 157 | char *uid; |
125 | char *gid; | 158 | char *gid; |
@@ -128,25 +161,24 @@ struct v9fs_stat { | |||
128 | u32 n_uid; /* 9p2000.u extensions */ | 161 | u32 n_uid; /* 9p2000.u extensions */ |
129 | u32 n_gid; /* 9p2000.u extensions */ | 162 | u32 n_gid; /* 9p2000.u extensions */ |
130 | u32 n_muid; /* 9p2000.u extensions */ | 163 | u32 n_muid; /* 9p2000.u extensions */ |
131 | char data[0]; | ||
132 | }; | 164 | }; |
133 | 165 | ||
134 | /* Structures for Protocol Operations */ | 166 | /* Structures for Protocol Operations */ |
135 | 167 | ||
136 | struct Tversion { | 168 | struct Tversion { |
137 | u32 msize; | 169 | u32 msize; |
138 | char *version; | 170 | struct v9fs_str version; |
139 | }; | 171 | }; |
140 | 172 | ||
141 | struct Rversion { | 173 | struct Rversion { |
142 | u32 msize; | 174 | u32 msize; |
143 | char *version; | 175 | struct v9fs_str version; |
144 | }; | 176 | }; |
145 | 177 | ||
146 | struct Tauth { | 178 | struct Tauth { |
147 | u32 afid; | 179 | u32 afid; |
148 | char *uname; | 180 | struct v9fs_str uname; |
149 | char *aname; | 181 | struct v9fs_str aname; |
150 | }; | 182 | }; |
151 | 183 | ||
152 | struct Rauth { | 184 | struct Rauth { |
@@ -154,12 +186,12 @@ struct Rauth { | |||
154 | }; | 186 | }; |
155 | 187 | ||
156 | struct Rerror { | 188 | struct Rerror { |
157 | char *error; | 189 | struct v9fs_str error; |
158 | u32 errno; /* 9p2000.u extension */ | 190 | u32 errno; /* 9p2000.u extension */ |
159 | }; | 191 | }; |
160 | 192 | ||
161 | struct Tflush { | 193 | struct Tflush { |
162 | u32 oldtag; | 194 | u16 oldtag; |
163 | }; | 195 | }; |
164 | 196 | ||
165 | struct Rflush { | 197 | struct Rflush { |
@@ -168,8 +200,8 @@ struct Rflush { | |||
168 | struct Tattach { | 200 | struct Tattach { |
169 | u32 fid; | 201 | u32 fid; |
170 | u32 afid; | 202 | u32 afid; |
171 | char *uname; | 203 | struct v9fs_str uname; |
172 | char *aname; | 204 | struct v9fs_str aname; |
173 | }; | 205 | }; |
174 | 206 | ||
175 | struct Rattach { | 207 | struct Rattach { |
@@ -179,13 +211,13 @@ struct Rattach { | |||
179 | struct Twalk { | 211 | struct Twalk { |
180 | u32 fid; | 212 | u32 fid; |
181 | u32 newfid; | 213 | u32 newfid; |
182 | u32 nwname; | 214 | u16 nwname; |
183 | char **wnames; | 215 | struct v9fs_str wnames[16]; |
184 | }; | 216 | }; |
185 | 217 | ||
186 | struct Rwalk { | 218 | struct Rwalk { |
187 | u32 nwqid; | 219 | u16 nwqid; |
188 | struct v9fs_qid *wqids; | 220 | struct v9fs_qid wqids[16]; |
189 | }; | 221 | }; |
190 | 222 | ||
191 | struct Topen { | 223 | struct Topen { |
@@ -200,7 +232,7 @@ struct Ropen { | |||
200 | 232 | ||
201 | struct Tcreate { | 233 | struct Tcreate { |
202 | u32 fid; | 234 | u32 fid; |
203 | char *name; | 235 | struct v9fs_str name; |
204 | u32 perm; | 236 | u32 perm; |
205 | u8 mode; | 237 | u8 mode; |
206 | }; | 238 | }; |
@@ -251,12 +283,12 @@ struct Tstat { | |||
251 | }; | 283 | }; |
252 | 284 | ||
253 | struct Rstat { | 285 | struct Rstat { |
254 | struct v9fs_stat *stat; | 286 | struct v9fs_stat stat; |
255 | }; | 287 | }; |
256 | 288 | ||
257 | struct Twstat { | 289 | struct Twstat { |
258 | u32 fid; | 290 | u32 fid; |
259 | struct v9fs_stat *stat; | 291 | struct v9fs_stat stat; |
260 | }; | 292 | }; |
261 | 293 | ||
262 | struct Rwstat { | 294 | struct Rwstat { |
@@ -271,6 +303,7 @@ struct v9fs_fcall { | |||
271 | u32 size; | 303 | u32 size; |
272 | u8 id; | 304 | u8 id; |
273 | u16 tag; | 305 | u16 tag; |
306 | void *sdata; | ||
274 | 307 | ||
275 | union { | 308 | union { |
276 | struct Tversion tversion; | 309 | struct Tversion tversion; |
@@ -303,7 +336,9 @@ struct v9fs_fcall { | |||
303 | } params; | 336 | } params; |
304 | }; | 337 | }; |
305 | 338 | ||
306 | #define FCALL_ERROR(fcall) (fcall ? fcall->params.rerror.error : "") | 339 | #define PRINT_FCALL_ERROR(s, fcall) dprintk(DEBUG_ERROR, "%s: %.*s\n", s, \ |
340 | fcall?fcall->params.rerror.error.len:0, \ | ||
341 | fcall?fcall->params.rerror.error.str:""); | ||
307 | 342 | ||
308 | int v9fs_t_version(struct v9fs_session_info *v9ses, u32 msize, | 343 | int v9fs_t_version(struct v9fs_session_info *v9ses, u32 msize, |
309 | char *version, struct v9fs_fcall **rcall); | 344 | char *version, struct v9fs_fcall **rcall); |
@@ -311,8 +346,7 @@ int v9fs_t_version(struct v9fs_session_info *v9ses, u32 msize, | |||
311 | int v9fs_t_attach(struct v9fs_session_info *v9ses, char *uname, char *aname, | 346 | int v9fs_t_attach(struct v9fs_session_info *v9ses, char *uname, char *aname, |
312 | u32 fid, u32 afid, struct v9fs_fcall **rcall); | 347 | u32 fid, u32 afid, struct v9fs_fcall **rcall); |
313 | 348 | ||
314 | int v9fs_t_clunk(struct v9fs_session_info *v9ses, u32 fid, | 349 | int v9fs_t_clunk(struct v9fs_session_info *v9ses, u32 fid); |
315 | struct v9fs_fcall **rcall); | ||
316 | 350 | ||
317 | int v9fs_t_flush(struct v9fs_session_info *v9ses, u16 oldtag); | 351 | int v9fs_t_flush(struct v9fs_session_info *v9ses, u16 oldtag); |
318 | 352 | ||
@@ -320,7 +354,7 @@ int v9fs_t_stat(struct v9fs_session_info *v9ses, u32 fid, | |||
320 | struct v9fs_fcall **rcall); | 354 | struct v9fs_fcall **rcall); |
321 | 355 | ||
322 | int v9fs_t_wstat(struct v9fs_session_info *v9ses, u32 fid, | 356 | int v9fs_t_wstat(struct v9fs_session_info *v9ses, u32 fid, |
323 | struct v9fs_stat *stat, struct v9fs_fcall **rcall); | 357 | struct v9fs_wstat *wstat, struct v9fs_fcall **rcall); |
324 | 358 | ||
325 | int v9fs_t_walk(struct v9fs_session_info *v9ses, u32 fid, u32 newfid, | 359 | int v9fs_t_walk(struct v9fs_session_info *v9ses, u32 fid, u32 newfid, |
326 | char *name, struct v9fs_fcall **rcall); | 360 | char *name, struct v9fs_fcall **rcall); |
@@ -338,4 +372,5 @@ int v9fs_t_read(struct v9fs_session_info *v9ses, u32 fid, | |||
338 | u64 offset, u32 count, struct v9fs_fcall **rcall); | 372 | u64 offset, u32 count, struct v9fs_fcall **rcall); |
339 | 373 | ||
340 | int v9fs_t_write(struct v9fs_session_info *v9ses, u32 fid, u64 offset, | 374 | int v9fs_t_write(struct v9fs_session_info *v9ses, u32 fid, u64 offset, |
341 | u32 count, void *data, struct v9fs_fcall **rcall); | 375 | u32 count, const char __user * data, |
376 | struct v9fs_fcall **rcall); | ||
diff --git a/fs/9p/Makefile b/fs/9p/Makefile index e4e4ffe5a7dc..3d023089707e 100644 --- a/fs/9p/Makefile +++ b/fs/9p/Makefile | |||
@@ -1,17 +1,17 @@ | |||
1 | obj-$(CONFIG_9P_FS) := 9p2000.o | 1 | obj-$(CONFIG_9P_FS) := 9p2000.o |
2 | 2 | ||
3 | 9p2000-objs := \ | 3 | 9p2000-objs := \ |
4 | trans_fd.o \ | ||
5 | trans_sock.o \ | ||
6 | mux.o \ | ||
7 | 9p.o \ | ||
8 | conv.o \ | ||
4 | vfs_super.o \ | 9 | vfs_super.o \ |
5 | vfs_inode.o \ | 10 | vfs_inode.o \ |
6 | vfs_file.o \ | 11 | vfs_file.o \ |
7 | vfs_dir.o \ | 12 | vfs_dir.o \ |
8 | vfs_dentry.o \ | 13 | vfs_dentry.o \ |
9 | error.o \ | 14 | error.o \ |
10 | mux.o \ | ||
11 | trans_fd.o \ | ||
12 | trans_sock.o \ | ||
13 | 9p.o \ | ||
14 | conv.o \ | ||
15 | v9fs.o \ | 15 | v9fs.o \ |
16 | fid.o | 16 | fid.o |
17 | 17 | ||
diff --git a/fs/9p/conv.c b/fs/9p/conv.c index 18121af99d3e..55ccfa10ee9e 100644 --- a/fs/9p/conv.c +++ b/fs/9p/conv.c | |||
@@ -30,7 +30,7 @@ | |||
30 | #include <linux/errno.h> | 30 | #include <linux/errno.h> |
31 | #include <linux/fs.h> | 31 | #include <linux/fs.h> |
32 | #include <linux/idr.h> | 32 | #include <linux/idr.h> |
33 | 33 | #include <asm/uaccess.h> | |
34 | #include "debug.h" | 34 | #include "debug.h" |
35 | #include "v9fs.h" | 35 | #include "v9fs.h" |
36 | #include "9p.h" | 36 | #include "9p.h" |
@@ -58,12 +58,15 @@ static inline int buf_check_overflow(struct cbuf *buf) | |||
58 | 58 | ||
59 | static inline int buf_check_size(struct cbuf *buf, int len) | 59 | static inline int buf_check_size(struct cbuf *buf, int len) |
60 | { | 60 | { |
61 | if (buf->p+len > buf->ep) { | 61 | if (buf->p + len > buf->ep) { |
62 | if (buf->p < buf->ep) { | 62 | if (buf->p < buf->ep) { |
63 | eprintk(KERN_ERR, "buffer overflow\n"); | 63 | eprintk(KERN_ERR, "buffer overflow: want %d has %d\n", |
64 | len, (int)(buf->ep - buf->p)); | ||
65 | dump_stack(); | ||
64 | buf->p = buf->ep + 1; | 66 | buf->p = buf->ep + 1; |
65 | return 0; | ||
66 | } | 67 | } |
68 | |||
69 | return 0; | ||
67 | } | 70 | } |
68 | 71 | ||
69 | return 1; | 72 | return 1; |
@@ -127,14 +130,6 @@ static inline void buf_put_string(struct cbuf *buf, const char *s) | |||
127 | buf_put_stringn(buf, s, strlen(s)); | 130 | buf_put_stringn(buf, s, strlen(s)); |
128 | } | 131 | } |
129 | 132 | ||
130 | static inline void buf_put_data(struct cbuf *buf, void *data, u32 datalen) | ||
131 | { | ||
132 | if (buf_check_size(buf, datalen)) { | ||
133 | memcpy(buf->p, data, datalen); | ||
134 | buf->p += datalen; | ||
135 | } | ||
136 | } | ||
137 | |||
138 | static inline u8 buf_get_int8(struct cbuf *buf) | 133 | static inline u8 buf_get_int8(struct cbuf *buf) |
139 | { | 134 | { |
140 | u8 ret = 0; | 135 | u8 ret = 0; |
@@ -183,86 +178,37 @@ static inline u64 buf_get_int64(struct cbuf *buf) | |||
183 | return ret; | 178 | return ret; |
184 | } | 179 | } |
185 | 180 | ||
186 | static inline int | 181 | static inline void buf_get_str(struct cbuf *buf, struct v9fs_str *vstr) |
187 | buf_get_string(struct cbuf *buf, char *data, unsigned int datalen) | ||
188 | { | ||
189 | u16 len = 0; | ||
190 | |||
191 | len = buf_get_int16(buf); | ||
192 | if (!buf_check_overflow(buf) && buf_check_size(buf, len) && len+1>datalen) { | ||
193 | memcpy(data, buf->p, len); | ||
194 | data[len] = 0; | ||
195 | buf->p += len; | ||
196 | len++; | ||
197 | } | ||
198 | |||
199 | return len; | ||
200 | } | ||
201 | |||
202 | static inline char *buf_get_stringb(struct cbuf *buf, struct cbuf *sbuf) | ||
203 | { | ||
204 | char *ret; | ||
205 | u16 len; | ||
206 | |||
207 | ret = NULL; | ||
208 | len = buf_get_int16(buf); | ||
209 | |||
210 | if (!buf_check_overflow(buf) && buf_check_size(buf, len) && | ||
211 | buf_check_size(sbuf, len+1)) { | ||
212 | |||
213 | memcpy(sbuf->p, buf->p, len); | ||
214 | sbuf->p[len] = 0; | ||
215 | ret = sbuf->p; | ||
216 | buf->p += len; | ||
217 | sbuf->p += len + 1; | ||
218 | } | ||
219 | |||
220 | return ret; | ||
221 | } | ||
222 | |||
223 | static inline int buf_get_data(struct cbuf *buf, void *data, int datalen) | ||
224 | { | 182 | { |
225 | int ret = 0; | 183 | vstr->len = buf_get_int16(buf); |
226 | 184 | if (!buf_check_overflow(buf) && buf_check_size(buf, vstr->len)) { | |
227 | if (buf_check_size(buf, datalen)) { | 185 | vstr->str = buf->p; |
228 | memcpy(data, buf->p, datalen); | 186 | buf->p += vstr->len; |
229 | buf->p += datalen; | 187 | } else { |
230 | ret = datalen; | 188 | vstr->len = 0; |
189 | vstr->str = NULL; | ||
231 | } | 190 | } |
232 | |||
233 | return ret; | ||
234 | } | 191 | } |
235 | 192 | ||
236 | static inline void *buf_get_datab(struct cbuf *buf, struct cbuf *dbuf, | 193 | static inline void buf_get_qid(struct cbuf *bufp, struct v9fs_qid *qid) |
237 | int datalen) | ||
238 | { | 194 | { |
239 | char *ret = NULL; | 195 | qid->type = buf_get_int8(bufp); |
240 | int n = 0; | 196 | qid->version = buf_get_int32(bufp); |
241 | 197 | qid->path = buf_get_int64(bufp); | |
242 | if (buf_check_size(dbuf, datalen)) { | ||
243 | n = buf_get_data(buf, dbuf->p, datalen); | ||
244 | if (n > 0) { | ||
245 | ret = dbuf->p; | ||
246 | dbuf->p += n; | ||
247 | } | ||
248 | } | ||
249 | |||
250 | return ret; | ||
251 | } | 198 | } |
252 | 199 | ||
253 | /** | 200 | /** |
254 | * v9fs_size_stat - calculate the size of a variable length stat struct | 201 | * v9fs_size_wstat - calculate the size of a variable length stat struct |
255 | * @v9ses: session information | ||
256 | * @stat: metadata (stat) structure | 202 | * @stat: metadata (stat) structure |
203 | * @extended: non-zero if 9P2000.u | ||
257 | * | 204 | * |
258 | */ | 205 | */ |
259 | 206 | ||
260 | static int v9fs_size_stat(struct v9fs_session_info *v9ses, | 207 | static int v9fs_size_wstat(struct v9fs_wstat *wstat, int extended) |
261 | struct v9fs_stat *stat) | ||
262 | { | 208 | { |
263 | int size = 0; | 209 | int size = 0; |
264 | 210 | ||
265 | if (stat == NULL) { | 211 | if (wstat == NULL) { |
266 | eprintk(KERN_ERR, "v9fs_size_stat: got a NULL stat pointer\n"); | 212 | eprintk(KERN_ERR, "v9fs_size_stat: got a NULL stat pointer\n"); |
267 | return 0; | 213 | return 0; |
268 | } | 214 | } |
@@ -279,82 +225,38 @@ static int v9fs_size_stat(struct v9fs_session_info *v9ses, | |||
279 | 8 + /* length[8] */ | 225 | 8 + /* length[8] */ |
280 | 8; /* minimum sum of string lengths */ | 226 | 8; /* minimum sum of string lengths */ |
281 | 227 | ||
282 | if (stat->name) | 228 | if (wstat->name) |
283 | size += strlen(stat->name); | 229 | size += strlen(wstat->name); |
284 | if (stat->uid) | 230 | if (wstat->uid) |
285 | size += strlen(stat->uid); | 231 | size += strlen(wstat->uid); |
286 | if (stat->gid) | 232 | if (wstat->gid) |
287 | size += strlen(stat->gid); | 233 | size += strlen(wstat->gid); |
288 | if (stat->muid) | 234 | if (wstat->muid) |
289 | size += strlen(stat->muid); | 235 | size += strlen(wstat->muid); |
290 | 236 | ||
291 | if (v9ses->extended) { | 237 | if (extended) { |
292 | size += 4 + /* n_uid[4] */ | 238 | size += 4 + /* n_uid[4] */ |
293 | 4 + /* n_gid[4] */ | 239 | 4 + /* n_gid[4] */ |
294 | 4 + /* n_muid[4] */ | 240 | 4 + /* n_muid[4] */ |
295 | 2; /* string length of extension[4] */ | 241 | 2; /* string length of extension[4] */ |
296 | if (stat->extension) | 242 | if (wstat->extension) |
297 | size += strlen(stat->extension); | 243 | size += strlen(wstat->extension); |
298 | } | 244 | } |
299 | 245 | ||
300 | return size; | 246 | return size; |
301 | } | 247 | } |
302 | 248 | ||
303 | /** | 249 | /** |
304 | * serialize_stat - safely format a stat structure for transmission | 250 | * buf_get_stat - safely decode a recieved metadata (stat) structure |
305 | * @v9ses: session info | ||
306 | * @stat: metadata (stat) structure | ||
307 | * @bufp: buffer to serialize structure into | ||
308 | * | ||
309 | */ | ||
310 | |||
311 | static int | ||
312 | serialize_stat(struct v9fs_session_info *v9ses, struct v9fs_stat *stat, | ||
313 | struct cbuf *bufp) | ||
314 | { | ||
315 | buf_put_int16(bufp, stat->size); | ||
316 | buf_put_int16(bufp, stat->type); | ||
317 | buf_put_int32(bufp, stat->dev); | ||
318 | buf_put_int8(bufp, stat->qid.type); | ||
319 | buf_put_int32(bufp, stat->qid.version); | ||
320 | buf_put_int64(bufp, stat->qid.path); | ||
321 | buf_put_int32(bufp, stat->mode); | ||
322 | buf_put_int32(bufp, stat->atime); | ||
323 | buf_put_int32(bufp, stat->mtime); | ||
324 | buf_put_int64(bufp, stat->length); | ||
325 | |||
326 | buf_put_string(bufp, stat->name); | ||
327 | buf_put_string(bufp, stat->uid); | ||
328 | buf_put_string(bufp, stat->gid); | ||
329 | buf_put_string(bufp, stat->muid); | ||
330 | |||
331 | if (v9ses->extended) { | ||
332 | buf_put_string(bufp, stat->extension); | ||
333 | buf_put_int32(bufp, stat->n_uid); | ||
334 | buf_put_int32(bufp, stat->n_gid); | ||
335 | buf_put_int32(bufp, stat->n_muid); | ||
336 | } | ||
337 | |||
338 | if (buf_check_overflow(bufp)) | ||
339 | return 0; | ||
340 | |||
341 | return stat->size; | ||
342 | } | ||
343 | |||
344 | /** | ||
345 | * deserialize_stat - safely decode a recieved metadata (stat) structure | ||
346 | * @v9ses: session info | ||
347 | * @bufp: buffer to deserialize | 251 | * @bufp: buffer to deserialize |
348 | * @stat: metadata (stat) structure | 252 | * @stat: metadata (stat) structure |
349 | * @dbufp: buffer to deserialize variable strings into | 253 | * @extended: non-zero if 9P2000.u |
350 | * | 254 | * |
351 | */ | 255 | */ |
352 | 256 | ||
353 | static inline int | 257 | static inline void |
354 | deserialize_stat(struct v9fs_session_info *v9ses, struct cbuf *bufp, | 258 | buf_get_stat(struct cbuf *bufp, struct v9fs_stat *stat, int extended) |
355 | struct v9fs_stat *stat, struct cbuf *dbufp) | ||
356 | { | 259 | { |
357 | |||
358 | stat->size = buf_get_int16(bufp); | 260 | stat->size = buf_get_int16(bufp); |
359 | stat->type = buf_get_int16(bufp); | 261 | stat->type = buf_get_int16(bufp); |
360 | stat->dev = buf_get_int32(bufp); | 262 | stat->dev = buf_get_int32(bufp); |
@@ -365,282 +267,82 @@ deserialize_stat(struct v9fs_session_info *v9ses, struct cbuf *bufp, | |||
365 | stat->atime = buf_get_int32(bufp); | 267 | stat->atime = buf_get_int32(bufp); |
366 | stat->mtime = buf_get_int32(bufp); | 268 | stat->mtime = buf_get_int32(bufp); |
367 | stat->length = buf_get_int64(bufp); | 269 | stat->length = buf_get_int64(bufp); |
368 | stat->name = buf_get_stringb(bufp, dbufp); | 270 | buf_get_str(bufp, &stat->name); |
369 | stat->uid = buf_get_stringb(bufp, dbufp); | 271 | buf_get_str(bufp, &stat->uid); |
370 | stat->gid = buf_get_stringb(bufp, dbufp); | 272 | buf_get_str(bufp, &stat->gid); |
371 | stat->muid = buf_get_stringb(bufp, dbufp); | 273 | buf_get_str(bufp, &stat->muid); |
372 | 274 | ||
373 | if (v9ses->extended) { | 275 | if (extended) { |
374 | stat->extension = buf_get_stringb(bufp, dbufp); | 276 | buf_get_str(bufp, &stat->extension); |
375 | stat->n_uid = buf_get_int32(bufp); | 277 | stat->n_uid = buf_get_int32(bufp); |
376 | stat->n_gid = buf_get_int32(bufp); | 278 | stat->n_gid = buf_get_int32(bufp); |
377 | stat->n_muid = buf_get_int32(bufp); | 279 | stat->n_muid = buf_get_int32(bufp); |
378 | } | 280 | } |
379 | |||
380 | if (buf_check_overflow(bufp) || buf_check_overflow(dbufp)) | ||
381 | return 0; | ||
382 | |||
383 | return stat->size + 2; | ||
384 | } | ||
385 | |||
386 | /** | ||
387 | * deserialize_statb - wrapper for decoding a received metadata structure | ||
388 | * @v9ses: session info | ||
389 | * @bufp: buffer to deserialize | ||
390 | * @dbufp: buffer to deserialize variable strings into | ||
391 | * | ||
392 | */ | ||
393 | |||
394 | static inline struct v9fs_stat *deserialize_statb(struct v9fs_session_info | ||
395 | *v9ses, struct cbuf *bufp, | ||
396 | struct cbuf *dbufp) | ||
397 | { | ||
398 | struct v9fs_stat *ret = buf_alloc(dbufp, sizeof(struct v9fs_stat)); | ||
399 | |||
400 | if (ret) { | ||
401 | int n = deserialize_stat(v9ses, bufp, ret, dbufp); | ||
402 | if (n <= 0) | ||
403 | return NULL; | ||
404 | } | ||
405 | |||
406 | return ret; | ||
407 | } | 281 | } |
408 | 282 | ||
409 | /** | 283 | /** |
410 | * v9fs_deserialize_stat - decode a received metadata structure | 284 | * v9fs_deserialize_stat - decode a received metadata structure |
411 | * @v9ses: session info | ||
412 | * @buf: buffer to deserialize | 285 | * @buf: buffer to deserialize |
413 | * @buflen: length of received buffer | 286 | * @buflen: length of received buffer |
414 | * @stat: metadata structure to decode into | 287 | * @stat: metadata structure to decode into |
415 | * @statlen: length of destination metadata structure | 288 | * @extended: non-zero if 9P2000.u |
416 | * | 289 | * |
290 | * Note: stat will point to the buf region. | ||
417 | */ | 291 | */ |
418 | 292 | ||
419 | int | 293 | int |
420 | v9fs_deserialize_stat(struct v9fs_session_info *v9ses, void *buf, | 294 | v9fs_deserialize_stat(void *buf, u32 buflen, struct v9fs_stat *stat, |
421 | u32 buflen, struct v9fs_stat *stat, u32 statlen) | 295 | int extended) |
422 | { | 296 | { |
423 | struct cbuf buffer; | 297 | struct cbuf buffer; |
424 | struct cbuf *bufp = &buffer; | 298 | struct cbuf *bufp = &buffer; |
425 | struct cbuf dbuffer; | 299 | unsigned char *p; |
426 | struct cbuf *dbufp = &dbuffer; | ||
427 | 300 | ||
428 | buf_init(bufp, buf, buflen); | 301 | buf_init(bufp, buf, buflen); |
429 | buf_init(dbufp, (char *)stat + sizeof(struct v9fs_stat), | 302 | p = bufp->p; |
430 | statlen - sizeof(struct v9fs_stat)); | 303 | buf_get_stat(bufp, stat, extended); |
431 | |||
432 | return deserialize_stat(v9ses, bufp, stat, dbufp); | ||
433 | } | ||
434 | |||
435 | static inline int | ||
436 | v9fs_size_fcall(struct v9fs_session_info *v9ses, struct v9fs_fcall *fcall) | ||
437 | { | ||
438 | int size = 4 + 1 + 2; /* size[4] msg[1] tag[2] */ | ||
439 | int i = 0; | ||
440 | |||
441 | switch (fcall->id) { | ||
442 | default: | ||
443 | eprintk(KERN_ERR, "bad msg type %d\n", fcall->id); | ||
444 | return 0; | ||
445 | case TVERSION: /* msize[4] version[s] */ | ||
446 | size += 4 + 2 + strlen(fcall->params.tversion.version); | ||
447 | break; | ||
448 | case TAUTH: /* afid[4] uname[s] aname[s] */ | ||
449 | size += 4 + 2 + strlen(fcall->params.tauth.uname) + | ||
450 | 2 + strlen(fcall->params.tauth.aname); | ||
451 | break; | ||
452 | case TFLUSH: /* oldtag[2] */ | ||
453 | size += 2; | ||
454 | break; | ||
455 | case TATTACH: /* fid[4] afid[4] uname[s] aname[s] */ | ||
456 | size += 4 + 4 + 2 + strlen(fcall->params.tattach.uname) + | ||
457 | 2 + strlen(fcall->params.tattach.aname); | ||
458 | break; | ||
459 | case TWALK: /* fid[4] newfid[4] nwname[2] nwname*(wname[s]) */ | ||
460 | size += 4 + 4 + 2; | ||
461 | /* now compute total for the array of names */ | ||
462 | for (i = 0; i < fcall->params.twalk.nwname; i++) | ||
463 | size += 2 + strlen(fcall->params.twalk.wnames[i]); | ||
464 | break; | ||
465 | case TOPEN: /* fid[4] mode[1] */ | ||
466 | size += 4 + 1; | ||
467 | break; | ||
468 | case TCREATE: /* fid[4] name[s] perm[4] mode[1] */ | ||
469 | size += 4 + 2 + strlen(fcall->params.tcreate.name) + 4 + 1; | ||
470 | break; | ||
471 | case TREAD: /* fid[4] offset[8] count[4] */ | ||
472 | size += 4 + 8 + 4; | ||
473 | break; | ||
474 | case TWRITE: /* fid[4] offset[8] count[4] data[count] */ | ||
475 | size += 4 + 8 + 4 + fcall->params.twrite.count; | ||
476 | break; | ||
477 | case TCLUNK: /* fid[4] */ | ||
478 | size += 4; | ||
479 | break; | ||
480 | case TREMOVE: /* fid[4] */ | ||
481 | size += 4; | ||
482 | break; | ||
483 | case TSTAT: /* fid[4] */ | ||
484 | size += 4; | ||
485 | break; | ||
486 | case TWSTAT: /* fid[4] stat[n] */ | ||
487 | fcall->params.twstat.stat->size = | ||
488 | v9fs_size_stat(v9ses, fcall->params.twstat.stat); | ||
489 | size += 4 + 2 + 2 + fcall->params.twstat.stat->size; | ||
490 | } | ||
491 | return size; | ||
492 | } | ||
493 | |||
494 | /* | ||
495 | * v9fs_serialize_fcall - marshall fcall struct into a packet | ||
496 | * @v9ses: session information | ||
497 | * @fcall: structure to convert | ||
498 | * @data: buffer to serialize fcall into | ||
499 | * @datalen: length of buffer to serialize fcall into | ||
500 | * | ||
501 | */ | ||
502 | |||
503 | int | ||
504 | v9fs_serialize_fcall(struct v9fs_session_info *v9ses, struct v9fs_fcall *fcall, | ||
505 | void *data, u32 datalen) | ||
506 | { | ||
507 | int i = 0; | ||
508 | struct v9fs_stat *stat = NULL; | ||
509 | struct cbuf buffer; | ||
510 | struct cbuf *bufp = &buffer; | ||
511 | |||
512 | buf_init(bufp, data, datalen); | ||
513 | |||
514 | if (!fcall) { | ||
515 | eprintk(KERN_ERR, "no fcall\n"); | ||
516 | return -EINVAL; | ||
517 | } | ||
518 | |||
519 | fcall->size = v9fs_size_fcall(v9ses, fcall); | ||
520 | |||
521 | buf_put_int32(bufp, fcall->size); | ||
522 | buf_put_int8(bufp, fcall->id); | ||
523 | buf_put_int16(bufp, fcall->tag); | ||
524 | |||
525 | dprintk(DEBUG_CONV, "size %d id %d tag %d\n", fcall->size, fcall->id, | ||
526 | fcall->tag); | ||
527 | |||
528 | /* now encode it */ | ||
529 | switch (fcall->id) { | ||
530 | default: | ||
531 | eprintk(KERN_ERR, "bad msg type: %d\n", fcall->id); | ||
532 | return -EPROTO; | ||
533 | case TVERSION: | ||
534 | buf_put_int32(bufp, fcall->params.tversion.msize); | ||
535 | buf_put_string(bufp, fcall->params.tversion.version); | ||
536 | break; | ||
537 | case TAUTH: | ||
538 | buf_put_int32(bufp, fcall->params.tauth.afid); | ||
539 | buf_put_string(bufp, fcall->params.tauth.uname); | ||
540 | buf_put_string(bufp, fcall->params.tauth.aname); | ||
541 | break; | ||
542 | case TFLUSH: | ||
543 | buf_put_int16(bufp, fcall->params.tflush.oldtag); | ||
544 | break; | ||
545 | case TATTACH: | ||
546 | buf_put_int32(bufp, fcall->params.tattach.fid); | ||
547 | buf_put_int32(bufp, fcall->params.tattach.afid); | ||
548 | buf_put_string(bufp, fcall->params.tattach.uname); | ||
549 | buf_put_string(bufp, fcall->params.tattach.aname); | ||
550 | break; | ||
551 | case TWALK: | ||
552 | buf_put_int32(bufp, fcall->params.twalk.fid); | ||
553 | buf_put_int32(bufp, fcall->params.twalk.newfid); | ||
554 | buf_put_int16(bufp, fcall->params.twalk.nwname); | ||
555 | for (i = 0; i < fcall->params.twalk.nwname; i++) | ||
556 | buf_put_string(bufp, fcall->params.twalk.wnames[i]); | ||
557 | break; | ||
558 | case TOPEN: | ||
559 | buf_put_int32(bufp, fcall->params.topen.fid); | ||
560 | buf_put_int8(bufp, fcall->params.topen.mode); | ||
561 | break; | ||
562 | case TCREATE: | ||
563 | buf_put_int32(bufp, fcall->params.tcreate.fid); | ||
564 | buf_put_string(bufp, fcall->params.tcreate.name); | ||
565 | buf_put_int32(bufp, fcall->params.tcreate.perm); | ||
566 | buf_put_int8(bufp, fcall->params.tcreate.mode); | ||
567 | break; | ||
568 | case TREAD: | ||
569 | buf_put_int32(bufp, fcall->params.tread.fid); | ||
570 | buf_put_int64(bufp, fcall->params.tread.offset); | ||
571 | buf_put_int32(bufp, fcall->params.tread.count); | ||
572 | break; | ||
573 | case TWRITE: | ||
574 | buf_put_int32(bufp, fcall->params.twrite.fid); | ||
575 | buf_put_int64(bufp, fcall->params.twrite.offset); | ||
576 | buf_put_int32(bufp, fcall->params.twrite.count); | ||
577 | buf_put_data(bufp, fcall->params.twrite.data, | ||
578 | fcall->params.twrite.count); | ||
579 | break; | ||
580 | case TCLUNK: | ||
581 | buf_put_int32(bufp, fcall->params.tclunk.fid); | ||
582 | break; | ||
583 | case TREMOVE: | ||
584 | buf_put_int32(bufp, fcall->params.tremove.fid); | ||
585 | break; | ||
586 | case TSTAT: | ||
587 | buf_put_int32(bufp, fcall->params.tstat.fid); | ||
588 | break; | ||
589 | case TWSTAT: | ||
590 | buf_put_int32(bufp, fcall->params.twstat.fid); | ||
591 | stat = fcall->params.twstat.stat; | ||
592 | |||
593 | buf_put_int16(bufp, stat->size + 2); | ||
594 | serialize_stat(v9ses, stat, bufp); | ||
595 | break; | ||
596 | } | ||
597 | 304 | ||
598 | if (buf_check_overflow(bufp)) | 305 | if (buf_check_overflow(bufp)) |
599 | return -EIO; | 306 | return 0; |
600 | 307 | else | |
601 | return fcall->size; | 308 | return bufp->p - p; |
602 | } | 309 | } |
603 | 310 | ||
604 | /** | 311 | /** |
605 | * deserialize_fcall - unmarshal a response | 312 | * deserialize_fcall - unmarshal a response |
606 | * @v9ses: session information | ||
607 | * @msgsize: size of rcall message | ||
608 | * @buf: recieved buffer | 313 | * @buf: recieved buffer |
609 | * @buflen: length of received buffer | 314 | * @buflen: length of received buffer |
610 | * @rcall: fcall structure to populate | 315 | * @rcall: fcall structure to populate |
611 | * @rcalllen: length of fcall structure to populate | 316 | * @rcalllen: length of fcall structure to populate |
317 | * @extended: non-zero if 9P2000.u | ||
612 | * | 318 | * |
613 | */ | 319 | */ |
614 | 320 | ||
615 | int | 321 | int |
616 | v9fs_deserialize_fcall(struct v9fs_session_info *v9ses, u32 msgsize, | 322 | v9fs_deserialize_fcall(void *buf, u32 buflen, struct v9fs_fcall *rcall, |
617 | void *buf, u32 buflen, struct v9fs_fcall *rcall, | 323 | int extended) |
618 | int rcalllen) | ||
619 | { | 324 | { |
620 | 325 | ||
621 | struct cbuf buffer; | 326 | struct cbuf buffer; |
622 | struct cbuf *bufp = &buffer; | 327 | struct cbuf *bufp = &buffer; |
623 | struct cbuf dbuffer; | ||
624 | struct cbuf *dbufp = &dbuffer; | ||
625 | int i = 0; | 328 | int i = 0; |
626 | 329 | ||
627 | buf_init(bufp, buf, buflen); | 330 | buf_init(bufp, buf, buflen); |
628 | buf_init(dbufp, (char *)rcall + sizeof(struct v9fs_fcall), | ||
629 | rcalllen - sizeof(struct v9fs_fcall)); | ||
630 | 331 | ||
631 | rcall->size = msgsize; | 332 | rcall->size = buf_get_int32(bufp); |
632 | rcall->id = buf_get_int8(bufp); | 333 | rcall->id = buf_get_int8(bufp); |
633 | rcall->tag = buf_get_int16(bufp); | 334 | rcall->tag = buf_get_int16(bufp); |
634 | 335 | ||
635 | dprintk(DEBUG_CONV, "size %d id %d tag %d\n", rcall->size, rcall->id, | 336 | dprintk(DEBUG_CONV, "size %d id %d tag %d\n", rcall->size, rcall->id, |
636 | rcall->tag); | 337 | rcall->tag); |
338 | |||
637 | switch (rcall->id) { | 339 | switch (rcall->id) { |
638 | default: | 340 | default: |
639 | eprintk(KERN_ERR, "unknown message type: %d\n", rcall->id); | 341 | eprintk(KERN_ERR, "unknown message type: %d\n", rcall->id); |
640 | return -EPROTO; | 342 | return -EPROTO; |
641 | case RVERSION: | 343 | case RVERSION: |
642 | rcall->params.rversion.msize = buf_get_int32(bufp); | 344 | rcall->params.rversion.msize = buf_get_int32(bufp); |
643 | rcall->params.rversion.version = buf_get_stringb(bufp, dbufp); | 345 | buf_get_str(bufp, &rcall->params.rversion.version); |
644 | break; | 346 | break; |
645 | case RFLUSH: | 347 | case RFLUSH: |
646 | break; | 348 | break; |
@@ -651,34 +353,27 @@ v9fs_deserialize_fcall(struct v9fs_session_info *v9ses, u32 msgsize, | |||
651 | break; | 353 | break; |
652 | case RWALK: | 354 | case RWALK: |
653 | rcall->params.rwalk.nwqid = buf_get_int16(bufp); | 355 | rcall->params.rwalk.nwqid = buf_get_int16(bufp); |
654 | rcall->params.rwalk.wqids = buf_alloc(dbufp, | 356 | if (rcall->params.rwalk.nwqid > V9FS_MAXWELEM) { |
655 | rcall->params.rwalk.nwqid * sizeof(struct v9fs_qid)); | 357 | eprintk(KERN_ERR, "Rwalk with more than %d qids: %d\n", |
656 | if (rcall->params.rwalk.wqids) | 358 | V9FS_MAXWELEM, rcall->params.rwalk.nwqid); |
657 | for (i = 0; i < rcall->params.rwalk.nwqid; i++) { | 359 | return -EPROTO; |
658 | rcall->params.rwalk.wqids[i].type = | 360 | } |
659 | buf_get_int8(bufp); | 361 | |
660 | rcall->params.rwalk.wqids[i].version = | 362 | for (i = 0; i < rcall->params.rwalk.nwqid; i++) |
661 | buf_get_int16(bufp); | 363 | buf_get_qid(bufp, &rcall->params.rwalk.wqids[i]); |
662 | rcall->params.rwalk.wqids[i].path = | ||
663 | buf_get_int64(bufp); | ||
664 | } | ||
665 | break; | 364 | break; |
666 | case ROPEN: | 365 | case ROPEN: |
667 | rcall->params.ropen.qid.type = buf_get_int8(bufp); | 366 | buf_get_qid(bufp, &rcall->params.ropen.qid); |
668 | rcall->params.ropen.qid.version = buf_get_int32(bufp); | ||
669 | rcall->params.ropen.qid.path = buf_get_int64(bufp); | ||
670 | rcall->params.ropen.iounit = buf_get_int32(bufp); | 367 | rcall->params.ropen.iounit = buf_get_int32(bufp); |
671 | break; | 368 | break; |
672 | case RCREATE: | 369 | case RCREATE: |
673 | rcall->params.rcreate.qid.type = buf_get_int8(bufp); | 370 | buf_get_qid(bufp, &rcall->params.rcreate.qid); |
674 | rcall->params.rcreate.qid.version = buf_get_int32(bufp); | ||
675 | rcall->params.rcreate.qid.path = buf_get_int64(bufp); | ||
676 | rcall->params.rcreate.iounit = buf_get_int32(bufp); | 371 | rcall->params.rcreate.iounit = buf_get_int32(bufp); |
677 | break; | 372 | break; |
678 | case RREAD: | 373 | case RREAD: |
679 | rcall->params.rread.count = buf_get_int32(bufp); | 374 | rcall->params.rread.count = buf_get_int32(bufp); |
680 | rcall->params.rread.data = buf_get_datab(bufp, dbufp, | 375 | rcall->params.rread.data = bufp->p; |
681 | rcall->params.rread.count); | 376 | buf_check_size(bufp, rcall->params.rread.count); |
682 | break; | 377 | break; |
683 | case RWRITE: | 378 | case RWRITE: |
684 | rcall->params.rwrite.count = buf_get_int32(bufp); | 379 | rcall->params.rwrite.count = buf_get_int32(bufp); |
@@ -689,20 +384,443 @@ v9fs_deserialize_fcall(struct v9fs_session_info *v9ses, u32 msgsize, | |||
689 | break; | 384 | break; |
690 | case RSTAT: | 385 | case RSTAT: |
691 | buf_get_int16(bufp); | 386 | buf_get_int16(bufp); |
692 | rcall->params.rstat.stat = | 387 | buf_get_stat(bufp, &rcall->params.rstat.stat, extended); |
693 | deserialize_statb(v9ses, bufp, dbufp); | ||
694 | break; | 388 | break; |
695 | case RWSTAT: | 389 | case RWSTAT: |
696 | break; | 390 | break; |
697 | case RERROR: | 391 | case RERROR: |
698 | rcall->params.rerror.error = buf_get_stringb(bufp, dbufp); | 392 | buf_get_str(bufp, &rcall->params.rerror.error); |
699 | if (v9ses->extended) | 393 | if (extended) |
700 | rcall->params.rerror.errno = buf_get_int16(bufp); | 394 | rcall->params.rerror.errno = buf_get_int16(bufp); |
701 | break; | 395 | break; |
702 | } | 396 | } |
703 | 397 | ||
704 | if (buf_check_overflow(bufp) || buf_check_overflow(dbufp)) | 398 | if (buf_check_overflow(bufp)) { |
399 | dprintk(DEBUG_ERROR, "buffer overflow\n"); | ||
705 | return -EIO; | 400 | return -EIO; |
401 | } | ||
402 | |||
403 | return bufp->p - bufp->sp; | ||
404 | } | ||
405 | |||
406 | static inline void v9fs_put_int8(struct cbuf *bufp, u8 val, u8 * p) | ||
407 | { | ||
408 | *p = val; | ||
409 | buf_put_int8(bufp, val); | ||
410 | } | ||
411 | |||
412 | static inline void v9fs_put_int16(struct cbuf *bufp, u16 val, u16 * p) | ||
413 | { | ||
414 | *p = val; | ||
415 | buf_put_int16(bufp, val); | ||
416 | } | ||
417 | |||
418 | static inline void v9fs_put_int32(struct cbuf *bufp, u32 val, u32 * p) | ||
419 | { | ||
420 | *p = val; | ||
421 | buf_put_int32(bufp, val); | ||
422 | } | ||
423 | |||
424 | static inline void v9fs_put_int64(struct cbuf *bufp, u64 val, u64 * p) | ||
425 | { | ||
426 | *p = val; | ||
427 | buf_put_int64(bufp, val); | ||
428 | } | ||
706 | 429 | ||
707 | return rcall->size; | 430 | static inline void |
431 | v9fs_put_str(struct cbuf *bufp, char *data, struct v9fs_str *str) | ||
432 | { | ||
433 | if (data) { | ||
434 | str->len = strlen(data); | ||
435 | str->str = bufp->p; | ||
436 | } else { | ||
437 | str->len = 0; | ||
438 | str->str = NULL; | ||
439 | } | ||
440 | |||
441 | buf_put_stringn(bufp, data, str->len); | ||
442 | } | ||
443 | |||
444 | static inline int | ||
445 | v9fs_put_user_data(struct cbuf *bufp, const char __user * data, int count, | ||
446 | unsigned char **pdata) | ||
447 | { | ||
448 | *pdata = buf_alloc(bufp, count); | ||
449 | return copy_from_user(*pdata, data, count); | ||
450 | } | ||
451 | |||
452 | static void | ||
453 | v9fs_put_wstat(struct cbuf *bufp, struct v9fs_wstat *wstat, | ||
454 | struct v9fs_stat *stat, int statsz, int extended) | ||
455 | { | ||
456 | v9fs_put_int16(bufp, statsz, &stat->size); | ||
457 | v9fs_put_int16(bufp, wstat->type, &stat->type); | ||
458 | v9fs_put_int32(bufp, wstat->dev, &stat->dev); | ||
459 | v9fs_put_int8(bufp, wstat->qid.type, &stat->qid.type); | ||
460 | v9fs_put_int32(bufp, wstat->qid.version, &stat->qid.version); | ||
461 | v9fs_put_int64(bufp, wstat->qid.path, &stat->qid.path); | ||
462 | v9fs_put_int32(bufp, wstat->mode, &stat->mode); | ||
463 | v9fs_put_int32(bufp, wstat->atime, &stat->atime); | ||
464 | v9fs_put_int32(bufp, wstat->mtime, &stat->mtime); | ||
465 | v9fs_put_int64(bufp, wstat->length, &stat->length); | ||
466 | |||
467 | v9fs_put_str(bufp, wstat->name, &stat->name); | ||
468 | v9fs_put_str(bufp, wstat->uid, &stat->uid); | ||
469 | v9fs_put_str(bufp, wstat->gid, &stat->gid); | ||
470 | v9fs_put_str(bufp, wstat->muid, &stat->muid); | ||
471 | |||
472 | if (extended) { | ||
473 | v9fs_put_str(bufp, wstat->extension, &stat->extension); | ||
474 | v9fs_put_int32(bufp, wstat->n_uid, &stat->n_uid); | ||
475 | v9fs_put_int32(bufp, wstat->n_gid, &stat->n_gid); | ||
476 | v9fs_put_int32(bufp, wstat->n_muid, &stat->n_muid); | ||
477 | } | ||
478 | } | ||
479 | |||
480 | static struct v9fs_fcall * | ||
481 | v9fs_create_common(struct cbuf *bufp, u32 size, u8 id) | ||
482 | { | ||
483 | struct v9fs_fcall *fc; | ||
484 | |||
485 | size += 4 + 1 + 2; /* size[4] id[1] tag[2] */ | ||
486 | fc = kmalloc(sizeof(struct v9fs_fcall) + size, GFP_KERNEL); | ||
487 | if (!fc) | ||
488 | return ERR_PTR(-ENOMEM); | ||
489 | |||
490 | fc->sdata = (char *)fc + sizeof(*fc); | ||
491 | |||
492 | buf_init(bufp, (char *)fc->sdata, size); | ||
493 | v9fs_put_int32(bufp, size, &fc->size); | ||
494 | v9fs_put_int8(bufp, id, &fc->id); | ||
495 | v9fs_put_int16(bufp, V9FS_NOTAG, &fc->tag); | ||
496 | |||
497 | return fc; | ||
498 | } | ||
499 | |||
500 | void v9fs_set_tag(struct v9fs_fcall *fc, u16 tag) | ||
501 | { | ||
502 | fc->tag = tag; | ||
503 | *(__le16 *) (fc->sdata + 5) = cpu_to_le16(tag); | ||
504 | } | ||
505 | |||
506 | struct v9fs_fcall *v9fs_create_tversion(u32 msize, char *version) | ||
507 | { | ||
508 | int size; | ||
509 | struct v9fs_fcall *fc; | ||
510 | struct cbuf buffer; | ||
511 | struct cbuf *bufp = &buffer; | ||
512 | |||
513 | size = 4 + 2 + strlen(version); /* msize[4] version[s] */ | ||
514 | fc = v9fs_create_common(bufp, size, TVERSION); | ||
515 | if (IS_ERR(fc)) | ||
516 | goto error; | ||
517 | |||
518 | v9fs_put_int32(bufp, msize, &fc->params.tversion.msize); | ||
519 | v9fs_put_str(bufp, version, &fc->params.tversion.version); | ||
520 | |||
521 | if (buf_check_overflow(bufp)) { | ||
522 | kfree(fc); | ||
523 | fc = ERR_PTR(-ENOMEM); | ||
524 | } | ||
525 | error: | ||
526 | return fc; | ||
527 | } | ||
528 | |||
529 | struct v9fs_fcall *v9fs_create_tauth(u32 afid, char *uname, char *aname) | ||
530 | { | ||
531 | int size; | ||
532 | struct v9fs_fcall *fc; | ||
533 | struct cbuf buffer; | ||
534 | struct cbuf *bufp = &buffer; | ||
535 | |||
536 | size = 4 + 2 + strlen(uname) + 2 + strlen(aname); /* afid[4] uname[s] aname[s] */ | ||
537 | fc = v9fs_create_common(bufp, size, TAUTH); | ||
538 | if (IS_ERR(fc)) | ||
539 | goto error; | ||
540 | |||
541 | v9fs_put_int32(bufp, afid, &fc->params.tauth.afid); | ||
542 | v9fs_put_str(bufp, uname, &fc->params.tauth.uname); | ||
543 | v9fs_put_str(bufp, aname, &fc->params.tauth.aname); | ||
544 | |||
545 | if (buf_check_overflow(bufp)) { | ||
546 | kfree(fc); | ||
547 | fc = ERR_PTR(-ENOMEM); | ||
548 | } | ||
549 | error: | ||
550 | return fc; | ||
551 | } | ||
552 | |||
553 | struct v9fs_fcall * | ||
554 | v9fs_create_tattach(u32 fid, u32 afid, char *uname, char *aname) | ||
555 | { | ||
556 | int size; | ||
557 | struct v9fs_fcall *fc; | ||
558 | struct cbuf buffer; | ||
559 | struct cbuf *bufp = &buffer; | ||
560 | |||
561 | size = 4 + 4 + 2 + strlen(uname) + 2 + strlen(aname); /* fid[4] afid[4] uname[s] aname[s] */ | ||
562 | fc = v9fs_create_common(bufp, size, TATTACH); | ||
563 | if (IS_ERR(fc)) | ||
564 | goto error; | ||
565 | |||
566 | v9fs_put_int32(bufp, fid, &fc->params.tattach.fid); | ||
567 | v9fs_put_int32(bufp, afid, &fc->params.tattach.afid); | ||
568 | v9fs_put_str(bufp, uname, &fc->params.tattach.uname); | ||
569 | v9fs_put_str(bufp, aname, &fc->params.tattach.aname); | ||
570 | |||
571 | error: | ||
572 | return fc; | ||
573 | } | ||
574 | |||
575 | struct v9fs_fcall *v9fs_create_tflush(u16 oldtag) | ||
576 | { | ||
577 | int size; | ||
578 | struct v9fs_fcall *fc; | ||
579 | struct cbuf buffer; | ||
580 | struct cbuf *bufp = &buffer; | ||
581 | |||
582 | size = 2; /* oldtag[2] */ | ||
583 | fc = v9fs_create_common(bufp, size, TFLUSH); | ||
584 | if (IS_ERR(fc)) | ||
585 | goto error; | ||
586 | |||
587 | v9fs_put_int16(bufp, oldtag, &fc->params.tflush.oldtag); | ||
588 | |||
589 | if (buf_check_overflow(bufp)) { | ||
590 | kfree(fc); | ||
591 | fc = ERR_PTR(-ENOMEM); | ||
592 | } | ||
593 | error: | ||
594 | return fc; | ||
595 | } | ||
596 | |||
597 | struct v9fs_fcall *v9fs_create_twalk(u32 fid, u32 newfid, u16 nwname, | ||
598 | char **wnames) | ||
599 | { | ||
600 | int i, size; | ||
601 | struct v9fs_fcall *fc; | ||
602 | struct cbuf buffer; | ||
603 | struct cbuf *bufp = &buffer; | ||
604 | |||
605 | if (nwname > V9FS_MAXWELEM) { | ||
606 | dprintk(DEBUG_ERROR, "nwname > %d\n", V9FS_MAXWELEM); | ||
607 | return NULL; | ||
608 | } | ||
609 | |||
610 | size = 4 + 4 + 2; /* fid[4] newfid[4] nwname[2] ... */ | ||
611 | for (i = 0; i < nwname; i++) { | ||
612 | size += 2 + strlen(wnames[i]); /* wname[s] */ | ||
613 | } | ||
614 | |||
615 | fc = v9fs_create_common(bufp, size, TWALK); | ||
616 | if (IS_ERR(fc)) | ||
617 | goto error; | ||
618 | |||
619 | v9fs_put_int32(bufp, fid, &fc->params.twalk.fid); | ||
620 | v9fs_put_int32(bufp, newfid, &fc->params.twalk.newfid); | ||
621 | v9fs_put_int16(bufp, nwname, &fc->params.twalk.nwname); | ||
622 | for (i = 0; i < nwname; i++) { | ||
623 | v9fs_put_str(bufp, wnames[i], &fc->params.twalk.wnames[i]); | ||
624 | } | ||
625 | |||
626 | if (buf_check_overflow(bufp)) { | ||
627 | kfree(fc); | ||
628 | fc = ERR_PTR(-ENOMEM); | ||
629 | } | ||
630 | error: | ||
631 | return fc; | ||
632 | } | ||
633 | |||
634 | struct v9fs_fcall *v9fs_create_topen(u32 fid, u8 mode) | ||
635 | { | ||
636 | int size; | ||
637 | struct v9fs_fcall *fc; | ||
638 | struct cbuf buffer; | ||
639 | struct cbuf *bufp = &buffer; | ||
640 | |||
641 | size = 4 + 1; /* fid[4] mode[1] */ | ||
642 | fc = v9fs_create_common(bufp, size, TOPEN); | ||
643 | if (IS_ERR(fc)) | ||
644 | goto error; | ||
645 | |||
646 | v9fs_put_int32(bufp, fid, &fc->params.topen.fid); | ||
647 | v9fs_put_int8(bufp, mode, &fc->params.topen.mode); | ||
648 | |||
649 | if (buf_check_overflow(bufp)) { | ||
650 | kfree(fc); | ||
651 | fc = ERR_PTR(-ENOMEM); | ||
652 | } | ||
653 | error: | ||
654 | return fc; | ||
655 | } | ||
656 | |||
657 | struct v9fs_fcall *v9fs_create_tcreate(u32 fid, char *name, u32 perm, u8 mode) | ||
658 | { | ||
659 | int size; | ||
660 | struct v9fs_fcall *fc; | ||
661 | struct cbuf buffer; | ||
662 | struct cbuf *bufp = &buffer; | ||
663 | |||
664 | size = 4 + 2 + strlen(name) + 4 + 1; /* fid[4] name[s] perm[4] mode[1] */ | ||
665 | fc = v9fs_create_common(bufp, size, TCREATE); | ||
666 | if (IS_ERR(fc)) | ||
667 | goto error; | ||
668 | |||
669 | v9fs_put_int32(bufp, fid, &fc->params.tcreate.fid); | ||
670 | v9fs_put_str(bufp, name, &fc->params.tcreate.name); | ||
671 | v9fs_put_int32(bufp, perm, &fc->params.tcreate.perm); | ||
672 | v9fs_put_int8(bufp, mode, &fc->params.tcreate.mode); | ||
673 | |||
674 | if (buf_check_overflow(bufp)) { | ||
675 | kfree(fc); | ||
676 | fc = ERR_PTR(-ENOMEM); | ||
677 | } | ||
678 | error: | ||
679 | return fc; | ||
680 | } | ||
681 | |||
682 | struct v9fs_fcall *v9fs_create_tread(u32 fid, u64 offset, u32 count) | ||
683 | { | ||
684 | int size; | ||
685 | struct v9fs_fcall *fc; | ||
686 | struct cbuf buffer; | ||
687 | struct cbuf *bufp = &buffer; | ||
688 | |||
689 | size = 4 + 8 + 4; /* fid[4] offset[8] count[4] */ | ||
690 | fc = v9fs_create_common(bufp, size, TREAD); | ||
691 | if (IS_ERR(fc)) | ||
692 | goto error; | ||
693 | |||
694 | v9fs_put_int32(bufp, fid, &fc->params.tread.fid); | ||
695 | v9fs_put_int64(bufp, offset, &fc->params.tread.offset); | ||
696 | v9fs_put_int32(bufp, count, &fc->params.tread.count); | ||
697 | |||
698 | if (buf_check_overflow(bufp)) { | ||
699 | kfree(fc); | ||
700 | fc = ERR_PTR(-ENOMEM); | ||
701 | } | ||
702 | error: | ||
703 | return fc; | ||
704 | } | ||
705 | |||
706 | struct v9fs_fcall *v9fs_create_twrite(u32 fid, u64 offset, u32 count, | ||
707 | const char __user * data) | ||
708 | { | ||
709 | int size, err; | ||
710 | struct v9fs_fcall *fc; | ||
711 | struct cbuf buffer; | ||
712 | struct cbuf *bufp = &buffer; | ||
713 | |||
714 | size = 4 + 8 + 4 + count; /* fid[4] offset[8] count[4] data[count] */ | ||
715 | fc = v9fs_create_common(bufp, size, TWRITE); | ||
716 | if (IS_ERR(fc)) | ||
717 | goto error; | ||
718 | |||
719 | v9fs_put_int32(bufp, fid, &fc->params.twrite.fid); | ||
720 | v9fs_put_int64(bufp, offset, &fc->params.twrite.offset); | ||
721 | v9fs_put_int32(bufp, count, &fc->params.twrite.count); | ||
722 | err = v9fs_put_user_data(bufp, data, count, &fc->params.twrite.data); | ||
723 | if (err) { | ||
724 | kfree(fc); | ||
725 | fc = ERR_PTR(err); | ||
726 | } | ||
727 | |||
728 | if (buf_check_overflow(bufp)) { | ||
729 | kfree(fc); | ||
730 | fc = ERR_PTR(-ENOMEM); | ||
731 | } | ||
732 | error: | ||
733 | return fc; | ||
734 | } | ||
735 | |||
736 | struct v9fs_fcall *v9fs_create_tclunk(u32 fid) | ||
737 | { | ||
738 | int size; | ||
739 | struct v9fs_fcall *fc; | ||
740 | struct cbuf buffer; | ||
741 | struct cbuf *bufp = &buffer; | ||
742 | |||
743 | size = 4; /* fid[4] */ | ||
744 | fc = v9fs_create_common(bufp, size, TCLUNK); | ||
745 | if (IS_ERR(fc)) | ||
746 | goto error; | ||
747 | |||
748 | v9fs_put_int32(bufp, fid, &fc->params.tclunk.fid); | ||
749 | |||
750 | if (buf_check_overflow(bufp)) { | ||
751 | kfree(fc); | ||
752 | fc = ERR_PTR(-ENOMEM); | ||
753 | } | ||
754 | error: | ||
755 | return fc; | ||
756 | } | ||
757 | |||
758 | struct v9fs_fcall *v9fs_create_tremove(u32 fid) | ||
759 | { | ||
760 | int size; | ||
761 | struct v9fs_fcall *fc; | ||
762 | struct cbuf buffer; | ||
763 | struct cbuf *bufp = &buffer; | ||
764 | |||
765 | size = 4; /* fid[4] */ | ||
766 | fc = v9fs_create_common(bufp, size, TREMOVE); | ||
767 | if (IS_ERR(fc)) | ||
768 | goto error; | ||
769 | |||
770 | v9fs_put_int32(bufp, fid, &fc->params.tremove.fid); | ||
771 | |||
772 | if (buf_check_overflow(bufp)) { | ||
773 | kfree(fc); | ||
774 | fc = ERR_PTR(-ENOMEM); | ||
775 | } | ||
776 | error: | ||
777 | return fc; | ||
778 | } | ||
779 | |||
780 | struct v9fs_fcall *v9fs_create_tstat(u32 fid) | ||
781 | { | ||
782 | int size; | ||
783 | struct v9fs_fcall *fc; | ||
784 | struct cbuf buffer; | ||
785 | struct cbuf *bufp = &buffer; | ||
786 | |||
787 | size = 4; /* fid[4] */ | ||
788 | fc = v9fs_create_common(bufp, size, TSTAT); | ||
789 | if (IS_ERR(fc)) | ||
790 | goto error; | ||
791 | |||
792 | v9fs_put_int32(bufp, fid, &fc->params.tstat.fid); | ||
793 | |||
794 | if (buf_check_overflow(bufp)) { | ||
795 | kfree(fc); | ||
796 | fc = ERR_PTR(-ENOMEM); | ||
797 | } | ||
798 | error: | ||
799 | return fc; | ||
800 | } | ||
801 | |||
802 | struct v9fs_fcall *v9fs_create_twstat(u32 fid, struct v9fs_wstat *wstat, | ||
803 | int extended) | ||
804 | { | ||
805 | int size, statsz; | ||
806 | struct v9fs_fcall *fc; | ||
807 | struct cbuf buffer; | ||
808 | struct cbuf *bufp = &buffer; | ||
809 | |||
810 | statsz = v9fs_size_wstat(wstat, extended); | ||
811 | size = 4 + 2 + 2 + statsz; /* fid[4] stat[n] */ | ||
812 | fc = v9fs_create_common(bufp, size, TWSTAT); | ||
813 | if (IS_ERR(fc)) | ||
814 | goto error; | ||
815 | |||
816 | v9fs_put_int32(bufp, fid, &fc->params.twstat.fid); | ||
817 | buf_put_int16(bufp, statsz + 2); | ||
818 | v9fs_put_wstat(bufp, wstat, &fc->params.twstat.stat, statsz, extended); | ||
819 | |||
820 | if (buf_check_overflow(bufp)) { | ||
821 | kfree(fc); | ||
822 | fc = ERR_PTR(-ENOMEM); | ||
823 | } | ||
824 | error: | ||
825 | return fc; | ||
708 | } | 826 | } |
diff --git a/fs/9p/conv.h b/fs/9p/conv.h index ee849613c61a..26a736e4a2e7 100644 --- a/fs/9p/conv.h +++ b/fs/9p/conv.h | |||
@@ -1,8 +1,9 @@ | |||
1 | /* | 1 | /* |
2 | * linux/fs/9p/conv.h | 2 | * linux/fs/9p/conv.h |
3 | * | 3 | * |
4 | * 9P protocol conversion definitions | 4 | * 9P protocol conversion definitions. |
5 | * | 5 | * |
6 | * Copyright (C) 2005 by Latchesar Ionkov <lucho@ionkov.net> | ||
6 | * Copyright (C) 2004 by Eric Van Hensbergen <ericvh@gmail.com> | 7 | * Copyright (C) 2004 by Eric Van Hensbergen <ericvh@gmail.com> |
7 | * Copyright (C) 2002 by Ron Minnich <rminnich@lanl.gov> | 8 | * Copyright (C) 2002 by Ron Minnich <rminnich@lanl.gov> |
8 | * | 9 | * |
@@ -24,13 +25,27 @@ | |||
24 | * | 25 | * |
25 | */ | 26 | */ |
26 | 27 | ||
27 | int v9fs_deserialize_stat(struct v9fs_session_info *, void *buf, | 28 | int v9fs_deserialize_stat(void *buf, u32 buflen, struct v9fs_stat *stat, |
28 | u32 buflen, struct v9fs_stat *stat, u32 statlen); | 29 | int extended); |
29 | int v9fs_serialize_fcall(struct v9fs_session_info *, struct v9fs_fcall *tcall, | 30 | int v9fs_deserialize_fcall(void *buf, u32 buflen, struct v9fs_fcall *rcall, |
30 | void *buf, u32 buflen); | 31 | int extended); |
31 | int v9fs_deserialize_fcall(struct v9fs_session_info *, u32 msglen, | ||
32 | void *buf, u32 buflen, struct v9fs_fcall *rcall, | ||
33 | int rcalllen); | ||
34 | 32 | ||
35 | /* this one is actually in error.c right now */ | 33 | void v9fs_set_tag(struct v9fs_fcall *fc, u16 tag); |
36 | int v9fs_errstr2errno(char *errstr); | 34 | |
35 | struct v9fs_fcall *v9fs_create_tversion(u32 msize, char *version); | ||
36 | struct v9fs_fcall *v9fs_create_tauth(u32 afid, char *uname, char *aname); | ||
37 | struct v9fs_fcall *v9fs_create_tattach(u32 fid, u32 afid, char *uname, | ||
38 | char *aname); | ||
39 | struct v9fs_fcall *v9fs_create_tflush(u16 oldtag); | ||
40 | struct v9fs_fcall *v9fs_create_twalk(u32 fid, u32 newfid, u16 nwname, | ||
41 | char **wnames); | ||
42 | struct v9fs_fcall *v9fs_create_topen(u32 fid, u8 mode); | ||
43 | struct v9fs_fcall *v9fs_create_tcreate(u32 fid, char *name, u32 perm, u8 mode); | ||
44 | struct v9fs_fcall *v9fs_create_tread(u32 fid, u64 offset, u32 count); | ||
45 | struct v9fs_fcall *v9fs_create_twrite(u32 fid, u64 offset, u32 count, | ||
46 | const char __user *data); | ||
47 | struct v9fs_fcall *v9fs_create_tclunk(u32 fid); | ||
48 | struct v9fs_fcall *v9fs_create_tremove(u32 fid); | ||
49 | struct v9fs_fcall *v9fs_create_tstat(u32 fid); | ||
50 | struct v9fs_fcall *v9fs_create_twstat(u32 fid, struct v9fs_wstat *wstat, | ||
51 | int extended); | ||
diff --git a/fs/9p/debug.h b/fs/9p/debug.h index 4445f06919d9..fe551032788b 100644 --- a/fs/9p/debug.h +++ b/fs/9p/debug.h | |||
@@ -51,16 +51,23 @@ do { \ | |||
51 | #if DEBUG_DUMP_PKT | 51 | #if DEBUG_DUMP_PKT |
52 | static inline void dump_data(const unsigned char *data, unsigned int datalen) | 52 | static inline void dump_data(const unsigned char *data, unsigned int datalen) |
53 | { | 53 | { |
54 | int i, j; | 54 | int i, n; |
55 | int len = datalen; | 55 | char buf[5*8]; |
56 | 56 | ||
57 | printk(KERN_DEBUG "data "); | 57 | n = 0; |
58 | for (i = 0; i < len; i += 4) { | 58 | i = 0; |
59 | for (j = 0; (j < 4) && (i + j < len); j++) | 59 | while (i < datalen) { |
60 | printk(KERN_DEBUG "%02x", data[i + j]); | 60 | n += snprintf(buf+n, sizeof(buf)-n, "%02x", data[i++]); |
61 | printk(KERN_DEBUG " "); | 61 | if (i%4 == 0) |
62 | n += snprintf(buf+n, sizeof(buf)-n, " "); | ||
63 | |||
64 | if (i%16 == 0) { | ||
65 | dprintk(DEBUG_ERROR, "%s\n", buf); | ||
66 | n = 0; | ||
67 | } | ||
62 | } | 68 | } |
63 | printk(KERN_DEBUG "\n"); | 69 | |
70 | dprintk(DEBUG_ERROR, "%s\n", buf); | ||
64 | } | 71 | } |
65 | #else /* DEBUG_DUMP_PKT */ | 72 | #else /* DEBUG_DUMP_PKT */ |
66 | static inline void dump_data(const unsigned char *data, unsigned int datalen) | 73 | static inline void dump_data(const unsigned char *data, unsigned int datalen) |
diff --git a/fs/9p/error.c b/fs/9p/error.c index 834cb179e388..e4b6f8f38b6f 100644 --- a/fs/9p/error.c +++ b/fs/9p/error.c | |||
@@ -33,7 +33,6 @@ | |||
33 | 33 | ||
34 | #include <linux/list.h> | 34 | #include <linux/list.h> |
35 | #include <linux/jhash.h> | 35 | #include <linux/jhash.h> |
36 | #include <linux/string.h> | ||
37 | 36 | ||
38 | #include "debug.h" | 37 | #include "debug.h" |
39 | #include "error.h" | 38 | #include "error.h" |
@@ -55,7 +54,8 @@ int v9fs_error_init(void) | |||
55 | 54 | ||
56 | /* load initial error map into hash table */ | 55 | /* load initial error map into hash table */ |
57 | for (c = errmap; c->name != NULL; c++) { | 56 | for (c = errmap; c->name != NULL; c++) { |
58 | bucket = jhash(c->name, strlen(c->name), 0) % ERRHASHSZ; | 57 | c->namelen = strlen(c->name); |
58 | bucket = jhash(c->name, c->namelen, 0) % ERRHASHSZ; | ||
59 | INIT_HLIST_NODE(&c->list); | 59 | INIT_HLIST_NODE(&c->list); |
60 | hlist_add_head(&c->list, &hash_errmap[bucket]); | 60 | hlist_add_head(&c->list, &hash_errmap[bucket]); |
61 | } | 61 | } |
@@ -69,15 +69,15 @@ int v9fs_error_init(void) | |||
69 | * | 69 | * |
70 | */ | 70 | */ |
71 | 71 | ||
72 | int v9fs_errstr2errno(char *errstr) | 72 | int v9fs_errstr2errno(char *errstr, int len) |
73 | { | 73 | { |
74 | int errno = 0; | 74 | int errno = 0; |
75 | struct hlist_node *p = NULL; | 75 | struct hlist_node *p = NULL; |
76 | struct errormap *c = NULL; | 76 | struct errormap *c = NULL; |
77 | int bucket = jhash(errstr, strlen(errstr), 0) % ERRHASHSZ; | 77 | int bucket = jhash(errstr, len, 0) % ERRHASHSZ; |
78 | 78 | ||
79 | hlist_for_each_entry(c, p, &hash_errmap[bucket], list) { | 79 | hlist_for_each_entry(c, p, &hash_errmap[bucket], list) { |
80 | if (!strcmp(c->name, errstr)) { | 80 | if (c->namelen==len && !memcmp(c->name, errstr, len)) { |
81 | errno = c->val; | 81 | errno = c->val; |
82 | break; | 82 | break; |
83 | } | 83 | } |
diff --git a/fs/9p/error.h b/fs/9p/error.h index 78f89acf7c9a..a9794e85fe51 100644 --- a/fs/9p/error.h +++ b/fs/9p/error.h | |||
@@ -36,6 +36,7 @@ struct errormap { | |||
36 | char *name; | 36 | char *name; |
37 | int val; | 37 | int val; |
38 | 38 | ||
39 | int namelen; | ||
39 | struct hlist_node list; | 40 | struct hlist_node list; |
40 | }; | 41 | }; |
41 | 42 | ||
@@ -175,4 +176,3 @@ static struct errormap errmap[] = { | |||
175 | }; | 176 | }; |
176 | 177 | ||
177 | extern int v9fs_error_init(void); | 178 | extern int v9fs_error_init(void); |
178 | extern int v9fs_errstr2errno(char *errstr); | ||
diff --git a/fs/9p/fid.c b/fs/9p/fid.c index d95f8626d170..eda449778fa5 100644 --- a/fs/9p/fid.c +++ b/fs/9p/fid.c | |||
@@ -31,9 +31,6 @@ | |||
31 | #include "v9fs.h" | 31 | #include "v9fs.h" |
32 | #include "9p.h" | 32 | #include "9p.h" |
33 | #include "v9fs_vfs.h" | 33 | #include "v9fs_vfs.h" |
34 | #include "transport.h" | ||
35 | #include "mux.h" | ||
36 | #include "conv.h" | ||
37 | #include "fid.h" | 34 | #include "fid.h" |
38 | 35 | ||
39 | /** | 36 | /** |
@@ -164,7 +161,7 @@ static struct v9fs_fid *v9fs_fid_walk_up(struct dentry *dentry) | |||
164 | return v9fs_fid_create(dentry, v9ses, fidnum, 0); | 161 | return v9fs_fid_create(dentry, v9ses, fidnum, 0); |
165 | 162 | ||
166 | clunk_fid: | 163 | clunk_fid: |
167 | v9fs_t_clunk(v9ses, fidnum, NULL); | 164 | v9fs_t_clunk(v9ses, fidnum); |
168 | return ERR_PTR(err); | 165 | return ERR_PTR(err); |
169 | } | 166 | } |
170 | 167 | ||
diff --git a/fs/9p/mux.c b/fs/9p/mux.c index 8835b576f744..945cb368d451 100644 --- a/fs/9p/mux.c +++ b/fs/9p/mux.c | |||
@@ -4,7 +4,7 @@ | |||
4 | * Protocol Multiplexer | 4 | * Protocol Multiplexer |
5 | * | 5 | * |
6 | * Copyright (C) 2004 by Eric Van Hensbergen <ericvh@gmail.com> | 6 | * Copyright (C) 2004 by Eric Van Hensbergen <ericvh@gmail.com> |
7 | * Copyright (C) 2004 by Latchesar Ionkov <lucho@ionkov.net> | 7 | * Copyright (C) 2004-2005 by Latchesar Ionkov <lucho@ionkov.net> |
8 | * | 8 | * |
9 | * This program is free software; you can redistribute it and/or modify | 9 | * This program is free software; you can redistribute it and/or modify |
10 | * it under the terms of the GNU General Public License as published by | 10 | * it under the terms of the GNU General Public License as published by |
@@ -28,448 +28,943 @@ | |||
28 | #include <linux/module.h> | 28 | #include <linux/module.h> |
29 | #include <linux/errno.h> | 29 | #include <linux/errno.h> |
30 | #include <linux/fs.h> | 30 | #include <linux/fs.h> |
31 | #include <linux/poll.h> | ||
31 | #include <linux/kthread.h> | 32 | #include <linux/kthread.h> |
32 | #include <linux/idr.h> | 33 | #include <linux/idr.h> |
33 | 34 | ||
34 | #include "debug.h" | 35 | #include "debug.h" |
35 | #include "v9fs.h" | 36 | #include "v9fs.h" |
36 | #include "9p.h" | 37 | #include "9p.h" |
37 | #include "transport.h" | ||
38 | #include "conv.h" | 38 | #include "conv.h" |
39 | #include "transport.h" | ||
39 | #include "mux.h" | 40 | #include "mux.h" |
40 | 41 | ||
42 | #define ERREQFLUSH 1 | ||
43 | #define SCHED_TIMEOUT 10 | ||
44 | #define MAXPOLLWADDR 2 | ||
45 | |||
46 | enum { | ||
47 | Rworksched = 1, /* read work scheduled or running */ | ||
48 | Rpending = 2, /* can read */ | ||
49 | Wworksched = 4, /* write work scheduled or running */ | ||
50 | Wpending = 8, /* can write */ | ||
51 | }; | ||
52 | |||
53 | struct v9fs_mux_poll_task; | ||
54 | |||
55 | struct v9fs_req { | ||
56 | int tag; | ||
57 | struct v9fs_fcall *tcall; | ||
58 | struct v9fs_fcall *rcall; | ||
59 | int err; | ||
60 | v9fs_mux_req_callback cb; | ||
61 | void *cba; | ||
62 | struct list_head req_list; | ||
63 | }; | ||
64 | |||
65 | struct v9fs_mux_data { | ||
66 | spinlock_t lock; | ||
67 | struct list_head mux_list; | ||
68 | struct v9fs_mux_poll_task *poll_task; | ||
69 | int msize; | ||
70 | unsigned char *extended; | ||
71 | struct v9fs_transport *trans; | ||
72 | struct v9fs_idpool tidpool; | ||
73 | int err; | ||
74 | wait_queue_head_t equeue; | ||
75 | struct list_head req_list; | ||
76 | struct list_head unsent_req_list; | ||
77 | struct v9fs_fcall *rcall; | ||
78 | int rpos; | ||
79 | char *rbuf; | ||
80 | int wpos; | ||
81 | int wsize; | ||
82 | char *wbuf; | ||
83 | wait_queue_t poll_wait[MAXPOLLWADDR]; | ||
84 | wait_queue_head_t *poll_waddr[MAXPOLLWADDR]; | ||
85 | poll_table pt; | ||
86 | struct work_struct rq; | ||
87 | struct work_struct wq; | ||
88 | unsigned long wsched; | ||
89 | }; | ||
90 | |||
91 | struct v9fs_mux_poll_task { | ||
92 | struct task_struct *task; | ||
93 | struct list_head mux_list; | ||
94 | int muxnum; | ||
95 | }; | ||
96 | |||
97 | struct v9fs_mux_rpc { | ||
98 | struct v9fs_mux_data *m; | ||
99 | struct v9fs_req *req; | ||
100 | int err; | ||
101 | struct v9fs_fcall *rcall; | ||
102 | wait_queue_head_t wqueue; | ||
103 | }; | ||
104 | |||
105 | static int v9fs_poll_proc(void *); | ||
106 | static void v9fs_read_work(void *); | ||
107 | static void v9fs_write_work(void *); | ||
108 | static void v9fs_pollwait(struct file *filp, wait_queue_head_t * wait_address, | ||
109 | poll_table * p); | ||
110 | static u16 v9fs_mux_get_tag(struct v9fs_mux_data *); | ||
111 | static void v9fs_mux_put_tag(struct v9fs_mux_data *, u16); | ||
112 | |||
113 | static DECLARE_MUTEX(v9fs_mux_task_lock); | ||
114 | static struct workqueue_struct *v9fs_mux_wq; | ||
115 | |||
116 | static int v9fs_mux_num; | ||
117 | static int v9fs_mux_poll_task_num; | ||
118 | static struct v9fs_mux_poll_task v9fs_mux_poll_tasks[100]; | ||
119 | |||
120 | int v9fs_mux_global_init(void) | ||
121 | { | ||
122 | int i; | ||
123 | |||
124 | for (i = 0; i < ARRAY_SIZE(v9fs_mux_poll_tasks); i++) | ||
125 | v9fs_mux_poll_tasks[i].task = NULL; | ||
126 | |||
127 | v9fs_mux_wq = create_workqueue("v9fs"); | ||
128 | if (!v9fs_mux_wq) | ||
129 | return -ENOMEM; | ||
130 | |||
131 | return 0; | ||
132 | } | ||
133 | |||
134 | void v9fs_mux_global_exit(void) | ||
135 | { | ||
136 | destroy_workqueue(v9fs_mux_wq); | ||
137 | } | ||
138 | |||
41 | /** | 139 | /** |
42 | * dprintcond - print condition of session info | 140 | * v9fs_mux_calc_poll_procs - calculates the number of polling procs |
43 | * @v9ses: session info structure | 141 | * based on the number of mounted v9fs filesystems. |
44 | * @req: RPC request structure | ||
45 | * | 142 | * |
143 | * The current implementation returns sqrt of the number of mounts. | ||
46 | */ | 144 | */ |
145 | inline int v9fs_mux_calc_poll_procs(int muxnum) | ||
146 | { | ||
147 | int n; | ||
148 | |||
149 | if (v9fs_mux_poll_task_num) | ||
150 | n = muxnum / v9fs_mux_poll_task_num + | ||
151 | (muxnum % v9fs_mux_poll_task_num ? 1 : 0); | ||
152 | else | ||
153 | n = 1; | ||
154 | |||
155 | if (n > ARRAY_SIZE(v9fs_mux_poll_tasks)) | ||
156 | n = ARRAY_SIZE(v9fs_mux_poll_tasks); | ||
157 | |||
158 | return n; | ||
159 | } | ||
47 | 160 | ||
48 | static inline int | 161 | static int v9fs_mux_poll_start(struct v9fs_mux_data *m) |
49 | dprintcond(struct v9fs_session_info *v9ses, struct v9fs_rpcreq *req) | ||
50 | { | 162 | { |
51 | dprintk(DEBUG_MUX, "condition: %d, %p\n", v9ses->transport->status, | 163 | int i, n; |
52 | req->rcall); | 164 | struct v9fs_mux_poll_task *vpt, *vptlast; |
165 | struct task_struct *pproc; | ||
166 | |||
167 | dprintk(DEBUG_MUX, "mux %p muxnum %d procnum %d\n", m, v9fs_mux_num, | ||
168 | v9fs_mux_poll_task_num); | ||
169 | up(&v9fs_mux_task_lock); | ||
170 | |||
171 | n = v9fs_mux_calc_poll_procs(v9fs_mux_num + 1); | ||
172 | if (n > v9fs_mux_poll_task_num) { | ||
173 | for (i = 0; i < ARRAY_SIZE(v9fs_mux_poll_tasks); i++) { | ||
174 | if (v9fs_mux_poll_tasks[i].task == NULL) { | ||
175 | vpt = &v9fs_mux_poll_tasks[i]; | ||
176 | dprintk(DEBUG_MUX, "create proc %p\n", vpt); | ||
177 | pproc = kthread_create(v9fs_poll_proc, vpt, | ||
178 | "v9fs-poll"); | ||
179 | |||
180 | if (!IS_ERR(pproc)) { | ||
181 | vpt->task = pproc; | ||
182 | INIT_LIST_HEAD(&vpt->mux_list); | ||
183 | vpt->muxnum = 0; | ||
184 | v9fs_mux_poll_task_num++; | ||
185 | wake_up_process(vpt->task); | ||
186 | } | ||
187 | break; | ||
188 | } | ||
189 | } | ||
190 | |||
191 | if (i >= ARRAY_SIZE(v9fs_mux_poll_tasks)) | ||
192 | dprintk(DEBUG_ERROR, "warning: no free poll slots\n"); | ||
193 | } | ||
194 | |||
195 | n = (v9fs_mux_num + 1) / v9fs_mux_poll_task_num + | ||
196 | ((v9fs_mux_num + 1) % v9fs_mux_poll_task_num ? 1 : 0); | ||
197 | |||
198 | vptlast = NULL; | ||
199 | for (i = 0; i < ARRAY_SIZE(v9fs_mux_poll_tasks); i++) { | ||
200 | vpt = &v9fs_mux_poll_tasks[i]; | ||
201 | if (vpt->task != NULL) { | ||
202 | vptlast = vpt; | ||
203 | if (vpt->muxnum < n) { | ||
204 | dprintk(DEBUG_MUX, "put in proc %d\n", i); | ||
205 | list_add(&m->mux_list, &vpt->mux_list); | ||
206 | vpt->muxnum++; | ||
207 | m->poll_task = vpt; | ||
208 | memset(&m->poll_waddr, 0, sizeof(m->poll_waddr)); | ||
209 | init_poll_funcptr(&m->pt, v9fs_pollwait); | ||
210 | break; | ||
211 | } | ||
212 | } | ||
213 | } | ||
214 | |||
215 | if (i >= ARRAY_SIZE(v9fs_mux_poll_tasks)) { | ||
216 | if (vptlast == NULL) | ||
217 | return -ENOMEM; | ||
218 | |||
219 | dprintk(DEBUG_MUX, "put in proc %d\n", i); | ||
220 | list_add(&m->mux_list, &vptlast->mux_list); | ||
221 | vptlast->muxnum++; | ||
222 | m->poll_task = vptlast; | ||
223 | memset(&m->poll_waddr, 0, sizeof(m->poll_waddr)); | ||
224 | init_poll_funcptr(&m->pt, v9fs_pollwait); | ||
225 | } | ||
226 | |||
227 | v9fs_mux_num++; | ||
228 | down(&v9fs_mux_task_lock); | ||
229 | |||
53 | return 0; | 230 | return 0; |
54 | } | 231 | } |
55 | 232 | ||
233 | static void v9fs_mux_poll_stop(struct v9fs_mux_data *m) | ||
234 | { | ||
235 | int i; | ||
236 | struct v9fs_mux_poll_task *vpt; | ||
237 | |||
238 | up(&v9fs_mux_task_lock); | ||
239 | vpt = m->poll_task; | ||
240 | list_del(&m->mux_list); | ||
241 | for(i = 0; i < ARRAY_SIZE(m->poll_waddr); i++) { | ||
242 | if (m->poll_waddr[i] != NULL) { | ||
243 | remove_wait_queue(m->poll_waddr[i], &m->poll_wait[i]); | ||
244 | m->poll_waddr[i] = NULL; | ||
245 | } | ||
246 | } | ||
247 | vpt->muxnum--; | ||
248 | if (!vpt->muxnum) { | ||
249 | dprintk(DEBUG_MUX, "destroy proc %p\n", vpt); | ||
250 | send_sig(SIGKILL, vpt->task, 1); | ||
251 | vpt->task = NULL; | ||
252 | v9fs_mux_poll_task_num--; | ||
253 | } | ||
254 | v9fs_mux_num--; | ||
255 | down(&v9fs_mux_task_lock); | ||
256 | } | ||
257 | |||
56 | /** | 258 | /** |
57 | * xread - force read of a certain number of bytes | 259 | * v9fs_mux_init - allocate and initialize the per-session mux data |
58 | * @v9ses: session info structure | 260 | * Creates the polling task if this is the first session. |
59 | * @ptr: pointer to buffer | ||
60 | * @sz: number of bytes to read | ||
61 | * | 261 | * |
62 | * Chuck Cranor CS-533 project1 | 262 | * @trans - transport structure |
263 | * @msize - maximum message size | ||
264 | * @extended - pointer to the extended flag | ||
63 | */ | 265 | */ |
64 | 266 | struct v9fs_mux_data *v9fs_mux_init(struct v9fs_transport *trans, int msize, | |
65 | static int xread(struct v9fs_session_info *v9ses, void *ptr, unsigned long sz) | 267 | unsigned char *extended) |
66 | { | 268 | { |
67 | int rd = 0; | 269 | int i, n; |
68 | int ret = 0; | 270 | struct v9fs_mux_data *m, *mtmp; |
69 | while (rd < sz) { | 271 | |
70 | ret = v9ses->transport->read(v9ses->transport, ptr, sz - rd); | 272 | dprintk(DEBUG_MUX, "transport %p msize %d\n", trans, msize); |
71 | if (ret <= 0) { | 273 | m = kmalloc(sizeof(struct v9fs_mux_data), GFP_KERNEL); |
72 | dprintk(DEBUG_ERROR, "xread errno %d\n", ret); | 274 | if (!m) |
73 | return ret; | 275 | return ERR_PTR(-ENOMEM); |
276 | |||
277 | spin_lock_init(&m->lock); | ||
278 | INIT_LIST_HEAD(&m->mux_list); | ||
279 | m->msize = msize; | ||
280 | m->extended = extended; | ||
281 | m->trans = trans; | ||
282 | idr_init(&m->tidpool.pool); | ||
283 | init_MUTEX(&m->tidpool.lock); | ||
284 | m->err = 0; | ||
285 | init_waitqueue_head(&m->equeue); | ||
286 | INIT_LIST_HEAD(&m->req_list); | ||
287 | INIT_LIST_HEAD(&m->unsent_req_list); | ||
288 | m->rcall = NULL; | ||
289 | m->rpos = 0; | ||
290 | m->rbuf = NULL; | ||
291 | m->wpos = m->wsize = 0; | ||
292 | m->wbuf = NULL; | ||
293 | INIT_WORK(&m->rq, v9fs_read_work, m); | ||
294 | INIT_WORK(&m->wq, v9fs_write_work, m); | ||
295 | m->wsched = 0; | ||
296 | memset(&m->poll_waddr, 0, sizeof(m->poll_waddr)); | ||
297 | m->poll_task = NULL; | ||
298 | n = v9fs_mux_poll_start(m); | ||
299 | if (n) | ||
300 | return ERR_PTR(n); | ||
301 | |||
302 | n = trans->poll(trans, &m->pt); | ||
303 | if (n & POLLIN) { | ||
304 | dprintk(DEBUG_MUX, "mux %p can read\n", m); | ||
305 | set_bit(Rpending, &m->wsched); | ||
306 | } | ||
307 | |||
308 | if (n & POLLOUT) { | ||
309 | dprintk(DEBUG_MUX, "mux %p can write\n", m); | ||
310 | set_bit(Wpending, &m->wsched); | ||
311 | } | ||
312 | |||
313 | for(i = 0; i < ARRAY_SIZE(m->poll_waddr); i++) { | ||
314 | if (IS_ERR(m->poll_waddr[i])) { | ||
315 | v9fs_mux_poll_stop(m); | ||
316 | mtmp = (void *)m->poll_waddr; /* the error code */ | ||
317 | kfree(m); | ||
318 | m = mtmp; | ||
319 | break; | ||
74 | } | 320 | } |
75 | rd += ret; | ||
76 | ptr += ret; | ||
77 | } | 321 | } |
78 | return (rd); | 322 | |
323 | return m; | ||
79 | } | 324 | } |
80 | 325 | ||
81 | /** | 326 | /** |
82 | * read_message - read a full 9P2000 fcall packet | 327 | * v9fs_mux_destroy - cancels all pending requests and frees mux resources |
83 | * @v9ses: session info structure | ||
84 | * @rcall: fcall structure to read into | ||
85 | * @rcalllen: size of fcall buffer | ||
86 | * | ||
87 | */ | 328 | */ |
329 | void v9fs_mux_destroy(struct v9fs_mux_data *m) | ||
330 | { | ||
331 | dprintk(DEBUG_MUX, "mux %p prev %p next %p\n", m, | ||
332 | m->mux_list.prev, m->mux_list.next); | ||
333 | v9fs_mux_cancel(m, -ECONNRESET); | ||
334 | |||
335 | if (!list_empty(&m->req_list)) { | ||
336 | /* wait until all processes waiting on this session exit */ | ||
337 | dprintk(DEBUG_MUX, "mux %p waiting for empty request queue\n", | ||
338 | m); | ||
339 | wait_event_timeout(m->equeue, (list_empty(&m->req_list)), 5000); | ||
340 | dprintk(DEBUG_MUX, "mux %p request queue empty: %d\n", m, | ||
341 | list_empty(&m->req_list)); | ||
342 | } | ||
343 | |||
344 | v9fs_mux_poll_stop(m); | ||
345 | m->trans = NULL; | ||
346 | |||
347 | kfree(m); | ||
348 | } | ||
88 | 349 | ||
89 | static int | 350 | /** |
90 | read_message(struct v9fs_session_info *v9ses, | 351 | * v9fs_pollwait - called by files poll operation to add v9fs-poll task |
91 | struct v9fs_fcall *rcall, int rcalllen) | 352 | * to files wait queue |
353 | */ | ||
354 | static void | ||
355 | v9fs_pollwait(struct file *filp, wait_queue_head_t * wait_address, | ||
356 | poll_table * p) | ||
92 | { | 357 | { |
93 | unsigned char buf[4]; | 358 | int i; |
94 | void *data; | 359 | struct v9fs_mux_data *m; |
95 | int size = 0; | 360 | |
96 | int res = 0; | 361 | m = container_of(p, struct v9fs_mux_data, pt); |
97 | 362 | for(i = 0; i < ARRAY_SIZE(m->poll_waddr); i++) | |
98 | res = xread(v9ses, buf, sizeof(buf)); | 363 | if (m->poll_waddr[i] == NULL) |
99 | if (res < 0) { | 364 | break; |
100 | dprintk(DEBUG_ERROR, | 365 | |
101 | "Reading of count field failed returned: %d\n", res); | 366 | if (i >= ARRAY_SIZE(m->poll_waddr)) { |
102 | return res; | 367 | dprintk(DEBUG_ERROR, "not enough wait_address slots\n"); |
368 | return; | ||
103 | } | 369 | } |
104 | 370 | ||
105 | if (res < 4) { | 371 | m->poll_waddr[i] = wait_address; |
106 | dprintk(DEBUG_ERROR, | 372 | |
107 | "Reading of count field failed returned: %d\n", res); | 373 | if (!wait_address) { |
108 | return -EIO; | 374 | dprintk(DEBUG_ERROR, "no wait_address\n"); |
375 | m->poll_waddr[i] = ERR_PTR(-EIO); | ||
376 | return; | ||
109 | } | 377 | } |
110 | 378 | ||
111 | size = buf[0] | (buf[1] << 8) | (buf[2] << 16) | (buf[3] << 24); | 379 | init_waitqueue_entry(&m->poll_wait[i], m->poll_task->task); |
112 | dprintk(DEBUG_MUX, "got a packet count: %d\n", size); | 380 | add_wait_queue(wait_address, &m->poll_wait[i]); |
381 | } | ||
382 | |||
383 | /** | ||
384 | * v9fs_poll_mux - polls a mux and schedules read or write works if necessary | ||
385 | */ | ||
386 | static inline void v9fs_poll_mux(struct v9fs_mux_data *m) | ||
387 | { | ||
388 | int n; | ||
113 | 389 | ||
114 | /* adjust for the four bytes of size */ | 390 | if (m->err < 0) |
115 | size -= 4; | 391 | return; |
116 | 392 | ||
117 | if (size > v9ses->maxdata) { | 393 | n = m->trans->poll(m->trans, NULL); |
118 | dprintk(DEBUG_ERROR, "packet too big: %d\n", size); | 394 | if (n < 0 || n & (POLLERR | POLLHUP | POLLNVAL)) { |
119 | return -E2BIG; | 395 | dprintk(DEBUG_MUX, "error mux %p err %d\n", m, n); |
396 | if (n >= 0) | ||
397 | n = -ECONNRESET; | ||
398 | v9fs_mux_cancel(m, n); | ||
120 | } | 399 | } |
121 | 400 | ||
122 | data = kmalloc(size, GFP_KERNEL); | 401 | if (n & POLLIN) { |
123 | if (!data) { | 402 | set_bit(Rpending, &m->wsched); |
124 | eprintk(KERN_WARNING, "out of memory\n"); | 403 | dprintk(DEBUG_MUX, "mux %p can read\n", m); |
125 | return -ENOMEM; | 404 | if (!test_and_set_bit(Rworksched, &m->wsched)) { |
405 | dprintk(DEBUG_MUX, "schedule read work mux %p\n", m); | ||
406 | queue_work(v9fs_mux_wq, &m->rq); | ||
407 | } | ||
126 | } | 408 | } |
127 | 409 | ||
128 | res = xread(v9ses, data, size); | 410 | if (n & POLLOUT) { |
129 | if (res < size) { | 411 | set_bit(Wpending, &m->wsched); |
130 | dprintk(DEBUG_ERROR, "Reading of fcall failed returned: %d\n", | 412 | dprintk(DEBUG_MUX, "mux %p can write\n", m); |
131 | res); | 413 | if ((m->wsize || !list_empty(&m->unsent_req_list)) |
132 | kfree(data); | 414 | && !test_and_set_bit(Wworksched, &m->wsched)) { |
133 | return res; | 415 | dprintk(DEBUG_MUX, "schedule write work mux %p\n", m); |
416 | queue_work(v9fs_mux_wq, &m->wq); | ||
417 | } | ||
134 | } | 418 | } |
419 | } | ||
420 | |||
421 | /** | ||
422 | * v9fs_poll_proc - polls all v9fs transports for new events and queues | ||
423 | * the appropriate work to the work queue | ||
424 | */ | ||
425 | static int v9fs_poll_proc(void *a) | ||
426 | { | ||
427 | struct v9fs_mux_data *m, *mtmp; | ||
428 | struct v9fs_mux_poll_task *vpt; | ||
135 | 429 | ||
136 | /* we now have an in-memory string that is the reply. | 430 | vpt = a; |
137 | * deserialize it. There is very little to go wrong at this point | 431 | dprintk(DEBUG_MUX, "start %p %p\n", current, vpt); |
138 | * save for v9fs_alloc errors. | 432 | allow_signal(SIGKILL); |
139 | */ | 433 | while (!kthread_should_stop()) { |
140 | res = v9fs_deserialize_fcall(v9ses, size, data, v9ses->maxdata, | 434 | set_current_state(TASK_INTERRUPTIBLE); |
141 | rcall, rcalllen); | 435 | if (signal_pending(current)) |
436 | break; | ||
142 | 437 | ||
143 | kfree(data); | 438 | list_for_each_entry_safe(m, mtmp, &vpt->mux_list, mux_list) { |
439 | v9fs_poll_mux(m); | ||
440 | } | ||
144 | 441 | ||
145 | if (res < 0) | 442 | dprintk(DEBUG_MUX, "sleeping...\n"); |
146 | return res; | 443 | schedule_timeout(SCHED_TIMEOUT * HZ); |
444 | } | ||
147 | 445 | ||
446 | __set_current_state(TASK_RUNNING); | ||
447 | dprintk(DEBUG_MUX, "finish\n"); | ||
148 | return 0; | 448 | return 0; |
149 | } | 449 | } |
150 | 450 | ||
151 | /** | 451 | /** |
152 | * v9fs_recv - receive an RPC response for a particular tag | 452 | * v9fs_write_work - called when a transport can send some data |
153 | * @v9ses: session info structure | ||
154 | * @req: RPC request structure | ||
155 | * | ||
156 | */ | 453 | */ |
157 | 454 | static void v9fs_write_work(void *a) | |
158 | static int v9fs_recv(struct v9fs_session_info *v9ses, struct v9fs_rpcreq *req) | ||
159 | { | 455 | { |
160 | int ret = 0; | 456 | int n, err; |
457 | struct v9fs_mux_data *m; | ||
458 | struct v9fs_req *req; | ||
161 | 459 | ||
162 | dprintk(DEBUG_MUX, "waiting for response: %d\n", req->tcall->tag); | 460 | m = a; |
163 | ret = wait_event_interruptible(v9ses->read_wait, | ||
164 | ((v9ses->transport->status != Connected) || | ||
165 | (req->rcall != 0) || (req->err < 0) || | ||
166 | dprintcond(v9ses, req))); | ||
167 | 461 | ||
168 | dprintk(DEBUG_MUX, "got it: rcall %p\n", req->rcall); | 462 | if (m->err < 0) { |
463 | clear_bit(Wworksched, &m->wsched); | ||
464 | return; | ||
465 | } | ||
169 | 466 | ||
170 | spin_lock(&v9ses->muxlock); | 467 | if (!m->wsize) { |
171 | list_del(&req->next); | 468 | if (list_empty(&m->unsent_req_list)) { |
172 | spin_unlock(&v9ses->muxlock); | 469 | clear_bit(Wworksched, &m->wsched); |
470 | return; | ||
471 | } | ||
173 | 472 | ||
174 | if (req->err < 0) | 473 | spin_lock(&m->lock); |
175 | return req->err; | 474 | req = |
475 | list_entry(m->unsent_req_list.next, struct v9fs_req, | ||
476 | req_list); | ||
477 | list_move_tail(&req->req_list, &m->req_list); | ||
478 | m->wbuf = req->tcall->sdata; | ||
479 | m->wsize = req->tcall->size; | ||
480 | m->wpos = 0; | ||
481 | dump_data(m->wbuf, m->wsize); | ||
482 | spin_unlock(&m->lock); | ||
483 | } | ||
176 | 484 | ||
177 | if (v9ses->transport->status == Disconnected) | 485 | dprintk(DEBUG_MUX, "mux %p pos %d size %d\n", m, m->wpos, m->wsize); |
178 | return -ECONNRESET; | 486 | clear_bit(Wpending, &m->wsched); |
487 | err = m->trans->write(m->trans, m->wbuf + m->wpos, m->wsize - m->wpos); | ||
488 | dprintk(DEBUG_MUX, "mux %p sent %d bytes\n", m, err); | ||
489 | if (err == -EAGAIN) { | ||
490 | clear_bit(Wworksched, &m->wsched); | ||
491 | return; | ||
492 | } | ||
179 | 493 | ||
180 | return ret; | 494 | if (err <= 0) |
181 | } | 495 | goto error; |
182 | 496 | ||
183 | /** | 497 | m->wpos += err; |
184 | * v9fs_send - send a 9P request | 498 | if (m->wpos == m->wsize) |
185 | * @v9ses: session info structure | 499 | m->wpos = m->wsize = 0; |
186 | * @req: RPC request to send | 500 | |
187 | * | 501 | if (m->wsize == 0 && !list_empty(&m->unsent_req_list)) { |
188 | */ | 502 | if (test_and_clear_bit(Wpending, &m->wsched)) |
503 | n = POLLOUT; | ||
504 | else | ||
505 | n = m->trans->poll(m->trans, NULL); | ||
506 | |||
507 | if (n & POLLOUT) { | ||
508 | dprintk(DEBUG_MUX, "schedule write work mux %p\n", m); | ||
509 | queue_work(v9fs_mux_wq, &m->wq); | ||
510 | } else | ||
511 | clear_bit(Wworksched, &m->wsched); | ||
512 | } else | ||
513 | clear_bit(Wworksched, &m->wsched); | ||
514 | |||
515 | return; | ||
189 | 516 | ||
190 | static int v9fs_send(struct v9fs_session_info *v9ses, struct v9fs_rpcreq *req) | 517 | error: |
518 | v9fs_mux_cancel(m, err); | ||
519 | clear_bit(Wworksched, &m->wsched); | ||
520 | } | ||
521 | |||
522 | static void process_request(struct v9fs_mux_data *m, struct v9fs_req *req) | ||
191 | { | 523 | { |
192 | int ret = -1; | 524 | int ecode, tag; |
193 | void *data = NULL; | 525 | struct v9fs_str *ename; |
194 | struct v9fs_fcall *tcall = req->tcall; | ||
195 | 526 | ||
196 | data = kmalloc(v9ses->maxdata + V9FS_IOHDRSZ, GFP_KERNEL); | 527 | tag = req->tag; |
197 | if (!data) | 528 | if (req->rcall->id == RERROR && !req->err) { |
198 | return -ENOMEM; | 529 | ecode = req->rcall->params.rerror.errno; |
530 | ename = &req->rcall->params.rerror.error; | ||
199 | 531 | ||
200 | tcall->size = 0; /* enforce size recalculation */ | 532 | dprintk(DEBUG_MUX, "Rerror %.*s\n", ename->len, ename->str); |
201 | ret = | ||
202 | v9fs_serialize_fcall(v9ses, tcall, data, | ||
203 | v9ses->maxdata + V9FS_IOHDRSZ); | ||
204 | if (ret < 0) | ||
205 | goto free_data; | ||
206 | 533 | ||
207 | spin_lock(&v9ses->muxlock); | 534 | if (*m->extended) |
208 | list_add(&req->next, &v9ses->mux_fcalls); | 535 | req->err = -ecode; |
209 | spin_unlock(&v9ses->muxlock); | ||
210 | 536 | ||
211 | dprintk(DEBUG_MUX, "sending message: tag %d size %d\n", tcall->tag, | 537 | if (!req->err) { |
212 | tcall->size); | 538 | req->err = v9fs_errstr2errno(ename->str, ename->len); |
213 | ret = v9ses->transport->write(v9ses->transport, data, tcall->size); | ||
214 | 539 | ||
215 | if (ret != tcall->size) { | 540 | if (!req->err) { /* string match failed */ |
216 | spin_lock(&v9ses->muxlock); | 541 | PRINT_FCALL_ERROR("unknown error", req->rcall); |
217 | list_del(&req->next); | 542 | } |
218 | kfree(req->rcall); | 543 | |
544 | if (!req->err) | ||
545 | req->err = -ESERVERFAULT; | ||
546 | } | ||
547 | } else if (req->tcall && req->rcall->id != req->tcall->id + 1) { | ||
548 | dprintk(DEBUG_ERROR, "fcall mismatch: expected %d, got %d\n", | ||
549 | req->tcall->id + 1, req->rcall->id); | ||
550 | if (!req->err) | ||
551 | req->err = -EIO; | ||
552 | } | ||
219 | 553 | ||
220 | spin_unlock(&v9ses->muxlock); | 554 | if (req->cb && req->err != ERREQFLUSH) { |
221 | if (ret >= 0) | 555 | dprintk(DEBUG_MUX, "calling callback tcall %p rcall %p\n", |
222 | ret = -EREMOTEIO; | 556 | req->tcall, req->rcall); |
557 | |||
558 | (*req->cb) (req->cba, req->tcall, req->rcall, req->err); | ||
559 | req->cb = NULL; | ||
223 | } else | 560 | } else |
224 | ret = 0; | 561 | kfree(req->rcall); |
225 | 562 | ||
226 | free_data: | 563 | v9fs_mux_put_tag(m, tag); |
227 | kfree(data); | 564 | |
228 | return ret; | 565 | wake_up(&m->equeue); |
566 | kfree(req); | ||
229 | } | 567 | } |
230 | 568 | ||
231 | /** | 569 | /** |
232 | * v9fs_mux_rpc - send a request, receive a response | 570 | * v9fs_read_work - called when there is some data to be read from a transport |
233 | * @v9ses: session info structure | ||
234 | * @tcall: fcall to send | ||
235 | * @rcall: buffer to place response into | ||
236 | * | ||
237 | */ | 571 | */ |
238 | 572 | static void v9fs_read_work(void *a) | |
239 | long | ||
240 | v9fs_mux_rpc(struct v9fs_session_info *v9ses, struct v9fs_fcall *tcall, | ||
241 | struct v9fs_fcall **rcall) | ||
242 | { | 573 | { |
243 | int tid = -1; | 574 | int n, err; |
244 | struct v9fs_fcall *fcall = NULL; | 575 | struct v9fs_mux_data *m; |
245 | struct v9fs_rpcreq req; | 576 | struct v9fs_req *req, *rptr, *rreq; |
246 | int ret = -1; | 577 | struct v9fs_fcall *rcall; |
247 | 578 | char *rbuf; | |
248 | if (!v9ses) | 579 | |
249 | return -EINVAL; | 580 | m = a; |
250 | 581 | ||
251 | if (!v9ses->transport || v9ses->transport->status != Connected) | 582 | if (m->err < 0) |
252 | return -EIO; | 583 | return; |
584 | |||
585 | rcall = NULL; | ||
586 | dprintk(DEBUG_MUX, "start mux %p pos %d\n", m, m->rpos); | ||
587 | |||
588 | if (!m->rcall) { | ||
589 | m->rcall = | ||
590 | kmalloc(sizeof(struct v9fs_fcall) + m->msize, GFP_KERNEL); | ||
591 | if (!m->rcall) { | ||
592 | err = -ENOMEM; | ||
593 | goto error; | ||
594 | } | ||
253 | 595 | ||
254 | if (rcall) | 596 | m->rbuf = (char *)m->rcall + sizeof(struct v9fs_fcall); |
255 | *rcall = NULL; | 597 | m->rpos = 0; |
598 | } | ||
256 | 599 | ||
257 | if (tcall->id != TVERSION) { | 600 | clear_bit(Rpending, &m->wsched); |
258 | tid = v9fs_get_idpool(&v9ses->tidpool); | 601 | err = m->trans->read(m->trans, m->rbuf + m->rpos, m->msize - m->rpos); |
259 | if (tid < 0) | 602 | dprintk(DEBUG_MUX, "mux %p got %d bytes\n", m, err); |
260 | return -ENOMEM; | 603 | if (err == -EAGAIN) { |
604 | clear_bit(Rworksched, &m->wsched); | ||
605 | return; | ||
261 | } | 606 | } |
262 | 607 | ||
263 | tcall->tag = tid; | 608 | if (err <= 0) |
609 | goto error; | ||
264 | 610 | ||
265 | req.tcall = tcall; | 611 | m->rpos += err; |
266 | req.err = 0; | 612 | while (m->rpos > 4) { |
267 | req.rcall = NULL; | 613 | n = le32_to_cpu(*(__le32 *) m->rbuf); |
614 | if (n >= m->msize) { | ||
615 | dprintk(DEBUG_ERROR, | ||
616 | "requested packet size too big: %d\n", n); | ||
617 | err = -EIO; | ||
618 | goto error; | ||
619 | } | ||
268 | 620 | ||
269 | ret = v9fs_send(v9ses, &req); | 621 | if (m->rpos < n) |
622 | break; | ||
270 | 623 | ||
271 | if (ret < 0) { | 624 | dump_data(m->rbuf, n); |
272 | if (tcall->id != TVERSION) | 625 | err = |
273 | v9fs_put_idpool(tid, &v9ses->tidpool); | 626 | v9fs_deserialize_fcall(m->rbuf, n, m->rcall, *m->extended); |
274 | dprintk(DEBUG_MUX, "error %d\n", ret); | 627 | if (err < 0) { |
275 | return ret; | 628 | goto error; |
276 | } | 629 | } |
630 | |||
631 | rcall = m->rcall; | ||
632 | rbuf = m->rbuf; | ||
633 | if (m->rpos > n) { | ||
634 | m->rcall = kmalloc(sizeof(struct v9fs_fcall) + m->msize, | ||
635 | GFP_KERNEL); | ||
636 | if (!m->rcall) { | ||
637 | err = -ENOMEM; | ||
638 | goto error; | ||
639 | } | ||
277 | 640 | ||
278 | ret = v9fs_recv(v9ses, &req); | 641 | m->rbuf = (char *)m->rcall + sizeof(struct v9fs_fcall); |
279 | 642 | memmove(m->rbuf, rbuf + n, m->rpos - n); | |
280 | fcall = req.rcall; | 643 | m->rpos -= n; |
281 | 644 | } else { | |
282 | dprintk(DEBUG_MUX, "received: tag=%x, ret=%d\n", tcall->tag, ret); | 645 | m->rcall = NULL; |
283 | if (ret == -ERESTARTSYS) { | 646 | m->rbuf = NULL; |
284 | if (v9ses->transport->status != Disconnected | 647 | m->rpos = 0; |
285 | && tcall->id != TFLUSH) { | ||
286 | unsigned long flags; | ||
287 | |||
288 | dprintk(DEBUG_MUX, "flushing the tag: %d\n", | ||
289 | tcall->tag); | ||
290 | clear_thread_flag(TIF_SIGPENDING); | ||
291 | v9fs_t_flush(v9ses, tcall->tag); | ||
292 | spin_lock_irqsave(¤t->sighand->siglock, flags); | ||
293 | recalc_sigpending(); | ||
294 | spin_unlock_irqrestore(¤t->sighand->siglock, | ||
295 | flags); | ||
296 | dprintk(DEBUG_MUX, "flushing done\n"); | ||
297 | } | 648 | } |
298 | 649 | ||
299 | goto release_req; | 650 | dprintk(DEBUG_MUX, "mux %p fcall id %d tag %d\n", m, rcall->id, |
300 | } else if (ret < 0) | 651 | rcall->tag); |
301 | goto release_req; | 652 | |
302 | 653 | req = NULL; | |
303 | if (!fcall) | 654 | spin_lock(&m->lock); |
304 | ret = -EIO; | 655 | list_for_each_entry_safe(rreq, rptr, &m->req_list, req_list) { |
305 | else { | 656 | if (rreq->tag == rcall->tag) { |
306 | if (fcall->id == RERROR) { | 657 | req = rreq; |
307 | ret = v9fs_errstr2errno(fcall->params.rerror.error); | 658 | req->rcall = rcall; |
308 | if (ret == 0) { /* string match failed */ | 659 | list_del(&req->req_list); |
309 | if (fcall->params.rerror.errno) | 660 | spin_unlock(&m->lock); |
310 | ret = -(fcall->params.rerror.errno); | 661 | process_request(m, req); |
311 | else | 662 | break; |
312 | ret = -ESERVERFAULT; | ||
313 | } | 663 | } |
314 | } else if (fcall->id != tcall->id + 1) { | 664 | |
315 | dprintk(DEBUG_ERROR, | 665 | } |
316 | "fcall mismatch: expected %d, got %d\n", | 666 | |
317 | tcall->id + 1, fcall->id); | 667 | if (!req) { |
318 | ret = -EIO; | 668 | spin_unlock(&m->lock); |
669 | if (err >= 0 && rcall->id != RFLUSH) | ||
670 | dprintk(DEBUG_ERROR, | ||
671 | "unexpected response mux %p id %d tag %d\n", | ||
672 | m, rcall->id, rcall->tag); | ||
673 | kfree(rcall); | ||
319 | } | 674 | } |
320 | } | 675 | } |
321 | 676 | ||
322 | release_req: | 677 | if (!list_empty(&m->req_list)) { |
323 | if (tcall->id != TVERSION) | 678 | if (test_and_clear_bit(Rpending, &m->wsched)) |
324 | v9fs_put_idpool(tid, &v9ses->tidpool); | 679 | n = POLLIN; |
325 | if (rcall) | 680 | else |
326 | *rcall = fcall; | 681 | n = m->trans->poll(m->trans, NULL); |
327 | else | 682 | |
328 | kfree(fcall); | 683 | if (n & POLLIN) { |
684 | dprintk(DEBUG_MUX, "schedule read work mux %p\n", m); | ||
685 | queue_work(v9fs_mux_wq, &m->rq); | ||
686 | } else | ||
687 | clear_bit(Rworksched, &m->wsched); | ||
688 | } else | ||
689 | clear_bit(Rworksched, &m->wsched); | ||
690 | |||
691 | return; | ||
329 | 692 | ||
330 | return ret; | 693 | error: |
694 | v9fs_mux_cancel(m, err); | ||
695 | clear_bit(Rworksched, &m->wsched); | ||
331 | } | 696 | } |
332 | 697 | ||
333 | /** | 698 | /** |
334 | * v9fs_mux_cancel_requests - cancels all pending requests | 699 | * v9fs_send_request - send 9P request |
700 | * The function can sleep until the request is scheduled for sending. | ||
701 | * The function can be interrupted. Return from the function is not | ||
702 | * a guarantee that the request is sent succesfully. Can return errors | ||
703 | * that can be retrieved by PTR_ERR macros. | ||
335 | * | 704 | * |
336 | * @v9ses: session info structure | 705 | * @m: mux data |
337 | * @err: error code to return to the requests | 706 | * @tc: request to be sent |
707 | * @cb: callback function to call when response is received | ||
708 | * @cba: parameter to pass to the callback function | ||
338 | */ | 709 | */ |
339 | void v9fs_mux_cancel_requests(struct v9fs_session_info *v9ses, int err) | 710 | static struct v9fs_req *v9fs_send_request(struct v9fs_mux_data *m, |
711 | struct v9fs_fcall *tc, | ||
712 | v9fs_mux_req_callback cb, void *cba) | ||
340 | { | 713 | { |
341 | struct v9fs_rpcreq *rptr; | 714 | int n; |
342 | struct v9fs_rpcreq *rreq; | 715 | struct v9fs_req *req; |
343 | 716 | ||
344 | dprintk(DEBUG_MUX, " %d\n", err); | 717 | dprintk(DEBUG_MUX, "mux %p task %p tcall %p id %d\n", m, current, |
345 | spin_lock(&v9ses->muxlock); | 718 | tc, tc->id); |
346 | list_for_each_entry_safe(rreq, rptr, &v9ses->mux_fcalls, next) { | 719 | if (m->err < 0) |
347 | rreq->err = err; | 720 | return ERR_PTR(m->err); |
348 | } | ||
349 | spin_unlock(&v9ses->muxlock); | ||
350 | wake_up_all(&v9ses->read_wait); | ||
351 | } | ||
352 | 721 | ||
353 | /** | 722 | req = kmalloc(sizeof(struct v9fs_req), GFP_KERNEL); |
354 | * v9fs_recvproc - kproc to handle demultiplexing responses | 723 | if (!req) |
355 | * @data: session info structure | 724 | return ERR_PTR(-ENOMEM); |
356 | * | ||
357 | */ | ||
358 | 725 | ||
359 | static int v9fs_recvproc(void *data) | 726 | if (tc->id == TVERSION) |
360 | { | 727 | n = V9FS_NOTAG; |
361 | struct v9fs_session_info *v9ses = (struct v9fs_session_info *)data; | 728 | else |
362 | struct v9fs_fcall *rcall = NULL; | 729 | n = v9fs_mux_get_tag(m); |
363 | struct v9fs_rpcreq *rptr; | ||
364 | struct v9fs_rpcreq *req; | ||
365 | struct v9fs_rpcreq *rreq; | ||
366 | int err = 0; | ||
367 | 730 | ||
368 | allow_signal(SIGKILL); | 731 | if (n < 0) |
369 | set_current_state(TASK_INTERRUPTIBLE); | 732 | return ERR_PTR(-ENOMEM); |
370 | complete(&v9ses->proccmpl); | ||
371 | while (!kthread_should_stop() && err >= 0) { | ||
372 | req = rptr = rreq = NULL; | ||
373 | |||
374 | rcall = kmalloc(v9ses->maxdata + V9FS_IOHDRSZ, GFP_KERNEL); | ||
375 | if (!rcall) { | ||
376 | eprintk(KERN_ERR, "no memory for buffers\n"); | ||
377 | break; | ||
378 | } | ||
379 | 733 | ||
380 | err = read_message(v9ses, rcall, v9ses->maxdata + V9FS_IOHDRSZ); | 734 | v9fs_set_tag(tc, n); |
381 | spin_lock(&v9ses->muxlock); | ||
382 | if (err < 0) { | ||
383 | list_for_each_entry_safe(rreq, rptr, &v9ses->mux_fcalls, next) { | ||
384 | rreq->err = err; | ||
385 | } | ||
386 | if(err != -ERESTARTSYS) | ||
387 | eprintk(KERN_ERR, | ||
388 | "Transport error while reading message %d\n", err); | ||
389 | } else { | ||
390 | list_for_each_entry_safe(rreq, rptr, &v9ses->mux_fcalls, next) { | ||
391 | if (rreq->tcall->tag == rcall->tag) { | ||
392 | req = rreq; | ||
393 | req->rcall = rcall; | ||
394 | break; | ||
395 | } | ||
396 | } | ||
397 | } | ||
398 | 735 | ||
399 | if (req && (req->tcall->id == TFLUSH)) { | 736 | req->tag = n; |
400 | struct v9fs_rpcreq *treq = NULL; | 737 | req->tcall = tc; |
401 | list_for_each_entry_safe(treq, rptr, &v9ses->mux_fcalls, next) { | 738 | req->rcall = NULL; |
402 | if (treq->tcall->tag == | 739 | req->err = 0; |
403 | req->tcall->params.tflush.oldtag) { | 740 | req->cb = cb; |
404 | list_del(&rptr->next); | 741 | req->cba = cba; |
405 | kfree(treq->rcall); | 742 | |
406 | break; | 743 | spin_lock(&m->lock); |
407 | } | 744 | list_add_tail(&req->req_list, &m->unsent_req_list); |
745 | spin_unlock(&m->lock); | ||
746 | |||
747 | if (test_and_clear_bit(Wpending, &m->wsched)) | ||
748 | n = POLLOUT; | ||
749 | else | ||
750 | n = m->trans->poll(m->trans, NULL); | ||
751 | |||
752 | if (n & POLLOUT && !test_and_set_bit(Wworksched, &m->wsched)) | ||
753 | queue_work(v9fs_mux_wq, &m->wq); | ||
754 | |||
755 | return req; | ||
756 | } | ||
757 | |||
758 | static inline void | ||
759 | v9fs_mux_flush_cb(void *a, struct v9fs_fcall *tc, struct v9fs_fcall *rc, | ||
760 | int err) | ||
761 | { | ||
762 | v9fs_mux_req_callback cb; | ||
763 | int tag; | ||
764 | struct v9fs_mux_data *m; | ||
765 | struct v9fs_req *req, *rptr; | ||
766 | |||
767 | m = a; | ||
768 | dprintk(DEBUG_MUX, "mux %p tc %p rc %p err %d oldtag %d\n", m, tc, | ||
769 | rc, err, tc->params.tflush.oldtag); | ||
770 | |||
771 | spin_lock(&m->lock); | ||
772 | cb = NULL; | ||
773 | tag = tc->params.tflush.oldtag; | ||
774 | list_for_each_entry_safe(req, rptr, &m->req_list, req_list) { | ||
775 | if (req->tag == tag) { | ||
776 | list_del(&req->req_list); | ||
777 | if (req->cb) { | ||
778 | cb = req->cb; | ||
779 | req->cb = NULL; | ||
780 | spin_unlock(&m->lock); | ||
781 | (*cb) (req->cba, req->tcall, req->rcall, | ||
782 | req->err); | ||
408 | } | 783 | } |
784 | kfree(req); | ||
785 | wake_up(&m->equeue); | ||
786 | break; | ||
409 | } | 787 | } |
788 | } | ||
410 | 789 | ||
411 | spin_unlock(&v9ses->muxlock); | 790 | if (!cb) |
791 | spin_unlock(&m->lock); | ||
412 | 792 | ||
413 | if (!req) { | 793 | v9fs_mux_put_tag(m, tag); |
414 | if (err >= 0) | 794 | kfree(tc); |
415 | dprintk(DEBUG_ERROR, | 795 | kfree(rc); |
416 | "unexpected response: id %d tag %d\n", | 796 | } |
417 | rcall->id, rcall->tag); | ||
418 | 797 | ||
419 | kfree(rcall); | 798 | static void |
420 | } | 799 | v9fs_mux_flush_request(struct v9fs_mux_data *m, struct v9fs_req *req) |
800 | { | ||
801 | struct v9fs_fcall *fc; | ||
421 | 802 | ||
422 | wake_up_all(&v9ses->read_wait); | 803 | dprintk(DEBUG_MUX, "mux %p req %p tag %d\n", m, req, req->tag); |
423 | set_current_state(TASK_INTERRUPTIBLE); | 804 | |
805 | fc = v9fs_create_tflush(req->tag); | ||
806 | v9fs_send_request(m, fc, v9fs_mux_flush_cb, m); | ||
807 | } | ||
808 | |||
809 | static void | ||
810 | v9fs_mux_rpc_cb(void *a, struct v9fs_fcall *tc, struct v9fs_fcall *rc, int err) | ||
811 | { | ||
812 | struct v9fs_mux_rpc *r; | ||
813 | |||
814 | if (err == ERREQFLUSH) { | ||
815 | dprintk(DEBUG_MUX, "err req flush\n"); | ||
816 | return; | ||
424 | } | 817 | } |
425 | 818 | ||
426 | v9ses->transport->close(v9ses->transport); | 819 | r = a; |
820 | dprintk(DEBUG_MUX, "mux %p req %p tc %p rc %p err %d\n", r->m, r->req, | ||
821 | tc, rc, err); | ||
822 | r->rcall = rc; | ||
823 | r->err = err; | ||
824 | wake_up(&r->wqueue); | ||
825 | } | ||
427 | 826 | ||
428 | /* Inform all pending processes about the failure */ | 827 | /** |
429 | wake_up_all(&v9ses->read_wait); | 828 | * v9fs_mux_rpc - sends 9P request and waits until a response is available. |
829 | * The function can be interrupted. | ||
830 | * @m: mux data | ||
831 | * @tc: request to be sent | ||
832 | * @rc: pointer where a pointer to the response is stored | ||
833 | */ | ||
834 | int | ||
835 | v9fs_mux_rpc(struct v9fs_mux_data *m, struct v9fs_fcall *tc, | ||
836 | struct v9fs_fcall **rc) | ||
837 | { | ||
838 | int err; | ||
839 | unsigned long flags; | ||
840 | struct v9fs_req *req; | ||
841 | struct v9fs_mux_rpc r; | ||
842 | |||
843 | r.err = 0; | ||
844 | r.rcall = NULL; | ||
845 | r.m = m; | ||
846 | init_waitqueue_head(&r.wqueue); | ||
847 | |||
848 | if (rc) | ||
849 | *rc = NULL; | ||
850 | |||
851 | req = v9fs_send_request(m, tc, v9fs_mux_rpc_cb, &r); | ||
852 | if (IS_ERR(req)) { | ||
853 | err = PTR_ERR(req); | ||
854 | dprintk(DEBUG_MUX, "error %d\n", err); | ||
855 | return PTR_ERR(req); | ||
856 | } | ||
430 | 857 | ||
431 | if (signal_pending(current)) | 858 | r.req = req; |
432 | complete(&v9ses->proccmpl); | 859 | dprintk(DEBUG_MUX, "mux %p tc %p tag %d rpc %p req %p\n", m, tc, |
860 | req->tag, &r, req); | ||
861 | err = wait_event_interruptible(r.wqueue, r.rcall != NULL || r.err < 0); | ||
862 | if (r.err < 0) | ||
863 | err = r.err; | ||
864 | |||
865 | if (err == -ERESTARTSYS && m->trans->status == Connected && m->err == 0) { | ||
866 | spin_lock(&m->lock); | ||
867 | req->tcall = NULL; | ||
868 | req->err = ERREQFLUSH; | ||
869 | spin_unlock(&m->lock); | ||
870 | |||
871 | clear_thread_flag(TIF_SIGPENDING); | ||
872 | v9fs_mux_flush_request(m, req); | ||
873 | spin_lock_irqsave(¤t->sighand->siglock, flags); | ||
874 | recalc_sigpending(); | ||
875 | spin_unlock_irqrestore(¤t->sighand->siglock, flags); | ||
876 | } | ||
433 | 877 | ||
434 | dprintk(DEBUG_MUX, "recvproc: end\n"); | 878 | if (!err) { |
435 | v9ses->recvproc = NULL; | 879 | if (r.rcall) |
880 | dprintk(DEBUG_MUX, "got response id %d tag %d\n", | ||
881 | r.rcall->id, r.rcall->tag); | ||
882 | |||
883 | if (rc) | ||
884 | *rc = r.rcall; | ||
885 | else | ||
886 | kfree(r.rcall); | ||
887 | } else { | ||
888 | kfree(r.rcall); | ||
889 | dprintk(DEBUG_MUX, "got error %d\n", err); | ||
890 | if (err > 0) | ||
891 | err = -EIO; | ||
892 | } | ||
436 | 893 | ||
437 | return err >= 0; | 894 | return err; |
438 | } | 895 | } |
439 | 896 | ||
440 | /** | 897 | /** |
441 | * v9fs_mux_init - initialize multiplexer (spawn kproc) | 898 | * v9fs_mux_rpcnb - sends 9P request without waiting for response. |
442 | * @v9ses: session info structure | 899 | * @m: mux data |
443 | * @dev_name: mount device information (to create unique kproc) | 900 | * @tc: request to be sent |
444 | * | 901 | * @cb: callback function to be called when response arrives |
902 | * @cba: value to pass to the callback function | ||
445 | */ | 903 | */ |
904 | int v9fs_mux_rpcnb(struct v9fs_mux_data *m, struct v9fs_fcall *tc, | ||
905 | v9fs_mux_req_callback cb, void *a) | ||
906 | { | ||
907 | int err; | ||
908 | struct v9fs_req *req; | ||
909 | |||
910 | req = v9fs_send_request(m, tc, cb, a); | ||
911 | if (IS_ERR(req)) { | ||
912 | err = PTR_ERR(req); | ||
913 | dprintk(DEBUG_MUX, "error %d\n", err); | ||
914 | return PTR_ERR(req); | ||
915 | } | ||
916 | |||
917 | dprintk(DEBUG_MUX, "mux %p tc %p tag %d\n", m, tc, req->tag); | ||
918 | return 0; | ||
919 | } | ||
446 | 920 | ||
447 | int v9fs_mux_init(struct v9fs_session_info *v9ses, const char *dev_name) | 921 | /** |
922 | * v9fs_mux_cancel - cancel all pending requests with error | ||
923 | * @m: mux data | ||
924 | * @err: error code | ||
925 | */ | ||
926 | void v9fs_mux_cancel(struct v9fs_mux_data *m, int err) | ||
448 | { | 927 | { |
449 | char procname[60]; | 928 | struct v9fs_req *req, *rtmp; |
450 | 929 | LIST_HEAD(cancel_list); | |
451 | strncpy(procname, dev_name, sizeof(procname)); | 930 | |
452 | procname[sizeof(procname) - 1] = 0; | 931 | dprintk(DEBUG_MUX, "mux %p err %d\n", m, err); |
453 | 932 | m->err = err; | |
454 | init_waitqueue_head(&v9ses->read_wait); | 933 | spin_lock(&m->lock); |
455 | init_completion(&v9ses->fcread); | 934 | list_for_each_entry_safe(req, rtmp, &m->req_list, req_list) { |
456 | init_completion(&v9ses->proccmpl); | 935 | list_move(&req->req_list, &cancel_list); |
457 | spin_lock_init(&v9ses->muxlock); | ||
458 | INIT_LIST_HEAD(&v9ses->mux_fcalls); | ||
459 | v9ses->recvproc = NULL; | ||
460 | v9ses->curfcall = NULL; | ||
461 | |||
462 | v9ses->recvproc = kthread_create(v9fs_recvproc, v9ses, | ||
463 | "v9fs_recvproc %s", procname); | ||
464 | |||
465 | if (IS_ERR(v9ses->recvproc)) { | ||
466 | eprintk(KERN_ERR, "cannot create receiving thread\n"); | ||
467 | v9fs_session_close(v9ses); | ||
468 | return -ECONNABORTED; | ||
469 | } | 936 | } |
937 | spin_unlock(&m->lock); | ||
470 | 938 | ||
471 | wake_up_process(v9ses->recvproc); | 939 | list_for_each_entry_safe(req, rtmp, &cancel_list, req_list) { |
472 | wait_for_completion(&v9ses->proccmpl); | 940 | list_del(&req->req_list); |
941 | if (!req->err) | ||
942 | req->err = err; | ||
473 | 943 | ||
474 | return 0; | 944 | if (req->cb) |
945 | (*req->cb) (req->cba, req->tcall, req->rcall, req->err); | ||
946 | else | ||
947 | kfree(req->rcall); | ||
948 | |||
949 | kfree(req); | ||
950 | } | ||
951 | |||
952 | wake_up(&m->equeue); | ||
953 | } | ||
954 | |||
955 | static u16 v9fs_mux_get_tag(struct v9fs_mux_data *m) | ||
956 | { | ||
957 | int tag; | ||
958 | |||
959 | tag = v9fs_get_idpool(&m->tidpool); | ||
960 | if (tag < 0) | ||
961 | return V9FS_NOTAG; | ||
962 | else | ||
963 | return (u16) tag; | ||
964 | } | ||
965 | |||
966 | static void v9fs_mux_put_tag(struct v9fs_mux_data *m, u16 tag) | ||
967 | { | ||
968 | if (tag != V9FS_NOTAG && v9fs_check_idpool(tag, &m->tidpool)) | ||
969 | v9fs_put_idpool(tag, &m->tidpool); | ||
475 | } | 970 | } |
diff --git a/fs/9p/mux.h b/fs/9p/mux.h index 4994cb10badf..9473b84f24b2 100644 --- a/fs/9p/mux.h +++ b/fs/9p/mux.h | |||
@@ -3,6 +3,7 @@ | |||
3 | * | 3 | * |
4 | * Multiplexer Definitions | 4 | * Multiplexer Definitions |
5 | * | 5 | * |
6 | * Copyright (C) 2005 by Latchesar Ionkov <lucho@ionkov.net> | ||
6 | * Copyright (C) 2004 by Eric Van Hensbergen <ericvh@gmail.com> | 7 | * Copyright (C) 2004 by Eric Van Hensbergen <ericvh@gmail.com> |
7 | * | 8 | * |
8 | * This program is free software; you can redistribute it and/or modify | 9 | * This program is free software; you can redistribute it and/or modify |
@@ -23,19 +24,35 @@ | |||
23 | * | 24 | * |
24 | */ | 25 | */ |
25 | 26 | ||
26 | /* structure to manage each RPC transaction */ | 27 | struct v9fs_mux_data; |
27 | 28 | ||
28 | struct v9fs_rpcreq { | 29 | /** |
29 | struct v9fs_fcall *tcall; | 30 | * v9fs_mux_req_callback - callback function that is called when the |
30 | struct v9fs_fcall *rcall; | 31 | * response of a request is received. The callback is called from |
31 | int err; /* error code if response failed */ | 32 | * a workqueue and shouldn't block. |
33 | * | ||
34 | * @a - the pointer that was specified when the request was send to be | ||
35 | * passed to the callback | ||
36 | * @tc - request call | ||
37 | * @rc - response call | ||
38 | * @err - error code (non-zero if error occured) | ||
39 | */ | ||
40 | typedef void (*v9fs_mux_req_callback)(void *a, struct v9fs_fcall *tc, | ||
41 | struct v9fs_fcall *rc, int err); | ||
42 | |||
43 | int v9fs_mux_global_init(void); | ||
44 | void v9fs_mux_global_exit(void); | ||
32 | 45 | ||
33 | /* XXX - could we put scatter/gather buffers here? */ | 46 | struct v9fs_mux_data *v9fs_mux_init(struct v9fs_transport *trans, int msize, |
47 | unsigned char *extended); | ||
48 | void v9fs_mux_destroy(struct v9fs_mux_data *); | ||
34 | 49 | ||
35 | struct list_head next; | 50 | int v9fs_mux_send(struct v9fs_mux_data *m, struct v9fs_fcall *tc); |
36 | }; | 51 | struct v9fs_fcall *v9fs_mux_recv(struct v9fs_mux_data *m); |
52 | int v9fs_mux_rpc(struct v9fs_mux_data *m, struct v9fs_fcall *tc, struct v9fs_fcall **rc); | ||
53 | int v9fs_mux_rpcnb(struct v9fs_mux_data *m, struct v9fs_fcall *tc, | ||
54 | v9fs_mux_req_callback cb, void *a); | ||
37 | 55 | ||
38 | int v9fs_mux_init(struct v9fs_session_info *v9ses, const char *dev_name); | 56 | void v9fs_mux_flush(struct v9fs_mux_data *m, int sendflush); |
39 | long v9fs_mux_rpc(struct v9fs_session_info *v9ses, | 57 | void v9fs_mux_cancel(struct v9fs_mux_data *m, int err); |
40 | struct v9fs_fcall *tcall, struct v9fs_fcall **rcall); | 58 | int v9fs_errstr2errno(char *errstr, int len); |
41 | void v9fs_mux_cancel_requests(struct v9fs_session_info *v9ses, int err); | ||
diff --git a/fs/9p/trans_fd.c b/fs/9p/trans_fd.c index 63b58ce98ff4..1a28ef97a3d1 100644 --- a/fs/9p/trans_fd.c +++ b/fs/9p/trans_fd.c | |||
@@ -3,6 +3,7 @@ | |||
3 | * | 3 | * |
4 | * File Descriptor Transport Layer | 4 | * File Descriptor Transport Layer |
5 | * | 5 | * |
6 | * Copyright (C) 2005 by Latchesar Ionkov <lucho@ionkov.net> | ||
6 | * Copyright (C) 2005 by Eric Van Hensbergen <ericvh@gmail.com> | 7 | * Copyright (C) 2005 by Eric Van Hensbergen <ericvh@gmail.com> |
7 | * | 8 | * |
8 | * This program is free software; you can redistribute it and/or modify | 9 | * This program is free software; you can redistribute it and/or modify |
@@ -106,9 +107,6 @@ v9fs_fd_init(struct v9fs_session_info *v9ses, const char *addr, char *data) | |||
106 | return -ENOPROTOOPT; | 107 | return -ENOPROTOOPT; |
107 | } | 108 | } |
108 | 109 | ||
109 | sema_init(&trans->writelock, 1); | ||
110 | sema_init(&trans->readlock, 1); | ||
111 | |||
112 | ts = kmalloc(sizeof(struct v9fs_trans_fd), GFP_KERNEL); | 110 | ts = kmalloc(sizeof(struct v9fs_trans_fd), GFP_KERNEL); |
113 | 111 | ||
114 | if (!ts) | 112 | if (!ts) |
@@ -148,12 +146,12 @@ static void v9fs_fd_close(struct v9fs_transport *trans) | |||
148 | if (!trans) | 146 | if (!trans) |
149 | return; | 147 | return; |
150 | 148 | ||
151 | trans->status = Disconnected; | 149 | ts = xchg(&trans->priv, NULL); |
152 | ts = trans->priv; | ||
153 | 150 | ||
154 | if (!ts) | 151 | if (!ts) |
155 | return; | 152 | return; |
156 | 153 | ||
154 | trans->status = Disconnected; | ||
157 | if (ts->in_file) | 155 | if (ts->in_file) |
158 | fput(ts->in_file); | 156 | fput(ts->in_file); |
159 | 157 | ||
@@ -163,10 +161,55 @@ static void v9fs_fd_close(struct v9fs_transport *trans) | |||
163 | kfree(ts); | 161 | kfree(ts); |
164 | } | 162 | } |
165 | 163 | ||
164 | static unsigned int | ||
165 | v9fs_fd_poll(struct v9fs_transport *trans, struct poll_table_struct *pt) | ||
166 | { | ||
167 | int ret, n; | ||
168 | struct v9fs_trans_fd *ts; | ||
169 | mm_segment_t oldfs; | ||
170 | |||
171 | if (!trans) | ||
172 | return -EIO; | ||
173 | |||
174 | ts = trans->priv; | ||
175 | if (trans->status != Connected || !ts) | ||
176 | return -EIO; | ||
177 | |||
178 | oldfs = get_fs(); | ||
179 | set_fs(get_ds()); | ||
180 | |||
181 | if (!ts->in_file->f_op || !ts->in_file->f_op->poll) { | ||
182 | ret = -EIO; | ||
183 | goto end; | ||
184 | } | ||
185 | |||
186 | ret = ts->in_file->f_op->poll(ts->in_file, pt); | ||
187 | |||
188 | if (ts->out_file != ts->in_file) { | ||
189 | if (!ts->out_file->f_op || !ts->out_file->f_op->poll) { | ||
190 | ret = -EIO; | ||
191 | goto end; | ||
192 | } | ||
193 | |||
194 | n = ts->out_file->f_op->poll(ts->out_file, pt); | ||
195 | |||
196 | ret &= ~POLLOUT; | ||
197 | n &= ~POLLIN; | ||
198 | |||
199 | ret |= n; | ||
200 | } | ||
201 | |||
202 | end: | ||
203 | set_fs(oldfs); | ||
204 | return ret; | ||
205 | } | ||
206 | |||
207 | |||
166 | struct v9fs_transport v9fs_trans_fd = { | 208 | struct v9fs_transport v9fs_trans_fd = { |
167 | .init = v9fs_fd_init, | 209 | .init = v9fs_fd_init, |
168 | .write = v9fs_fd_send, | 210 | .write = v9fs_fd_send, |
169 | .read = v9fs_fd_recv, | 211 | .read = v9fs_fd_recv, |
170 | .close = v9fs_fd_close, | 212 | .close = v9fs_fd_close, |
213 | .poll = v9fs_fd_poll, | ||
171 | }; | 214 | }; |
172 | 215 | ||
diff --git a/fs/9p/trans_sock.c b/fs/9p/trans_sock.c index a93c2bf94c33..44e830697acb 100644 --- a/fs/9p/trans_sock.c +++ b/fs/9p/trans_sock.c | |||
@@ -3,6 +3,7 @@ | |||
3 | * | 3 | * |
4 | * Socket Transport Layer | 4 | * Socket Transport Layer |
5 | * | 5 | * |
6 | * Copyright (C) 2004-2005 by Latchesar Ionkov <lucho@ionkov.net> | ||
6 | * Copyright (C) 2004 by Eric Van Hensbergen <ericvh@gmail.com> | 7 | * Copyright (C) 2004 by Eric Van Hensbergen <ericvh@gmail.com> |
7 | * Copyright (C) 1997-2002 by Ron Minnich <rminnich@sarnoff.com> | 8 | * Copyright (C) 1997-2002 by Ron Minnich <rminnich@sarnoff.com> |
8 | * Copyright (C) 1995, 1996 by Olaf Kirch <okir@monad.swb.de> | 9 | * Copyright (C) 1995, 1996 by Olaf Kirch <okir@monad.swb.de> |
@@ -26,6 +27,7 @@ | |||
26 | */ | 27 | */ |
27 | 28 | ||
28 | #include <linux/config.h> | 29 | #include <linux/config.h> |
30 | #include <linux/in.h> | ||
29 | #include <linux/module.h> | 31 | #include <linux/module.h> |
30 | #include <linux/net.h> | 32 | #include <linux/net.h> |
31 | #include <linux/ipv6.h> | 33 | #include <linux/ipv6.h> |
@@ -35,6 +37,7 @@ | |||
35 | #include <asm/uaccess.h> | 37 | #include <asm/uaccess.h> |
36 | #include <linux/inet.h> | 38 | #include <linux/inet.h> |
37 | #include <linux/idr.h> | 39 | #include <linux/idr.h> |
40 | #include <linux/file.h> | ||
38 | 41 | ||
39 | #include "debug.h" | 42 | #include "debug.h" |
40 | #include "v9fs.h" | 43 | #include "v9fs.h" |
@@ -44,6 +47,7 @@ | |||
44 | 47 | ||
45 | struct v9fs_trans_sock { | 48 | struct v9fs_trans_sock { |
46 | struct socket *s; | 49 | struct socket *s; |
50 | struct file *filp; | ||
47 | }; | 51 | }; |
48 | 52 | ||
49 | /** | 53 | /** |
@@ -56,41 +60,26 @@ struct v9fs_trans_sock { | |||
56 | 60 | ||
57 | static int v9fs_sock_recv(struct v9fs_transport *trans, void *v, int len) | 61 | static int v9fs_sock_recv(struct v9fs_transport *trans, void *v, int len) |
58 | { | 62 | { |
59 | struct msghdr msg; | 63 | int ret; |
60 | struct kvec iov; | 64 | struct v9fs_trans_sock *ts; |
61 | int result; | ||
62 | mm_segment_t oldfs; | ||
63 | struct v9fs_trans_sock *ts = trans ? trans->priv : NULL; | ||
64 | 65 | ||
65 | if (trans->status == Disconnected) | 66 | if (!trans || trans->status == Disconnected) { |
67 | dprintk(DEBUG_ERROR, "disconnected ...\n"); | ||
66 | return -EREMOTEIO; | 68 | return -EREMOTEIO; |
69 | } | ||
67 | 70 | ||
68 | result = -EINVAL; | 71 | ts = trans->priv; |
69 | |||
70 | oldfs = get_fs(); | ||
71 | set_fs(get_ds()); | ||
72 | |||
73 | iov.iov_base = v; | ||
74 | iov.iov_len = len; | ||
75 | msg.msg_name = NULL; | ||
76 | msg.msg_namelen = 0; | ||
77 | msg.msg_iovlen = 1; | ||
78 | msg.msg_control = NULL; | ||
79 | msg.msg_controllen = 0; | ||
80 | msg.msg_namelen = 0; | ||
81 | msg.msg_flags = MSG_NOSIGNAL; | ||
82 | 72 | ||
83 | result = kernel_recvmsg(ts->s, &msg, &iov, 1, len, 0); | 73 | if (!(ts->filp->f_flags & O_NONBLOCK)) |
74 | dprintk(DEBUG_ERROR, "blocking read ...\n"); | ||
84 | 75 | ||
85 | dprintk(DEBUG_TRANS, "socket state %d\n", ts->s->state); | 76 | ret = kernel_read(ts->filp, ts->filp->f_pos, v, len); |
86 | set_fs(oldfs); | 77 | if (ret <= 0) { |
87 | 78 | if (ret != -ERESTARTSYS && ret != -EAGAIN) | |
88 | if (result <= 0) { | ||
89 | if (result != -ERESTARTSYS) | ||
90 | trans->status = Disconnected; | 79 | trans->status = Disconnected; |
91 | } | 80 | } |
92 | 81 | ||
93 | return result; | 82 | return ret; |
94 | } | 83 | } |
95 | 84 | ||
96 | /** | 85 | /** |
@@ -103,40 +92,72 @@ static int v9fs_sock_recv(struct v9fs_transport *trans, void *v, int len) | |||
103 | 92 | ||
104 | static int v9fs_sock_send(struct v9fs_transport *trans, void *v, int len) | 93 | static int v9fs_sock_send(struct v9fs_transport *trans, void *v, int len) |
105 | { | 94 | { |
106 | struct kvec iov; | 95 | int ret; |
107 | struct msghdr msg; | ||
108 | int result = -1; | ||
109 | mm_segment_t oldfs; | 96 | mm_segment_t oldfs; |
110 | struct v9fs_trans_sock *ts = trans ? trans->priv : NULL; | 97 | struct v9fs_trans_sock *ts; |
111 | 98 | ||
112 | dprintk(DEBUG_TRANS, "Sending packet size %d (%x)\n", len, len); | 99 | if (!trans || trans->status == Disconnected) { |
113 | dump_data(v, len); | 100 | dprintk(DEBUG_ERROR, "disconnected ...\n"); |
101 | return -EREMOTEIO; | ||
102 | } | ||
103 | |||
104 | ts = trans->priv; | ||
105 | if (!ts) { | ||
106 | dprintk(DEBUG_ERROR, "no transport ...\n"); | ||
107 | return -EREMOTEIO; | ||
108 | } | ||
114 | 109 | ||
115 | down(&trans->writelock); | 110 | if (!(ts->filp->f_flags & O_NONBLOCK)) |
111 | dprintk(DEBUG_ERROR, "blocking write ...\n"); | ||
116 | 112 | ||
117 | oldfs = get_fs(); | 113 | oldfs = get_fs(); |
118 | set_fs(get_ds()); | 114 | set_fs(get_ds()); |
119 | iov.iov_base = v; | 115 | ret = vfs_write(ts->filp, (void __user *)v, len, &ts->filp->f_pos); |
120 | iov.iov_len = len; | ||
121 | msg.msg_name = NULL; | ||
122 | msg.msg_namelen = 0; | ||
123 | msg.msg_iovlen = 1; | ||
124 | msg.msg_control = NULL; | ||
125 | msg.msg_controllen = 0; | ||
126 | msg.msg_namelen = 0; | ||
127 | msg.msg_flags = MSG_NOSIGNAL; | ||
128 | result = kernel_sendmsg(ts->s, &msg, &iov, 1, len); | ||
129 | set_fs(oldfs); | 116 | set_fs(oldfs); |
130 | 117 | ||
131 | if (result < 0) { | 118 | if (ret < 0) { |
132 | if (result != -ERESTARTSYS) | 119 | if (ret != -ERESTARTSYS) |
133 | trans->status = Disconnected; | 120 | trans->status = Disconnected; |
134 | } | 121 | } |
135 | 122 | ||
136 | up(&trans->writelock); | 123 | return ret; |
137 | return result; | 124 | } |
125 | |||
126 | static unsigned int v9fs_sock_poll(struct v9fs_transport *trans, | ||
127 | struct poll_table_struct *pt) { | ||
128 | |||
129 | int ret; | ||
130 | struct v9fs_trans_sock *ts; | ||
131 | mm_segment_t oldfs; | ||
132 | |||
133 | if (!trans) { | ||
134 | dprintk(DEBUG_ERROR, "no transport\n"); | ||
135 | return -EIO; | ||
136 | } | ||
137 | |||
138 | ts = trans->priv; | ||
139 | if (trans->status != Connected || !ts) { | ||
140 | dprintk(DEBUG_ERROR, "transport disconnected: %d\n", trans->status); | ||
141 | return -EIO; | ||
142 | } | ||
143 | |||
144 | oldfs = get_fs(); | ||
145 | set_fs(get_ds()); | ||
146 | |||
147 | if (!ts->filp->f_op || !ts->filp->f_op->poll) { | ||
148 | dprintk(DEBUG_ERROR, "no poll operation\n"); | ||
149 | ret = -EIO; | ||
150 | goto end; | ||
151 | } | ||
152 | |||
153 | ret = ts->filp->f_op->poll(ts->filp, pt); | ||
154 | |||
155 | end: | ||
156 | set_fs(oldfs); | ||
157 | return ret; | ||
138 | } | 158 | } |
139 | 159 | ||
160 | |||
140 | /** | 161 | /** |
141 | * v9fs_tcp_init - initialize TCP socket | 162 | * v9fs_tcp_init - initialize TCP socket |
142 | * @v9ses: session information | 163 | * @v9ses: session information |
@@ -153,9 +174,9 @@ v9fs_tcp_init(struct v9fs_session_info *v9ses, const char *addr, char *data) | |||
153 | int rc = 0; | 174 | int rc = 0; |
154 | struct v9fs_trans_sock *ts = NULL; | 175 | struct v9fs_trans_sock *ts = NULL; |
155 | struct v9fs_transport *trans = v9ses->transport; | 176 | struct v9fs_transport *trans = v9ses->transport; |
177 | int fd; | ||
156 | 178 | ||
157 | sema_init(&trans->writelock, 1); | 179 | trans->status = Disconnected; |
158 | sema_init(&trans->readlock, 1); | ||
159 | 180 | ||
160 | ts = kmalloc(sizeof(struct v9fs_trans_sock), GFP_KERNEL); | 181 | ts = kmalloc(sizeof(struct v9fs_trans_sock), GFP_KERNEL); |
161 | 182 | ||
@@ -164,6 +185,7 @@ v9fs_tcp_init(struct v9fs_session_info *v9ses, const char *addr, char *data) | |||
164 | 185 | ||
165 | trans->priv = ts; | 186 | trans->priv = ts; |
166 | ts->s = NULL; | 187 | ts->s = NULL; |
188 | ts->filp = NULL; | ||
167 | 189 | ||
168 | if (!addr) | 190 | if (!addr) |
169 | return -EINVAL; | 191 | return -EINVAL; |
@@ -184,7 +206,18 @@ v9fs_tcp_init(struct v9fs_session_info *v9ses, const char *addr, char *data) | |||
184 | return rc; | 206 | return rc; |
185 | } | 207 | } |
186 | csocket->sk->sk_allocation = GFP_NOIO; | 208 | csocket->sk->sk_allocation = GFP_NOIO; |
209 | |||
210 | fd = sock_map_fd(csocket); | ||
211 | if (fd < 0) { | ||
212 | sock_release(csocket); | ||
213 | kfree(ts); | ||
214 | trans->priv = NULL; | ||
215 | return fd; | ||
216 | } | ||
217 | |||
187 | ts->s = csocket; | 218 | ts->s = csocket; |
219 | ts->filp = fget(fd); | ||
220 | ts->filp->f_flags |= O_NONBLOCK; | ||
188 | trans->status = Connected; | 221 | trans->status = Connected; |
189 | 222 | ||
190 | return 0; | 223 | return 0; |
@@ -202,7 +235,7 @@ static int | |||
202 | v9fs_unix_init(struct v9fs_session_info *v9ses, const char *dev_name, | 235 | v9fs_unix_init(struct v9fs_session_info *v9ses, const char *dev_name, |
203 | char *data) | 236 | char *data) |
204 | { | 237 | { |
205 | int rc; | 238 | int rc, fd; |
206 | struct socket *csocket; | 239 | struct socket *csocket; |
207 | struct sockaddr_un sun_server; | 240 | struct sockaddr_un sun_server; |
208 | struct v9fs_transport *trans; | 241 | struct v9fs_transport *trans; |
@@ -212,6 +245,8 @@ v9fs_unix_init(struct v9fs_session_info *v9ses, const char *dev_name, | |||
212 | csocket = NULL; | 245 | csocket = NULL; |
213 | trans = v9ses->transport; | 246 | trans = v9ses->transport; |
214 | 247 | ||
248 | trans->status = Disconnected; | ||
249 | |||
215 | if (strlen(dev_name) > UNIX_PATH_MAX) { | 250 | if (strlen(dev_name) > UNIX_PATH_MAX) { |
216 | eprintk(KERN_ERR, "v9fs_trans_unix: address too long: %s\n", | 251 | eprintk(KERN_ERR, "v9fs_trans_unix: address too long: %s\n", |
217 | dev_name); | 252 | dev_name); |
@@ -224,9 +259,7 @@ v9fs_unix_init(struct v9fs_session_info *v9ses, const char *dev_name, | |||
224 | 259 | ||
225 | trans->priv = ts; | 260 | trans->priv = ts; |
226 | ts->s = NULL; | 261 | ts->s = NULL; |
227 | 262 | ts->filp = NULL; | |
228 | sema_init(&trans->writelock, 1); | ||
229 | sema_init(&trans->readlock, 1); | ||
230 | 263 | ||
231 | sun_server.sun_family = PF_UNIX; | 264 | sun_server.sun_family = PF_UNIX; |
232 | strcpy(sun_server.sun_path, dev_name); | 265 | strcpy(sun_server.sun_path, dev_name); |
@@ -240,7 +273,18 @@ v9fs_unix_init(struct v9fs_session_info *v9ses, const char *dev_name, | |||
240 | return rc; | 273 | return rc; |
241 | } | 274 | } |
242 | csocket->sk->sk_allocation = GFP_NOIO; | 275 | csocket->sk->sk_allocation = GFP_NOIO; |
276 | |||
277 | fd = sock_map_fd(csocket); | ||
278 | if (fd < 0) { | ||
279 | sock_release(csocket); | ||
280 | kfree(ts); | ||
281 | trans->priv = NULL; | ||
282 | return fd; | ||
283 | } | ||
284 | |||
243 | ts->s = csocket; | 285 | ts->s = csocket; |
286 | ts->filp = fget(fd); | ||
287 | ts->filp->f_flags |= O_NONBLOCK; | ||
244 | trans->status = Connected; | 288 | trans->status = Connected; |
245 | 289 | ||
246 | return 0; | 290 | return 0; |
@@ -261,12 +305,11 @@ static void v9fs_sock_close(struct v9fs_transport *trans) | |||
261 | 305 | ||
262 | ts = trans->priv; | 306 | ts = trans->priv; |
263 | 307 | ||
264 | if ((ts) && (ts->s)) { | 308 | if ((ts) && (ts->filp)) { |
265 | dprintk(DEBUG_TRANS, "closing the socket %p\n", ts->s); | 309 | fput(ts->filp); |
266 | sock_release(ts->s); | 310 | ts->filp = NULL; |
267 | ts->s = NULL; | 311 | ts->s = NULL; |
268 | trans->status = Disconnected; | 312 | trans->status = Disconnected; |
269 | dprintk(DEBUG_TRANS, "socket closed\n"); | ||
270 | } | 313 | } |
271 | 314 | ||
272 | kfree(ts); | 315 | kfree(ts); |
@@ -279,6 +322,7 @@ struct v9fs_transport v9fs_trans_tcp = { | |||
279 | .write = v9fs_sock_send, | 322 | .write = v9fs_sock_send, |
280 | .read = v9fs_sock_recv, | 323 | .read = v9fs_sock_recv, |
281 | .close = v9fs_sock_close, | 324 | .close = v9fs_sock_close, |
325 | .poll = v9fs_sock_poll, | ||
282 | }; | 326 | }; |
283 | 327 | ||
284 | struct v9fs_transport v9fs_trans_unix = { | 328 | struct v9fs_transport v9fs_trans_unix = { |
@@ -286,4 +330,5 @@ struct v9fs_transport v9fs_trans_unix = { | |||
286 | .write = v9fs_sock_send, | 330 | .write = v9fs_sock_send, |
287 | .read = v9fs_sock_recv, | 331 | .read = v9fs_sock_recv, |
288 | .close = v9fs_sock_close, | 332 | .close = v9fs_sock_close, |
333 | .poll = v9fs_sock_poll, | ||
289 | }; | 334 | }; |
diff --git a/fs/9p/transport.h b/fs/9p/transport.h index 9e9cd418efd5..91fcdb94b361 100644 --- a/fs/9p/transport.h +++ b/fs/9p/transport.h | |||
@@ -3,6 +3,7 @@ | |||
3 | * | 3 | * |
4 | * Transport Definition | 4 | * Transport Definition |
5 | * | 5 | * |
6 | * Copyright (C) 2005 by Latchesar Ionkov <lucho@ionkov.net> | ||
6 | * Copyright (C) 2004 by Eric Van Hensbergen <ericvh@gmail.com> | 7 | * Copyright (C) 2004 by Eric Van Hensbergen <ericvh@gmail.com> |
7 | * | 8 | * |
8 | * This program is free software; you can redistribute it and/or modify | 9 | * This program is free software; you can redistribute it and/or modify |
@@ -31,14 +32,13 @@ enum v9fs_transport_status { | |||
31 | 32 | ||
32 | struct v9fs_transport { | 33 | struct v9fs_transport { |
33 | enum v9fs_transport_status status; | 34 | enum v9fs_transport_status status; |
34 | struct semaphore writelock; | ||
35 | struct semaphore readlock; | ||
36 | void *priv; | 35 | void *priv; |
37 | 36 | ||
38 | int (*init) (struct v9fs_session_info *, const char *, char *); | 37 | int (*init) (struct v9fs_session_info *, const char *, char *); |
39 | int (*write) (struct v9fs_transport *, void *, int); | 38 | int (*write) (struct v9fs_transport *, void *, int); |
40 | int (*read) (struct v9fs_transport *, void *, int); | 39 | int (*read) (struct v9fs_transport *, void *, int); |
41 | void (*close) (struct v9fs_transport *); | 40 | void (*close) (struct v9fs_transport *); |
41 | unsigned int (*poll)(struct v9fs_transport *, struct poll_table_struct *); | ||
42 | }; | 42 | }; |
43 | 43 | ||
44 | extern struct v9fs_transport v9fs_trans_tcp; | 44 | extern struct v9fs_transport v9fs_trans_tcp; |
diff --git a/fs/9p/v9fs.c b/fs/9p/v9fs.c index 418c3743fdee..5250c428fc1f 100644 --- a/fs/9p/v9fs.c +++ b/fs/9p/v9fs.c | |||
@@ -37,7 +37,6 @@ | |||
37 | #include "v9fs_vfs.h" | 37 | #include "v9fs_vfs.h" |
38 | #include "transport.h" | 38 | #include "transport.h" |
39 | #include "mux.h" | 39 | #include "mux.h" |
40 | #include "conv.h" | ||
41 | 40 | ||
42 | /* TODO: sysfs or debugfs interface */ | 41 | /* TODO: sysfs or debugfs interface */ |
43 | int v9fs_debug_level = 0; /* feature-rific global debug level */ | 42 | int v9fs_debug_level = 0; /* feature-rific global debug level */ |
@@ -213,7 +212,8 @@ retry: | |||
213 | return -1; | 212 | return -1; |
214 | } | 213 | } |
215 | 214 | ||
216 | error = idr_get_new(&p->pool, NULL, &i); | 215 | /* no need to store exactly p, we just need something non-null */ |
216 | error = idr_get_new(&p->pool, p, &i); | ||
217 | up(&p->lock); | 217 | up(&p->lock); |
218 | 218 | ||
219 | if (error == -EAGAIN) | 219 | if (error == -EAGAIN) |
@@ -243,6 +243,16 @@ void v9fs_put_idpool(int id, struct v9fs_idpool *p) | |||
243 | } | 243 | } |
244 | 244 | ||
245 | /** | 245 | /** |
246 | * v9fs_check_idpool - check if the specified id is available | ||
247 | * @id - id to check | ||
248 | * @p - pool | ||
249 | */ | ||
250 | int v9fs_check_idpool(int id, struct v9fs_idpool *p) | ||
251 | { | ||
252 | return idr_find(&p->pool, id) != NULL; | ||
253 | } | ||
254 | |||
255 | /** | ||
246 | * v9fs_session_init - initialize session | 256 | * v9fs_session_init - initialize session |
247 | * @v9ses: session information structure | 257 | * @v9ses: session information structure |
248 | * @dev_name: device being mounted | 258 | * @dev_name: device being mounted |
@@ -259,6 +269,7 @@ v9fs_session_init(struct v9fs_session_info *v9ses, | |||
259 | int n = 0; | 269 | int n = 0; |
260 | int newfid = -1; | 270 | int newfid = -1; |
261 | int retval = -EINVAL; | 271 | int retval = -EINVAL; |
272 | struct v9fs_str *version; | ||
262 | 273 | ||
263 | v9ses->name = __getname(); | 274 | v9ses->name = __getname(); |
264 | if (!v9ses->name) | 275 | if (!v9ses->name) |
@@ -281,9 +292,6 @@ v9fs_session_init(struct v9fs_session_info *v9ses, | |||
281 | /* id pools that are session-dependent: FIDs and TIDs */ | 292 | /* id pools that are session-dependent: FIDs and TIDs */ |
282 | idr_init(&v9ses->fidpool.pool); | 293 | idr_init(&v9ses->fidpool.pool); |
283 | init_MUTEX(&v9ses->fidpool.lock); | 294 | init_MUTEX(&v9ses->fidpool.lock); |
284 | idr_init(&v9ses->tidpool.pool); | ||
285 | init_MUTEX(&v9ses->tidpool.lock); | ||
286 | |||
287 | 295 | ||
288 | switch (v9ses->proto) { | 296 | switch (v9ses->proto) { |
289 | case PROTO_TCP: | 297 | case PROTO_TCP: |
@@ -320,7 +328,12 @@ v9fs_session_init(struct v9fs_session_info *v9ses, | |||
320 | v9ses->shutdown = 0; | 328 | v9ses->shutdown = 0; |
321 | v9ses->session_hung = 0; | 329 | v9ses->session_hung = 0; |
322 | 330 | ||
323 | if ((retval = v9fs_mux_init(v9ses, dev_name)) < 0) { | 331 | v9ses->mux = v9fs_mux_init(v9ses->transport, v9ses->maxdata + V9FS_IOHDRSZ, |
332 | &v9ses->extended); | ||
333 | |||
334 | if (IS_ERR(v9ses->mux)) { | ||
335 | retval = PTR_ERR(v9ses->mux); | ||
336 | v9ses->mux = NULL; | ||
324 | dprintk(DEBUG_ERROR, "problem initializing mux\n"); | 337 | dprintk(DEBUG_ERROR, "problem initializing mux\n"); |
325 | goto SessCleanUp; | 338 | goto SessCleanUp; |
326 | } | 339 | } |
@@ -339,13 +352,16 @@ v9fs_session_init(struct v9fs_session_info *v9ses, | |||
339 | goto FreeFcall; | 352 | goto FreeFcall; |
340 | } | 353 | } |
341 | 354 | ||
342 | /* Really should check for 9P1 and report error */ | 355 | version = &fcall->params.rversion.version; |
343 | if (!strcmp(fcall->params.rversion.version, "9P2000.u")) { | 356 | if (version->len==8 && !memcmp(version->str, "9P2000.u", 8)) { |
344 | dprintk(DEBUG_9P, "9P2000 UNIX extensions enabled\n"); | 357 | dprintk(DEBUG_9P, "9P2000 UNIX extensions enabled\n"); |
345 | v9ses->extended = 1; | 358 | v9ses->extended = 1; |
346 | } else { | 359 | } else if (version->len==6 && !memcmp(version->str, "9P2000", 6)) { |
347 | dprintk(DEBUG_9P, "9P2000 legacy mode enabled\n"); | 360 | dprintk(DEBUG_9P, "9P2000 legacy mode enabled\n"); |
348 | v9ses->extended = 0; | 361 | v9ses->extended = 0; |
362 | } else { | ||
363 | retval = -EREMOTEIO; | ||
364 | goto FreeFcall; | ||
349 | } | 365 | } |
350 | 366 | ||
351 | n = fcall->params.rversion.msize; | 367 | n = fcall->params.rversion.msize; |
@@ -381,7 +397,7 @@ v9fs_session_init(struct v9fs_session_info *v9ses, | |||
381 | } | 397 | } |
382 | 398 | ||
383 | if (v9ses->afid != ~0) { | 399 | if (v9ses->afid != ~0) { |
384 | if (v9fs_t_clunk(v9ses, v9ses->afid, NULL)) | 400 | if (v9fs_t_clunk(v9ses, v9ses->afid)) |
385 | dprintk(DEBUG_ERROR, "clunk failed\n"); | 401 | dprintk(DEBUG_ERROR, "clunk failed\n"); |
386 | } | 402 | } |
387 | 403 | ||
@@ -403,13 +419,16 @@ v9fs_session_init(struct v9fs_session_info *v9ses, | |||
403 | 419 | ||
404 | void v9fs_session_close(struct v9fs_session_info *v9ses) | 420 | void v9fs_session_close(struct v9fs_session_info *v9ses) |
405 | { | 421 | { |
406 | if (v9ses->recvproc) { | 422 | if (v9ses->mux) { |
407 | send_sig(SIGKILL, v9ses->recvproc, 1); | 423 | v9fs_mux_destroy(v9ses->mux); |
408 | wait_for_completion(&v9ses->proccmpl); | 424 | v9ses->mux = NULL; |
409 | } | 425 | } |
410 | 426 | ||
411 | if (v9ses->transport) | 427 | if (v9ses->transport) { |
412 | v9ses->transport->close(v9ses->transport); | 428 | v9ses->transport->close(v9ses->transport); |
429 | kfree(v9ses->transport); | ||
430 | v9ses->transport = NULL; | ||
431 | } | ||
413 | 432 | ||
414 | __putname(v9ses->name); | 433 | __putname(v9ses->name); |
415 | __putname(v9ses->remotename); | 434 | __putname(v9ses->remotename); |
@@ -420,8 +439,9 @@ void v9fs_session_close(struct v9fs_session_info *v9ses) | |||
420 | * and cancel all pending requests. | 439 | * and cancel all pending requests. |
421 | */ | 440 | */ |
422 | void v9fs_session_cancel(struct v9fs_session_info *v9ses) { | 441 | void v9fs_session_cancel(struct v9fs_session_info *v9ses) { |
442 | dprintk(DEBUG_ERROR, "cancel session %p\n", v9ses); | ||
423 | v9ses->transport->status = Disconnected; | 443 | v9ses->transport->status = Disconnected; |
424 | v9fs_mux_cancel_requests(v9ses, -EIO); | 444 | v9fs_mux_cancel(v9ses->mux, -EIO); |
425 | } | 445 | } |
426 | 446 | ||
427 | extern int v9fs_error_init(void); | 447 | extern int v9fs_error_init(void); |
@@ -433,11 +453,17 @@ extern int v9fs_error_init(void); | |||
433 | 453 | ||
434 | static int __init init_v9fs(void) | 454 | static int __init init_v9fs(void) |
435 | { | 455 | { |
456 | int ret; | ||
457 | |||
436 | v9fs_error_init(); | 458 | v9fs_error_init(); |
437 | 459 | ||
438 | printk(KERN_INFO "Installing v9fs 9P2000 file system support\n"); | 460 | printk(KERN_INFO "Installing v9fs 9P2000 file system support\n"); |
439 | 461 | ||
440 | return register_filesystem(&v9fs_fs_type); | 462 | ret = v9fs_mux_global_init(); |
463 | if (!ret) | ||
464 | ret = register_filesystem(&v9fs_fs_type); | ||
465 | |||
466 | return ret; | ||
441 | } | 467 | } |
442 | 468 | ||
443 | /** | 469 | /** |
@@ -447,6 +473,7 @@ static int __init init_v9fs(void) | |||
447 | 473 | ||
448 | static void __exit exit_v9fs(void) | 474 | static void __exit exit_v9fs(void) |
449 | { | 475 | { |
476 | v9fs_mux_global_exit(); | ||
450 | unregister_filesystem(&v9fs_fs_type); | 477 | unregister_filesystem(&v9fs_fs_type); |
451 | } | 478 | } |
452 | 479 | ||
diff --git a/fs/9p/v9fs.h b/fs/9p/v9fs.h index 45dcef42bdd6..f337da7a0eec 100644 --- a/fs/9p/v9fs.h +++ b/fs/9p/v9fs.h | |||
@@ -57,24 +57,14 @@ struct v9fs_session_info { | |||
57 | 57 | ||
58 | /* book keeping */ | 58 | /* book keeping */ |
59 | struct v9fs_idpool fidpool; /* The FID pool for file descriptors */ | 59 | struct v9fs_idpool fidpool; /* The FID pool for file descriptors */ |
60 | struct v9fs_idpool tidpool; /* The TID pool for transactions ids */ | ||
61 | 60 | ||
62 | /* transport information */ | ||
63 | struct v9fs_transport *transport; | 61 | struct v9fs_transport *transport; |
62 | struct v9fs_mux_data *mux; | ||
64 | 63 | ||
65 | int inprogress; /* session in progress => true */ | 64 | int inprogress; /* session in progress => true */ |
66 | int shutdown; /* session shutting down. no more attaches. */ | 65 | int shutdown; /* session shutting down. no more attaches. */ |
67 | unsigned char session_hung; | 66 | unsigned char session_hung; |
68 | 67 | struct dentry *debugfs_dir; | |
69 | /* mux private data */ | ||
70 | struct v9fs_fcall *curfcall; | ||
71 | wait_queue_head_t read_wait; | ||
72 | struct completion fcread; | ||
73 | struct completion proccmpl; | ||
74 | struct task_struct *recvproc; | ||
75 | |||
76 | spinlock_t muxlock; | ||
77 | struct list_head mux_fcalls; | ||
78 | }; | 68 | }; |
79 | 69 | ||
80 | /* possible values of ->proto */ | 70 | /* possible values of ->proto */ |
@@ -84,11 +74,14 @@ enum { | |||
84 | PROTO_FD, | 74 | PROTO_FD, |
85 | }; | 75 | }; |
86 | 76 | ||
77 | extern struct dentry *v9fs_debugfs_root; | ||
78 | |||
87 | int v9fs_session_init(struct v9fs_session_info *, const char *, char *); | 79 | int v9fs_session_init(struct v9fs_session_info *, const char *, char *); |
88 | struct v9fs_session_info *v9fs_inode2v9ses(struct inode *); | 80 | struct v9fs_session_info *v9fs_inode2v9ses(struct inode *); |
89 | void v9fs_session_close(struct v9fs_session_info *v9ses); | 81 | void v9fs_session_close(struct v9fs_session_info *v9ses); |
90 | int v9fs_get_idpool(struct v9fs_idpool *p); | 82 | int v9fs_get_idpool(struct v9fs_idpool *p); |
91 | void v9fs_put_idpool(int id, struct v9fs_idpool *p); | 83 | void v9fs_put_idpool(int id, struct v9fs_idpool *p); |
84 | int v9fs_check_idpool(int id, struct v9fs_idpool *p); | ||
92 | void v9fs_session_cancel(struct v9fs_session_info *v9ses); | 85 | void v9fs_session_cancel(struct v9fs_session_info *v9ses); |
93 | 86 | ||
94 | #define V9FS_MAGIC 0x01021997 | 87 | #define V9FS_MAGIC 0x01021997 |
diff --git a/fs/9p/v9fs_vfs.h b/fs/9p/v9fs_vfs.h index 2f2cea7ee3e7..c78502ad00ed 100644 --- a/fs/9p/v9fs_vfs.h +++ b/fs/9p/v9fs_vfs.h | |||
@@ -45,9 +45,8 @@ extern struct dentry_operations v9fs_dentry_operations; | |||
45 | 45 | ||
46 | struct inode *v9fs_get_inode(struct super_block *sb, int mode); | 46 | struct inode *v9fs_get_inode(struct super_block *sb, int mode); |
47 | ino_t v9fs_qid2ino(struct v9fs_qid *qid); | 47 | ino_t v9fs_qid2ino(struct v9fs_qid *qid); |
48 | void v9fs_mistat2inode(struct v9fs_stat *, struct inode *, | 48 | void v9fs_stat2inode(struct v9fs_stat *, struct inode *, struct super_block *); |
49 | struct super_block *); | ||
50 | int v9fs_dir_release(struct inode *inode, struct file *filp); | 49 | int v9fs_dir_release(struct inode *inode, struct file *filp); |
51 | int v9fs_file_open(struct inode *inode, struct file *file); | 50 | int v9fs_file_open(struct inode *inode, struct file *file); |
52 | void v9fs_inode2mistat(struct inode *inode, struct v9fs_stat *mistat); | 51 | void v9fs_inode2stat(struct inode *inode, struct v9fs_stat *stat); |
53 | void v9fs_dentry_release(struct dentry *); | 52 | void v9fs_dentry_release(struct dentry *); |
diff --git a/fs/9p/vfs_dentry.c b/fs/9p/vfs_dentry.c index a6aa947de0f9..2dd806dac9f1 100644 --- a/fs/9p/vfs_dentry.c +++ b/fs/9p/vfs_dentry.c | |||
@@ -40,7 +40,6 @@ | |||
40 | #include "v9fs.h" | 40 | #include "v9fs.h" |
41 | #include "9p.h" | 41 | #include "9p.h" |
42 | #include "v9fs_vfs.h" | 42 | #include "v9fs_vfs.h" |
43 | #include "conv.h" | ||
44 | #include "fid.h" | 43 | #include "fid.h" |
45 | 44 | ||
46 | /** | 45 | /** |
@@ -95,24 +94,22 @@ static int v9fs_dentry_validate(struct dentry *dentry, struct nameidata *nd) | |||
95 | 94 | ||
96 | void v9fs_dentry_release(struct dentry *dentry) | 95 | void v9fs_dentry_release(struct dentry *dentry) |
97 | { | 96 | { |
97 | int err; | ||
98 | |||
98 | dprintk(DEBUG_VFS, " dentry: %s (%p)\n", dentry->d_iname, dentry); | 99 | dprintk(DEBUG_VFS, " dentry: %s (%p)\n", dentry->d_iname, dentry); |
99 | 100 | ||
100 | if (dentry->d_fsdata != NULL) { | 101 | if (dentry->d_fsdata != NULL) { |
101 | struct list_head *fid_list = dentry->d_fsdata; | 102 | struct list_head *fid_list = dentry->d_fsdata; |
102 | struct v9fs_fid *temp = NULL; | 103 | struct v9fs_fid *temp = NULL; |
103 | struct v9fs_fid *current_fid = NULL; | 104 | struct v9fs_fid *current_fid = NULL; |
104 | struct v9fs_fcall *fcall = NULL; | ||
105 | 105 | ||
106 | list_for_each_entry_safe(current_fid, temp, fid_list, list) { | 106 | list_for_each_entry_safe(current_fid, temp, fid_list, list) { |
107 | if (v9fs_t_clunk | 107 | err = v9fs_t_clunk(current_fid->v9ses, current_fid->fid); |
108 | (current_fid->v9ses, current_fid->fid, &fcall)) | ||
109 | dprintk(DEBUG_ERROR, "clunk failed: %s\n", | ||
110 | FCALL_ERROR(fcall)); | ||
111 | 108 | ||
112 | v9fs_put_idpool(current_fid->fid, | 109 | if (err < 0) |
113 | ¤t_fid->v9ses->fidpool); | 110 | dprintk(DEBUG_ERROR, "clunk failed: %d name %s\n", |
111 | err, dentry->d_iname); | ||
114 | 112 | ||
115 | kfree(fcall); | ||
116 | v9fs_fid_destroy(current_fid); | 113 | v9fs_fid_destroy(current_fid); |
117 | } | 114 | } |
118 | 115 | ||
diff --git a/fs/9p/vfs_dir.c b/fs/9p/vfs_dir.c index 57a43b8feef5..ae6d032b9b59 100644 --- a/fs/9p/vfs_dir.c +++ b/fs/9p/vfs_dir.c | |||
@@ -37,8 +37,8 @@ | |||
37 | #include "debug.h" | 37 | #include "debug.h" |
38 | #include "v9fs.h" | 38 | #include "v9fs.h" |
39 | #include "9p.h" | 39 | #include "9p.h" |
40 | #include "v9fs_vfs.h" | ||
41 | #include "conv.h" | 40 | #include "conv.h" |
41 | #include "v9fs_vfs.h" | ||
42 | #include "fid.h" | 42 | #include "fid.h" |
43 | 43 | ||
44 | /** | 44 | /** |
@@ -74,20 +74,16 @@ static int v9fs_dir_readdir(struct file *filp, void *dirent, filldir_t filldir) | |||
74 | struct inode *inode = filp->f_dentry->d_inode; | 74 | struct inode *inode = filp->f_dentry->d_inode; |
75 | struct v9fs_session_info *v9ses = v9fs_inode2v9ses(inode); | 75 | struct v9fs_session_info *v9ses = v9fs_inode2v9ses(inode); |
76 | struct v9fs_fid *file = filp->private_data; | 76 | struct v9fs_fid *file = filp->private_data; |
77 | unsigned int i, n; | 77 | unsigned int i, n, s; |
78 | int fid = -1; | 78 | int fid = -1; |
79 | int ret = 0; | 79 | int ret = 0; |
80 | struct v9fs_stat *mi = NULL; | 80 | struct v9fs_stat stat; |
81 | int over = 0; | 81 | int over = 0; |
82 | 82 | ||
83 | dprintk(DEBUG_VFS, "name %s\n", filp->f_dentry->d_name.name); | 83 | dprintk(DEBUG_VFS, "name %s\n", filp->f_dentry->d_name.name); |
84 | 84 | ||
85 | fid = file->fid; | 85 | fid = file->fid; |
86 | 86 | ||
87 | mi = kmalloc(v9ses->maxdata, GFP_KERNEL); | ||
88 | if (!mi) | ||
89 | return -ENOMEM; | ||
90 | |||
91 | if (file->rdir_fcall && (filp->f_pos != file->rdir_pos)) { | 87 | if (file->rdir_fcall && (filp->f_pos != file->rdir_pos)) { |
92 | kfree(file->rdir_fcall); | 88 | kfree(file->rdir_fcall); |
93 | file->rdir_fcall = NULL; | 89 | file->rdir_fcall = NULL; |
@@ -97,20 +93,20 @@ static int v9fs_dir_readdir(struct file *filp, void *dirent, filldir_t filldir) | |||
97 | n = file->rdir_fcall->params.rread.count; | 93 | n = file->rdir_fcall->params.rread.count; |
98 | i = file->rdir_fpos; | 94 | i = file->rdir_fpos; |
99 | while (i < n) { | 95 | while (i < n) { |
100 | int s = v9fs_deserialize_stat(v9ses, | 96 | s = v9fs_deserialize_stat( |
101 | file->rdir_fcall->params.rread.data + i, | 97 | file->rdir_fcall->params.rread.data + i, |
102 | n - i, mi, v9ses->maxdata); | 98 | n - i, &stat, v9ses->extended); |
103 | 99 | ||
104 | if (s == 0) { | 100 | if (s == 0) { |
105 | dprintk(DEBUG_ERROR, | 101 | dprintk(DEBUG_ERROR, |
106 | "error while deserializing mistat\n"); | 102 | "error while deserializing stat\n"); |
107 | ret = -EIO; | 103 | ret = -EIO; |
108 | goto FreeStructs; | 104 | goto FreeStructs; |
109 | } | 105 | } |
110 | 106 | ||
111 | over = filldir(dirent, mi->name, strlen(mi->name), | 107 | over = filldir(dirent, stat.name.str, stat.name.len, |
112 | filp->f_pos, v9fs_qid2ino(&mi->qid), | 108 | filp->f_pos, v9fs_qid2ino(&stat.qid), |
113 | dt_type(mi)); | 109 | dt_type(&stat)); |
114 | 110 | ||
115 | if (over) { | 111 | if (over) { |
116 | file->rdir_fpos = i; | 112 | file->rdir_fpos = i; |
@@ -130,7 +126,7 @@ static int v9fs_dir_readdir(struct file *filp, void *dirent, filldir_t filldir) | |||
130 | 126 | ||
131 | while (!over) { | 127 | while (!over) { |
132 | ret = v9fs_t_read(v9ses, fid, filp->f_pos, | 128 | ret = v9fs_t_read(v9ses, fid, filp->f_pos, |
133 | v9ses->maxdata-V9FS_IOHDRSZ, &fcall); | 129 | v9ses->maxdata-V9FS_IOHDRSZ, &fcall); |
134 | if (ret < 0) { | 130 | if (ret < 0) { |
135 | dprintk(DEBUG_ERROR, "error while reading: %d: %p\n", | 131 | dprintk(DEBUG_ERROR, "error while reading: %d: %p\n", |
136 | ret, fcall); | 132 | ret, fcall); |
@@ -141,19 +137,18 @@ static int v9fs_dir_readdir(struct file *filp, void *dirent, filldir_t filldir) | |||
141 | n = ret; | 137 | n = ret; |
142 | i = 0; | 138 | i = 0; |
143 | while (i < n) { | 139 | while (i < n) { |
144 | int s = v9fs_deserialize_stat(v9ses, | 140 | s = v9fs_deserialize_stat(fcall->params.rread.data + i, |
145 | fcall->params.rread.data + i, n - i, mi, | 141 | n - i, &stat, v9ses->extended); |
146 | v9ses->maxdata); | ||
147 | 142 | ||
148 | if (s == 0) { | 143 | if (s == 0) { |
149 | dprintk(DEBUG_ERROR, | 144 | dprintk(DEBUG_ERROR, |
150 | "error while deserializing mistat\n"); | 145 | "error while deserializing stat\n"); |
151 | return -EIO; | 146 | return -EIO; |
152 | } | 147 | } |
153 | 148 | ||
154 | over = filldir(dirent, mi->name, strlen(mi->name), | 149 | over = filldir(dirent, stat.name.str, stat.name.len, |
155 | filp->f_pos, v9fs_qid2ino(&mi->qid), | 150 | filp->f_pos, v9fs_qid2ino(&stat.qid), |
156 | dt_type(mi)); | 151 | dt_type(&stat)); |
157 | 152 | ||
158 | if (over) { | 153 | if (over) { |
159 | file->rdir_fcall = fcall; | 154 | file->rdir_fcall = fcall; |
@@ -172,7 +167,6 @@ static int v9fs_dir_readdir(struct file *filp, void *dirent, filldir_t filldir) | |||
172 | 167 | ||
173 | FreeStructs: | 168 | FreeStructs: |
174 | kfree(fcall); | 169 | kfree(fcall); |
175 | kfree(mi); | ||
176 | return ret; | 170 | return ret; |
177 | } | 171 | } |
178 | 172 | ||
@@ -193,18 +187,15 @@ int v9fs_dir_release(struct inode *inode, struct file *filp) | |||
193 | fid->fid); | 187 | fid->fid); |
194 | fidnum = fid->fid; | 188 | fidnum = fid->fid; |
195 | 189 | ||
196 | filemap_fdatawrite(inode->i_mapping); | 190 | filemap_write_and_wait(inode->i_mapping); |
197 | filemap_fdatawait(inode->i_mapping); | ||
198 | 191 | ||
199 | if (fidnum >= 0) { | 192 | if (fidnum >= 0) { |
200 | dprintk(DEBUG_VFS, "fidopen: %d v9f->fid: %d\n", fid->fidopen, | 193 | dprintk(DEBUG_VFS, "fidopen: %d v9f->fid: %d\n", fid->fidopen, |
201 | fid->fid); | 194 | fid->fid); |
202 | 195 | ||
203 | if (v9fs_t_clunk(v9ses, fidnum, NULL)) | 196 | if (v9fs_t_clunk(v9ses, fidnum)) |
204 | dprintk(DEBUG_ERROR, "clunk failed\n"); | 197 | dprintk(DEBUG_ERROR, "clunk failed\n"); |
205 | 198 | ||
206 | v9fs_put_idpool(fid->fid, &v9ses->fidpool); | ||
207 | |||
208 | kfree(fid->rdir_fcall); | 199 | kfree(fid->rdir_fcall); |
209 | kfree(fid); | 200 | kfree(fid); |
210 | 201 | ||
diff --git a/fs/9p/vfs_file.c b/fs/9p/vfs_file.c index 89c849da8504..6852f0eb96ed 100644 --- a/fs/9p/vfs_file.c +++ b/fs/9p/vfs_file.c | |||
@@ -32,6 +32,7 @@ | |||
32 | #include <linux/string.h> | 32 | #include <linux/string.h> |
33 | #include <linux/smp_lock.h> | 33 | #include <linux/smp_lock.h> |
34 | #include <linux/inet.h> | 34 | #include <linux/inet.h> |
35 | #include <linux/version.h> | ||
35 | #include <linux/list.h> | 36 | #include <linux/list.h> |
36 | #include <asm/uaccess.h> | 37 | #include <asm/uaccess.h> |
37 | #include <linux/idr.h> | 38 | #include <linux/idr.h> |
@@ -117,9 +118,7 @@ int v9fs_file_open(struct inode *inode, struct file *file) | |||
117 | 118 | ||
118 | result = v9fs_t_open(v9ses, newfid, open_mode, &fcall); | 119 | result = v9fs_t_open(v9ses, newfid, open_mode, &fcall); |
119 | if (result < 0) { | 120 | if (result < 0) { |
120 | dprintk(DEBUG_ERROR, | 121 | PRINT_FCALL_ERROR("open failed", fcall); |
121 | "open failed, open_mode 0x%x: %s\n", open_mode, | ||
122 | FCALL_ERROR(fcall)); | ||
123 | kfree(fcall); | 122 | kfree(fcall); |
124 | return result; | 123 | return result; |
125 | } | 124 | } |
@@ -165,8 +164,7 @@ static int v9fs_file_lock(struct file *filp, int cmd, struct file_lock *fl) | |||
165 | return -ENOLCK; | 164 | return -ENOLCK; |
166 | 165 | ||
167 | if ((IS_SETLK(cmd) || IS_SETLKW(cmd)) && fl->fl_type != F_UNLCK) { | 166 | if ((IS_SETLK(cmd) || IS_SETLKW(cmd)) && fl->fl_type != F_UNLCK) { |
168 | filemap_fdatawrite(inode->i_mapping); | 167 | filemap_write_and_wait(inode->i_mapping); |
169 | filemap_fdatawait(inode->i_mapping); | ||
170 | invalidate_inode_pages(&inode->i_data); | 168 | invalidate_inode_pages(&inode->i_data); |
171 | } | 169 | } |
172 | 170 | ||
@@ -257,7 +255,6 @@ v9fs_file_write(struct file *filp, const char __user * data, | |||
257 | int result = -EIO; | 255 | int result = -EIO; |
258 | int rsize = 0; | 256 | int rsize = 0; |
259 | int total = 0; | 257 | int total = 0; |
260 | char *buf; | ||
261 | 258 | ||
262 | dprintk(DEBUG_VFS, "data %p count %d offset %x\n", data, (int)count, | 259 | dprintk(DEBUG_VFS, "data %p count %d offset %x\n", data, (int)count, |
263 | (int)*offset); | 260 | (int)*offset); |
@@ -265,28 +262,14 @@ v9fs_file_write(struct file *filp, const char __user * data, | |||
265 | if (v9fid->iounit != 0 && rsize > v9fid->iounit) | 262 | if (v9fid->iounit != 0 && rsize > v9fid->iounit) |
266 | rsize = v9fid->iounit; | 263 | rsize = v9fid->iounit; |
267 | 264 | ||
268 | buf = kmalloc(v9ses->maxdata - V9FS_IOHDRSZ, GFP_KERNEL); | ||
269 | if (!buf) | ||
270 | return -ENOMEM; | ||
271 | |||
272 | do { | 265 | do { |
273 | if (count < rsize) | 266 | if (count < rsize) |
274 | rsize = count; | 267 | rsize = count; |
275 | 268 | ||
276 | result = copy_from_user(buf, data, rsize); | 269 | result = v9fs_t_write(v9ses, fid, *offset, rsize, data, &fcall); |
277 | if (result) { | ||
278 | dprintk(DEBUG_ERROR, "Problem copying from user\n"); | ||
279 | kfree(buf); | ||
280 | return -EFAULT; | ||
281 | } | ||
282 | |||
283 | dump_data(buf, rsize); | ||
284 | result = v9fs_t_write(v9ses, fid, *offset, rsize, buf, &fcall); | ||
285 | if (result < 0) { | 270 | if (result < 0) { |
286 | eprintk(KERN_ERR, "error while writing: %s(%d)\n", | 271 | PRINT_FCALL_ERROR("error while writing", fcall); |
287 | FCALL_ERROR(fcall), result); | ||
288 | kfree(fcall); | 272 | kfree(fcall); |
289 | kfree(buf); | ||
290 | return result; | 273 | return result; |
291 | } else | 274 | } else |
292 | *offset += result; | 275 | *offset += result; |
@@ -306,7 +289,6 @@ v9fs_file_write(struct file *filp, const char __user * data, | |||
306 | total += result; | 289 | total += result; |
307 | } while (count); | 290 | } while (count); |
308 | 291 | ||
309 | kfree(buf); | ||
310 | return total; | 292 | return total; |
311 | } | 293 | } |
312 | 294 | ||
diff --git a/fs/9p/vfs_inode.c b/fs/9p/vfs_inode.c index 0ea965c3bb7d..a17b28854288 100644 --- a/fs/9p/vfs_inode.c +++ b/fs/9p/vfs_inode.c | |||
@@ -40,7 +40,6 @@ | |||
40 | #include "v9fs.h" | 40 | #include "v9fs.h" |
41 | #include "9p.h" | 41 | #include "9p.h" |
42 | #include "v9fs_vfs.h" | 42 | #include "v9fs_vfs.h" |
43 | #include "conv.h" | ||
44 | #include "fid.h" | 43 | #include "fid.h" |
45 | 44 | ||
46 | static struct inode_operations v9fs_dir_inode_operations; | 45 | static struct inode_operations v9fs_dir_inode_operations; |
@@ -127,100 +126,32 @@ static int p9mode2unixmode(struct v9fs_session_info *v9ses, int mode) | |||
127 | } | 126 | } |
128 | 127 | ||
129 | /** | 128 | /** |
130 | * v9fs_blank_mistat - helper function to setup a 9P stat structure | 129 | * v9fs_blank_wstat - helper function to setup a 9P stat structure |
131 | * @v9ses: 9P session info (for determining extended mode) | 130 | * @v9ses: 9P session info (for determining extended mode) |
132 | * @mistat: structure to initialize | 131 | * @wstat: structure to initialize |
133 | * | 132 | * |
134 | */ | 133 | */ |
135 | 134 | ||
136 | static void | 135 | static void |
137 | v9fs_blank_mistat(struct v9fs_session_info *v9ses, struct v9fs_stat *mistat) | 136 | v9fs_blank_wstat(struct v9fs_wstat *wstat) |
138 | { | 137 | { |
139 | mistat->type = ~0; | 138 | wstat->type = ~0; |
140 | mistat->dev = ~0; | 139 | wstat->dev = ~0; |
141 | mistat->qid.type = ~0; | 140 | wstat->qid.type = ~0; |
142 | mistat->qid.version = ~0; | 141 | wstat->qid.version = ~0; |
143 | *((long long *)&mistat->qid.path) = ~0; | 142 | *((long long *)&wstat->qid.path) = ~0; |
144 | mistat->mode = ~0; | 143 | wstat->mode = ~0; |
145 | mistat->atime = ~0; | 144 | wstat->atime = ~0; |
146 | mistat->mtime = ~0; | 145 | wstat->mtime = ~0; |
147 | mistat->length = ~0; | 146 | wstat->length = ~0; |
148 | mistat->name = mistat->data; | 147 | wstat->name = NULL; |
149 | mistat->uid = mistat->data; | 148 | wstat->uid = NULL; |
150 | mistat->gid = mistat->data; | 149 | wstat->gid = NULL; |
151 | mistat->muid = mistat->data; | 150 | wstat->muid = NULL; |
152 | if (v9ses->extended) { | 151 | wstat->n_uid = ~0; |
153 | mistat->n_uid = ~0; | 152 | wstat->n_gid = ~0; |
154 | mistat->n_gid = ~0; | 153 | wstat->n_muid = ~0; |
155 | mistat->n_muid = ~0; | 154 | wstat->extension = NULL; |
156 | mistat->extension = mistat->data; | ||
157 | } | ||
158 | *mistat->data = 0; | ||
159 | } | ||
160 | |||
161 | /** | ||
162 | * v9fs_mistat2unix - convert mistat to unix stat | ||
163 | * @mistat: Plan 9 metadata (mistat) structure | ||
164 | * @buf: unix metadata (stat) structure to populate | ||
165 | * @sb: superblock | ||
166 | * | ||
167 | */ | ||
168 | |||
169 | static void | ||
170 | v9fs_mistat2unix(struct v9fs_stat *mistat, struct stat *buf, | ||
171 | struct super_block *sb) | ||
172 | { | ||
173 | struct v9fs_session_info *v9ses = sb ? sb->s_fs_info : NULL; | ||
174 | |||
175 | buf->st_nlink = 1; | ||
176 | |||
177 | buf->st_atime = mistat->atime; | ||
178 | buf->st_mtime = mistat->mtime; | ||
179 | buf->st_ctime = mistat->mtime; | ||
180 | |||
181 | buf->st_uid = (unsigned short)-1; | ||
182 | buf->st_gid = (unsigned short)-1; | ||
183 | |||
184 | if (v9ses && v9ses->extended) { | ||
185 | /* TODO: string to uid mapping via user-space daemon */ | ||
186 | if (mistat->n_uid != -1) | ||
187 | sscanf(mistat->uid, "%x", (unsigned int *)&buf->st_uid); | ||
188 | |||
189 | if (mistat->n_gid != -1) | ||
190 | sscanf(mistat->gid, "%x", (unsigned int *)&buf->st_gid); | ||
191 | } | ||
192 | |||
193 | if (buf->st_uid == (unsigned short)-1) | ||
194 | buf->st_uid = v9ses->uid; | ||
195 | if (buf->st_gid == (unsigned short)-1) | ||
196 | buf->st_gid = v9ses->gid; | ||
197 | |||
198 | buf->st_mode = p9mode2unixmode(v9ses, mistat->mode); | ||
199 | if ((S_ISBLK(buf->st_mode)) || (S_ISCHR(buf->st_mode))) { | ||
200 | char type = 0; | ||
201 | int major = -1; | ||
202 | int minor = -1; | ||
203 | sscanf(mistat->extension, "%c %u %u", &type, &major, &minor); | ||
204 | switch (type) { | ||
205 | case 'c': | ||
206 | buf->st_mode &= ~S_IFBLK; | ||
207 | buf->st_mode |= S_IFCHR; | ||
208 | break; | ||
209 | case 'b': | ||
210 | break; | ||
211 | default: | ||
212 | dprintk(DEBUG_ERROR, "Unknown special type %c (%s)\n", | ||
213 | type, mistat->extension); | ||
214 | }; | ||
215 | buf->st_rdev = MKDEV(major, minor); | ||
216 | } else | ||
217 | buf->st_rdev = 0; | ||
218 | |||
219 | buf->st_size = mistat->length; | ||
220 | |||
221 | buf->st_blksize = sb->s_blocksize; | ||
222 | buf->st_blocks = | ||
223 | (buf->st_size + buf->st_blksize - 1) >> sb->s_blocksize_bits; | ||
224 | } | 155 | } |
225 | 156 | ||
226 | /** | 157 | /** |
@@ -312,12 +243,12 @@ v9fs_create(struct inode *dir, | |||
312 | struct inode *file_inode = NULL; | 243 | struct inode *file_inode = NULL; |
313 | struct v9fs_fcall *fcall = NULL; | 244 | struct v9fs_fcall *fcall = NULL; |
314 | struct v9fs_qid qid; | 245 | struct v9fs_qid qid; |
315 | struct stat newstat; | ||
316 | int dirfidnum = -1; | 246 | int dirfidnum = -1; |
317 | long newfid = -1; | 247 | long newfid = -1; |
318 | int result = 0; | 248 | int result = 0; |
319 | unsigned int iounit = 0; | 249 | unsigned int iounit = 0; |
320 | int wfidno = -1; | 250 | int wfidno = -1; |
251 | int err; | ||
321 | 252 | ||
322 | perm = unixmode2p9mode(v9ses, perm); | 253 | perm = unixmode2p9mode(v9ses, perm); |
323 | 254 | ||
@@ -349,57 +280,64 @@ v9fs_create(struct inode *dir, | |||
349 | 280 | ||
350 | result = v9fs_t_walk(v9ses, dirfidnum, newfid, NULL, &fcall); | 281 | result = v9fs_t_walk(v9ses, dirfidnum, newfid, NULL, &fcall); |
351 | if (result < 0) { | 282 | if (result < 0) { |
352 | dprintk(DEBUG_ERROR, "clone error: %s\n", FCALL_ERROR(fcall)); | 283 | PRINT_FCALL_ERROR("clone error", fcall); |
353 | v9fs_put_idpool(newfid, &v9ses->fidpool); | 284 | v9fs_put_idpool(newfid, &v9ses->fidpool); |
354 | newfid = -1; | 285 | newfid = -1; |
355 | goto CleanUpFid; | 286 | goto CleanUpFid; |
356 | } | 287 | } |
357 | 288 | ||
358 | kfree(fcall); | 289 | kfree(fcall); |
290 | fcall = NULL; | ||
359 | 291 | ||
360 | result = v9fs_t_create(v9ses, newfid, (char *)file_dentry->d_name.name, | 292 | result = v9fs_t_create(v9ses, newfid, (char *)file_dentry->d_name.name, |
361 | perm, open_mode, &fcall); | 293 | perm, open_mode, &fcall); |
362 | if (result < 0) { | 294 | if (result < 0) { |
363 | dprintk(DEBUG_ERROR, "create fails: %s(%d)\n", | 295 | PRINT_FCALL_ERROR("create fails", fcall); |
364 | FCALL_ERROR(fcall), result); | ||
365 | |||
366 | goto CleanUpFid; | 296 | goto CleanUpFid; |
367 | } | 297 | } |
368 | 298 | ||
369 | iounit = fcall->params.rcreate.iounit; | 299 | iounit = fcall->params.rcreate.iounit; |
370 | qid = fcall->params.rcreate.qid; | 300 | qid = fcall->params.rcreate.qid; |
371 | kfree(fcall); | 301 | kfree(fcall); |
302 | fcall = NULL; | ||
372 | 303 | ||
373 | fid = v9fs_fid_create(file_dentry, v9ses, newfid, 1); | 304 | if (!(perm&V9FS_DMDIR)) { |
374 | dprintk(DEBUG_VFS, "fid %p %d\n", fid, fid->fidcreate); | 305 | fid = v9fs_fid_create(file_dentry, v9ses, newfid, 1); |
375 | if (!fid) { | 306 | dprintk(DEBUG_VFS, "fid %p %d\n", fid, fid->fidcreate); |
376 | result = -ENOMEM; | 307 | if (!fid) { |
377 | goto CleanUpFid; | 308 | result = -ENOMEM; |
378 | } | 309 | goto CleanUpFid; |
310 | } | ||
379 | 311 | ||
380 | fid->qid = qid; | 312 | fid->qid = qid; |
381 | fid->iounit = iounit; | 313 | fid->iounit = iounit; |
314 | } else { | ||
315 | err = v9fs_t_clunk(v9ses, newfid); | ||
316 | newfid = -1; | ||
317 | if (err < 0) | ||
318 | dprintk(DEBUG_ERROR, "clunk for mkdir failed: %d\n", err); | ||
319 | } | ||
382 | 320 | ||
383 | /* walk to the newly created file and put the fid in the dentry */ | 321 | /* walk to the newly created file and put the fid in the dentry */ |
384 | wfidno = v9fs_get_idpool(&v9ses->fidpool); | 322 | wfidno = v9fs_get_idpool(&v9ses->fidpool); |
385 | if (newfid < 0) { | 323 | if (wfidno < 0) { |
386 | eprintk(KERN_WARNING, "no free fids available\n"); | 324 | eprintk(KERN_WARNING, "no free fids available\n"); |
387 | return -ENOSPC; | 325 | return -ENOSPC; |
388 | } | 326 | } |
389 | 327 | ||
390 | result = v9fs_t_walk(v9ses, dirfidnum, wfidno, | 328 | result = v9fs_t_walk(v9ses, dirfidnum, wfidno, |
391 | (char *) file_dentry->d_name.name, NULL); | 329 | (char *) file_dentry->d_name.name, &fcall); |
392 | if (result < 0) { | 330 | if (result < 0) { |
393 | dprintk(DEBUG_ERROR, "clone error: %s\n", FCALL_ERROR(fcall)); | 331 | PRINT_FCALL_ERROR("clone error", fcall); |
394 | v9fs_put_idpool(wfidno, &v9ses->fidpool); | 332 | v9fs_put_idpool(wfidno, &v9ses->fidpool); |
395 | wfidno = -1; | 333 | wfidno = -1; |
396 | goto CleanUpFid; | 334 | goto CleanUpFid; |
397 | } | 335 | } |
336 | kfree(fcall); | ||
337 | fcall = NULL; | ||
398 | 338 | ||
399 | if (!v9fs_fid_create(file_dentry, v9ses, wfidno, 0)) { | 339 | if (!v9fs_fid_create(file_dentry, v9ses, wfidno, 0)) { |
400 | if (!v9fs_t_clunk(v9ses, newfid, &fcall)) { | 340 | v9fs_put_idpool(wfidno, &v9ses->fidpool); |
401 | v9fs_put_idpool(wfidno, &v9ses->fidpool); | ||
402 | } | ||
403 | 341 | ||
404 | goto CleanUpFid; | 342 | goto CleanUpFid; |
405 | } | 343 | } |
@@ -409,62 +347,43 @@ v9fs_create(struct inode *dir, | |||
409 | (perm & V9FS_DMDEVICE)) | 347 | (perm & V9FS_DMDEVICE)) |
410 | return 0; | 348 | return 0; |
411 | 349 | ||
412 | result = v9fs_t_stat(v9ses, newfid, &fcall); | 350 | result = v9fs_t_stat(v9ses, wfidno, &fcall); |
413 | if (result < 0) { | 351 | if (result < 0) { |
414 | dprintk(DEBUG_ERROR, "stat error: %s(%d)\n", FCALL_ERROR(fcall), | 352 | PRINT_FCALL_ERROR("stat error", fcall); |
415 | result); | ||
416 | goto CleanUpFid; | 353 | goto CleanUpFid; |
417 | } | 354 | } |
418 | 355 | ||
419 | v9fs_mistat2unix(fcall->params.rstat.stat, &newstat, sb); | ||
420 | 356 | ||
421 | file_inode = v9fs_get_inode(sb, newstat.st_mode); | 357 | file_inode = v9fs_get_inode(sb, |
358 | p9mode2unixmode(v9ses, fcall->params.rstat.stat.mode)); | ||
359 | |||
422 | if ((!file_inode) || IS_ERR(file_inode)) { | 360 | if ((!file_inode) || IS_ERR(file_inode)) { |
423 | dprintk(DEBUG_ERROR, "create inode failed\n"); | 361 | dprintk(DEBUG_ERROR, "create inode failed\n"); |
424 | result = -EBADF; | 362 | result = -EBADF; |
425 | goto CleanUpFid; | 363 | goto CleanUpFid; |
426 | } | 364 | } |
427 | 365 | ||
428 | v9fs_mistat2inode(fcall->params.rstat.stat, file_inode, sb); | 366 | v9fs_stat2inode(&fcall->params.rstat.stat, file_inode, sb); |
429 | kfree(fcall); | 367 | kfree(fcall); |
430 | fcall = NULL; | 368 | fcall = NULL; |
431 | file_dentry->d_op = &v9fs_dentry_operations; | 369 | file_dentry->d_op = &v9fs_dentry_operations; |
432 | d_instantiate(file_dentry, file_inode); | 370 | d_instantiate(file_dentry, file_inode); |
433 | 371 | ||
434 | if (perm & V9FS_DMDIR) { | ||
435 | if (!v9fs_t_clunk(v9ses, newfid, &fcall)) | ||
436 | v9fs_put_idpool(newfid, &v9ses->fidpool); | ||
437 | else | ||
438 | dprintk(DEBUG_ERROR, "clunk for mkdir failed: %s\n", | ||
439 | FCALL_ERROR(fcall)); | ||
440 | kfree(fcall); | ||
441 | fid->fidopen = 0; | ||
442 | fid->fidcreate = 0; | ||
443 | d_drop(file_dentry); | ||
444 | } | ||
445 | |||
446 | return 0; | 372 | return 0; |
447 | 373 | ||
448 | CleanUpFid: | 374 | CleanUpFid: |
449 | kfree(fcall); | 375 | kfree(fcall); |
376 | fcall = NULL; | ||
450 | 377 | ||
451 | if (newfid >= 0) { | 378 | if (newfid >= 0) { |
452 | if (!v9fs_t_clunk(v9ses, newfid, &fcall)) | 379 | err = v9fs_t_clunk(v9ses, newfid); |
453 | v9fs_put_idpool(newfid, &v9ses->fidpool); | 380 | if (err < 0) |
454 | else | 381 | dprintk(DEBUG_ERROR, "clunk failed: %d\n", err); |
455 | dprintk(DEBUG_ERROR, "clunk failed: %s\n", | ||
456 | FCALL_ERROR(fcall)); | ||
457 | |||
458 | kfree(fcall); | ||
459 | } | 382 | } |
460 | if (wfidno >= 0) { | 383 | if (wfidno >= 0) { |
461 | if (!v9fs_t_clunk(v9ses, wfidno, &fcall)) | 384 | err = v9fs_t_clunk(v9ses, wfidno); |
462 | v9fs_put_idpool(wfidno, &v9ses->fidpool); | 385 | if (err < 0) |
463 | else | 386 | dprintk(DEBUG_ERROR, "clunk failed: %d\n", err); |
464 | dprintk(DEBUG_ERROR, "clunk failed: %s\n", | ||
465 | FCALL_ERROR(fcall)); | ||
466 | |||
467 | kfree(fcall); | ||
468 | } | 387 | } |
469 | return result; | 388 | return result; |
470 | } | 389 | } |
@@ -509,10 +428,9 @@ static int v9fs_remove(struct inode *dir, struct dentry *file, int rmdir) | |||
509 | } | 428 | } |
510 | 429 | ||
511 | result = v9fs_t_remove(v9ses, fid, &fcall); | 430 | result = v9fs_t_remove(v9ses, fid, &fcall); |
512 | if (result < 0) | 431 | if (result < 0) { |
513 | dprintk(DEBUG_ERROR, "remove of file fails: %s(%d)\n", | 432 | PRINT_FCALL_ERROR("remove fails", fcall); |
514 | FCALL_ERROR(fcall), result); | 433 | } else { |
515 | else { | ||
516 | v9fs_put_idpool(fid, &v9ses->fidpool); | 434 | v9fs_put_idpool(fid, &v9ses->fidpool); |
517 | v9fs_fid_destroy(v9fid); | 435 | v9fs_fid_destroy(v9fid); |
518 | } | 436 | } |
@@ -567,7 +485,6 @@ static struct dentry *v9fs_vfs_lookup(struct inode *dir, struct dentry *dentry, | |||
567 | struct v9fs_fid *fid; | 485 | struct v9fs_fid *fid; |
568 | struct inode *inode; | 486 | struct inode *inode; |
569 | struct v9fs_fcall *fcall = NULL; | 487 | struct v9fs_fcall *fcall = NULL; |
570 | struct stat newstat; | ||
571 | int dirfidnum = -1; | 488 | int dirfidnum = -1; |
572 | int newfid = -1; | 489 | int newfid = -1; |
573 | int result = 0; | 490 | int result = 0; |
@@ -620,8 +537,8 @@ static struct dentry *v9fs_vfs_lookup(struct inode *dir, struct dentry *dentry, | |||
620 | goto FreeFcall; | 537 | goto FreeFcall; |
621 | } | 538 | } |
622 | 539 | ||
623 | v9fs_mistat2unix(fcall->params.rstat.stat, &newstat, sb); | 540 | inode = v9fs_get_inode(sb, p9mode2unixmode(v9ses, |
624 | inode = v9fs_get_inode(sb, newstat.st_mode); | 541 | fcall->params.rstat.stat.mode)); |
625 | 542 | ||
626 | if (IS_ERR(inode) && (PTR_ERR(inode) == -ENOSPC)) { | 543 | if (IS_ERR(inode) && (PTR_ERR(inode) == -ENOSPC)) { |
627 | eprintk(KERN_WARNING, "inode alloc failes, returns %ld\n", | 544 | eprintk(KERN_WARNING, "inode alloc failes, returns %ld\n", |
@@ -631,7 +548,7 @@ static struct dentry *v9fs_vfs_lookup(struct inode *dir, struct dentry *dentry, | |||
631 | goto FreeFcall; | 548 | goto FreeFcall; |
632 | } | 549 | } |
633 | 550 | ||
634 | inode->i_ino = v9fs_qid2ino(&fcall->params.rstat.stat->qid); | 551 | inode->i_ino = v9fs_qid2ino(&fcall->params.rstat.stat.qid); |
635 | 552 | ||
636 | fid = v9fs_fid_create(dentry, v9ses, newfid, 0); | 553 | fid = v9fs_fid_create(dentry, v9ses, newfid, 0); |
637 | if (fid == NULL) { | 554 | if (fid == NULL) { |
@@ -640,10 +557,10 @@ static struct dentry *v9fs_vfs_lookup(struct inode *dir, struct dentry *dentry, | |||
640 | goto FreeFcall; | 557 | goto FreeFcall; |
641 | } | 558 | } |
642 | 559 | ||
643 | fid->qid = fcall->params.rstat.stat->qid; | 560 | fid->qid = fcall->params.rstat.stat.qid; |
644 | 561 | ||
645 | dentry->d_op = &v9fs_dentry_operations; | 562 | dentry->d_op = &v9fs_dentry_operations; |
646 | v9fs_mistat2inode(fcall->params.rstat.stat, inode, inode->i_sb); | 563 | v9fs_stat2inode(&fcall->params.rstat.stat, inode, inode->i_sb); |
647 | 564 | ||
648 | d_add(dentry, inode); | 565 | d_add(dentry, inode); |
649 | kfree(fcall); | 566 | kfree(fcall); |
@@ -699,7 +616,7 @@ v9fs_vfs_rename(struct inode *old_dir, struct dentry *old_dentry, | |||
699 | v9fs_fid_lookup(old_dentry->d_parent); | 616 | v9fs_fid_lookup(old_dentry->d_parent); |
700 | struct v9fs_fid *newdirfid = | 617 | struct v9fs_fid *newdirfid = |
701 | v9fs_fid_lookup(new_dentry->d_parent); | 618 | v9fs_fid_lookup(new_dentry->d_parent); |
702 | struct v9fs_stat *mistat = kmalloc(v9ses->maxdata, GFP_KERNEL); | 619 | struct v9fs_wstat wstat; |
703 | struct v9fs_fcall *fcall = NULL; | 620 | struct v9fs_fcall *fcall = NULL; |
704 | int fid = -1; | 621 | int fid = -1; |
705 | int olddirfidnum = -1; | 622 | int olddirfidnum = -1; |
@@ -708,9 +625,6 @@ v9fs_vfs_rename(struct inode *old_dir, struct dentry *old_dentry, | |||
708 | 625 | ||
709 | dprintk(DEBUG_VFS, "\n"); | 626 | dprintk(DEBUG_VFS, "\n"); |
710 | 627 | ||
711 | if (!mistat) | ||
712 | return -ENOMEM; | ||
713 | |||
714 | if ((!oldfid) || (!olddirfid) || (!newdirfid)) { | 628 | if ((!oldfid) || (!olddirfid) || (!newdirfid)) { |
715 | dprintk(DEBUG_ERROR, "problem with arguments\n"); | 629 | dprintk(DEBUG_ERROR, "problem with arguments\n"); |
716 | return -EBADF; | 630 | return -EBADF; |
@@ -734,33 +648,22 @@ v9fs_vfs_rename(struct inode *old_dir, struct dentry *old_dentry, | |||
734 | goto FreeFcallnBail; | 648 | goto FreeFcallnBail; |
735 | } | 649 | } |
736 | 650 | ||
737 | v9fs_blank_mistat(v9ses, mistat); | 651 | v9fs_blank_wstat(&wstat); |
652 | wstat.muid = v9ses->name; | ||
653 | wstat.name = (char *) new_dentry->d_name.name; | ||
738 | 654 | ||
739 | strcpy(mistat->data + 1, v9ses->name); | 655 | retval = v9fs_t_wstat(v9ses, fid, &wstat, &fcall); |
740 | mistat->name = mistat->data + 1 + strlen(v9ses->name); | ||
741 | |||
742 | if (new_dentry->d_name.len > | ||
743 | (v9ses->maxdata - strlen(v9ses->name) - sizeof(struct v9fs_stat))) { | ||
744 | dprintk(DEBUG_ERROR, "new name too long\n"); | ||
745 | goto FreeFcallnBail; | ||
746 | } | ||
747 | |||
748 | strcpy(mistat->name, new_dentry->d_name.name); | ||
749 | retval = v9fs_t_wstat(v9ses, fid, mistat, &fcall); | ||
750 | 656 | ||
751 | FreeFcallnBail: | 657 | FreeFcallnBail: |
752 | kfree(mistat); | ||
753 | |||
754 | if (retval < 0) | 658 | if (retval < 0) |
755 | dprintk(DEBUG_ERROR, "v9fs_t_wstat error: %s\n", | 659 | PRINT_FCALL_ERROR("wstat error", fcall); |
756 | FCALL_ERROR(fcall)); | ||
757 | 660 | ||
758 | kfree(fcall); | 661 | kfree(fcall); |
759 | return retval; | 662 | return retval; |
760 | } | 663 | } |
761 | 664 | ||
762 | /** | 665 | /** |
763 | * v9fs_vfs_getattr - retreive file metadata | 666 | * v9fs_vfs_getattr - retrieve file metadata |
764 | * @mnt - mount information | 667 | * @mnt - mount information |
765 | * @dentry - file to get attributes on | 668 | * @dentry - file to get attributes on |
766 | * @stat - metadata structure to populate | 669 | * @stat - metadata structure to populate |
@@ -788,7 +691,7 @@ v9fs_vfs_getattr(struct vfsmount *mnt, struct dentry *dentry, | |||
788 | if (err < 0) | 691 | if (err < 0) |
789 | dprintk(DEBUG_ERROR, "stat error\n"); | 692 | dprintk(DEBUG_ERROR, "stat error\n"); |
790 | else { | 693 | else { |
791 | v9fs_mistat2inode(fcall->params.rstat.stat, dentry->d_inode, | 694 | v9fs_stat2inode(&fcall->params.rstat.stat, dentry->d_inode, |
792 | dentry->d_inode->i_sb); | 695 | dentry->d_inode->i_sb); |
793 | generic_fillattr(dentry->d_inode, stat); | 696 | generic_fillattr(dentry->d_inode, stat); |
794 | } | 697 | } |
@@ -809,57 +712,44 @@ static int v9fs_vfs_setattr(struct dentry *dentry, struct iattr *iattr) | |||
809 | struct v9fs_session_info *v9ses = v9fs_inode2v9ses(dentry->d_inode); | 712 | struct v9fs_session_info *v9ses = v9fs_inode2v9ses(dentry->d_inode); |
810 | struct v9fs_fid *fid = v9fs_fid_lookup(dentry); | 713 | struct v9fs_fid *fid = v9fs_fid_lookup(dentry); |
811 | struct v9fs_fcall *fcall = NULL; | 714 | struct v9fs_fcall *fcall = NULL; |
812 | struct v9fs_stat *mistat = kmalloc(v9ses->maxdata, GFP_KERNEL); | 715 | struct v9fs_wstat wstat; |
813 | int res = -EPERM; | 716 | int res = -EPERM; |
814 | 717 | ||
815 | dprintk(DEBUG_VFS, "\n"); | 718 | dprintk(DEBUG_VFS, "\n"); |
816 | 719 | ||
817 | if (!mistat) | ||
818 | return -ENOMEM; | ||
819 | |||
820 | if (!fid) { | 720 | if (!fid) { |
821 | dprintk(DEBUG_ERROR, | 721 | dprintk(DEBUG_ERROR, |
822 | "Couldn't find fid associated with dentry\n"); | 722 | "Couldn't find fid associated with dentry\n"); |
823 | return -EBADF; | 723 | return -EBADF; |
824 | } | 724 | } |
825 | 725 | ||
826 | v9fs_blank_mistat(v9ses, mistat); | 726 | v9fs_blank_wstat(&wstat); |
827 | if (iattr->ia_valid & ATTR_MODE) | 727 | if (iattr->ia_valid & ATTR_MODE) |
828 | mistat->mode = unixmode2p9mode(v9ses, iattr->ia_mode); | 728 | wstat.mode = unixmode2p9mode(v9ses, iattr->ia_mode); |
829 | 729 | ||
830 | if (iattr->ia_valid & ATTR_MTIME) | 730 | if (iattr->ia_valid & ATTR_MTIME) |
831 | mistat->mtime = iattr->ia_mtime.tv_sec; | 731 | wstat.mtime = iattr->ia_mtime.tv_sec; |
832 | 732 | ||
833 | if (iattr->ia_valid & ATTR_ATIME) | 733 | if (iattr->ia_valid & ATTR_ATIME) |
834 | mistat->atime = iattr->ia_atime.tv_sec; | 734 | wstat.atime = iattr->ia_atime.tv_sec; |
835 | 735 | ||
836 | if (iattr->ia_valid & ATTR_SIZE) | 736 | if (iattr->ia_valid & ATTR_SIZE) |
837 | mistat->length = iattr->ia_size; | 737 | wstat.length = iattr->ia_size; |
838 | 738 | ||
839 | if (v9ses->extended) { | 739 | if (v9ses->extended) { |
840 | char *ptr = mistat->data+1; | 740 | if (iattr->ia_valid & ATTR_UID) |
841 | 741 | wstat.n_uid = iattr->ia_uid; | |
842 | if (iattr->ia_valid & ATTR_UID) { | ||
843 | mistat->uid = ptr; | ||
844 | ptr += 1+sprintf(ptr, "%08x", iattr->ia_uid); | ||
845 | mistat->n_uid = iattr->ia_uid; | ||
846 | } | ||
847 | 742 | ||
848 | if (iattr->ia_valid & ATTR_GID) { | 743 | if (iattr->ia_valid & ATTR_GID) |
849 | mistat->gid = ptr; | 744 | wstat.n_gid = iattr->ia_gid; |
850 | ptr += 1+sprintf(ptr, "%08x", iattr->ia_gid); | ||
851 | mistat->n_gid = iattr->ia_gid; | ||
852 | } | ||
853 | } | 745 | } |
854 | 746 | ||
855 | res = v9fs_t_wstat(v9ses, fid->fid, mistat, &fcall); | 747 | res = v9fs_t_wstat(v9ses, fid->fid, &wstat, &fcall); |
856 | 748 | ||
857 | if (res < 0) | 749 | if (res < 0) |
858 | dprintk(DEBUG_ERROR, "wstat error: %s\n", FCALL_ERROR(fcall)); | 750 | PRINT_FCALL_ERROR("wstat error", fcall); |
859 | 751 | ||
860 | kfree(mistat); | ||
861 | kfree(fcall); | 752 | kfree(fcall); |
862 | |||
863 | if (res >= 0) | 753 | if (res >= 0) |
864 | res = inode_setattr(dentry->d_inode, iattr); | 754 | res = inode_setattr(dentry->d_inode, iattr); |
865 | 755 | ||
@@ -867,51 +757,47 @@ static int v9fs_vfs_setattr(struct dentry *dentry, struct iattr *iattr) | |||
867 | } | 757 | } |
868 | 758 | ||
869 | /** | 759 | /** |
870 | * v9fs_mistat2inode - populate an inode structure with mistat info | 760 | * v9fs_stat2inode - populate an inode structure with mistat info |
871 | * @mistat: Plan 9 metadata (mistat) structure | 761 | * @stat: Plan 9 metadata (mistat) structure |
872 | * @inode: inode to populate | 762 | * @inode: inode to populate |
873 | * @sb: superblock of filesystem | 763 | * @sb: superblock of filesystem |
874 | * | 764 | * |
875 | */ | 765 | */ |
876 | 766 | ||
877 | void | 767 | void |
878 | v9fs_mistat2inode(struct v9fs_stat *mistat, struct inode *inode, | 768 | v9fs_stat2inode(struct v9fs_stat *stat, struct inode *inode, |
879 | struct super_block *sb) | 769 | struct super_block *sb) |
880 | { | 770 | { |
771 | int n; | ||
772 | char ext[32]; | ||
881 | struct v9fs_session_info *v9ses = sb->s_fs_info; | 773 | struct v9fs_session_info *v9ses = sb->s_fs_info; |
882 | 774 | ||
883 | inode->i_nlink = 1; | 775 | inode->i_nlink = 1; |
884 | 776 | ||
885 | inode->i_atime.tv_sec = mistat->atime; | 777 | inode->i_atime.tv_sec = stat->atime; |
886 | inode->i_mtime.tv_sec = mistat->mtime; | 778 | inode->i_mtime.tv_sec = stat->mtime; |
887 | inode->i_ctime.tv_sec = mistat->mtime; | 779 | inode->i_ctime.tv_sec = stat->mtime; |
888 | 780 | ||
889 | inode->i_uid = -1; | 781 | inode->i_uid = v9ses->uid; |
890 | inode->i_gid = -1; | 782 | inode->i_gid = v9ses->gid; |
891 | 783 | ||
892 | if (v9ses->extended) { | 784 | if (v9ses->extended) { |
893 | /* TODO: string to uid mapping via user-space daemon */ | 785 | inode->i_uid = stat->n_uid; |
894 | inode->i_uid = mistat->n_uid; | 786 | inode->i_gid = stat->n_gid; |
895 | inode->i_gid = mistat->n_gid; | ||
896 | |||
897 | if (mistat->n_uid == -1) | ||
898 | sscanf(mistat->uid, "%x", &inode->i_uid); | ||
899 | |||
900 | if (mistat->n_gid == -1) | ||
901 | sscanf(mistat->gid, "%x", &inode->i_gid); | ||
902 | } | 787 | } |
903 | 788 | ||
904 | if (inode->i_uid == -1) | 789 | inode->i_mode = p9mode2unixmode(v9ses, stat->mode); |
905 | inode->i_uid = v9ses->uid; | ||
906 | if (inode->i_gid == -1) | ||
907 | inode->i_gid = v9ses->gid; | ||
908 | |||
909 | inode->i_mode = p9mode2unixmode(v9ses, mistat->mode); | ||
910 | if ((S_ISBLK(inode->i_mode)) || (S_ISCHR(inode->i_mode))) { | 790 | if ((S_ISBLK(inode->i_mode)) || (S_ISCHR(inode->i_mode))) { |
911 | char type = 0; | 791 | char type = 0; |
912 | int major = -1; | 792 | int major = -1; |
913 | int minor = -1; | 793 | int minor = -1; |
914 | sscanf(mistat->extension, "%c %u %u", &type, &major, &minor); | 794 | |
795 | n = stat->extension.len; | ||
796 | if (n > sizeof(ext)-1) | ||
797 | n = sizeof(ext)-1; | ||
798 | memmove(ext, stat->extension.str, n); | ||
799 | ext[n] = 0; | ||
800 | sscanf(ext, "%c %u %u", &type, &major, &minor); | ||
915 | switch (type) { | 801 | switch (type) { |
916 | case 'c': | 802 | case 'c': |
917 | inode->i_mode &= ~S_IFBLK; | 803 | inode->i_mode &= ~S_IFBLK; |
@@ -920,14 +806,14 @@ v9fs_mistat2inode(struct v9fs_stat *mistat, struct inode *inode, | |||
920 | case 'b': | 806 | case 'b': |
921 | break; | 807 | break; |
922 | default: | 808 | default: |
923 | dprintk(DEBUG_ERROR, "Unknown special type %c (%s)\n", | 809 | dprintk(DEBUG_ERROR, "Unknown special type %c (%.*s)\n", |
924 | type, mistat->extension); | 810 | type, stat->extension.len, stat->extension.str); |
925 | }; | 811 | }; |
926 | inode->i_rdev = MKDEV(major, minor); | 812 | inode->i_rdev = MKDEV(major, minor); |
927 | } else | 813 | } else |
928 | inode->i_rdev = 0; | 814 | inode->i_rdev = 0; |
929 | 815 | ||
930 | inode->i_size = mistat->length; | 816 | inode->i_size = stat->length; |
931 | 817 | ||
932 | inode->i_blksize = sb->s_blocksize; | 818 | inode->i_blksize = sb->s_blocksize; |
933 | inode->i_blocks = | 819 | inode->i_blocks = |
@@ -955,71 +841,6 @@ ino_t v9fs_qid2ino(struct v9fs_qid *qid) | |||
955 | } | 841 | } |
956 | 842 | ||
957 | /** | 843 | /** |
958 | * v9fs_vfs_symlink - helper function to create symlinks | ||
959 | * @dir: directory inode containing symlink | ||
960 | * @dentry: dentry for symlink | ||
961 | * @symname: symlink data | ||
962 | * | ||
963 | * See 9P2000.u RFC for more information | ||
964 | * | ||
965 | */ | ||
966 | |||
967 | static int | ||
968 | v9fs_vfs_symlink(struct inode *dir, struct dentry *dentry, const char *symname) | ||
969 | { | ||
970 | int retval = -EPERM; | ||
971 | struct v9fs_fid *newfid; | ||
972 | struct v9fs_session_info *v9ses = v9fs_inode2v9ses(dir); | ||
973 | struct v9fs_fcall *fcall = NULL; | ||
974 | struct v9fs_stat *mistat = kmalloc(v9ses->maxdata, GFP_KERNEL); | ||
975 | |||
976 | dprintk(DEBUG_VFS, " %lu,%s,%s\n", dir->i_ino, dentry->d_name.name, | ||
977 | symname); | ||
978 | |||
979 | if (!mistat) | ||
980 | return -ENOMEM; | ||
981 | |||
982 | if (!v9ses->extended) { | ||
983 | dprintk(DEBUG_ERROR, "not extended\n"); | ||
984 | goto FreeFcall; | ||
985 | } | ||
986 | |||
987 | /* issue a create */ | ||
988 | retval = v9fs_create(dir, dentry, S_IFLNK, 0); | ||
989 | if (retval != 0) | ||
990 | goto FreeFcall; | ||
991 | |||
992 | newfid = v9fs_fid_lookup(dentry); | ||
993 | |||
994 | /* issue a twstat */ | ||
995 | v9fs_blank_mistat(v9ses, mistat); | ||
996 | strcpy(mistat->data + 1, symname); | ||
997 | mistat->extension = mistat->data + 1; | ||
998 | retval = v9fs_t_wstat(v9ses, newfid->fid, mistat, &fcall); | ||
999 | if (retval < 0) { | ||
1000 | dprintk(DEBUG_ERROR, "v9fs_t_wstat error: %s\n", | ||
1001 | FCALL_ERROR(fcall)); | ||
1002 | goto FreeFcall; | ||
1003 | } | ||
1004 | |||
1005 | kfree(fcall); | ||
1006 | |||
1007 | if (v9fs_t_clunk(v9ses, newfid->fid, &fcall)) { | ||
1008 | dprintk(DEBUG_ERROR, "clunk for symlink failed: %s\n", | ||
1009 | FCALL_ERROR(fcall)); | ||
1010 | goto FreeFcall; | ||
1011 | } | ||
1012 | |||
1013 | d_drop(dentry); /* FID - will this also clunk? */ | ||
1014 | |||
1015 | FreeFcall: | ||
1016 | kfree(mistat); | ||
1017 | kfree(fcall); | ||
1018 | |||
1019 | return retval; | ||
1020 | } | ||
1021 | |||
1022 | /** | ||
1023 | * v9fs_readlink - read a symlink's location (internal version) | 844 | * v9fs_readlink - read a symlink's location (internal version) |
1024 | * @dentry: dentry for symlink | 845 | * @dentry: dentry for symlink |
1025 | * @buffer: buffer to load symlink location into | 846 | * @buffer: buffer to load symlink location into |
@@ -1058,16 +879,17 @@ static int v9fs_readlink(struct dentry *dentry, char *buffer, int buflen) | |||
1058 | if (!fcall) | 879 | if (!fcall) |
1059 | return -EIO; | 880 | return -EIO; |
1060 | 881 | ||
1061 | if (!(fcall->params.rstat.stat->mode & V9FS_DMSYMLINK)) { | 882 | if (!(fcall->params.rstat.stat.mode & V9FS_DMSYMLINK)) { |
1062 | retval = -EINVAL; | 883 | retval = -EINVAL; |
1063 | goto FreeFcall; | 884 | goto FreeFcall; |
1064 | } | 885 | } |
1065 | 886 | ||
1066 | /* copy extension buffer into buffer */ | 887 | /* copy extension buffer into buffer */ |
1067 | if (strlen(fcall->params.rstat.stat->extension) < buflen) | 888 | if (fcall->params.rstat.stat.extension.len < buflen) |
1068 | buflen = strlen(fcall->params.rstat.stat->extension); | 889 | buflen = fcall->params.rstat.stat.extension.len; |
1069 | 890 | ||
1070 | memcpy(buffer, fcall->params.rstat.stat->extension, buflen + 1); | 891 | memcpy(buffer, fcall->params.rstat.stat.extension.str, buflen - 1); |
892 | buffer[buflen-1] = 0; | ||
1071 | 893 | ||
1072 | retval = buflen; | 894 | retval = buflen; |
1073 | 895 | ||
@@ -1157,6 +979,77 @@ static void v9fs_vfs_put_link(struct dentry *dentry, struct nameidata *nd, void | |||
1157 | __putname(s); | 979 | __putname(s); |
1158 | } | 980 | } |
1159 | 981 | ||
982 | static int v9fs_vfs_mkspecial(struct inode *dir, struct dentry *dentry, | ||
983 | int mode, const char *extension) | ||
984 | { | ||
985 | int err, retval; | ||
986 | struct v9fs_session_info *v9ses; | ||
987 | struct v9fs_fcall *fcall; | ||
988 | struct v9fs_fid *fid; | ||
989 | struct v9fs_wstat wstat; | ||
990 | |||
991 | v9ses = v9fs_inode2v9ses(dir); | ||
992 | retval = -EPERM; | ||
993 | fcall = NULL; | ||
994 | |||
995 | if (!v9ses->extended) { | ||
996 | dprintk(DEBUG_ERROR, "not extended\n"); | ||
997 | goto free_mem; | ||
998 | } | ||
999 | |||
1000 | /* issue a create */ | ||
1001 | retval = v9fs_create(dir, dentry, mode, 0); | ||
1002 | if (retval != 0) | ||
1003 | goto free_mem; | ||
1004 | |||
1005 | fid = v9fs_fid_get_created(dentry); | ||
1006 | if (!fid) { | ||
1007 | dprintk(DEBUG_ERROR, "couldn't resolve fid from dentry\n"); | ||
1008 | goto free_mem; | ||
1009 | } | ||
1010 | |||
1011 | /* issue a Twstat */ | ||
1012 | v9fs_blank_wstat(&wstat); | ||
1013 | wstat.muid = v9ses->name; | ||
1014 | wstat.extension = (char *) extension; | ||
1015 | retval = v9fs_t_wstat(v9ses, fid->fid, &wstat, &fcall); | ||
1016 | if (retval < 0) { | ||
1017 | PRINT_FCALL_ERROR("wstat error", fcall); | ||
1018 | goto free_mem; | ||
1019 | } | ||
1020 | |||
1021 | err = v9fs_t_clunk(v9ses, fid->fid); | ||
1022 | if (err < 0) { | ||
1023 | dprintk(DEBUG_ERROR, "clunk failed: %d\n", err); | ||
1024 | goto free_mem; | ||
1025 | } | ||
1026 | |||
1027 | d_drop(dentry); /* FID - will this also clunk? */ | ||
1028 | |||
1029 | free_mem: | ||
1030 | kfree(fcall); | ||
1031 | return retval; | ||
1032 | } | ||
1033 | |||
1034 | /** | ||
1035 | * v9fs_vfs_symlink - helper function to create symlinks | ||
1036 | * @dir: directory inode containing symlink | ||
1037 | * @dentry: dentry for symlink | ||
1038 | * @symname: symlink data | ||
1039 | * | ||
1040 | * See 9P2000.u RFC for more information | ||
1041 | * | ||
1042 | */ | ||
1043 | |||
1044 | static int | ||
1045 | v9fs_vfs_symlink(struct inode *dir, struct dentry *dentry, const char *symname) | ||
1046 | { | ||
1047 | dprintk(DEBUG_VFS, " %lu,%s,%s\n", dir->i_ino, dentry->d_name.name, | ||
1048 | symname); | ||
1049 | |||
1050 | return v9fs_vfs_mkspecial(dir, dentry, S_IFLNK, symname); | ||
1051 | } | ||
1052 | |||
1160 | /** | 1053 | /** |
1161 | * v9fs_vfs_link - create a hardlink | 1054 | * v9fs_vfs_link - create a hardlink |
1162 | * @old_dentry: dentry for file to link to | 1055 | * @old_dentry: dentry for file to link to |
@@ -1173,64 +1066,24 @@ static int | |||
1173 | v9fs_vfs_link(struct dentry *old_dentry, struct inode *dir, | 1066 | v9fs_vfs_link(struct dentry *old_dentry, struct inode *dir, |
1174 | struct dentry *dentry) | 1067 | struct dentry *dentry) |
1175 | { | 1068 | { |
1176 | int retval = -EPERM; | 1069 | int retval; |
1177 | struct v9fs_session_info *v9ses = v9fs_inode2v9ses(dir); | 1070 | struct v9fs_fid *oldfid; |
1178 | struct v9fs_fcall *fcall = NULL; | 1071 | char *name; |
1179 | struct v9fs_stat *mistat = kmalloc(v9ses->maxdata, GFP_KERNEL); | ||
1180 | struct v9fs_fid *oldfid = v9fs_fid_lookup(old_dentry); | ||
1181 | struct v9fs_fid *newfid = NULL; | ||
1182 | char *symname = __getname(); | ||
1183 | 1072 | ||
1184 | dprintk(DEBUG_VFS, " %lu,%s,%s\n", dir->i_ino, dentry->d_name.name, | 1073 | dprintk(DEBUG_VFS, " %lu,%s,%s\n", dir->i_ino, dentry->d_name.name, |
1185 | old_dentry->d_name.name); | 1074 | old_dentry->d_name.name); |
1186 | 1075 | ||
1187 | if (!v9ses->extended) { | 1076 | oldfid = v9fs_fid_lookup(old_dentry); |
1188 | dprintk(DEBUG_ERROR, "not extended\n"); | 1077 | if (!oldfid) { |
1189 | goto FreeMem; | 1078 | dprintk(DEBUG_ERROR, "can't find oldfid\n"); |
1190 | } | 1079 | return -EPERM; |
1191 | |||
1192 | /* get fid of old_dentry */ | ||
1193 | sprintf(symname, "hardlink(%d)\n", oldfid->fid); | ||
1194 | |||
1195 | /* issue a create */ | ||
1196 | retval = v9fs_create(dir, dentry, V9FS_DMLINK, 0); | ||
1197 | if (retval != 0) | ||
1198 | goto FreeMem; | ||
1199 | |||
1200 | newfid = v9fs_fid_lookup(dentry); | ||
1201 | if (!newfid) { | ||
1202 | dprintk(DEBUG_ERROR, "couldn't resolve fid from dentry\n"); | ||
1203 | goto FreeMem; | ||
1204 | } | ||
1205 | |||
1206 | /* issue a twstat */ | ||
1207 | v9fs_blank_mistat(v9ses, mistat); | ||
1208 | strcpy(mistat->data + 1, symname); | ||
1209 | mistat->extension = mistat->data + 1; | ||
1210 | retval = v9fs_t_wstat(v9ses, newfid->fid, mistat, &fcall); | ||
1211 | if (retval < 0) { | ||
1212 | dprintk(DEBUG_ERROR, "v9fs_t_wstat error: %s\n", | ||
1213 | FCALL_ERROR(fcall)); | ||
1214 | goto FreeMem; | ||
1215 | } | ||
1216 | |||
1217 | kfree(fcall); | ||
1218 | |||
1219 | if (v9fs_t_clunk(v9ses, newfid->fid, &fcall)) { | ||
1220 | dprintk(DEBUG_ERROR, "clunk for symlink failed: %s\n", | ||
1221 | FCALL_ERROR(fcall)); | ||
1222 | goto FreeMem; | ||
1223 | } | 1080 | } |
1224 | 1081 | ||
1225 | d_drop(dentry); /* FID - will this also clunk? */ | 1082 | name = __getname(); |
1226 | 1083 | sprintf(name, "hardlink(%d)\n", oldfid->fid); | |
1227 | kfree(fcall); | 1084 | retval = v9fs_vfs_mkspecial(dir, dentry, V9FS_DMLINK, name); |
1228 | fcall = NULL; | 1085 | __putname(name); |
1229 | 1086 | ||
1230 | FreeMem: | ||
1231 | kfree(mistat); | ||
1232 | kfree(fcall); | ||
1233 | __putname(symname); | ||
1234 | return retval; | 1087 | return retval; |
1235 | } | 1088 | } |
1236 | 1089 | ||
@@ -1246,82 +1099,30 @@ v9fs_vfs_link(struct dentry *old_dentry, struct inode *dir, | |||
1246 | static int | 1099 | static int |
1247 | v9fs_vfs_mknod(struct inode *dir, struct dentry *dentry, int mode, dev_t rdev) | 1100 | v9fs_vfs_mknod(struct inode *dir, struct dentry *dentry, int mode, dev_t rdev) |
1248 | { | 1101 | { |
1249 | int retval = -EPERM; | 1102 | int retval; |
1250 | struct v9fs_fid *newfid; | 1103 | char *name; |
1251 | struct v9fs_session_info *v9ses = v9fs_inode2v9ses(dir); | ||
1252 | struct v9fs_fcall *fcall = NULL; | ||
1253 | struct v9fs_stat *mistat = kmalloc(v9ses->maxdata, GFP_KERNEL); | ||
1254 | char *symname = __getname(); | ||
1255 | 1104 | ||
1256 | dprintk(DEBUG_VFS, " %lu,%s mode: %x MAJOR: %u MINOR: %u\n", dir->i_ino, | 1105 | dprintk(DEBUG_VFS, " %lu,%s mode: %x MAJOR: %u MINOR: %u\n", dir->i_ino, |
1257 | dentry->d_name.name, mode, MAJOR(rdev), MINOR(rdev)); | 1106 | dentry->d_name.name, mode, MAJOR(rdev), MINOR(rdev)); |
1258 | 1107 | ||
1259 | if (!mistat) | 1108 | if (!new_valid_dev(rdev)) |
1260 | return -ENOMEM; | 1109 | return -EINVAL; |
1261 | |||
1262 | if (!new_valid_dev(rdev)) { | ||
1263 | retval = -EINVAL; | ||
1264 | goto FreeMem; | ||
1265 | } | ||
1266 | |||
1267 | if (!v9ses->extended) { | ||
1268 | dprintk(DEBUG_ERROR, "not extended\n"); | ||
1269 | goto FreeMem; | ||
1270 | } | ||
1271 | |||
1272 | /* issue a create */ | ||
1273 | retval = v9fs_create(dir, dentry, mode, 0); | ||
1274 | |||
1275 | if (retval != 0) | ||
1276 | goto FreeMem; | ||
1277 | |||
1278 | newfid = v9fs_fid_lookup(dentry); | ||
1279 | if (!newfid) { | ||
1280 | dprintk(DEBUG_ERROR, "coudn't resove fid from dentry\n"); | ||
1281 | retval = -EINVAL; | ||
1282 | goto FreeMem; | ||
1283 | } | ||
1284 | 1110 | ||
1111 | name = __getname(); | ||
1285 | /* build extension */ | 1112 | /* build extension */ |
1286 | if (S_ISBLK(mode)) | 1113 | if (S_ISBLK(mode)) |
1287 | sprintf(symname, "b %u %u", MAJOR(rdev), MINOR(rdev)); | 1114 | sprintf(name, "b %u %u", MAJOR(rdev), MINOR(rdev)); |
1288 | else if (S_ISCHR(mode)) | 1115 | else if (S_ISCHR(mode)) |
1289 | sprintf(symname, "c %u %u", MAJOR(rdev), MINOR(rdev)); | 1116 | sprintf(name, "c %u %u", MAJOR(rdev), MINOR(rdev)); |
1290 | else if (S_ISFIFO(mode)) | 1117 | else if (S_ISFIFO(mode)) |
1291 | ; /* DO NOTHING */ | 1118 | *name = 0; |
1292 | else { | 1119 | else { |
1293 | retval = -EINVAL; | 1120 | __putname(name); |
1294 | goto FreeMem; | 1121 | return -EINVAL; |
1295 | } | ||
1296 | |||
1297 | if (!S_ISFIFO(mode)) { | ||
1298 | /* issue a twstat */ | ||
1299 | v9fs_blank_mistat(v9ses, mistat); | ||
1300 | strcpy(mistat->data + 1, symname); | ||
1301 | mistat->extension = mistat->data + 1; | ||
1302 | retval = v9fs_t_wstat(v9ses, newfid->fid, mistat, &fcall); | ||
1303 | if (retval < 0) { | ||
1304 | dprintk(DEBUG_ERROR, "v9fs_t_wstat error: %s\n", | ||
1305 | FCALL_ERROR(fcall)); | ||
1306 | goto FreeMem; | ||
1307 | } | ||
1308 | } | 1122 | } |
1309 | 1123 | ||
1310 | /* need to update dcache so we show up */ | 1124 | retval = v9fs_vfs_mkspecial(dir, dentry, mode, name); |
1311 | kfree(fcall); | 1125 | __putname(name); |
1312 | |||
1313 | if (v9fs_t_clunk(v9ses, newfid->fid, &fcall)) { | ||
1314 | dprintk(DEBUG_ERROR, "clunk for symlink failed: %s\n", | ||
1315 | FCALL_ERROR(fcall)); | ||
1316 | goto FreeMem; | ||
1317 | } | ||
1318 | |||
1319 | d_drop(dentry); /* FID - will this also clunk? */ | ||
1320 | |||
1321 | FreeMem: | ||
1322 | kfree(mistat); | ||
1323 | kfree(fcall); | ||
1324 | __putname(symname); | ||
1325 | 1126 | ||
1326 | return retval; | 1127 | return retval; |
1327 | } | 1128 | } |
diff --git a/fs/9p/vfs_super.c b/fs/9p/vfs_super.c index 82c5b0084079..2c4fa75be025 100644 --- a/fs/9p/vfs_super.c +++ b/fs/9p/vfs_super.c | |||
@@ -44,7 +44,6 @@ | |||
44 | #include "v9fs.h" | 44 | #include "v9fs.h" |
45 | #include "9p.h" | 45 | #include "9p.h" |
46 | #include "v9fs_vfs.h" | 46 | #include "v9fs_vfs.h" |
47 | #include "conv.h" | ||
48 | #include "fid.h" | 47 | #include "fid.h" |
49 | 48 | ||
50 | static void v9fs_clear_inode(struct inode *); | 49 | static void v9fs_clear_inode(struct inode *); |
@@ -92,7 +91,7 @@ v9fs_fill_super(struct super_block *sb, struct v9fs_session_info *v9ses, | |||
92 | sb->s_op = &v9fs_super_ops; | 91 | sb->s_op = &v9fs_super_ops; |
93 | 92 | ||
94 | sb->s_flags = flags | MS_ACTIVE | MS_SYNCHRONOUS | MS_DIRSYNC | | 93 | sb->s_flags = flags | MS_ACTIVE | MS_SYNCHRONOUS | MS_DIRSYNC | |
95 | MS_NODIRATIME | MS_NOATIME; | 94 | MS_NOATIME; |
96 | } | 95 | } |
97 | 96 | ||
98 | /** | 97 | /** |
@@ -123,12 +122,13 @@ static struct super_block *v9fs_get_sb(struct file_system_type | |||
123 | 122 | ||
124 | dprintk(DEBUG_VFS, " \n"); | 123 | dprintk(DEBUG_VFS, " \n"); |
125 | 124 | ||
126 | v9ses = kcalloc(1, sizeof(struct v9fs_session_info), GFP_KERNEL); | 125 | v9ses = kzalloc(sizeof(struct v9fs_session_info), GFP_KERNEL); |
127 | if (!v9ses) | 126 | if (!v9ses) |
128 | return ERR_PTR(-ENOMEM); | 127 | return ERR_PTR(-ENOMEM); |
129 | 128 | ||
130 | if ((newfid = v9fs_session_init(v9ses, dev_name, data)) < 0) { | 129 | if ((newfid = v9fs_session_init(v9ses, dev_name, data)) < 0) { |
131 | dprintk(DEBUG_ERROR, "problem initiating session\n"); | 130 | dprintk(DEBUG_ERROR, "problem initiating session\n"); |
131 | kfree(v9ses); | ||
132 | return ERR_PTR(newfid); | 132 | return ERR_PTR(newfid); |
133 | } | 133 | } |
134 | 134 | ||
@@ -157,7 +157,7 @@ static struct super_block *v9fs_get_sb(struct file_system_type | |||
157 | stat_result = v9fs_t_stat(v9ses, newfid, &fcall); | 157 | stat_result = v9fs_t_stat(v9ses, newfid, &fcall); |
158 | if (stat_result < 0) { | 158 | if (stat_result < 0) { |
159 | dprintk(DEBUG_ERROR, "stat error\n"); | 159 | dprintk(DEBUG_ERROR, "stat error\n"); |
160 | v9fs_t_clunk(v9ses, newfid, NULL); | 160 | v9fs_t_clunk(v9ses, newfid); |
161 | v9fs_put_idpool(newfid, &v9ses->fidpool); | 161 | v9fs_put_idpool(newfid, &v9ses->fidpool); |
162 | } else { | 162 | } else { |
163 | /* Setup the Root Inode */ | 163 | /* Setup the Root Inode */ |
@@ -167,10 +167,10 @@ static struct super_block *v9fs_get_sb(struct file_system_type | |||
167 | goto put_back_sb; | 167 | goto put_back_sb; |
168 | } | 168 | } |
169 | 169 | ||
170 | root_fid->qid = fcall->params.rstat.stat->qid; | 170 | root_fid->qid = fcall->params.rstat.stat.qid; |
171 | root->d_inode->i_ino = | 171 | root->d_inode->i_ino = |
172 | v9fs_qid2ino(&fcall->params.rstat.stat->qid); | 172 | v9fs_qid2ino(&fcall->params.rstat.stat.qid); |
173 | v9fs_mistat2inode(fcall->params.rstat.stat, root->d_inode, sb); | 173 | v9fs_stat2inode(&fcall->params.rstat.stat, root->d_inode, sb); |
174 | } | 174 | } |
175 | 175 | ||
176 | kfree(fcall); | 176 | kfree(fcall); |