Received-SPF: pass (google.com: domain of linux-kernel+bounces-38276-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
Precedence: bulk
MIME-Version: 1.0
References: <20240125071901.3223188-1-zhaoyang.huang@unisoc.com>
 <6b2d5694-f802-43a4-a0fd-1c8e34f8e69a@kernel.org> <CAGWkznHK5UPajY2PG24Jm7+A0c9q+tyQzrPdd=n3tp0dgX+T0w@mail.gmail.com>
 <95082224-a61d-4f4b-bc96-1beea8aa93a9@kernel.org>
In-Reply-To: <95082224-a61d-4f4b-bc96-1beea8aa93a9@kernel.org>
From: Zhaoyang Huang <huangzhaoyang@gmail.com>
Date: Thu, 25 Jan 2024 17:32:47 +0800
Message-ID: <CAGWkznGe+37K7_E34G_MZQXNxoLuM3E0mY=BZCGpyiq+TB_PoA@mail.gmail.com>
Subject: Re: [PATCHv3 1/1] block: introduce content activity based ioprio
To: Damien Le Moal <dlemoal@kernel.org>
Cc: "zhaoyang.huang" <zhaoyang.huang@unisoc.com>, Andrew Morton <akpm@linux-foundation.org>, 
	Jens Axboe <axboe@kernel.dk>, Yu Zhao <yuzhao@google.com>, 
	Niklas Cassel <niklas.cassel@wdc.com>, "Martin K . Petersen" <martin.petersen@oracle.com>, 
	Hannes Reinecke <hare@suse.de>, Linus Walleij <linus.walleij@linaro.org>, linux-mm@kvack.org, 
	linux-block@vger.kernel.org, linux-kernel@vger.kernel.org, 
	steve.kang@unisoc.com
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

On Thu, Jan 25, 2024 at 4:26=E2=80=AFPM Damien Le Moal <dlemoal@kernel.org>=
 wrote:
>
> On 1/25/24 16:52, Zhaoyang Huang wrote:
> > On Thu, Jan 25, 2024 at 3:40=E2=80=AFPM Damien Le Moal <dlemoal@kernel.=
org> wrote:
> >>
> >> On 1/25/24 16:19, zhaoyang.huang wrote:
> >>> From: Zhaoyang Huang <zhaoyang.huang@unisoc.com>
> >>>
> >>> Currently, request's ioprio are set via task's schedule priority(when=
 no
> >>> blkcg configured), which has high priority tasks possess the privileg=
e on
> >>> both of CPU and IO scheduling.
> >>> This commit works as a hint of original policy by promoting the reque=
st ioprio
> >>> based on the page/folio's activity. The original idea comes from LRU_=
GEN
> >>> which provides more precised folio activity than before. This commit =
try
> >>> to adjust the request's ioprio when certain part of its folios are ho=
t,
> >>> which indicate that this request carry important contents and need be
> >>> scheduled ealier.
> >>>
> >>> This commit is verified on a v6.6 6GB RAM android14 system via 4 test=
 cases
> >>> by changing the bio_add_page/folio API in ext4 and f2fs.
> >>
> >> And as mentioned already by Chrisoph and Jens, why don't you just simp=
ly set
> >> bio->bi_ioprio to the value you want before calling submit_bio() in th=
ese file
> >> systems ? Why all the hacking of the priority code for that ? That is =
not
> >> justified at all.
> >>
> >> Furthermore, the activity things reduces the ioprio hint bits to the b=
are
> >> minimum 3 bits necessary for command duration limits. Not great. But i=
f you
> >> simply set the prio class based on your activity algorithm, you do not=
 need to
> >> change all that.
> > That is because bio->io_prio changes during bio grows with adding
> > different activity pages in. I have to wrap these into an API which
> > has both of fs and block be transparent to the process.
>
> Pages are not added to BIOs on the fly. The FS does bio_add_page() or sim=
ilar
> (it can be a get user pages for direct IOs) and then calls bio_submit(). =
Between
> these 2, you can set your IO priority according to how many pages you hav=
e.
Please correct me if I am wrong. So you suggest iterating the
request->bios->bvecs(pages) before final submit_bio? Is it too costly
and introduces too many modifications on each fs.
>
> You can even likely do all of this based on the iocb (and use iocb->ki_io=
prio to
> set the prio), so before one starts allocating and setting up BIOs to pro=
cess
> the user IO.
Actually, the activity information comes from page's history (recorded
at page cache's slot) instead of user space in step(1) and can be
associate with bio in step(2) or iterate the bio in step(3)

page fault        \
(1)
                              (2)                       (3)
                          allocate_pages=3D=3D>add_page_to_page_cache(get
activity information)=3D=3D>xxx_readpage=3D=3D>bio_add_page=3D=3D>submit_bi=
o
vfs read/write  /
>
> --
> Damien Le Moal
> Western Digital Research
>