Dan Kenigsberg has posted comments on this change.
Change subject: Move the SANLock SDM lease to a different offset
......................................................................
Patch Set 16: (2 inline comments)
....................................................
File vdsm/storage/safelease.py
Line 37: # orfer to preserve the ability to acquire both locks (e.g.: during the
orfer->order
Line 40: SDM_LEASE_OFFSET = 512 * 2048
where does this number come from? what limitations does it put on MAX_HOST_ID ?
--
To view, visit
http://gerrit.ovirt.org/3840
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I45db6790acffecfda006b513b784f8138605ad81
Gerrit-PatchSet: 16
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <fsimonce(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com>
Gerrit-Reviewer: Igor Lvovsky <ilvovsky(a)redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi(a)redhat.com>