Orangefs: clean up orangefs_kernel_op_s comments.
authorMike Marshall <hubcap@omnibond.com>
Wed, 24 Feb 2016 18:24:14 +0000 (13:24 -0500)
committerMike Marshall <hubcap@omnibond.com>
Wed, 24 Feb 2016 18:24:14 +0000 (13:24 -0500)
Signed-off-by: Mike Marshall <hubcap@omnibond.com>
fs/orangefs/orangefs-kernel.h

index 6290c24d82703f7efd16472c1814caa65bc4c9d6..4ed64e555ca03e116b22ce4be711eb3569ebf9df 100644 (file)
@@ -188,11 +188,12 @@ struct orangefs_kernel_op_s {
        __u64 tag;
 
        /*
-        * Set uses_shared_memory to 1 if this operation uses shared memory.
-        * If true, then a retry on the op must also get a new shared memory
-        * buffer and re-populate it.  Cancels don't care - it only matters
-        * for service_operation() retry logics and cancels don't go through
-        * it anymore.
+        * Set uses_shared_memory to non zero if this operation uses
+        * shared memory. If true, then a retry on the op must also
+        * get a new shared memory buffer and re-populate it.
+        * Cancels don't care - it only matters for service_operation()
+        * retry logics and cancels don't go through it anymore. It
+        * safely stays non-zero when we use it as slot_to_free.
         */
        union {
                int uses_shared_memory;
@@ -205,8 +206,6 @@ struct orangefs_kernel_op_s {
        struct completion waitq;
        spinlock_t lock;
 
-       /* VFS aio fields */
-
        int attempts;
 
        struct list_head list;