drbd: Update some outdated comments to match the code
authorAndreas Gruenbacher <agruen@linbit.com>
Wed, 17 Aug 2011 10:43:25 +0000 (12:43 +0200)
committerPhilipp Reisner <philipp.reisner@linbit.com>
Thu, 8 Nov 2012 15:58:02 +0000 (16:58 +0100)
Signed-off-by: Philipp Reisner <philipp.reisner@linbit.com>
Signed-off-by: Lars Ellenberg <lars.ellenberg@linbit.com>
drivers/block/drbd/drbd_bitmap.c
drivers/block/drbd/drbd_nl.c
drivers/block/drbd/drbd_req.c
drivers/block/drbd/drbd_worker.c

index 706e522..092f827 100644 (file)
@@ -389,8 +389,8 @@ static struct page **bm_realloc_pages(struct drbd_bitmap *b, unsigned long want)
 
        /* Trying kmalloc first, falling back to vmalloc.
         * GFP_KERNEL is ok, as this is done when a lower level disk is
-        * "attached" to the drbd.  Context is receiver thread or cqueue
-        * thread.  As we have no disk yet, we are not in the IO path,
+        * "attached" to the drbd.  Context is receiver thread or drbdsetup /
+        * netlink process.  As we have no disk yet, we are not in the IO path,
         * not even the IO path of the peer. */
        bytes = sizeof(struct page *)*want;
        new_pages = kmalloc(bytes, GFP_KERNEL);
index 92adb3d..684e2e4 100644 (file)
@@ -2038,7 +2038,7 @@ int drbd_adm_connect(struct sk_buff *skb, struct genl_info *info)
                goto fail;
        }
 
-       /* allocation not in the IO path, cqueue thread context */
+       /* allocation not in the IO path, drbdsetup / netlink process context */
        new_conf = kzalloc(sizeof(*new_conf), GFP_KERNEL);
        if (!new_conf) {
                retcode = ERR_NOMEM;
index 60fc186..d61309d 100644 (file)
@@ -292,13 +292,12 @@ void _req_may_be_done(struct drbd_request *req, struct bio_and_error *m)
 
        if ((s & RQ_NET_MASK) == 0 || (s & RQ_NET_DONE)) {
                /* this is disconnected (local only) operation,
-                * or protocol C P_WRITE_ACK,
-                * or protocol A or B P_BARRIER_ACK,
+                * or protocol A, B, or C P_BARRIER_ACK,
                 * or killed from the transfer log due to connection loss. */
                _req_is_done(mdev, req, rw);
        }
        /* else: network part and not DONE yet. that is
-        * protocol A or B, barrier ack still pending... */
+        * protocol A, B, or C, barrier ack still pending... */
 }
 
 static void _req_may_be_done_not_susp(struct drbd_request *req, struct bio_and_error *m)
@@ -668,7 +667,7 @@ int __req_mod(struct drbd_request *req, enum drbd_req_event what,
                        break;
 
                if (req->rq_state & RQ_NET_PENDING) {
-                       /* barrier came in before all requests have been acked.
+                       /* barrier came in before all requests were acked.
                         * this is bad, because if the connection is lost now,
                         * we won't be able to clean them up... */
                        dev_err(DEV, "FIXME (BARRIER_ACKED but pending)\n");
index c57e47c..34a6065 100644 (file)
@@ -969,7 +969,7 @@ int w_e_end_data_req(struct drbd_work *w, int cancel)
 }
 
 /**
- * w_e_end_rsdata_req() - Worker callback to send a P_RS_DATA_REPLY packet in response to a P_RS_DATA_REQUESTRS
+ * w_e_end_rsdata_req() - Worker callback to send a P_RS_DATA_REPLY packet in response to a P_RS_DATA_REQUEST
  * @mdev:      DRBD device.
  * @w:         work object.
  * @cancel:    The connection will be closed anyways