Re: [PATCH v2 2/3] blk-ioprio: remove ioprio_blkcg_from_bio()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Yu Kuai <yukuai1@xxxxxxxxxxxxxxx>, jack@xxxxxxx, hch@xxxxxxxxxxxxx, tj@xxxxxxxxxx, josef@xxxxxxxxxxxxxx, axboe@xxxxxxxxx
Subject: Re: [PATCH v2 2/3] blk-ioprio: remove ioprio_blkcg_from_bio()
From: Bart Van Assche <bvanassche@xxxxxxx>
Date: Fri, 19 Jul 2024 09:26:40 -0700
Cc: cgroups@xxxxxxxxxxxxxxx, linux-block@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, yukuai3@xxxxxxxxxx, yi.zhang@xxxxxxxxxx, yangerkun@xxxxxxxxxx
In-reply-to: <20240719023431.3800647-3-yukuai1@huaweicloud.com>
References: <20240719023431.3800647-1-yukuai1@huaweicloud.com> <20240719023431.3800647-3-yukuai1@huaweicloud.com>
User-agent: Mozilla Thunderbird

On 7/18/24 7:34 PM, Yu Kuai wrote:

Currently, if config is enabled, then ioprio is always enabled by
default from blkcg_init_disk(), hence there is no point to check if
the policy is enabled from blkg in ioprio_blkcg_from_bio(). Hence remove
it and get blkcg directly from bio.


Reviewed-by: Bart Van Assche <bvanassche@xxxxxxx>

References:
- [PATCH v2 0/3] blk-ioprio: remove per-disk structure
  - From: Yu Kuai
- [PATCH v2 2/3] blk-ioprio: remove ioprio_blkcg_from_bio()
  - From: Yu Kuai

Prev by Date: Re: [PATCH v2 3/3] blk-ioprio: remove per-disk structure
Next by Date: Re: [PATCH v7 2/3] btrfs: always uses root memcgroup for filemap_add_folio()
Previous by thread: Re: [PATCH v2 2/3] blk-ioprio: remove ioprio_blkcg_from_bio()
Next by thread: [PATCH -v2] cgroup: fix deadlock caused by cgroup_mutex and cpu_hotplug_lock
Index(es):
- Date
- Thread

[Index of Archives] [Linux ARM Kernel] [Linux ARM] [Linux Omap] [Fedora ARM] [IETF Annouce] [Security] [Bugtraq] [Linux OMAP] [Linux MIPS] [eCos] [Asterisk Internet PBX] [Linux API] [Monitors]

Powered by Linux