Sukadev Bhattiprolu wrote:
Oren Laadan [orenl@xxxxxxxxxxxxxxx] wrote:
|
|
| > + if (lock) {
| > + h->fl_start = lock->fl_start;
| > + h->fl_end = lock->fl_end;
| > + h->fl_type = lock->fl_type;
| > + h->fl_flags = lock->fl_flags;
| > + } else {
| > + /* Checkpoint a dummy lock as a marker */
| > + h->fl_start = -1;
|
| Maybe designate some constant for this ? e.g. CKPT_FLOCK_NONE ?
| In any case, you need a (loff_t) -1 (like in the restore code).
Ok. Defining macros CKPT_HDR_SET_MARKER_LOCK() and
CKPT_HDR_CHECK_MARKER_LOCK(). Also added the missing (loff_t).
The nice thing about #define CKPT_FLOCK_NONE -1 - see also
the CKPT_PID_NULL, when defined in checkpoint_hdr.h, is that
they are intentionally visible to userspace too.
Oren.
--
To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
