Changeset 8489

Timestamp:

09/02/10 11:54:33 (3 years ago)

Author:

bligon

Message:

Modified pjmp code to use a task-id of GETATTR instead of 0, because 0
causes the state machine processor to free the frame before returning
to the calling machine.

Files:

• branches/Orange-Branch/src/server/list-attr.sm (modified) (5 diffs)

branches/Orange-Branch/src/server/list-attr.sm

@@ -25 +25 @@
 #include "pvfs2-internal.h"
 
+#define GETATTR 100
+
+
 %%
 
@@ -48 +51 @@
         pjmp listattr_setup_getattr
         {
-            success => pvfs2_pjmp_get_attr_work_sm;
+            GETATTR => pvfs2_pjmp_get_attr_work_sm;
         }
         success => interpret_getattrs;
@@ -172 +175 @@
         getattr_op->u.getattr.attrmask = s_op->req->u.listattr.attrmask;
 
-        ret = PINT_sm_push_frame(smcb, 0, getattr_op);
+        ret = PINT_sm_push_frame(smcb, GETATTR, getattr_op);
         if(ret < 0)
         {
@@ -207 +210 @@
     job_status_s *js_p)
 {    
-    struct PINT_server_op *getattr_op;
+    struct PINT_server_op *getattr_op = NULL;
     /* note: this gives us a pointer to the base frame (list_attr), 
      * _not_ the getattr frames that were previously pushed.
@@ -229 +232 @@
         getattr_op = PINT_sm_pop_frame(smcb, &task_id, &tmp_err, 
             &remaining);
+        gossip_debug(GOSSIP_SERVER_DEBUG, "getattr_op is %s.\n",(getattr_op ? "okay" : "NULL"));
         gossip_debug(GOSSIP_SERVER_DEBUG, "listattr: nested sm returned error code: %d\n", tmp_err);
+        gossip_debug(GOSSIP_SERVER_DEBUG, "s_op is %p \t getattr_op is %p",s_op,getattr_op );
         /* match it up with the correct array entry */
         for(j=0; j<s_op->req->u.listattr.nhandles; j++)