OSDN Git Service

dm space maps: don't reset space map allocation cursor when committing
authorJoe Thornber <ejt@redhat.com>
Tue, 13 Apr 2021 08:03:49 +0000 (09:03 +0100)
committerMike Snitzer <snitzer@redhat.com>
Fri, 4 Jun 2021 16:07:21 +0000 (12:07 -0400)
commit5faafc77f7de69147d1e818026b9a0cbf036a7b2
tree07331ee521ebc8d3fb15dc3c98acbb71907526d8
parent4eafdb1515a708d97e4659bd488ddac19f274c4f
dm space maps: don't reset space map allocation cursor when committing

Current commit code resets the place where the search for free blocks
will begin back to the start of the metadata device.  There are a couple
of repercussions to this:

- The first allocation after the commit is likely to take longer than
  normal as it searches for a free block in an area that is likely to
  have very few free blocks (if any).

- Any free blocks it finds will have been recently freed.  Reusing them
  means we have fewer old copies of the metadata to aid recovery from
  hardware error.

Fix these issues by leaving the cursor alone, only resetting when the
search hits the end of the metadata device.

Signed-off-by: Joe Thornber <ejt@redhat.com>
Signed-off-by: Mike Snitzer <snitzer@redhat.com>
drivers/md/persistent-data/dm-space-map-disk.c
drivers/md/persistent-data/dm-space-map-metadata.c