[net/9p] Handle Zero Copy TREAD/RERROR case in !dotl case.
authorVenkateswararao Jujjuri (JV) <jvrao@linux.vnet.ibm.com>
Wed, 2 Feb 2011 04:04:59 +0000 (20:04 -0800)
committerEric Van Hensbergen <ericvh@gmail.com>
Tue, 15 Mar 2011 14:57:36 +0000 (09:57 -0500)
This takes care of copying out error buffers from user buffer
payloads when we are using zero copy.  This happens because the
only payload buffer the server has to respond to the request is
the user buffer given for the zero copy read.

Because we only use zerocopy when the amount of data to transfer
is greater than a certain size (currently 4K) and error strings are
limited to ERRMAX (currently 128) we don't need to worry about there
being sufficient space for the error to fit in the payload.

Signed-off-by: Venkateswararao Jujjuri <jvrao@linux.vnet.ibm.com>
Signed-off-by: Eric Van Hensbergen <ericvh@gmail.com>
net/9p/client.c

index 6e07ef494ff251f22a46c3aae023d3bdf729962a..251abb1699c4169a834bcb7951d862fba9cb105f 100644 (file)
@@ -443,6 +443,7 @@ static int p9_check_errors(struct p9_client *c, struct p9_req_t *req)
 {
        int8_t type;
        int err;
+       int ecode;
 
        err = p9_parse_header(req->rc, NULL, &type, NULL, 0);
        if (err) {
@@ -450,36 +451,53 @@ static int p9_check_errors(struct p9_client *c, struct p9_req_t *req)
                return err;
        }
 
-       if (type == P9_RERROR || type == P9_RLERROR) {
-               int ecode;
-
-               if (!p9_is_proto_dotl(c)) {
-                       char *ename;
+       if (type != P9_RERROR && type != P9_RLERROR)
+               return 0;
 
-                       err = p9pdu_readf(req->rc, c->proto_version, "s?d",
-                                                               &ename, &ecode);
-                       if (err)
-                               goto out_err;
+       if (!p9_is_proto_dotl(c)) {
+               char *ename;
+
+               if (req->tc->pbuf_size) {
+                       /* Handle user buffers */
+                       size_t len = req->rc->size - req->rc->offset;
+                       if (req->tc->pubuf) {
+                               /* User Buffer */
+                               err = copy_from_user(
+                                       &req->rc->sdata[req->rc->offset],
+                                       req->tc->pubuf, len);
+                               if (err) {
+                                       err = -EFAULT;
+                                       goto out_err;
+                               }
+                       } else {
+                               /* Kernel Buffer */
+                               memmove(&req->rc->sdata[req->rc->offset],
+                                               req->tc->pkbuf, len);
+                       }
+               }
+               err = p9pdu_readf(req->rc, c->proto_version, "s?d",
+                               &ename, &ecode);
+               if (err)
+                       goto out_err;
 
-                       if (p9_is_proto_dotu(c))
-                               err = -ecode;
+               if (p9_is_proto_dotu(c))
+                       err = -ecode;
 
-                       if (!err || !IS_ERR_VALUE(err)) {
-                               err = p9_errstr2errno(ename, strlen(ename));
+               if (!err || !IS_ERR_VALUE(err)) {
+                       err = p9_errstr2errno(ename, strlen(ename));
 
-                               P9_DPRINTK(P9_DEBUG_9P, "<<< RERROR (%d) %s\n", -ecode, ename);
+                       P9_DPRINTK(P9_DEBUG_9P, "<<< RERROR (%d) %s\n", -ecode,
+                                       ename);
 
-                               kfree(ename);
-                       }
-               } else {
-                       err = p9pdu_readf(req->rc, c->proto_version, "d", &ecode);
-                       err = -ecode;
-
-                       P9_DPRINTK(P9_DEBUG_9P, "<<< RLERROR (%d)\n", -ecode);
+                       kfree(ename);
                }
+       } else {
+               err = p9pdu_readf(req->rc, c->proto_version, "d", &ecode);
+               err = -ecode;
+
+               P9_DPRINTK(P9_DEBUG_9P, "<<< RLERROR (%d)\n", -ecode);
+       }
 
-       } else
-               err = 0;
 
        return err;