Commit 337366e0 authored by Yu Kuai's avatar Yu Kuai Committed by Jens Axboe
Browse files

block, bfq: replace 0/1 with false/true in bic apis 



Just to make the code a litter cleaner, there are no functional changes.

Signed-off-by: default avatarYu Kuai <yukuai3@huawei.com>
Reviewed-by: default avatarJan Kara <jack@suse.cz>
Link: https://lore.kernel.org/r/20221214033155.3455754-3-yukuai1@huaweicloud.com


Signed-off-by: default avatarJens Axboe <axboe@kernel.dk>
parent 452af7dc

block/bfq-cgroup.c

+4 −4
Original line number Diff line number Diff line
@@ -728,15 +728,15 @@ static void __bfq_bic_change_cgroup(struct bfq_data *bfqd, 
				    struct bfq_io_cq *bic,

				    struct bfq_group *bfqg)

{

	struct bfq_queue *async_bfqq = bic_to_bfqq(bic, 0);

	struct bfq_queue *sync_bfqq = bic_to_bfqq(bic, 1);

	struct bfq_queue *async_bfqq = bic_to_bfqq(bic, false);

	struct bfq_queue *sync_bfqq = bic_to_bfqq(bic, true);

	struct bfq_entity *entity;



	if (async_bfqq) {

		entity = &async_bfqq->entity;



		if (entity->sched_data != &bfqg->sched_data) {

			bic_set_bfqq(bic, NULL, 0);

			bic_set_bfqq(bic, NULL, false);

			bfq_release_process_ref(bfqd, async_bfqq);

		}

	}
@@ -772,7 +772,7 @@ static void __bfq_bic_change_cgroup(struct bfq_data *bfqd, 
				 */

				bfq_put_cooperator(sync_bfqq);

				bfq_release_process_ref(bfqd, sync_bfqq);

				bic_set_bfqq(bic, NULL, 1);

				bic_set_bfqq(bic, NULL, true);

			}

		}

	}

block/bfq-iosched.c

+2 −2
Original line number Diff line number Diff line
@@ -3114,7 +3114,7 @@ bfq_merge_bfqqs(struct bfq_data *bfqd, struct bfq_io_cq *bic, 
	/*

	 * Merge queues (that is, let bic redirect its requests to new_bfqq)

	 */

	bic_set_bfqq(bic, new_bfqq, 1);

	bic_set_bfqq(bic, new_bfqq, true);

	bfq_mark_bfqq_coop(new_bfqq);

	/*

	 * new_bfqq now belongs to at least two bics (it is a shared queue):
@@ -6562,7 +6562,7 @@ bfq_split_bfqq(struct bfq_io_cq *bic, struct bfq_queue *bfqq) 
		return bfqq;

	}



	bic_set_bfqq(bic, NULL, 1);

	bic_set_bfqq(bic, NULL, true);



	bfq_put_cooperator(bfqq);