Bug 173254 - [zfs] [patch] Upgrade requests used in ZFS trim map based on ashift
Summary: [zfs] [patch] Upgrade requests used in ZFS trim map based on ashift
Status: Closed FIXED
Alias: None
Product: Base System
Classification: Unclassified
Component: kern (show other bugs)
Version: 8.3-RELEASE
Hardware: Any Any
: Normal Affects Only Me
Assignee: Steven Hartland
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-11-01 04:10 UTC by Steven Hartland
Modified: 2012-12-13 17:08 UTC (History)
0 users

See Also:


Attachments
file.txt (2.47 KB, text/plain)
2012-11-01 04:10 UTC, Steven Hartland
no flags Details
zz-zfstrim-block-perf.txt (1.99 KB, text/plain; format=flowed)
2012-11-01 23:15 UTC, Steven Hartland &
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Steven Hartland 2012-11-01 04:10:01 UTC
Upgrades trim free request sizes before inserting them into to free map, making range consolidation much more effective particularly for small deletes.

This reduces memory used by the free map as well as reducing the number of bio requests down to geom required to process all deletes.

In tests this achieved a factor of 10 reduction of trim ranges / geom call downs.

Fix: Apply the attached patch

Patch attached with submission follows:
How-To-Repeat: N/A
Comment 1 Eitan Adler freebsd_committer freebsd_triage 2012-11-01 19:02:08 UTC
Responsible Changed
From-To: freebsd-bugs->freebsd-fs

set synopsis and assign
Comment 2 Steven Hartland & 2012-11-01 23:15:56 UTC
Updated patched which simplifies / optimises logic

================================================
This e.mail is private and confidential between Multiplay (UK) Ltd. and the person or entity to whom it is addressed. In the event of misdirection, the recipient is prohibited from using, copying, printing or otherwise disseminating it or any information contained in it. 

In the event of misdirection, illegible or incomplete transmission please telephone +44 845 868 1337
or return the E.mail to postmaster@multiplay.co.uk.
Comment 3 Steven Hartland freebsd_committer freebsd_triage 2012-11-17 18:47:50 UTC
Responsible Changed
From-To: freebsd-fs->smh

I'll take it.
Comment 4 Steven Hartland freebsd_committer freebsd_triage 2012-12-13 17:08:46 UTC
State Changed
From-To: open->closed

patched in head/ with r244187