]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
block, bfq: fix some typos in comments
authorAngelo Ruocco <angelo.ruocco.90@gmail.com>
Mon, 8 Apr 2019 15:35:34 +0000 (17:35 +0200)
committerJens Axboe <axboe@kernel.dk>
Mon, 8 Apr 2019 16:05:43 +0000 (10:05 -0600)
Some of the comments in the bfq files had typos. This patch fixes them.

Signed-off-by: Angelo Ruocco <angeloruocco90@gmail.com>
Signed-off-by: Paolo Valente <paolo.valente@linaro.org>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
block/bfq-cgroup.c
block/bfq-iosched.c
block/bfq-iosched.h
block/bfq-wf2q.c

index 2a74a3f2a8f7aec8ac7aaed2724529d0e38b3a19..793c027ca60e8b58872a3418174cdab7619b1a11 100644 (file)
@@ -1103,7 +1103,7 @@ struct cftype bfq_blkcg_legacy_files[] = {
        },
 #endif /* CONFIG_DEBUG_BLK_CGROUP */
 
-       /* the same statictics which cover the bfqg and its descendants */
+       /* the same statistics which cover the bfqg and its descendants */
        {
                .name = "bfq.io_service_bytes_recursive",
                .private = (unsigned long)&blkcg_policy_bfq,
index ceb06abd73df3e93f6a6dfc9e7d26adfbe004a29..bcaa4bde3961d161aa0b9fc7c9f209f50da7bd00 100644 (file)
@@ -189,7 +189,7 @@ static const int bfq_default_max_budget = 16 * 1024;
 /*
  * When a sync request is dispatched, the queue that contains that
  * request, and all the ancestor entities of that queue, are charged
- * with the number of sectors of the request. In constrast, if the
+ * with the number of sectors of the request. In contrast, if the
  * request is async, then the queue and its ancestor entities are
  * charged with the number of sectors of the request, multiplied by
  * the factor below. This throttles the bandwidth for async I/O,
@@ -217,7 +217,7 @@ const int bfq_timeout = HZ / 8;
  * queue merging.
  *
  * As can be deduced from the low time limit below, queue merging, if
- * successful, happens at the very beggining of the I/O of the involved
+ * successful, happens at the very beginning of the I/O of the involved
  * cooperating processes, as a consequence of the arrival of the very
  * first requests from each cooperator.  After that, there is very
  * little chance to find cooperators.
@@ -441,7 +441,7 @@ void bfq_schedule_dispatch(struct bfq_data *bfqd)
 
 /*
  * Lifted from AS - choose which of rq1 and rq2 that is best served now.
- * We choose the request that is closesr to the head right now.  Distance
+ * We choose the request that is closer to the head right now.  Distance
  * behind the head is penalized and only allowed to a certain extent.
  */
 static struct request *bfq_choose_req(struct bfq_data *bfqd,
@@ -989,7 +989,7 @@ static unsigned int bfq_wr_duration(struct bfq_data *bfqd)
         *   of several files
         * mplayer took 23 seconds to start, if constantly weight-raised.
         *
-        * As for higher values than that accomodating the above bad
+        * As for higher values than that accommodating the above bad
         * scenario, tests show that higher values would often yield
         * the opposite of the desired result, i.e., would worsen
         * responsiveness by allowing non-interactive applications to
@@ -2636,8 +2636,8 @@ static bool bfq_allow_bio_merge(struct request_queue *q, struct request *rq,
                /*
                 * bic still points to bfqq, then it has not yet been
                 * redirected to some other bfq_queue, and a queue
-                * merge beween bfqq and new_bfqq can be safely
-                * fulfillled, i.e., bic can be redirected to new_bfqq
+                * merge between bfqq and new_bfqq can be safely
+                * fulfilled, i.e., bic can be redirected to new_bfqq
                 * and bfqq can be put.
                 */
                bfq_merge_bfqqs(bfqd, bfqd->bio_bic, bfqq,
@@ -3089,7 +3089,7 @@ static void __bfq_bfqq_expire(struct bfq_data *bfqd, struct bfq_queue *bfqq)
        /*
         * All in-service entities must have been properly deactivated
         * or requeued before executing the next function, which
-        * resets all in-service entites as no more in service.
+        * resets all in-service entities as no more in service.
         */
        __bfq_bfqd_reset_in_service(bfqd);
 }
@@ -5632,7 +5632,7 @@ static void bfq_prepare_request(struct request *rq, struct bio *bio)
  * preparation is that, after the prepare_request hook is invoked for
  * rq, rq may still be transformed into a request with no icq, i.e., a
  * request not associated with any queue. No bfq hook is invoked to
- * signal this tranformation. As a consequence, should these
+ * signal this transformation. As a consequence, should these
  * preparation operations be performed when the prepare_request hook
  * is invoked, and should rq be transformed one moment later, bfq
  * would end up in an inconsistent state, because it would have
index 60c148728cc5013e0c112af019b76e4457dfcb5e..e7dc07cf9a57bd3b3560d1faa51beb360f1869bb 100644 (file)
@@ -91,7 +91,7 @@ struct bfq_service_tree {
  * expiration. This peculiar definition allows for the following
  * optimization, not yet exploited: while a given entity is still in
  * service, we already know which is the best candidate for next
- * service among the other active entitities in the same parent
+ * service among the other active entities in the same parent
  * entity. We can then quickly compare the timestamps of the
  * in-service entity with those of such best candidate.
  *
@@ -142,7 +142,7 @@ struct bfq_weight_counter {
  *
  * Unless cgroups are used, the weight value is calculated from the
  * ioprio to export the same interface as CFQ.  When dealing with
- * ``well-behaved'' queues (i.e., queues that do not spend too much
+ * "well-behaved" queues (i.e., queues that do not spend too much
  * time to consume their budget and have true sequential behavior, and
  * when there are no external factors breaking anticipation) the
  * relative weights at each level of the cgroups hierarchy should be
index 51ef1f00df802de9780193733bb9f94fcc383ad9..d2ea98ef26a3918d4d1056db6a45781af81228f6 100644 (file)
@@ -59,7 +59,7 @@ static bool bfq_update_parent_budget(struct bfq_entity *next_in_service);
  * bfq_update_next_in_service - update sd->next_in_service
  * @sd: sched_data for which to perform the update.
  * @new_entity: if not NULL, pointer to the entity whose activation,
- *             requeueing or repositionig triggered the invocation of
+ *             requeueing or repositioning triggered the invocation of
  *             this function.
  * @expiration: id true, this function is being invoked after the
  *             expiration of the in-service entity
@@ -90,7 +90,7 @@ static bool bfq_update_next_in_service(struct bfq_sched_data *sd,
 
        /*
         * If this update is triggered by the activation, requeueing
-        * or repositiong of an entity that does not coincide with
+        * or repositioning of an entity that does not coincide with
         * sd->next_in_service, then a full lookup in the active tree
         * can be avoided. In fact, it is enough to check whether the
         * just-modified entity has the same priority as
@@ -1396,7 +1396,7 @@ left:
  * In this first case, update the virtual time in @st too (see the
  * comments on this update inside the function).
  *
- * In constrast, if there is an in-service entity, then return the
+ * In contrast, if there is an in-service entity, then return the
  * entity that would be set in service if not only the above
  * conditions, but also the next one held true: the currently
  * in-service entity, on expiration,
@@ -1479,12 +1479,12 @@ static struct bfq_entity *bfq_lookup_next_entity(struct bfq_sched_data *sd,
                 * is being invoked as a part of the expiration path
                 * of the in-service queue. In this case, even if
                 * sd->in_service_entity is not NULL,
-                * sd->in_service_entiy at this point is actually not
+                * sd->in_service_entity at this point is actually not
                 * in service any more, and, if needed, has already
                 * been properly queued or requeued into the right
                 * tree. The reason why sd->in_service_entity is still
                 * not NULL here, even if expiration is true, is that
-                * sd->in_service_entiy is reset as a last step in the
+                * sd->in_service_entity is reset as a last step in the
                 * expiration path. So, if expiration is true, tell
                 * __bfq_lookup_next_entity that there is no
                 * sd->in_service_entity.