Skip to content
This repository has been archived by the owner on Oct 22, 2024. It is now read-only.

pmd-ndctl.go: implement FlushDeviceData #123

Closed
wants to merge 1 commit into from

Conversation

okartau
Copy link
Contributor

@okartau okartau commented Jan 9, 2019

call flushDevice on block device, this improves pass rate
of csi-sanity in direct-nvdimm mode.

@okartau okartau requested a review from avalluri January 9, 2019 13:19
@okartau
Copy link
Contributor Author

okartau commented Jan 10, 2019

verified in direct-nvdimm mode, unified-mode on single host, in qemu emulation so far, running util/sanity and util/lifecycle

Copy link
Contributor

@avalluri avalluri left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ndctl's Namespace.nullify() can't sever this purpose?

@okartau
Copy link
Contributor Author

okartau commented Jan 10, 2019

yes, true, I think while I was testing, nullify was not working properly.
csi-sanity test rate should get improved by nullify as well (I need to verify that separately)
But nullify is not replacement of Flush, because nullify zeros 4k of device start, while Flush wipes all data, and thats needed to fulfill privacy requirement.

@okartau okartau force-pushed the add-flush-ndctl-mode branch from 7cc5f26 to 40675f5 Compare January 10, 2019 12:43
call flushDevice on block device, this improves pass rate
of csi-sanity in direct-nvdimm mode.
@okartau okartau force-pushed the add-flush-ndctl-mode branch from 40675f5 to 52e6264 Compare January 15, 2019 11:45
@okartau okartau mentioned this pull request Jan 15, 2019
@okartau
Copy link
Contributor Author

okartau commented Jan 15, 2019

change from this PR was moved into #133, closing

@okartau okartau closed this Jan 15, 2019
@okartau okartau deleted the add-flush-ndctl-mode branch March 11, 2019 12:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants