diff options
author | George Wilson <[email protected]> | 2020-09-18 14:13:47 -0500 |
---|---|---|
committer | GitHub <[email protected]> | 2020-09-18 12:13:47 -0700 |
commit | c494aa7f578d5cb844b770f679bd46495242edad (patch) | |
tree | 6abcc94bc41285c5157817062b600c81986b5e7e /module/zfs/lzjb.c | |
parent | 908d43d0a9f736af62c0f4b179950bb1262dfd7d (diff) |
vdev_ashift should only be set once
== Motivation and Context
The new vdev ashift optimization prevents the removal of devices when
a zfs configuration is comprised of disks which have different logical
and physical block sizes. This is caused because we set 'spa_min_ashift'
in vdev_open and then later call 'vdev_ashift_optimize'. This would
result in an inconsistency between spa's ashift calculations and that
of the top-level vdev.
In addition, the optimization logical ignores the overridden ashift
value that would be provided by '-o ashift=<val>'.
== Description
This change reworks the vdev ashift optimization so that it's only
set the first time the device is configured. It still allows the
physical and logical ahsift values to be set every time the device
is opened but those values are only consulted on first open.
Reviewed-by: Matthew Ahrens <[email protected]>
Reviewed-by: Brian Behlendorf <[email protected]>
Reviewed-by: Cedric Berger <[email protected]>
Signed-off-by: George Wilson <[email protected]>
External-Issue: DLPX-71831
Closes #10932
Diffstat (limited to 'module/zfs/lzjb.c')
0 files changed, 0 insertions, 0 deletions