Yaniv Kaul has posted comments on this change. Change subject: block: add blkdiscard on zero image ......................................................................
Patch Set 1: (1 comment) https://gerrit.ovirt.org/#/c/35631/1/vdsm/storage/blockSD.py File vdsm/storage/blockSD.py: Line 219: size = multipath.getDeviceSize(lvm.lvDmDev(sdUUID, volUUID)) Line 220: Line 221: try: Line 222: misc.ddWatchCopy("/dev/zero", path, aborting, size) Line 223: utillinux.blkdiscard(path) Are we discarding AFTER we perform the 'dd'? What's the point? Line 224: except Exception as e: Line 225: log.exception('zeroing operation failed') Line 226: raise se.VolumesZeroingError(path) Line 227: -- To view, visit https://gerrit.ovirt.org/35631 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I4e059c556c550440727b36b8af8e5dfc29ce2ccb Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli <fsimo...@redhat.com> Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: Tal Nisan <tni...@redhat.com> Gerrit-Reviewer: Yaniv Kaul <yk...@redhat.com> Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org> Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches