Align DISCARD requests on zvols.
authorEtienne Dechamps <etienne.dechamps@ovh.net>
Thu, 4 Oct 2012 08:38:55 +0000 (10:38 +0200)
committerBrian Behlendorf <behlendorf1@llnl.gov>
Thu, 4 Oct 2012 23:01:44 +0000 (16:01 -0700)
commit089fa91bc51dc10da5a75a6d9fff77f186fccb23
tree3bd79d76d353df0b931f9ce2818404b7a2c5a040
parent31ab19429787d04d2291851b8178bcee817c15ca
Align DISCARD requests on zvols.

Currently, when processing DISCARD requests, zvol_discard() calls
dmu_free_long_range() with the precise offset and size of the request.

Unfortunately, this is not optimal for requests that are not aligned to
the zvol block boundaries. Indeed, in the case of an unaligned range,
dnode_free_range() will zero out the unaligned parts. Not only is this
useless since we are not freeing any space by doing so, it is also slow
because it translates to a read-modify-write operation.

This patch fixes the issue by rounding up the discard start offset to
the next volume block boundary, and rounding down the discard end
offset to the previous volume block boundary.

Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
Closes #1010
module/zfs/zvol.c