diff options
author | Dan Williams <dan.j.williams@intel.com> | 2019-08-26 17:55:00 -0700 |
---|---|---|
committer | Dan Williams <dan.j.williams@intel.com> | 2019-08-29 13:51:57 -0700 |
commit | bc4f2199ca3107809df96cf72f618b9559b00a21 (patch) | |
tree | 45cdf145a18f3ecae8c06ee1f78c53f0359fb07c /drivers/nvdimm/pfn_devs.c | |
parent | d78c620a2e824d7b01a6e991208a8aa2c938cabe (diff) | |
download | lwn-bc4f2199ca3107809df96cf72f618b9559b00a21.tar.gz lwn-bc4f2199ca3107809df96cf72f618b9559b00a21.zip |
libnvdimm/security: Tighten scope of nvdimm->busy vs security operations
An attempt to freeze DIMMs currently runs afoul of default blocking of
all security operations in the entry to the 'store' routine for the
'security' sysfs attribute.
The blanket blocking of all security operations while the DIMM is in
active use in a region is too restrictive. The only security operations
that need to be aware of the ->busy state are those that mutate the
state of data, i.e. erase and overwrite.
Refactor the ->busy checks to be applied at the entry common entry point
in __security_store() rather than each of the helper routines to enable
freeze to be run regardless of busy state.
Reviewed-by: Dave Jiang <dave.jiang@intel.com>
Reviewed-by: Jeff Moyer <jmoyer@redhat.com>
Link: https://lore.kernel.org/r/156686729996.184120.3458026302402493937.stgit@dwillia2-desk3.amr.corp.intel.com
Signed-off-by: Dan Williams <dan.j.williams@intel.com>
Diffstat (limited to 'drivers/nvdimm/pfn_devs.c')
0 files changed, 0 insertions, 0 deletions