Skip to content

Conversation

@kakra
Copy link
Owner

@kakra kakra commented Nov 23, 2024

Export patch series: https://github.com/kakra/linux/pull/35.patch

  • based on an idea by Eric Wheeler
  • low priority IO requests bypass the cache to reduce SSD wear from backups and maintenance jobs

Eric Wheeler and others added 3 commits November 18, 2024 15:28
…ack hints

Add sysfs entries to support to hint for bypass/writeback by the ioprio
assigned to the bio.  If the bio is unassigned, use current's io-context
ioprio for cache writeback or bypass (configured per-process with
`ionice`).

Having idle IOs bypass the cache can increase performance elsewhere
since you probably don't care about their performance.  In addition,
this prevents idle IOs from promoting into (polluting) your cache and
evicting blocks that are more important elsewhere.

If you really nead the performance at the expense of SSD wearout,
then configure ioprio_writeback and set your `ionice` appropriately.

For example:
	echo 2,7 > /sys/block/bcache0/bcache/ioprio_bypass
	echo 2,0 > /sys/block/bcache0/bcache/ioprio_writeback

See the documentation commit for details.

v2: Removed calls to get_task_io_context()

Signed-off-by: Eric Wheeler <bcache@linux.ewheeler.net>
Acked-by: Kent Overstreet <kent.overstreet@gmail.com>
Tested-by: Kai Krakow <kai@kaishome.de>
Cc: nix@esperi.org.uk
Signed-off-by: Kai Krakow <kai@kaishome.de>
v2: get_task_io_context() no longer exists in the kernel

Signed-off-by: Eric Wheeler <bcache@linux.ewheeler.net>
Signed-off-by: Kai Krakow <kai@kaishome.de>
Even if ioprio_bypass hints bypassing the request, we still allow it
for REQ_META|REQ_PRIO bio.

Cc: Eric Wheeler <bcache@linux.ewheeler.net>
Signed-off-by: Kai Krakow <kai@kaishome.de>
@kakra kakra added the done To be superseded by next LTS label Dec 13, 2025
@kakra
Copy link
Owner Author

kakra commented Dec 14, 2025

Superseded by #41

@kakra kakra closed this Dec 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

done To be superseded by next LTS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants