<div dir="ltr"><div>Applied to kinetic and lunar linux master-next</div><div><br></div><div>Thanks Andrea!</div><div><br></div><div>- Luke<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, May 9, 2023 at 7:19 AM Andrea Righi <<a href="mailto:andrea.righi@canonical.com">andrea.righi@canonical.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">BugLink: <a href="https://bugs.launchpad.net/bugs/2016186" rel="noreferrer" target="_blank">https://bugs.launchpad.net/bugs/2016186</a><br>
<br>
[Impact]<br>
<br>
Commit f382fb0bcef4 ("block: remove legacy IO schedulers") introduced a<br>
behavior change in the blkio throttle cgroup subsystem: IO statistics<br>
are not reported anymore unless a throttling rule is explicitly defined,<br>
because the current code only counts bios that are actually throttled.<br>
<br>
This behavior change is potentially breaking some user-space<br>
applications that are relying on the old behavior (see original bug<br>
report).<br>
<br>
[Test case]<br>
<br>
 - mount cgroup v1<br>
 - create a blkio cgroup<br>
 - move a task into the blkio cgroup<br>
 - perform some I/O (i.e., dd)<br>
 - read the IO stats for the cgroup (blkio.throttle.io_serviced and<br>
   blkio.throttle.io_service_bytes in cgroupfs)<br>
 - IO stats are all 0, unless a throttle rule is defined<br>
<br>
Previous behavior (kernel 5.15) was showing I/O statistics even without<br>
throttling rules defined.<br>
<br>
[Fix]<br>
<br>
Apply / backport this fix:<br>
<a href="https://lore.kernel.org/lkml/20230507170631.89607-1-hanjinke.666@bytedance.com/t/" rel="noreferrer" target="_blank">https://lore.kernel.org/lkml/20230507170631.89607-1-hanjinke.666@bytedance.com/t/</a><br>
<br>
[Regression potential]<br>
<br>
The fix is affecting the block IO cgroup subsystem, we may see potential<br>
regressions in this particular cgroup subsystem with this fix applied.<br>
<br>
[Additional notes]<br>
<br>
This fix is provided as SAUCE patch for now, but it is likely to be<br>
applied to the upstream stable trees at this point (it has been<br>
submitted upstream and well tested).<br>
<br>
<br>
-- <br>
kernel-team mailing list<br>
<a href="mailto:kernel-team@lists.ubuntu.com" target="_blank">kernel-team@lists.ubuntu.com</a><br>
<a href="https://lists.ubuntu.com/mailman/listinfo/kernel-team" rel="noreferrer" target="_blank">https://lists.ubuntu.com/mailman/listinfo/kernel-team</a><br>
</blockquote></div></div>